cast to pointer from integer of different size.

[Kaiser K. (kingkong_18)]

hallo,
ich möchte Daten per Rs232 übertragen. dafür habe ich mir diese untere 
Funktion geschrieben. Bevor ich ein Kommando schicken möchte ich erst 
die VT und CRC drauf hängen.

beim compilieren komme ich auf die Fehlermeldung "cast to pointer from 
integer of different size". ich habe viele printf reingeschrieben, um 
nur zu gucken wo mein program hängt. ich hast schon mehr mal gegoogelt 
und komme aber immer noch nicht weiter...

wie könnte ich sie Prüfsumme casten um sie in VT anhägen zu können.

Danke im Voraus

void SendViaRS232(int ComPort, unsigned char *string,   uint8_t CRC)
{
  int commandLen, i = 0;
  char VT[50] = "\v";
  char c= (unsigned char)CRC;

  printf("\nhahahaaa\n");
  strcat((char *)VT, (const char*)c); // hier kommt die Meldung

        printf("\bbbbbbbbbbbbb\n");

  strcat((char *)string, (const char*)VT);
  commandLen = strlen((const char*)string);


  RS232_flushTX(ComPort);

  for (i = 0; i <= commandLen; ++i)
  {
    RS232_SendByte(ComPort, string[i]);
    Sleep(10);
    RS232_flushTX(ComPort);
  }
}

[Niklas G. (erlkoenig)]

Sicher, dass du den Eingabe-String verändern willst? So ginge es:

void SendViaRS232(int ComPort, const unsigned char *string,   uint8_t CRC)
{
  char VT[50] = "\v";

  VT [1] = CRC;
  strncpy (VT+2, string, sizeof(VT)-2);
  size_t commandLen = strlen(VT);
  if (commandLen > sizeof(VT)) commandLen=sizeof(VT); // Kann passieren wenn string zu lang

  RS232_flushTX(ComPort);

  for (size_t i = 0; i <= commandLen; ++i)
  {
    RS232_SendByte(ComPort, string[i]);
    Sleep(10);
    RS232_flushTX(ComPort);
  }
}

Variablen braucht man nicht am Funktionsanfang deklarieren. Das macht 
man am besten so spät wie möglich, dann kann man sie nicht versehentlich 
benutzen wenn sie noch keinen Wert haben. Verwende außerdem size_t für 
String/Array-Längen - z.B. auf AMD64 ist "int" meistens 32bit, und damit 
zu klein. size_t hat immer automatisch die richtige Größe. Nutze "const 
char*"  um den String nicht versehentlich zu verändern.

Da du eh jedes Byte einzeln sendest, kannst du dir den Puffer sparen:

void SendViaRS232(int ComPort, const unsigned char *string,   uint8_t CRC)
{
  RS232_flushTX(ComPort);
  RS232_SendByte(ComPort, '\v');
  Sleep(10);
  RS232_flushTX(ComPort);
  RS232_SendByte(ComPort, CRC);
  Sleep(10);
  RS232_flushTX(ComPort);
  while (*string) {
    RS232_SendByte(ComPort, *string);
    ++string;
    Sleep(10);
    RS232_flushTX(ComPort);
  }
}

Siehe ggf. auch Serialisierung.

[Kaiser K. (kingkong_18)]

Niklas G. schrieb:
>

void SendViaRS232(int ComPort, const unsigned char *string,   uint8_t 
> CRC)
> {
>   char VT[50] = "\v";
> 
>   VT [1] = CRC;
>   strncpy (VT+2, string, sizeof(VT)-2);
>   size_t commandLen = strlen(VT);
>   if (commandLen > sizeof(VT)) commandLen=sizeof(VT); // Kann passieren 
> wenn string zu lang
> 
>   RS232_flushTX(ComPort);
> 
>   for (size_t i = 0; i <= commandLen; ++i)
>   {
>     RS232_SendByte(ComPort, string[i]);
>     Sleep(10);
>     RS232_flushTX(ComPort);
>   }
> }

>

Danke für deine Rückmeldung.
ich hätte noch eine Bemerkung:

was ich mache möchte ist diese VT und CRC ganz am ende jede Kommando 
anhängen. das hießt man könnte V[1] = CRC nicht schreiben.

[Niklas G. (erlkoenig)]

Kaiser K. schrieb:
> was ich mache möchte ist diese VT und CRC ganz am ende jede Kommando
> anhängen. das hießt man könnte V[1] = CRC nicht schreiben.

Achso, das war an deinem Code nicht ersichtlich. Dann musst du die 
Anweisungen tauschen.

[Kaiser K. (kingkong_18)]

Niklas G. schrieb:
> Achso, das war an deinem Code nicht ersichtlich. Dann musst du die
> Anweisungen tauschen.

ja mache ich

Vielen Dank noch mal

[(prx) A. K. (prx)]

Ein char ist kein String und der Cast erzeugt einen Pointer ins 
Nirgendwo:
   char c= (unsigned char)CRC
   (const char*)c

[Dirk B. (dirkb2)]

Kaiser K. schrieb:
>   char VT[50] = "\v";
>   char c= (unsigned char)CRC;
>
>   strcat((char *)VT, (const char*)c); // hier kommt die Meldung

strcat erwartet zwei Adressen auf jeweils einen Speicherbereich, deren 
Inhalt Nullterminiert ist.
Die Adresse von einem Objekt bekommt man in C mit dem Adressoperator.

Das wäre dann &c.

Da aber c nur ein Zeichen aufnehmen kann, ist kein Platz für die '\0'. 
Daher funktioniert das auch nicht (obwohl keine Fehlermeldung mehr 
kommt).

[foobar (Gast)]

Wenn ich sowas sehen:

  for (i = 0; i <= commandLen; ++i)
  {
    RS232_SendByte(ComPort, string[i]);
    Sleep(10);
    RS232_flushTX(ComPort);
  }

klingen bei mir die Alarmglocken. In 99.99% der Fälle ist das ein 
Zeichen, dass die SendByte-Routine falsch implementiert ist. Mit ner 
korrekten Implementation wird deine Routine trivial:

void SendViaRS232(int ComPort, unsigned char *string, uint8_t CRC)
{
    while (*string)
        RS232_SendByte(ComPort, *string++);
    RS232_SendByte(ComPort, '\v');
    RS232_SendByte(ComPort, CRC);
}