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);
}
}
if(commandLen>sizeof(VT))commandLen=sizeof(VT);// Kann passieren wenn string zu lang
9
10
RS232_flushTX(ComPort);
11
12
for(size_ti=0;i<=commandLen;++i)
13
{
14
RS232_SendByte(ComPort,string[i]);
15
Sleep(10);
16
RS232_flushTX(ComPort);
17
}
18
}
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:
>if(commandLen>sizeof(VT))commandLen=sizeof(VT);// Kann passieren
10
>wennstringzulang
11
>
12
>RS232_flushTX(ComPort);
13
>
14
>for(size_ti=0;i<=commandLen;++i)
15
>{
16
>RS232_SendByte(ComPort,string[i]);
17
>Sleep(10);
18
>RS232_flushTX(ComPort);
19
>}
20
>}
>
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.
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. 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).
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:
