Arduino Serielle Schnittstelle Fehlersuche

[Flo D. (floflovaldo)]

Hallo zusammen,

ich habe folgender Code, der nicht so ganz funktioniert. Ich sende über 
die Schnittstelle vom PC "DHTT" und bekomme keine Antwort zurück. Die 
Ausgabe in der Methode loop bekomme ich, also "Testausgabe", aber 
irgendwie gibt er gar nichts in der Methode Einlesen aus. Nicht das 
"Echo: " und auch nicht den Beispielwert "xx.xx".

Würde mich über Hilfe freuen.

Gruß Florian

String Input="";                  // Input von Serieller Schnittstelle
String Befehl="NOP";              // Befehl aus Input
int Parameter=0;                  // Parameter aus Input
int Wert=0;                       // Rückgabewert Analog

//OneWire DSPin(2);                 // Datenleitung DS18xxx -> Pin 2
char buffer[20] = "";             // Speicherbereich für Datenkonvertierung

void setup() {
  Serial.begin(9600);
}

void loop() {
  if (Serial.available()) Einlesen();
  Serial.println("Testausgabe");
  delay(2000);
}

void Einlesen() {
  while (Serial.read());
    {
      char ch = Serial.read();
      Serial.print("Echo: ");
      Serial.println(ch);
      Input=Input+ch;
      Serial.println(Input);
      delay(10);
    }
    Befehl = Input.substring(0, 4);
    //Serial.println("Befehl: " + Befehl);
    Parameter = (Input.substring(4, 6)).toInt();
    //Serial.print("Parameter: ");
    //Serial.orintln("Parameter,DEC);
    ausfuehren();                                     
    //Serial.println(Befehl + Parameter + " = " + Wert);

    Input="";
}

void ausfuehren() {
  if (Befehl=="DHTT")
  {
    Serial.print("xx.xx");
  }
}

[Dieter S. (Gast)]

Ich habe mir nicht den kompletten Sketch angesehen, aber Strings musst 
du mit "strcmp" vergleichen, nicht mit "==".

[Flo D. (floflovaldo)]

und wie mache ich das dann ganz genau. Direkt ersetzten?

[Kaj (Gast)]

Einfach mal in die beschreibung der funktion gucken...
http://www.cplusplus.com/reference/cstring/strcmp/

[Norbert (Gast)]

Florian D. schrieb:
> Hallo zusammen,
>
> ich habe folgender Code, der nicht so ganz funktioniert. Ich sende über
> die Schnittstelle vom PC "DHTT" und bekomme keine Antwort zurück. Die
> Ausgabe in der Methode loop bekomme ich, also "Testausgabe", aber
> irgendwie gibt er gar nichts in der Methode Einlesen aus. Nicht das
> "Echo: " und auch nicht den Beispielwert "xx.xx".
>

void Einlesen() {
   while (Serial.read());

Nimm mal das Semikolon dort weg!

[Dieter S. (Gast)]

Florian D. schrieb:
> und wie mache ich das dann ganz genau. Direkt ersetzten?

Ganz einfach. Google benutzen und Basics lernen.

[Michael U. (amiga)]

Hallo,

  if (Serial.available()) Einlesen();
...
  while (Serial.read());
    {
      char ch = Serial.read();

Semikolen wurde ja schon gesagt, außerdem rufst Du bei einem verfügbaren 
Zeichen Einlesen() auf.
  while (Serial.read());
Diese Zeile holt jetzt das Zeichen.
      char ch = Serial.read();
Danach liest Du nochmal ein Zeichen, da dürfte aber keins mehr sein, 
weil Du das ja oben schon gelesen hast.

Gruß aus Berlin
Michael

[Flo D. (floflovaldo)]

Hallo,

Ja das mit dem Semikolon ist nun klar.

Aber wie würdest du das dann anderst machen. Ich versteh schon was du 
meinst aber ich wüsste jetzt nicht wie ich es ändern soll.

Gruß Florian

[Marc S. (marc_s86)]

char ch;
while (ch = Serial.read())
...

[Ulrich F. (Gast)]

Als Arduino Fan möchte ich dazu auch mal was sagen:

 while (Serial.available())
{
  // tuwas
}

Ansonsten fummelst du da mit String Objekten rum.
Das bedeutet dauernde "dynamische Speicherverwaltung". Problematisch!
Der String Instanz kann man auch einmal einen Buffer zuweisen. Dann 
lässt sie das sein.

Dieter S. schrieb:
> Ich habe mir nicht den kompletten Sketch angesehen, aber Strings musst
> du mit "strcmp" vergleichen, nicht mit "==".
Die String Klasse kann einiges....
https://www.arduino.cc/en/Reference/StringComparison

[Flo D. (floflovaldo)]

Hallo, ich habe das das Semikolon raus und hab geguckt das ich die 
Zeichen nicht zweimal auslesen:

String Input="";                  // Input von Serieller Schnittstelle
String Befehl="NOP";              // Befehl aus Input
int Parameter=0;                  // Parameter aus Input
int Wert=0;                       // Rückgabewert Analog
char ch;

//OneWire DSPin(2);                 // Datenleitung DS18xxx -> Pin 2
char buffer[20] = "";             // Speicherbereich für Datenkonvertierung

void setup() {
  Serial.begin(9600);
}

void loop() {
  if (Serial.available()) Einlesen();
  Serial.println("Testausgabe");
  delay(2000);
}

void Einlesen() {
  while (ch = Serial.read())
    {
      //char ch = Serial.read();
      //Serial.print("Echo: ");
      //Serial.println(ch);
      Input=Input+ch;
      //Serial.println(Input);
      delay(10);
    }
    Befehl = Input.substring(0, 4);
    //Serial.println("Befehl: " + Befehl);
    Parameter = (Input.substring(4, 6)).toInt();
    //Serial.print("Parameter: ");
    //Serial.orintln("Parameter,DEC);
    ausfuehren();                                     
    //Serial.println(Befehl + Parameter + " = " + Wert);

    Input="";
}

void ausfuehren() {
  if (Befehl=="DHTT")
  {
    Serial.print("xx.xx");
  }
}

Leider funktioniert es immer noch nicht. Ich bekomme genau die gleiche 
Ausgabe wie ganz am Anfang

[Marc S. (marc_s86)]

dann lass die debug ausgaben doch mal drin. kommt da was?

[Dieter F. (Gast)]

Probier mal so:

String Input="";                  // Input von Serieller Schnittstelle
boolean stringComplete = false;   // whether the string is complete
String Befehl="NOP";              // Befehl aus Input
int Parameter=0;                  // Parameter aus Input
int Wert=0;                       // Rückgabewert Analog


//OneWire DSPin(2);                 // Datenleitung DS18xxx -> Pin 2
char buffer[20] = "";             // Speicherbereich für Datenkonvertierung

void setup() 
{
  Serial.begin(9600);
  Serial.println("Testausgabe");
  Input.reserve(20);
}

void loop() 
{
  if (stringComplete) 
  {
    Serial.println(Input);
    // clear the string:
    ausfuehren();
    Input = "";
    stringComplete = false;
  }
}

/*
  SerialEvent occurs whenever a new data comes in the
 hardware serial RX.  This routine is run between each
 time loop() runs, so using delay inside loop can delay
 response.  Multiple bytes of data may be available.
 */
void serialEvent() {
  while (Serial.available()) 
  {
    // get the new byte:
    char inChar = (char)Serial.read();
    // add it to the inputString:
    Input += inChar;
    // if the incoming character is a newline, set a flag
    // so the main loop can do something about it:
    if (inChar == '\n') 
    {
      stringComplete = true;
    }
  }
}

void ausfuehren() 
{
  if (Input == "DHTT\n")
  {
    Serial.println("xx.xx");
  }
  else
    Serial.println("nop");
}

und stelle Deinen serialmonitor auf "Neue Zeile"

[Tom Thomsen (Gast)]

Mal 'ne ganz blöde Frage: Wozu gibt es in der Arduino IDE im Menü /Datei 
den Menüpunkt Beispiele.

Vielleicht sollte man erstmal mit einem funktionierenden einfachen 
Beispiel aus /04.Communication/ anfangen und das schrittweise in 
Richtung der gewünschten Funktion umbauen. Damit lassen sich viele 
grundlegende Fehler vermeiden.

[Flo D. (floflovaldo)]

Hallo,

ich habe es jetzt zum laufen gebracht, stehe jedoch jetzt vor einem 
anderen Problem in dem Sketch. Ich will ein DS18S20 Sensor einbinden. 
Jedoch bekomme ich folgendes angezeigt wenn ich es aufn Arduino 
Hochladen will: "'show_temperature' was not declared in this scope". 
Dies betrifft Zeile 62 im Code, diese wird rot markiert.

Ich will wenn ich nun OneWire01 an den Arduino sende eben die Temperatur 
des Sensors ausgelesen und angezeigt bekommen.

#include <OneWire.h>
#include <DallasTemperature.h>

#define DS18B20_PIN 2

OneWire oneWire(DS18B20_PIN);
DallasTemperature sensors(&oneWire);

int BlinkRate=0;                                         // Anzahl Blinken für Tests 
String Input="";                                         // Input von Serieller Schnittstelle 
String Befehl="NOP";                                     //Befehl aus Input 
int Parameter=0;                                         //Parameter aus Input 
int Wert=0;                                              //Rückgabewert Analog

char buffer[20] = "";                                    // Speicherbereich für Datenkonvertierung


void setup()
{
  Serial.begin(9600);
  sensors.begin();
}  

void loop()
{
  if (Serial.available()) Einlesen();
  //Serial.println("Testausgabe");
  //delay(2000);
}



void Einlesen()
{
    while (Serial.available())
       {
       char ch = Serial.read();
       //Serial.print("Echo: ");
       //Serial.println(ch);
       Input=Input+ch;
       //Serial.println(Input);
       delay(10);
       }
       Befehl = Input.substring(0, 4);
       //Serial.println("Befehl: " + Befehl);
       Parameter = (Input.substring(4, 6)).toInt();
       //Serial.print("Parameter: ");
       //Serial.println(Parameter,DEC);
       ausfuehren();
       //Serial.println(Befehl + Parameter + " = " + Wert);
       
       Input="";
}

void ausfuehren()
{

  if (Befehl=="OW01")
  {
   sensors.requestTemperatures();
  for(byte i=0;i<sensors.getDeviceCount();i++){ 
    show_temperature(i+1,sensors.getTempCByIndex(i));
  }
  else
  {
   Serial.println("ERROR");
  }
}

void show_temperature(byte num,float temp){
  //Serial.print("Sensor ");
  Serial.print(num);
  Serial.print(": ");
  Serial.println(temp);
  //Serial.print(" ");  
  //Serial.write(176);  
  //Serial.println("C");
}

[Ulrich F. (Gast)]

Florian D. schrieb:
> "'show_temperature' was not declared in this scope".

Die Meldung passt nicht zum Code!
> expected '}' before 'else'
sagts bei mir

[Flo D. (floflovaldo)]

Das ist was es bei mir anzeigt:

HANS:64: error: expected '}' before 'else'

   else

   ^

exit status 1
'show_temperature' was not declared in this scope

Und was ist mit dem ersten gemeint, Habe doch ein } vor dem else

[google (Gast)]

Zähl mal die Zahl der öffnenden geschweiften Klammern im IF-Teil und 
dann die Zahl der ... aaah ;-)

  {
   sensors.requestTemperatures();
  for(byte i=0;i<sensors.getDeviceCount();i++){
    show_temperature(i+1,sensors.getTempCByIndex(i));
  }

[Ulrich F. (Gast)]

Florian D. schrieb:
> Das ist was es bei mir anzeigt:

Tipp:
Wenn du 2 Fehlermeldungen bekommst, dann genau in dieser Reihenfolge 
bearbeiten. Denn der 2te könnte ein Folgefehler sein, welcher nur auf 
Grund des ersten Fehlers auftritt.

[Flo D. (floflovaldo)]

Danke, funktioniert.

[Flo D. (floflovaldo)]

Hat jemand schon oft mit den Sensoren DS18S20 gearbeitet. Im oben 
gesehen Code ist es ja so dass wenn ich OW01 an den Arduino sende dann 
der Temperatursensor ausgelesen wird.

Man kann ja mehrere von denen an den gleichen Pin anschließen, das habe 
ich ebenfalls schon gemacht (siehe Code oben, die werden ja nummeriert). 
Nun möchte ich aber zum Beispiel bei "OW01" den einen Sensor, der sich 
z.B. im Haus befindet auslesen und wenn ich Alternativ "OW02" an den 
Arduino sende, den zweiten Auslesen, der z.B. draußen ist.

Kann mir da jemand helfen oder wie der ausgelesen wird?

[Falk B. (falk)]

Beitrag "Onewire + DS18x20 Library"