RFM12B receive String

[Hi H. (hi_h)]

Guten Abend,

ich habe mein Projekt (drahtlose Temperaturmessstation mit RFM12B Chips) 
jetzt um ein drahtloses Display erweitert, auf dem die Temperatur 
angezeigt werden soll.

Mein aktueller Quellcode:

#include <RFM12B.h>
#include <avr/sleep.h>
#include <LiquidCrystal.h>
 
#define NODEID            30  //network ID used for this unit
#define NETWORKID        210  //the network ID we are on
#define ACK_TIME        2000  // # of ms to wait for an ack
 
#define LEDpin 1
 
#define SenderNODE 22
 
//Define LED-Ports
#define LCD_RS 9
#define LCD_enable 6
#define LCD_D4 5
#define LCD_D5 4
#define LCD_D6 3
#define LCD_D7 7
 
 
LiquidCrystal lcd(LCD_RS, LCD_enable, LCD_D4, LCD_D5, LCD_D6, LCD_D7);
RFM12B radio;
 
static void activityLED (byte mode) {
  pinMode(LEDpin, OUTPUT);
  digitalWrite(LEDpin, mode);
}
 
 
static void blink (byte pin, byte n = 3) {
  pinMode(pin, OUTPUT);
  for (byte i = 0; i < 2 * n; ++i) {
    delay(100);
    digitalWrite(pin, !digitalRead(pin));
  }
}
 
void setup() {
 
    if (LEDpin) {
    activityLED(1); // LED on
    delay(1000);
    activityLED(0); // LED off
  }
 
  radio.Initialize(NODEID, RF12_868MHZ, NETWORKID);
  lcd.begin(16, 2);
 
  lcd.print("Es funktioniert!");
  lcd.setCursor(0, 0);
 
  delay(1000);
  lcd.clear();
 
 
}
 
void loop() {
 
 
  if (radio.ReceiveComplete()) {
    if (radio.CRCPass()) {
      if (radio.GetSender() == SenderNODE) {
        int i;
        char cmd[*radio.DataLen];
        for (byte i = 0; i < *radio.DataLen; i++) //can also use radio.GetDataLen() if you don't like pointers
          cmd[i] = ((char) radio.Data[i]);
        lcd.print ((char) radio.Data[i]);
         
        cmd[i] = '\0';
       
       
       
        //PRINT TO DISPLAY HERE
        //lcd.print("test");
        lcd.setCursor(0, 1);  
 
        if (LEDpin) {
          blink(LEDpin, 2);
        }
     
   
   
   
   
   
   
    } }  }

Mein Ziel ist es jetzt, dass der String, der von dem Sender gesendet 
wird auf dem Display angezeigt wird. Ich bekomme nur nicht so richtig 
hin aus den empfangenen Daten wieder einen String zu machen.

Vielen Dank schon mal im Voraus
Paul

[Florian (Gast)]

Hi H. schrieb:
> for (byte i = 0; i < *radio.DataLen; i++)
>           cmd[i] = ((char) radio.Data[i]);
>         lcd.print ((char) radio.Data[i]);
>
>         cmd[i] = '\0';

Is zwar schon etwas her, aber ...

Was genau willst Du denn da machen?

// Willst Du hier die Daten aus "radio.Data[i]" in "cmd[i]" kopieren?
for (byte i = 0; i < *radio.DataLen; i++)
{
   cmd[i] = ((char) radio.Data[i]);
}

// Warum gibst Du dann "radio.Data[i]" aus, wozu dann das kopieren in "cmd[i]"? Wenn Du die Laufvariable i angibst landest Du auch im nirgendwo, da sie nach dem hochlaufen in der for-Schleife jetzt am Ende des Arrays steht. Das kann also gar nicht klappen :-) 
lcd.print ((char) radio.Data[i]);

// Wenn Du hier das String Ende setzt, was Grundsätzlich nicht falsch ist, warum machst Du dann mit cmd[i] nichts mehr?
cmd[i] = '\0';


// Du kannst auch einfach "cmd[i] = 0" schreiben.

logischer fände ich sowas:

for (byte i = 0; i < *radio.DataLen; i++)
{
   cmd[i] = ((char) radio.Data[i]);
}
cmd[i] = 0;

// Ein Array ist bereits ein Pointer auf das erste Element
lcd.print ((char) cmd);

// Du kannst auch schreiben:
lcd.print ((char) &cmd[0]);

// ob der cast auf (char) nötig ist kann ich Dir nicht sagen, das musst Du testen.

Mein code ist aber nur eine Grundlage, ich denke nicht das es so auf 
anhieb klappen wird.

Warum kopierst Du überhaupt um? Wenn Du einen String sendest sollte der 
doch auch eine 0-Terminierung haben. Kannst Du dann nicht sofort 
"radio.Data" ausgeben?

Gruß
Florian

[Hi H. (hi_h)]

Hallo Florian,

vielen Dank für die ausführliche Antwort! Habe jetzt leider auch schon 
länger nicht mehr hier rein geguckt, daher die späte Antwort.

Ich werde den Code direkt mal ausprobieren und gegebenfalls versuchen 
ans Laufen zu bringen.

Ich wollte einen String haben, damit ich diesen speichern und dann wenn 
ein neuer Messwert ankommt, den alten in der unteren Zeile des LCDs 
anzeigen kann.

Gruß
Paul