Probleme mit nRF24L01+

Zkall · August 22, 2018, 7:25pm

Hallo,

Ich habe ein kleines Problem, ich möchte über meine Handbewegung ein ferngesteuertes Auto steuern.
Dazu habe ich in einen Handschuh einen Arduino-nano, eine 9V Batterie, ein GY-521 Module und ein nRF24L01+ Modul eingebaut.

An sich funktioniert alles, Beispielprogramme zum senden und empfangen der Daten, und im Seriellen Monitor die Lageerkennung der Hand.
Mir stellt sich nur folgendes Problem bei der Programmierung, ich weiß nicht wie ich die nachricht über eine “if”-schleife auslesen lassen kann.

Code des Handschuhs:

(#include "Wire.h"
#include "SPI.h"
#include "RF24.h"

static const uint64_t pipes[6] = {0xF0F0F0F0E1LL, 0xF0F0F0F0D2LL, 0xF0F0F0F0C3LL, 0xF0F0F0F0B4LL, 0xF0F0F0F0A5LL, 0xF0F0F0F096LL};
 

float rechts = 10;
float gerade = 20;
float links = 30;
float vor = 40;
float stehen = 50;
float zurueck = 60;

bool radioNumber = 1;
RF24 radio(6,7);

byte addresses[][6] = {"1Node","2Node"};

const int MPU = 0x68;

int16_t acc_x, acc_y, acc_z;
int16_t gyro_x, gyro_y, gyro_z;
int16_t temp;

char tmp_str[7];

char* convert_int16_to_str(int16_t i)
{
  sprintf(tmp_str, "%6d", i);
  return tmp_str;
}


void setup() 
{
  radio.begin();
  delay(20);
  radio.setChannel(1);                
  radio.setAutoAck(0);
  radio.setRetries(15,15);    
  radio.setPALevel(RF24_PA_HIGH);

  
  Serial.begin(9600);
  Wire.begin();
  Wire.beginTransmission(MPU);
  Wire.write(0x6B);
  Wire.write(0);
  Wire.endTransmission(true);

  radio.begin();
  Serial.println("checking if chip connected");
  bool check = radio.isChipConnected();
  Serial.print("check-");
  Serial.println(check);

    if(radioNumber){
    radio.openWritingPipe(addresses[1]);
    radio.openReadingPipe(1,addresses[0]);
  }else{
    radio.openWritingPipe(addresses[0]);
    radio.openReadingPipe(1,addresses[1]);
  }

  radio.startListening();
  
}

void loop() 
{
  
  Wire.beginTransmission(MPU);
  Wire.write(0x3B);
  Wire.endTransmission(false);
  Wire.requestFrom(MPU, 7*2, true);


  acc_x = Wire.read()<<8 | Wire.read();
  acc_y = Wire.read()<<8 | Wire.read();
  acc_z = Wire.read()<<8 | Wire.read();

  temp = Wire.read()<<8 | Wire.read();

  gyro_x = Wire.read()<<8 | Wire.read();
  gyro_y = Wire.read()<<8 | Wire.read();
  gyro_z = Wire.read()<<8 | Wire.read();

  //Serial.print(" | aX = "); Serial.print(convert_int16_to_str(acc_x));
  //Serial.print(" | aY = "); Serial.print(convert_int16_to_str(acc_y));
  //Serial.print(" | aZ = "); Serial.print(convert_int16_to_str(acc_z));
  
  //Serial.print(" | tmp = "); Serial.print(temp/340.00+36.53);
  
  //Serial.print(" | gX = "); Serial.print(convert_int16_to_str(gyro_x));
  //Serial.print(" | gY = "); Serial.print(convert_int16_to_str(gyro_y));
  //Serial.print(" | gZ = "); Serial.print(convert_int16_to_str(gyro_z));

  //Serial.println();

  long message[6] = {long(rechts*100),long(gerade*100),long(links*100),long(vor*100),long(stehen*100),long(zurueck*100)};
  radio.stopListening();
  radio.write(&message, sizeof(message));
  radio.startListening();


     if (acc_x > 5300)
     {
        rechts++;
        Serial.print("| rechts |");
     }
     if(acc_x < 5300)
     {
        gerade++;
        Serial.print("| gradeaus |");
      }
      if(acc_x < -5300)
      {
        links++;
        Serial.print("| links |");
      }
      
    
      Serial.print(" ][ ");

      if (acc_y > 5300)
     {
        zurueck++;
        Serial.print("| zurück |");
     }
     if(acc_y < 5300)
     {
        stehen++;
        Serial.print("| stop |");
      }
      if(acc_y < -5300)
      {
        vor++;
        Serial.print("| vor |");
      }

      Serial.println();

   delay(100);
  
}
)
________________________
Code des Autos(unfertig):
________________________

[code]#include <Servo.h>
#include "SPI.h"
#include "RF24.h"


static const uint64_t pipes[6] = {0xF0F0F0F0E1LL, 0xF0F0F0F0D2LL, 0xF0F0F0F0C3LL, 0xF0F0F0F0B4LL, 0xF0F0F0F0A5LL, 0xF0F0F0F096LL};
 
uint8_t ClientNummer;
float rechts, gerade, links, vor, stehen, zurueck;
Servo gas;

bool radioNumber = 0;

RF24 radio(6,7);
byte addresses[][6] = {"1Node","2Node"};

int state = 0;
int state2 = 180;
int state3 = 90;

void setup()
{
  radio.begin();
  delay(20);
  radio.setChannel(1);               
  radio.setAutoAck(0);    
  radio.setPALevel(RF24_PA_HIGH);     
                                      
                                      
  radio.openReadingPipe(0,pipes[0]);
  radio.openReadingPipe(1,pipes[1]);
  radio.openReadingPipe(2,pipes[2]);
  radio.openReadingPipe(3,pipes[3]);
  radio.openReadingPipe(4,pipes[4]);
  radio.openReadingPipe(5,pipes[5]);  
 
  //servo
  gas.attach(3);
  Serial.begin(9600);

  Serial.println("checking if chip connected");
  bool check = radio.isChipConnected();
  Serial.print("check-");
  Serial.println(check);

  if(radioNumber){
    radio.openWritingPipe(addresses[1]);
    radio.openReadingPipe(1,addresses[0]);
  }else{
    radio.openWritingPipe(addresses[0]);
    radio.openReadingPipe(1,addresses[1]);
  }

  radio.startListening();
}

void loop() {
  if(radio.available(&ClientNummer))
  {
    long got_message[6] = {0,0,0,0,0,0};

    bool done = false;
    while(!done)
    {
      radio.read(&got_message, sizeof(got_message));
    }

    rechts = (float(got_message[0])/100);
    links = (float(got_message[1])/100);
    gerade = (float(got_message[2])/100);
    vor = (float(got_message[3])/100);
    stehen = (float(got_message[4])/100);
    zurueck = (float(got_message[5])/100);

    

  }
}

postmaster-ino · August 22, 2018, 7:43pm

Hi

Anscheinend fehlt ein Code?

IF ist keine Schleife, sondern eine Anfrage.
Auch wiederholt IF Nichts, egal, was passiert.
(... ok, man könnte mit goto Was stricken ... gibt aber gaaanz schlechtes Karma !!)

Ich verstehe nicht, Was Du auslesen möchtest.

MfG

Whandall · August 22, 2018, 7:43pm

Zkall:
ich weiß nicht wie ich die nachricht über eine "if"-schleife auslesen lassen kann.

Ich weiss nicht einmal was eine if-Schleife sein soll.

Aus wievielen Beispielen hast du den Kram zusammenkopiert?
Auch zwischendurch mal darüber nachgedacht, was du da tust?
Warum sollen sechs "Clients" ein Auto steuern?

  radio.setAutoAck(0);
  radio.setRetries(15,15);

Es gibt keine Timeouts oder Retries ohne Autoack.

Meinst du nicht ein Aufruf von radio.begin() würde genügen?

    bool done = false;
    while(!done)
    {
      radio.read(&got_message, sizeof(got_message));
    }

Das ist eine Schwachsinns Endlosschleife (hier wirklich eine Schleife)

Zkall · August 22, 2018, 8:25pm

Also erstmal wie mach ich 2 Codes in einen Beitrag?

Mit if-schleife meine ich eigentlich eine if-anfrage sorry für die Verwechslung.

Ich habe kein richtiges Tutorial gefunden wie man überhaupt mit dem funk Modul umgeht, also habe ich mir alles mögliche angeschaut und reinkopiert. Es waren viele unterschiedliche Programme.
Und ja ich habe über das nachgedacht was ich tue, aber ich kenne mich mit der Programmierung von funk Modulen nicht wirklich aus.

Zur Erklärung ich habe nie richtig programmieren gelernt, alles nur selbst beigebracht.

Whandall · August 22, 2018, 9:52pm

Code1:

#include "Wire.h"
#include "SPI.h"
#include "RF24.h"

static const uint64_t pipes[6] = {0xF0F0F0F0E1LL, 0xF0F0F0F0D2LL, 0xF0F0F0F0C3LL, 0xF0F0F0F0B4LL, 0xF0F0F0F0A5LL, 0xF0F0F0F096LL};
 

float rechts = 10;
float gerade = 20;
float links = 30;
float vor = 40;
float stehen = 50;
float zurueck = 60;

bool radioNumber = 1;
RF24 radio(6,7);

byte addresses[][6] = {"1Node","2Node"};

const int MPU = 0x68;

int16_t acc_x, acc_y, acc_z;
int16_t gyro_x, gyro_y, gyro_z;
int16_t temp;

char tmp_str[7];

char* convert_int16_to_str(int16_t i)
{
  sprintf(tmp_str, "%6d", i);
  return tmp_str;
}


void setup()
{
  radio.begin();
  delay(20);
  radio.setChannel(1);               
  radio.setAutoAck(0);
  radio.setRetries(15,15);   
  radio.setPALevel(RF24_PA_HIGH);

 
  Serial.begin(9600);
  Wire.begin();
  Wire.beginTransmission(MPU);
  Wire.write(0x6B);
  Wire.write(0);
  Wire.endTransmission(true);

  radio.begin();
  Serial.println("checking if chip connected");
  bool check = radio.isChipConnected();
  Serial.print("check-");
  Serial.println(check);

    if(radioNumber){
    radio.openWritingPipe(addresses[1]);
    radio.openReadingPipe(1,addresses[0]);
  }else{
    radio.openWritingPipe(addresses[0]);
    radio.openReadingPipe(1,addresses[1]);
  }

  radio.startListening();
 
}

void loop()
{
 
  Wire.beginTransmission(MPU);
  Wire.write(0x3B);
  Wire.endTransmission(false);
  Wire.requestFrom(MPU, 7*2, true);


  acc_x = Wire.read()<<8 | Wire.read();
  acc_y = Wire.read()<<8 | Wire.read();
  acc_z = Wire.read()<<8 | Wire.read();

  temp = Wire.read()<<8 | Wire.read();

  gyro_x = Wire.read()<<8 | Wire.read();
  gyro_y = Wire.read()<<8 | Wire.read();
  gyro_z = Wire.read()<<8 | Wire.read();

  //Serial.print(" | aX = "); Serial.print(convert_int16_to_str(acc_x));
  //Serial.print(" | aY = "); Serial.print(convert_int16_to_str(acc_y));
  //Serial.print(" | aZ = "); Serial.print(convert_int16_to_str(acc_z));
 
  //Serial.print(" | tmp = "); Serial.print(temp/340.00+36.53);
 
  //Serial.print(" | gX = "); Serial.print(convert_int16_to_str(gyro_x));
  //Serial.print(" | gY = "); Serial.print(convert_int16_to_str(gyro_y));
  //Serial.print(" | gZ = "); Serial.print(convert_int16_to_str(gyro_z));

  //Serial.println();

  long message[6] = {long(rechts*100),long(gerade*100),long(links*100),long(vor*100),long(stehen*100),long(zurueck*100)};
  radio.stopListening();
  radio.write(&message, sizeof(message));
  radio.startListening();


     if (acc_x > 5300)
     {
        rechts++;
        Serial.print("| rechts |");
     }
     if(acc_x < 5300)
     {
        gerade++;
        Serial.print("| gradeaus |");
      }
      if(acc_x < -5300)
      {
        links++;
        Serial.print("| links |");
      }
     
   
      Serial.print(" ][ ");

      if (acc_y > 5300)
     {
        zurueck++;
        Serial.print("| zurück |");
     }
     if(acc_y < 5300)
     {
        stehen++;
        Serial.print("| stop |");
      }
      if(acc_y < -5300)
      {
        vor++;
        Serial.print("| vor |");
      }

      Serial.println();

   delay(100);
 
}

Code 2

#include <Servo.h>
#include "SPI.h"
#include "RF24.h"


static const uint64_t pipes[6] = {0xF0F0F0F0E1LL, 0xF0F0F0F0D2LL, 0xF0F0F0F0C3LL, 0xF0F0F0F0B4LL, 0xF0F0F0F0A5LL, 0xF0F0F0F096LL};
 
uint8_t ClientNummer;
float rechts, gerade, links, vor, stehen, zurueck;
Servo gas;

bool radioNumber = 0;

RF24 radio(6,7);
byte addresses[][6] = {"1Node","2Node"};

int state = 0;
int state2 = 180;
int state3 = 90;

void setup()
{
  radio.begin();
  delay(20);
  radio.setChannel(1);               
  radio.setAutoAck(0);   
  radio.setPALevel(RF24_PA_HIGH);     
                                     
                                     
  radio.openReadingPipe(0,pipes[0]);
  radio.openReadingPipe(1,pipes[1]);
  radio.openReadingPipe(2,pipes[2]);
  radio.openReadingPipe(3,pipes[3]);
  radio.openReadingPipe(4,pipes[4]);
  radio.openReadingPipe(5,pipes[5]); 
 
  //servo
  gas.attach(3);
  Serial.begin(9600);

  Serial.println("checking if chip connected");
  bool check = radio.isChipConnected();
  Serial.print("check-");
  Serial.println(check);

  if(radioNumber){
    radio.openWritingPipe(addresses[1]);
    radio.openReadingPipe(1,addresses[0]);
  }else{
    radio.openWritingPipe(addresses[0]);
    radio.openReadingPipe(1,addresses[1]);
  }

  radio.startListening();
}

void loop() {
  if(radio.available(&ClientNummer))
  {
    long got_message[6] = {0,0,0,0,0,0};

    bool done = false;
    while(!done)
    {
      radio.read(&got_message, sizeof(got_message));
    }

    rechts = (float(got_message[0])/100);
    links = (float(got_message[1])/100);
    gerade = (float(got_message[2])/100);
    vor = (float(got_message[3])/100);
    stehen = (float(got_message[4])/100);
    zurueck = (float(got_message[5])/100);

   

  }
}

Da du auf meine anderen Punkte nicht eingehst füge ich noch ein paar allgemeine hinzu.

Knoten die nur senden, sollten nicht dauernd auf Empfang umschalten (loop Code 1).
delay sollte grundsätzlich vermieden werden, speziell als Voodoo-Statement (beide setups).
Reste von Rollentausch Beispielen haben hier nichts verloren, oder soll das Auto die Hand verdrehen?.
Das Verwenden einer Sorte von Adressen für die Pipes halte ich für verständlicher.
Ein Handschuh, ein Auto und eine Kommunikationsrichtung erfordert nur eine Pipe/Adresse.
AutoAck und DynamicPayload sind sinnvolle Konfigurationen.
Die Rückgabewerte von radio.write sind interessant und sollten ausgewertet werden (speziell bei AutoAck).
Eine niedrigere Übertragungsrate erhöht die Reichweite.

Zkall · August 23, 2018, 3:49pm

okay danke, ich werde die Codes nochmal nach deinen allgemeinen bearbeiten, und schauen ob ich es dann hinbekomme.

system · May 6, 2021, 1:37pm