Hochzählen mit 7 Segmentanzeige

[biede1987 (Gast)]

Hallo
ich möchte mit einer 7 Segmentanzeige und einem Taster die funktion 
darstellen, sobald ich den Taster drücke, soll die 7 Segmentanzeige 
hochzählen.

Ich programmier mit dem Board arduino duemilanove



ich hab aus dem Internent ein Programm gefunden (tutorial), bei dem ich 
das versuche zu verstehn. aber es kommt bei arduino immer eine 
fehlermeldung,

wisst ihr warum, wer schön wenn ihr mir helfen könntet

http://www.freeduino.de/books/arduino-tutorial-lady-ada/lektion-5-arduino-inputs/tastendr%C3%BCcke-z%C3%A4hlen

[Düsendieb (Gast)]

biede1987 schrieb:
> aber es kommt bei arduino immer eine
> fehlermeldung

Welche?
Wie ist die 7 Segmentanzeige angeschlossen?

[biede1987 (Gast)]

"
sketch_oct16a.ino: In function 'void setup()':
sketch_oct16a:19: error: a function-definition is not allowed here 
before '{' token
sketch_oct16a:29: error: expected `}' at end of input"

hast du auch ein arduino?

da passt irgendwas nicht mit den klammern?

[biede1987 (Gast)]

das programm von der seite

hat nichts mit der 7 segemntanzeige zu tun, das mach ich für mich selber

ich hab das nur versucht zu verstehn wie er addiert, da ich aber das 
programm nicht draufspielen kann, weiß ich nicht obs überhaupt geht!

[Karl H. (kbuchegg)]

biede1987 schrieb:
> "
> sketch_oct16a.ino: In function 'void setup()':
> sketch_oct16a:19: error: a function-definition is not allowed here
> before '{' token
> sketch_oct16a:29: error: expected `}' at end of input"
>
> hast du auch ein arduino?
>
> da passt irgendwas nicht mit den klammern?

Richtig.

Hier ist das Original, wenn man es mal ein wenig ordentlich formatiert

/* * Tastendrücke zählen */
int switchPin = 2;         // Schalter ist mit Pin 2 verbunden
int val;                   // Variable für den Pin Zustand
int buttonState;           // Variable für den letzten Schalterzustand
int buttonPresses = 0;     // Wie oft ist der Schalter gedrückt

void setup()
{
  pinMode(switchPin, INPUT);             // Schalter-Pin ist Input
  Serial.begin(9600);                    // Start der seriellen Kommunikation mit 9600bps
  buttonState = digitalRead(switchPin);  // Anfangszustand lesen
}

void loop()
{
  val = digitalRead(switchPin);          // Eingabewert lesen und in val speichern
  if (val != buttonState) {              // Der Zustand des Schalters hat sich verändert
    if (val == LOW) {                    // Ist der Schalter gedrückt?
      buttonPresses++;                   // Inkrementieren der Variablen buttonPresses
      Serial.print("Schalter wurde ");
      Serial.print(buttonPresses);
      Serial.println(" gedrueckt");
    }
  }

  buttonState = val;                     // Den Zustand merken
}

da ist nichts von fehlerhaften Klammern zu sehen.
D.h. es ist die Vermutung naheliegend, dass du selbst irgendwo einen 
Fehler gemacht hast, als du den Code übernommen, formatiert bzw. 
angepasst hast.

-> zeige nicht den Code den du von irgendwo übernommen hast, sondern 
zeige immer den Code, so wie DU ihn verwendest.


> ich hab das nur versucht zu verstehn wie er addiert, da
> ich aber das programm nicht draufspielen kann, weiß ich
> nicht obs überhaupt geht!

Lies die Fehlermeldung, versuch sie zu verstehen, bilde eine Hypothese 
und ändere das was du hast entsprechend ab. Das gehört genauso zum 
Programmieren dazu, wie alles andere, dass man mit Fehlermeldungen was 
anfangen kann.
PS: man konzentriert sich immer auf die erste bzw. auf die ersten paar 
Fehlermeldung.
Die erste lautet
> In function 'void setup()':
Das ist für sich alleine gesehen noch nicht sehr aussagekräftig. Aber es 
gibt zumindest mal einen Hinweis darauf, an welcher Stelle ungefähr das 
Problem existiert. Da ist von der Funktion setup die Rede. Also wird man 
dort mal näher nachsehen.

> sketch_oct16a:19: error: a function-definition is not allowed
> here before '{' token

Aha. da ist von der { die Rede.
Also sieht man sich da mal alle offnenden { an. Macht vielleicht erst 
mal fehlende Einrückungen um zu sehen, wo da was nicht stimmt.

[Karl H. (kbuchegg)]

> das programm von der seite
> hat nichts mit der 7 segemntanzeige zu tun, das mach ich für mich selber

Warum hast du dann genau dieses 'Detail' im Titel deiner Anfrage?
Deine Anfrage hat genau gar nichts mit 7-Segment-Anzeigen zu tun.

[biede1987 (Gast)]

super danke für die schnelle antwort ihr seit echt immer wieder super 
hier

[biede1987 (Gast)]

hab mein fehler auch gefunden, hab nicht richtig formatiert zum schluss

[Karl H. (kbuchegg)]

Lass mich raten.

Du hast beim Aufteilen des original Kauderwelsch an dieser Stelle

  buttonState = digitalRead(switchPin);  // Anfangszustand lesen
}

die } übersehen.

-> Es ist immer eine gute Idee, seinen Code ORDENTLICH zu formatieren. 
Inklusive Einrückungen.
Denn dann SIEHT man optisch, wie die Dinge zusammenhängen. Zum Beispiel 
findet man die zu einer schliessenden } gehörende entsprechende öffnende 
{. Denn die muss in der Zeile stehen, in der der Programmtext in genau 
der gleichen Spalte beginnt.

[biede1987 (Gast)]

ne ich hab ganz zum schluss die Klammer nicht runter gesetzt

 Serial.println(" gedrueckt");
}
}

  buttonState = val;                     // Den Zustand merken}



hätte ich eigentlich selber sehen müssen, trotzdem vielen dank

[biede1987 (Gast)]

hier ist mein Programm für die 7 Segment anzeige

  // Lichtschranke ist ein Taster und alle Segmente werden angesteuert bei 0 und bei 1 gehen sie aus

    int Lichtschranke = 11; // oder Taster
    int Runde; // Variable für den Pin Zustand 
    int buttonState; // Variable für den letzten Schalterzustand 
    int buttonPresses = 0; // Wie oft ist der Schalter gedrückt   

    int a = 2; // a
    int b = 3;
    int c = 4;
    int d = 5;
    int e = 6;
    int f = 7;
    int g = 8;



    void setup(){
 
  Serial.begin(9600); // Start der seriellen Kommunikation mit 9600bps
  pinMode(Lichtschranke,INPUT);
  pinMode(a,OUTPUT);
  pinMode(b,OUTPUT);
  pinMode(c,OUTPUT);
  pinMode(d,OUTPUT);
  pinMode(e,OUTPUT);
  pinMode(f,OUTPUT);
  pinMode(g,OUTPUT);
  
  buttonState = digitalRead(Lichtschranke); // Start der seriellen Kommunikation mit 9600bps 
  
}
  
  void loop() { 
    
    Runde = digitalRead(Lichtschranke);
    if (Runde != buttonState) { // Der Zustand des Schalters hat sich verändert 

        if (Runde == HIGH) { // Ist der Schalter gedrückt? 

        buttonPresses++; // Inkrementieren der Variablen buttonPresses 

                                   // Serial.println(Runde,DEC);
                                     // if (Runde == HIGH)
        Serial.print("Schalter wurde ");
        Serial.print(buttonPresses);
        Serial.println(" gedrueckt");
    
  
    if (Runde=0)
    {
      digitalWrite(a,LOW);
      digitalWrite(b,LOW);
      digitalWrite(c,LOW);
      digitalWrite(d,LOW);
      digitalWrite(e,LOW);
      digitalWrite(f,LOW);
      digitalWrite(g,HIGH);
      }

      if (Runde=1)
      {
      digitalWrite(a,1);
      digitalWrite(b,0);
      digitalWrite(c,0);
      digitalWrite(d,1);
      digitalWrite(e,1);
      digitalWrite(f,1);
      digitalWrite(g,1);
      }
      
      if (Runde=2)
      {
      digitalWrite(a,0);
      digitalWrite(b,0);
      digitalWrite(c,1);
      digitalWrite(d,0);
      digitalWrite(e,0);
      digitalWrite(f,1);
      digitalWrite(g,0);
      }
      
      if (Runde=3)
      {
      digitalWrite(a,0);
      digitalWrite(b,0);
      digitalWrite(c,0);
      digitalWrite(d,0);
      digitalWrite(e,1);
      digitalWrite(f,1);
      digitalWrite(g,0);
      }
      
      
      if (Runde=4)
      {
      digitalWrite(a,1);
      digitalWrite(b,0);
      digitalWrite(c,0);
      digitalWrite(d,1);
      digitalWrite(e,1);
      digitalWrite(f,0);
      digitalWrite(g,0);
      }
      
      if (Runde=5)
      {
      digitalWrite(a,0);
      digitalWrite(b,1);
      digitalWrite(c,0);
      digitalWrite(d,0);
      digitalWrite(e,1);
      digitalWrite(f,0);
      digitalWrite(g,0);
      }
      
      if (Runde=6)
      {
      digitalWrite(a,0);
      digitalWrite(b,1);
      digitalWrite(c,0);
      digitalWrite(d,0);
      digitalWrite(e,0);
      digitalWrite(f,0);
      digitalWrite(g,0);
      }
      
      if (Runde=7)
      {
      digitalWrite(a,0);
      digitalWrite(b,0);
      digitalWrite(c,0);
      digitalWrite(d,1);
      digitalWrite(e,1);
      digitalWrite(f,1);
      digitalWrite(g,1);
      }
      
      if (Runde=8)
      {
      digitalWrite(a,0);
      digitalWrite(b,0);
      digitalWrite(c,0);
      digitalWrite(d,1);
      digitalWrite(e,1);
      digitalWrite(f,1);
      digitalWrite(g,1);
      }
      
      if (Runde=9)
      {
      digitalWrite(a,0);
      digitalWrite(b,0);
      digitalWrite(c,0);
      digitalWrite(d,0);
      digitalWrite(e,1);
      digitalWrite(f,0);
      digitalWrite(g,0);
      }
      
        }
      }
      buttonState = Runde;
    }

[biede1987 (Gast)]

leider ist es so sobald eine 1 kommt, zählt er ganz schnell hoch, dieses 
sehe ich an meinem serial monitor,

ich bekomm dann gar nicht mit wie die anzeige hochzählt,

woran liegt das?

[Karl H. (kbuchegg)]

Das hier

    if (Runde=0)

sind keine Vergleiche sondern Zuweisungen.
Vergleiche schreiben sich mit ==

[Karl H. (kbuchegg)]

Ausserdem.
Überleg noch mal ganz stark, welche Variable im Originalcode welche 
Bedeutung hat. Was repräsentiert sie?

Und dann überlag mal ganz stark, was deine Variable 'Runde' eigentlich 
für eine Aufgabe hat. Kann es sinnvoll sein, in Runde das Ergebnis von

    Runde = digitalRead(Lichtschranke);

abzulegen?

Eher nicht.
'Runde' soll darüber Buch führen, in welcher Runde sich dein (Carrera-) 
Auto)?) sich befindet. Was bitte hat das damit zu tun, ob die 
Lichtschranke unterbrochen ist oder nicht?

In welcher Variablen wird denn hier

void loop()
{
  val = digitalRead(switchPin);          // Eingabewert lesen und in val speichern
  if (val != buttonState) {              // Der Zustand des Schalters hat sich verändert
    if (val == LOW) {                    // Ist der Schalter gedrückt?
      buttonPresses++;                   // Inkrementieren der Variablen buttonPresses
      Serial.print("Schalter wurde ");
      Serial.print(buttonPresses);
      Serial.println(" gedrueckt");
    }
  }

  buttonState = val;                     // Den Zustand merken
}

eigentlich gezählt?

Hinweis: Es ist NICHT 'val'! Die Variable 'val' sollte in diesem Code 
besser 'buttonStateNow' heißen. Der Name dieser Variablen ist nämlich 
irreführend.

void loop()
{
  buttonStateNow = digitalRead(switchPin);          // Eingabewert lesen und in val speichern
  if (buttonStateNow != buttonState) {              // Der Zustand des Schalters hat sich verändert
    if (buttonStateNow == LOW) {                    // Ist der Schalter gedrückt?
      buttonPresses++;                   // Inkrementieren der Variablen buttonPresses
      Serial.print("Schalter wurde ");
      Serial.print(buttonPresses);
      Serial.println(" gedrueckt");
    }
  }

  buttonState = buttonStateNow;                     // Den Zustand merken
}

Hier ....

    if (buttonStateNow == LOW) {                    // Ist der Schalter gedrückt?
      buttonPresses++;                   // Inkrementieren der Variablen buttonPresses

... steckt die Zählvariable! buttonPresses repräsentiert den Zähler! 
Erkennt man auch am Kommentar, bzw. daran, dass in weiterer Folge im 
Code mit dieser Variablen etwas gemacht wird ...

      Serial.print("Schalter wurde ");
      Serial.print(buttonPresses);
      Serial.println(" gedrueckt");

... in diesem konkreten Fall eben über die Serielle ausgeben.

[biede1987 (Gast)]

// Lichtschranke ist ein Taster und alle Segmente werden angesteuert bei 0 und bei 1 gehen sie aus

    int Lichtschranke = 11; // oder Taster
    int var; // Variable für den Pin Zustand 
    int buttonState; // Variable für den letzten Schalterzustand 
    int Runde = 0; // Wie oft ist der Schalter gedrückt   

    int a = 2; // a
    int b = 3;
    int c = 4;
    int d = 5;
    int e = 6;
    int f = 7;
    int g = 8;



    void setup(){
 
  Serial.begin(9600); // Start der seriellen Kommunikation mit 9600bps
  pinMode(Lichtschranke,INPUT);
  pinMode(a,OUTPUT);
  pinMode(b,OUTPUT);
  pinMode(c,OUTPUT);
  pinMode(d,OUTPUT);
  pinMode(e,OUTPUT);
  pinMode(f,OUTPUT);
  pinMode(g,OUTPUT);
  
  buttonState = digitalRead(Lichtschranke); // Start der seriellen Kommunikation mit 9600bps 
  
}
  
  void loop() { 
    
    var = digitalRead(Lichtschranke);
    if (var != buttonState) { // Der Zustand des Schalters hat sich verändert 

        if (var == HIGH) { // Ist der Schalter gedrückt? 

        Runde++; // Inkrementieren der Variablen Runde 

                                   // Serial.println(var,DEC);
                                     // if (var == HIGH)
        Serial.print("Schalter wurde ");
        Serial.print(Runde);
        Serial.println(" gedrueckt");
    
  
    if (Runde==0)
    {
      digitalWrite(a,1);
      digitalWrite(b,1);
      digitalWrite(c,1);
      digitalWrite(d,1);
      digitalWrite(e,LOW);
      digitalWrite(f,LOW);
      digitalWrite(g,HIGH);
      }

      if (Runde==1)
      {
      digitalWrite(a,1);
      digitalWrite(b,0);
      digitalWrite(c,0);
      digitalWrite(d,1);
      digitalWrite(e,1);
      digitalWrite(f,1);
      digitalWrite(g,1);
      }
      
      if (Runde==2)
      {
      digitalWrite(a,0);
      digitalWrite(b,0);
      digitalWrite(c,1);
      digitalWrite(d,0);
      digitalWrite(e,0);
      digitalWrite(f,1);
      digitalWrite(g,0);
      }
      
      if (Runde==3)
      {
      digitalWrite(a,0);
      digitalWrite(b,0);
      digitalWrite(c,0);
      digitalWrite(d,0);
      digitalWrite(e,1);
      digitalWrite(f,1);
      digitalWrite(g,0);
      }
      
      
      if (Runde==4)
      {
      digitalWrite(a,1);
      digitalWrite(b,0);
      digitalWrite(c,0);
      digitalWrite(d,1);
      digitalWrite(e,1);
      digitalWrite(f,0);
      digitalWrite(g,0);
      }
      
      if (Runde==5)
      {
      digitalWrite(a,0);
      digitalWrite(b,1);
      digitalWrite(c,0);
      digitalWrite(d,0);
      digitalWrite(e,1);
      digitalWrite(f,0);
      digitalWrite(g,0);
      }
      
      if (Runde==6)
      {
      digitalWrite(a,0);
      digitalWrite(b,1);
      digitalWrite(c,0);
      digitalWrite(d,0);
      digitalWrite(e,0);
      digitalWrite(f,0);
      digitalWrite(g,0);
      }
      
      if (Runde==7)
      {
      digitalWrite(a,0);
      digitalWrite(b,0);
      digitalWrite(c,0);
      digitalWrite(d,1);
      digitalWrite(e,1);
      digitalWrite(f,1);
      digitalWrite(g,1);
      }
      
      if (Runde==8)
      {
      digitalWrite(a,0);
      digitalWrite(b,0);
      digitalWrite(c,0);
      digitalWrite(d,1);
      digitalWrite(e,1);
      digitalWrite(f,1);
      digitalWrite(g,1);
      }
      
      if (Runde==9)
      {
      digitalWrite(a,0);
      digitalWrite(b,0);
      digitalWrite(c,0);
      digitalWrite(d,0);
      digitalWrite(e,1);
      digitalWrite(f,0);
      digitalWrite(g,0);
      }
      
        }
      }
      buttonState = var; //Den Zustand merken
    }

[biede1987 (Gast)]

ja du hattest recht ich hab die "runde" als "var" geschrieben das war 
falsch,

jetzt hab ich buttonpresses ersetzt durch Runde,

und meine alte Runde ist jetzt einfach ein variabler Wert.

ich glaub jetzt wirds logischer oder was meinst

[Karl H. (kbuchegg)]

Ja. Und funktionierts?

Weiter gehts!
Da gibt es noch viel zu tun und zu lernen.

[Karl H. (kbuchegg)]

biede1987 schrieb:

> ich glaub jetzt wirds logischer oder was meinst

Ich seh noch nicht, inwiefern mir beim lesen des Codes ein Variablenname 
von 'var' etwas über die Funktion der Variablen innerhalb des Programmes 
verrät.

var ist reichlich nichtssagend. Wenn du eine Variable brauchst, deren 
Aussage darin besteht, ob die Lichtschranke unterbrochen ist oder nicht, 
dann sollte wohl der Variablenname das auch irgendwie andeuten. Ich 
würde da irgendwie die Begriffe 'Lichtschranke' bzw. 'Unterbrochen' bzw. 
'Status' ins Spiel bringen. Selbiges für 'buttonState'. Hast du denn 
einen Button (einen Drückknopf)? Nein, hast du nicht. Du hast eine 
Lichtschranke. Eine Lichtschranke ist aber kein Button.


Du magst das jetzt als kleinlich empfinden. Aber in größeren Programmen 
ist es unabdingbar, dass mir der Code selber etwas erzählt. Zb durch gut 
gewählte Variablennamen. Eine Codestelle

  NettoPreis = BruttoPreis + BruttoPreis * Steuersatz;

braucht keinen Kommentar. Die Variablennamen sind so gewählt, dass mir 
der Code selber erzählt, was hier passiert. Und genau das ist das Ziel: 
Kommentare überflüssig machen, indem der Kommentar schon in der 
Formulierung des Codes integriert ist.

[biede1987 (Gast)]

jep funktioniert war noch ein fehler drin bei der

8

die hab falsch dargestellt,



und am anfang wenn 0 Runden sind,

zeigt er mir eine 8 statt eine 0 obwohl ich die ausgänge richtig gesetzt 
habe

weißt du warum?

[Karl H. (kbuchegg)]

Wenn du deinen Code ordentlich Einrücken würdest, würdest du es auch 
sehen.

Wenn am Anfang alles auf 0 ist, ist den Ausgängen noch nie irgendetwas 
zugewiesen worden.
Erst mit dem ersten Puls, den dein Programm mittels Lichtschranke 
feststellt, steuert dein Programm das erste mal die 7-Segment Ausgänge 
an.
Deine Anzeige zeigt daher am Anfang deswegen '8' an, weil es so ist, 
dass der Default-Zustand der Ausgänge nun mal 0 ist. D.h. alle deine 
Ausgänge sind auf 0, womit alle Segmente leuchten und du das als 8 
fehlinterpretierst. Wäre der Dezimalpunkt deiner Anzeige auch 
angeschlossen, dann könntest du sehen, dass der genauso leuchtet.

deine Programmlogik sieht zur Zeit so aus

void loop()
{
  if( Puls an der Lichtschranke )
  {
    erhöhe Zähler
    gib den Zähler aus
  }
}

Aber was ist am Anfang? Wenn noch kein Puls von der Lichtschranke 
festgestellt wurde? Dann ist auch nie irgendetwas ausgegeben worden.



-> deutlicher Hinweis darauf, dass die Größe der Funktion loop schon das 
Mass überschritten hast, das du noch überblicken kannst.
Es wird Zeit, dass du die Ansteurung der 7_Segment Anzeige in eine 
eigene Funktion auslagerst.

[biede1987 (Gast)]

ok ich versuch jetzt mal mit Hilfe eines zweiten Schalters die Runden 
immer am Anfang auf 0 zurücksetzten. Ein Resetschalter.


wenn das funktioniert, will ich mich informieren wie ich eine  4 
Segmentanzeige ansteuere.da kommen dann bestimmt wieder fragen auf!

ich versuch des jetzt mal mit dem Resettaster und tu nochmal mein 
Programm verbessern.

[biede1987 (Gast)]

Vielen Dank für die Hilfe

 um 10.00 Uhr war ich noch am verzweifeln

[biede1987 (Gast)]

was heißt !=

wenn sich der Zustand verändert oder?

[Karl H. (kbuchegg)]

biede1987 schrieb:
> was heißt !=
>

Vergiss bitte ganz schnell die Idee, du könntest C (oder C++) mit einem 
Online-Tutorial lernen.
Du brauchst ein ordentliches C-Buch! Und zwar schnell!


!=   ist 'ungleich'

[biede1987 (Gast)]

ok muss ich bei uns mal in der schule kucken,


ich hab mir das arduino kochbuch gekauft, ist super gut beschrieben.

macht echt spass