2 Sketche vereinen

[Markus (Gast)]

Hi Leute,


ich brauche dringend eure Hilfe.

Ich habe 2 Sketche, die unabhängig von einander relativ gut 
funktionieren, aber ich würde es gerne vereinen.

Das eine Programm ist der Empfänger von einem Sender/Empfänger Programm. 
Dieser empfängt int-Werte und schaltet LEDs an bzw. aus.

Das andere Programm erkennt XML Tag und Inhalt wie z.B. <LED1>An, 
Tag=<LED1> und Inhalt=An.

Diese 2 Programme habe ich schon versucht zu vereinen, aber durch 
mangels der Kenntnisse der Programmiersprache, wende ich mich nach mehr 
als einer mühen vollen Woche ohne Erfolg an euch.

Bitte hilft mir.

Hier die Programme

1. XML

#include <string.h>

char Tag[20] = "";
char Inhalt[20] = "";
char tmpStr[20] = "";
char endTag[3] = {'<', '/', '\0'};

// Flags fuer Daten und XML-Code
boolean TagFlag = false;
boolean InhaltFlag = false;

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

void loop() {
  while (Serial.available()) {
    XML(); 
  }
}
 
void XML() {

      char Zeichen = Serial.read();
  
   if (Zeichen == '<') {
      ZuLang(Zeichen, tmpStr);
      TagFlag = true;
      InhaltFlag = false;

   } else if (Zeichen == '>') {
      ZuLang(Zeichen, tmpStr);

      if (TagFlag) {      
         strncpy(Tag, tmpStr, strlen(tmpStr)+1);
      }
      // Tmp buffer wird geloescht
      Loschen(tmpStr);

      TagFlag = false;
      InhaltFlag = true;      
      
   } else if (Zeichen != 10) {
      if (TagFlag) {
         // fuegt den tag in den string bsp. <tempraum1>
         ZuLang(Zeichen, tmpStr);

         // Checkt </XML> end tag und ignoriert es
         if ( TagFlag && strcmp(tmpStr, endTag) == 0 ) {
            Loschen(tmpStr);
            TagFlag = false;
            InhaltFlag = false;
         }
      }
      if (InhaltFlag) {
         
         ZuLang(Zeichen, Inhalt);
      }
   }  
      Serial.print("Tag: ");
      Serial.println(Tag);
      Serial.print("Inahlt: ");
      Serial.println(Inhalt);
}

// Function im String zu loeschen
void Loschen (char* str) {
  
   int len = strlen(str);
   for (int c = 0; c < len; c++) {
      str[c] = 0;
   }
}

void ZuLang (char ch, char* str) {
  
   char *TagNachr  = "<TAG ZU LANG>";
   char *InhaltNachr = "-INHALT ZU LANG-";

//Wenn string zu lang wird, ignoriert er ihn
   if (strlen(str) > 20 - 2) {
      if (TagFlag) {
         Loschen(Tag);
         strcpy(Tag,TagNachr);
      }
      if (InhaltFlag) {
         Loschen(Inhalt);
         strcpy(Inhalt,InhaltNachr);
      }
      //loescht den tem buffer
      Loschen(tmpStr);
      TagFlag = false;
      InhaltFlag = false;

   } else {
        //fuegt ein eingelesenen Wert dem String hinzu
     str[strlen(str)] = ch;
   }
}

2. Empfänger

/**
 * MISO -> 12
 * MOSI -> 11
 * SCK -> 13
 * CE -> 8
 * CSN -> 7
 */

#include <SPI.h>
#include <Mirf.h>
#include <nRF24L01.h>
#include <MirfHardwareSpiDriver.h>

int fall=0;

void setup(){
  
  pinMode(4,OUTPUT);
  pinMode(3,OUTPUT);
  pinMode(2,OUTPUT);
  Serial.begin(9600);

  Mirf.spi = &MirfHardwareSpi;
  Mirf.init();
  Mirf.setRADDR((byte *)"serv1");// Name des Servers
  Mirf.payload = sizeof(unsigned long);
  Mirf.config();
  
  Serial.println("Empfangen..."); 
}

void loop(){

  byte data[Mirf.payload]; // Hier wird der empfangene byte gespeichert
  
  if(!Mirf.isSending() && Mirf.dataReady()){
    Serial.println("Got packet");

    Mirf.getData(data); // Packet (daten) empfangen
     
    Mirf.setTADDR((byte *)"clie1");//Bestimmen des Client Namens 
    
     Serial.println(data[0]);
    fall = data[0];
     switch (fall){
      
      case '1':
      digitalWrite(4,HIGH);break;
      case '2':
      digitalWrite(3,HIGH);break;
      case '3':
       digitalWrite(2,HIGH);break;    
      case '4':
      digitalWrite(4,LOW);break;
      case '5':
      digitalWrite(3,LOW);break;
      case '6':
       digitalWrite(2,LOW);break; 
     } 
    Mirf.send(data); // Sendet wert zurueck zum Clienten,
      
    Serial.println("Reply sent.");
  }
}

Wer die Module(Arduino Uno, nRF24l01 Funkmodul) zur Hand hat, kann das 
testen, dafür sende ich noch den Sender dazu

3. Sender

#include <SPI.h>
#include <Mirf.h>
#include <nRF24L01.h>
#include <MirfHardwareSpiDriver.h>

int hall; //Variable wird deklariert diese wird dann dem System Uebergeben

void setup(){
  Serial.begin(9600); 
  
  Mirf.spi = &MirfHardwareSpi; 
  Mirf.init(); 
  
  Mirf.setRADDR((byte *)"clie1"); //Der Name bzw. Adresse des Clienten wird erstellt

  Mirf.payload = sizeof(unsigned long); // Hier wird die Laenge des Buffers 

  Mirf.config(); //Laden der Konfigurationsdatein
  
  Serial.println("Beginn ... "); 
}

void loop(){
   unsigned long time = millis(); 

  int hall=0;
  
  if (Serial.available() > 0) {
                hall = Serial.read();} 
  Mirf.setTADDR((byte *)"serv1"); // Sucht den Server mit dem Namen "serv1"
  
  Mirf.send((byte *)&hall); 
  
  while(Mirf.isSending()){
  }
  Serial.println("Empfang beendet");
  delay(10);
  while(!Mirf.dataReady()){
    
    if ( ( millis() - time ) > 1000 ) {
      Serial.println("Server antwortet nicht!");
      return;
    }
  }
  Mirf.getData((byte *) &time);
  
  Serial.print("Zeit in mills: ");
  Serial.println((millis() - time));
  
  delay(1000);
}

Vielen Dank schon mal im Vorraus

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Lies bitte, was über jedem Eingabefeld steht:

Wichtige Regeln - erst lesen, dann posten!
  Längeren Sourcecode nicht im Text einfügen, sondern als Dateianhang

Deine Arduino Programme sind übrigens C-Verschnitt, deshalb wäre es 
sinnvoll, wenn du die Datei(en) nach *.c umbenennst. Dann gibt hier 
kostenlos nettes Syntaxhighlighting dazu...

Oder machs wenigstens so:

Formatierung 
    [c]C-Code[/c]

[Markus (Gast)]

Verzeih mir, habe das übersehen.

kann ich es noch ändern?

[Max H. (hartl192)]

Markus schrieb:
> kann ich es noch ändern?
Nur als Registrierter User. Ein Mod hat es für dich gemacht...

[Christian B. (chbalnuweit)]

Was funktioniert denn nicht? Gibt es Fehler beim kompilieren (oder wie 
das bei dem Arduino-Quatsch heissen mag)?

[Nachhilfe (Gast)]

Christian B. schrieb:
> Gibt es Fehler beim kompilieren (oder wie das bei dem Arduino-Quatsch
> heissen mag)?

"Übersetzen" heißt das da - komisches Wort ;-)

[Markus (Gast)]

HI Christian,


nein, Fehler gibt es nicht, die Programme  funktionieren unabhängig von 
einander relativ gut. Ich habe es nicht geschafft das XML Programm in 
den Empfänger zu integrieren.

Beispiel

<LED1>1</LED1> soll geschickt werden

und der Empfänger soll dann die LED1 einschalten

[Karl H. (kbuchegg)]

Ich denke, du wirst wieder mal den Fehler Nummero Uno in so einem Fall 
gemacht haben.

Du hast Programm A und Programm B und versuchst die beiden irgendwie 
miteinander zu vermischen.

So was geht praktisch immer schief.

Was du statt dessen machen musst, ist Programm C zu schreiben.

D.h. deine Strategie ist es nicht, die beiden Programm zu mischen, 
sondern zu allererst schreibst du erst mal ein komplett neues Programm.
Punkt.
In dieses neue Programm kannst du Ideen und Techniken aus den beiden 
anderen Programmen übernehmen. Übernehmen bedeutet in dem Fall: die Idee 
übernehmen, die Grundtechnik übernehmen. Ab und an kann man eventuell 
auch schon mal Codeteile übernehmen. Aber sich darauf zu versteifen, 
dass man den Code 1:1 übertragen kann, funktioniert nur sehr selten und 
auch nur dann, wenn die zu übernehmenden Codeteile ordentlich in ein 
Modul ausgelagert wurden.

Also:
Wenn du jedes der beiden Programme einzeln geschrieben hast, dann kennst 
du ja die Ideen die da drinn stecken und es sollte eigentlich nicht so 
schwer sein, ein komplett neues Programm zu schreiben, welches die 
Fähigkeiten der beiden Einzelprogramm vereint.


> aber durch mangels der Kenntnisse der Programmiersprache
und das ist der erste Punkt, den du abstellen musst.
Wer programmieren will, muss seine Programmiersprache können.
Punkt.

[Markus (Gast)]

Danke Karl Heinz für deine Antwort.


Ich nehme mir dein Vorschlag zur Herzen und werde versuchen ein neues 
Programm zuschreiben mit den Ideen von den anderen beiden.


Leider bleibt mir nicht viel Zeit, um die Programmiersprache komplett 
vorher zu erlernen. Kenntnisse in C sind vorhanden, aber keine 
überragende.

Nichtsdestotrotz wäre es cool, wenn mir jemand direkte Hinweise dafür 
geben würden.

Vielen Dank

[Düsendieb (Gast)]

welche Fehler kommen denn wenn Du mal gnadenlos mit control c und 
control v den Text aus dem einen Sketch in den anderen kopierst und dann 
das ganze unter einem neuen Namen abspeicherst und wieder übersetzt?

[Markus (Gast)]

Also wenn ich das ein so rein kopiere ohne variable namen zu ändern, 
dafür den rest an die richtige stelle setzte, was in Loop kommt habe ich 
in Loop kopiert usw. dann führt der nicht die XML Datei durch. dafür 
muss ich bestimmte variable unbennen und einiges mehr.

[Mark B. (markbrandis)]

Max H. schrieb:
> Markus schrieb:
>> kann ich es noch ändern?
> Nur als Registrierter User. Ein Mod hat es für dich gemacht...

Auch als registrierter Nutzer hätte er es an dieser Stelle nicht mehr 
ändern können ;-)

[Max H. (hartl192)]

Mark Brandis schrieb:
> Auch als registrierter Nutzer hätte er es an dieser Stelle nicht mehr
> ändern können ;-)
Den Eröffnungspost, kann ich auch später, nach einer Antwort, noch 
bearbeiten.

Als Demonstration: Schau dir an, wann dieser Post bearbeitet wurde: 
Beitrag "PIC24 ASM Tutorial"

[Karl H. (kbuchegg)]

Düsendieb schrieb:
> welche Fehler kommen denn wenn Du mal gnadenlos mit control c und
> control v den Text aus dem einen Sketch in den anderen kopierst und dann
> das ganze unter einem neuen Namen abspeicherst und wieder übersetzt?

Es bringt ihn nicht weiter.

Der erste Sketch parst offenbar einen Input, der von der seriellen 
Schnittstelle kommt. Der zweite Sketch ist offenbar die Übertragung von 
Kommandos über eine MIRF Schnittstelle (was immer das auch ist).
Die Zusamenführung der beiden: Übertragung von XML über MIRF und parsen 
dessen, was als XML aus dem MIRF dann rauskommt, gelingt nicht, indem 
man einfach naiv alles zusammenkopiert. Dazu muss man die beiden 
Einzelteile schon verstehen und zb auch verstehen, wie diese MIRF 
Übertragung funktioniert, solche Dinge, wie das hier

  Mirf.payload = sizeof(unsigned long); // Hier wird die Laenge des Buffers

müssen zb angepasst werden.
Hat man das erst mal, hat man die MIRF Übertragung auf Text umgestellt, 
dann kann man den XML Teil da hinten drann hängen. Denn der erste Sketch 
besteht ja auch aus 2 Teilen: dem Teil der das nächste Zeichen von der 
Schnittstelle holt

      char Zeichen = Serial.read();

und dem Rest, der dann mit dem Zeichen etwas macht.

Den Teil, der 1 Zeichen von der Schnittstelle holt, wenn eines da ist, 
den muss man eben durch dieses MIRF ersetzen.

In Summe nicht so schwer. Aber mit naiver Herangehensweise und schwachen 
Programmierfähigkeiten nicht zu machen.

[Markus (Gast)]

Ja, einfach rein kopieren geht leider nicht. Man muss bestimmte Dinge 
anpassen, wie Karl Heinz schon erwähnt hat.

Senden und empfangen funktioniert über ein Funkmodul(nRF24l01). Dieses 
Funkmodul hat 2 Bibs, eine davon ist die Mirf. Diese Mirf hat bestimmte 
Befehle, wie beispielsweise es Daten sendet und empfängt. Zu denen 
gehört auch diese von Karl Hein erwähnten Befehl.

Karl Heinz schrieb:
> In Summe nicht so schwer

Ich bin seit knapp 2 Wochen dabei.

[Mark B. (markbrandis)]

Max H. schrieb:
> Den Eröffnungspost, kann ich auch später, nach einer Antwort, noch
> bearbeiten.

Na sowas. Ich hatte den Satz "Eigene Beiträge können bis maximal 60 
Minuten nach dem Absenden bearbeitet werden, und nur wenn noch keine 
Antworten eingetroffen sind" wohl zu wörtlich genommen. ;-)

[Peter D. (peda)]

Am besten ist es, erstmal ausfürlich zu beschreiben, was die beiden 
Sketsche überhaupt machen sollen.
Einfach so drauf los zu programmieren ist der falsche Weg.

[Markus (Gast)]

HI Peter,
wie gewünscht, beschreibe ich die Programme genauer was die machen 
sollen.

Sender

Der Sender soll eine Zeichenkette senden, die man in die Konsole 
eingegeben hat, wie z.B. <LED1>1</LED1>


Empfänger

Der Empfänger soll diese Zeichenkette empfangen und durch das 
XML-Programm laufen lassen.
Dadurch bekommt man 2 Informationen, anhand des obigen Beispiel 
<LED1>1</LED1>

Tag: <LED1>
Inhalt: 1

Mit diesen Informationen läuft das Programm durch mehrere if-Schleifen, 
eine davon wäre

if(Tag==<LED1>){ 
   
   if(Inhalt==1){
   digitalWrite(4,HIGH);

   }else if(Inhalt==2){
   digitalWrite(4,LOW);

   }else {
   serialprint(Eingabe falsch)
   }
}

Natürlich vergleicht das Programm dann Zeichen für Zeichen,
also Tag[0]==<...

Das war denke ich alles.
Ich hoffe, das man jetzt ein besseres Verständnis dafür hat, was ich 
will und dringend brauche

[Peter D. (peda)]

Markus schrieb:
> Mit diesen Informationen läuft das Programm durch mehrere if-Schleifen,

Also das eine ist eine Unterfunktion des anderen.
Dann ruf doch das 2. am Ende des 1. auf.

Ich sehe allerdings nirgends, wo die 1. irgendwas ablegt, was dann die 
2. auswerten kann.

[Markus (Gast)]

Mit mehrere if-Schleifen, habe ich gemeint, das es mehrere Schleifen für 
den Tag gibt, und jede diese Schleife enthält weitere Schleifen

So ungefähr

if(Tag==<LED1>){ 
   
   if(Inhalt==1){
   digitalWrite(4,HIGH);

   }else if(Inhalt==2){
   digitalWrite(4,LOW);

   }else {
   serialprint(Eingabe falsch);
   }
}

if(Tag==<LED2>){ 
   
   if(Inhalt==1){
   digitalWrite(3,HIGH);

   }else if(Inhalt==2){
   digitalWrite(3,LOW);

   }else {
   serialprint(Eingabe falsch);
   }
}

if(Tag==<SERVO1>){ 
   
   if(Inhalt==1){
   digitalWrite(2,HIGH);

   }else if(Inhalt==2){
   digitalWrite(2,LOW);

   }else {
   serialprint(Eingabe falsch);
   }
}

so in etwa.

[Karl H. (kbuchegg)]

Markus schrieb:
> Mit mehrere if-Schleifen,

Es geht nicht mehr.
ES GIBT KEINE IF_SCHLEIFEN!

Das Wesen einer Schleife ist es, dass etwas wiederholt werden kann. 
Zumindest prinzipiell. Drum heißt es Schleife. Wie bei einem Tonband, 
das zu einer Schleife zusammen geklebt wird und das daher beim Abspielen 
immer wieder von vorne anfängt.

In einem if wird nichts wiederholt. Ein if trifft eine Auswahl aus 2 
Möglichkeiten. Das hat mit einer Schleife genau gar nichts zu tun.

[Karl H. (kbuchegg)]

Markus schrieb:

> so in etwa.

Ja.
Aber nur wenn man die Regeln der Programmiersprache sehr sehr frei 
interpretiert. Ausserdem fehlt da 80%. Die interessanten 80%: nämlich, 
wie kommt der Text erst mal in die String Variable "Tag" hinein.

[Mark B. (markbrandis)]

Karl Heinz schrieb:
> Markus schrieb:
>> Mit mehrere if-Schleifen,
>
> Es geht nicht mehr.
> ES GIBT KEINE IF_SCHLEIFEN!

In der Tat:

http://www.if-schleife.de/

Diese Fehlbezeichnung hält sich leider sehr hartnäckig... :-(

[Markus (Gast)]

Ja gut, dann keine Schleife, sondern eine if-Funktion.

das XML-Programm alleine Funktioniert relativ gut.

Aktuell ist es so, dass das XML-Programm eine Zeichenkette einließt und 
den Tag und Inhalt herausgibt.

dadurch könnte man diese dann durch die if-Funktion durch gehen lassen.

[A. W. (uracolix)]

Komplettes neu schreiben, das macht den Code immer besser!

Also, du hast zwei funktionierende Implementierungen, das ist gut um 
dagegen zu testen. Du hast das Problem vollstaendig verstanden, sonst 
gaebe es die
beiden Programme nicht.

Jetzt setz dich hin und schreib ein drittes Programm was a und b 
vereint.
Du wirst staunen wieviel Optimierungspotenzial deine aktuellen 
Implementierungen haben.

Der Wert deiner Arbeit steckt nicht in den Tastenanschlaegen mit denen 
du
den Code eingegeben hast, sondern im Durchdenken des Problems. Also 
schreibs
einfach neu und mach es besser (und bleib bei der Sprache die kannst, 
wozu was neues Lernen, das waere wirklich ein unnuetzes Nebenprojekt im 
Moment).

Ein Programm ist wie eine Packung Pralinen, die Idee ist die Schokolade 
und der Quelltext die Verpackung.

[Karl H. (kbuchegg)]

A. W. schrieb:

> Ein Programm ist wie eine Packung Pralinen, die Idee ist die Schokolade
> und der Quelltext die Verpackung.

Das hast du schön gesagt. Der Spruch ist es wert, gemerkt zu werden.