String aus Stringtabelle Pointer zeigt in den Wald!?

[Jan H. (janiiix3)]

Moin,

irgendwie habe ich heute ein Pointer Problem.
Wieso kann ich nicht den Inhalt mit "strcpy" kopieren?
Was mache ich bei der Pointer Verarbeitung falsch?

#include <stdio.h>
#include <stdlib.h>
#include <stdint.h>
#include <string.h>


#define HC05T_EOC  "\r\n" // end of command string



typedef struct{
  char *cmd;
}hc05t_t;

/*
*  command table for the bluetooth µc
*   attention, for some one at commands you need the '=' character
*/
const hc05t_t hc05tCmd[] = 
{
  "AT"      , //[0] test uart connection
  "AT+RESET"    , //[1] reset device
  "AT+VERSION"  , //[2] query firmware version
  "AT+ORGL"    , //[3] restore settings to factory defaults
  "AT+ADDR"    , //[4] query device bluetooth address
  "AT+NAME"    , //[5] query/set device name
  "AT+RNAME"    , //[6] query remote bluetooth device´s name
  "AT+ROLE"    , //[7] query/set device role
  "AT+CLASS"    , //[8] query/set class of device CoD
  "AT+IAC"    , //[9] query/set inquire access code
  "AT+INQM"    , //[10] query/set inquire access mode
  "AT+PSWD"    , //[11] query/set pairing passkey
  "AT+UART"    , //[12] query/set uart parameter
  "AT+CMODE"    , //[13] query/set communication mode
  "AT+BIND"    , //[14] query/set binding bluetooth address
  "AT+POLAR"    , //[15] query/set led output polarity
  "AT+PIO"    , //[16] set/reset a user i/o pin
  "AT+MPIO"    , //[17] set/reset multiple user i/0 pin
  "AT+MPIO?"    , //[18] query user i/o pin
  "AT+IPSCAN"      , //[19] query/set scanning parameters
  "AT+SNIFF"    , //[20] query/set sniff energy savings parameters
  "AT+SENM"    , //[21] query/set security & encryption modes
  "AT+RMSAD"    , //[22] delete authenticated device from list
  "AT+FSAD"    , //[23] find device from authenticated device list
  "AT+ADCN"    , //[24] query total number of device from authenticated device list
  "AT+MRAD"    , //[25] query most recently used authenticated device
  "AT+STATE"    , //[26] query current state of the device
  "AT+INIT"    , //[27] initialize spp profile
  "AT+INQ"    , //[28] query nearby discoverable devices
  "AT+INQC"    , //[29] cancel search for discoverable devices 
  "AT+PAIR"    , //[30] device pairing
  "AT+LINK"    , //[31] connect to a remote device
  "AT+DISC"    , //[32] disconnect from a remote device
  "AT+ENSNIFF"  , //[33] enter energy saving mode
  "AT+EXSNIFF"  , //[34] exit energy saving mode
};

void hc05tBuildCmd(const hc05t_t *cmd, char *cmdPara)
{
  char sendBuff[40] ="";
  strcpy(sendBuff,(const char *)cmd);
  strcat(sendBuff,cmdPara);
  
  printf(sendBuff);
}

int main(int argc, char *argv[]){
  
  hc05tBuildCmd(&hc05tCmd[10],"TEST_PARA");
  
  return 0;
}

[Stefan F. (Gast)]

Ich glaube es muss

> hc05tBuildCmd(hc05tCmd[10],"TEST_PARA");

heissen, ohne das & Zeichen. Weil das Array Zeiger auf Strings enthält. 
Mit dem & bekommst du aber die Adresse des Zeigers auf einen String.

[Markus F. (mfro)]

Jan H. schrieb:
> strcpy(sendBuff,(const char *)cmd);

spätestens wenn man an unvermuteter Stelle plötzlich einen Cast braucht, 
sollte beim Programmierer die rote Warnlampe angehen.

cmd ist ein Zeiger auf ein Element eines Arrays von structs. Das Element 
enthält einen Zeiger auf einen String. Anstatt den da rauszuholen, 
übergibst Du die Adresse der struct.

So wär's richtig:

strcpy(sendBuff, cmd->cmd);

Und plötzlich braucht man auch keinen Cast mehr.

[Jan H. (janiiix3)]

PEINLICH
Danke! Es ist Montag morgens. Habe es Mega verscheckt.
Danke!!!