problème erreur arduino " exit status 1 expected '}' before 'else' "

**sakloub** · 02/04/2020, 00h36

boujour j'ai développer un code pour une machine de réanimation un prototype,
mais j'obtient cette erreur " exit status 1
expected '}' before 'else' " voici mon code

Code:

const int IE=10;
int ie=1.2;
int marche = 9;
int valeurvt=500;
int valeurfr=10;
int course=67;
#include <Servo.h>
#include "LiquidCrystal.h"
LiquidCrystal lcd(6,7,12,11,4,5);

Servo moteur;

void setup() {
   attachInterrupt( 0, incrementVT, RISING );
   attachInterrupt( 1, incrementFR, RISING );
    moteur.attach(8);
  pinMode (IE, INPUT);
 
  lcd.begin(16, 2);
  lcd.print("****TDS****");
  delay(1000);
  Serial.begin(9600);
}

void loop() {

  if (digitalRead(marche)==0)// départ cycle
  {
    lcd.clear();
  lcd.setCursor(0,0);
  lcd.print("on");
 ////////
 if (digitalRead(IE)==0)
 {
 ie=1.2;
  if (valeurvt==500);
  {
    course = 67;
    float cycle= 60000/valeurfr;
    float i=cycle/(1+ie);
    float e= cycle -i;
    float tempsI=i/course;
    float tempsE=e/course;
  }
 
   else if (valeurvt==600)
   {
    course = 130;
    float cycle= 60000/valeurfr;
    float i=cycle/(1+ie);
    float e= cycle -i;
    float tempsI=i/course;
    float tempsE=e/course;
   }
   else  (valeurvt==700)
    {
    course = 180;
    float cycle= 60000/valeurfr;
    float i=cycle/(1+ie);
    float e= cycle -i;
    float tempsI=i/course;
    float tempsE=e/course;
   }
  }
  ////////////
   else (digitalRead(IE)==1)
 {
 ie=1.3;
  if (valeurvt==500);
  {course = 67;
    float cycle= 60000/valeurfr;
    float i=cycle/(1+ie);
    float e= cycle -i;
    float tempsI=i/course;
    float tempsE=e/course;
  }
   else if (valeurvt==600){
    course = 130;
    float cycle= 60000/valeurfr;
    float i=cycle/(1+ie);
    float e= cycle -i;
    float tempsI=i/course;
    float tempsE=e/course;
   }
   else (valeurvt==700)
    { course = 180;
    float cycle= 60000/valeurfr;
    float i=cycle/(1+ie);
    float e= cycle -i;
    float tempsI=i/course;
    float tempsE=e/course;
   }
  }
 
 
   lcd.setCursor(4,0);
  lcd.print("VT=");
  lcd.setCursor(7,0);
  lcd.print(valuervt/1000);
  lcd.print("L");
  lcd.setCursor(0,1);
  lcd.print("IE=");
  lcd.setCursor(3,1);
  lcd.print(ie);
  lcd.print("%");
  lcd.setCursor(7,1);
  lcd.print("Fr=");
  lcd.setCursor(11,1);
  lcd.print(valeurfr);
  for (int position = 0; position <= course; position++) {
    moteur.write(position);
    delay(tempsI);
  }
 
 
  for (int position = course; position >= 0; position--) {
    moteur.write(position);
    delay(tempsE);
  }
 
}
  else {
    lcd.clear();
    lcd.setCursor(0,0);
    lcd.print("off");
      delay(100);
       }
}

void incrementVT() {
  valeurvt = valeurvt + 500;
  if( valeurvt > 700 ) {
     valeurvt = 500;
  }
}
void incrementFR() {
  valeurfr = valeurfr + 1;
  if( valeurfr > 10 ) {
     valeurfr = 0;
  }
}

une idée !!
cordialement

**gienas** · 02/04/2020, 06h53

Bonjour sakloub et tout le groupe

Envoyé par sakloub

...
expected '}' before 'else' " voici mon code ...

On peut supposer que cette erreur sort lors de la compilation, en précisant la ligne à laquelle elle se produit.

Il doit s’agir d’un mauvais appairage des accolades ouvrantes et fermantes.

Le else arriverait avant que la fonction précédente ne soit fermée.

invite03481543 · 02/04/2020, 08h42

Il manque la parenthèse fermante du premier if apparemment.
Qui devrait donc survenir avant le else suivant ce if.

invite03481543 · 02/04/2020, 08h45

Le code est une assiette de "spaghetti"

il aurait mieux valut employer des switch case...

A voir en vidéo sur Futura · Aujourd'hui

**Seb.26** · 02/04/2020, 08h50

Code:

int ie=1.2;

-> c'est sensé faire quoi ??!!!

**Seb.26** · 02/04/2020, 08h52

Et pour ton problème de build :

Code:

}
  ////////////
   else (digitalRead(IE)==1)
 {

-> tu dois vouloir faire un "else if" je pense ...

... mais comme "digitalRead(IE)" ne peut renvoyer que 0 ou 1, un "else" suffira

**umfred** · 02/04/2020, 09h57

pour résumé, il doit manquer des if, des accolades; bref prends le temps de vérifier tes indentations pour trouver les erreurs (et le compilateur, comme dit plus haut, doit t'indiquer la ligne de l'erreur qu'il t'indique, donc le problème se situe en principe au dessus ou sur cette ligne.

**polo974** · 02/04/2020, 10h04

à mon avis:
valeurvt = valeurvt + 500;

devrait être:
valeurvt += 100;

entre autre...

**jiherve** · 02/04/2020, 10h05

Bonjour
Même analyse que Hulk.
Une habitude à prendre d'urgence : INDENTER!!!
tab de deux espace minimum cela aide à lire le code lorsque celui ci l'est par un visualisateur lambda et pas un éditeur ad hoc.
JR

**Seb.26** · 02/04/2020, 15h39

Envoyé par polo974

à mon avis:
valeurvt = valeurvt + 500;

devrait être:
valeurvt += 100;

entre autre...

je ne pense pas, non ...

invite03481543 · 02/04/2020, 15h55

Je n'ai pas regardé la pertinence du contenu du code mais ce qui est certain c'est que

expected '}' before 'else'

designe le fait qu'il manque une accolade et précisément à cet endroit:

Code:

void loop()
{


if (digitalRead(marche)==0)// départ cycle
 {
  lcd.clear();
  lcd.setCursor(0,0);
  lcd.print("on");
.....
.....
else  (valeurvt==700)
    {
    course = 180;
    float cycle= 60000/valeurfr;
    float i=cycle/(1+ie);
    float e= cycle -i;
    float tempsI=i/course;
    float tempsE=e/course;
   }
  }
 }  <----- accolade manquante
  ////////////
   else (digitalRead(IE)==1)
 {
 ie=1.3;
  if (valeurvt==500);
  {course = 67;
    float cycle= 60000/valeurfr;

....
.....

Il faut donc commencer par éliminer cette erreur, ensuite il pourra avancer sur la mise au point de son code.

**freepicbasic** · 02/04/2020, 17h28

Programme truffé d'Erreur !
(trop nombreuse pour détailler)

Maintenant ça compile OK!
Quand à la validité du code...

Je n'ai pas mis le switch ,
il n y a que 3 cas , c est d'ailleurs étonnant ou suspect !
On devrait sans doute tester des fourchettes entre 2 valeurs.

du genre;
if ( ( valeurvt >= 500) && (valeurvt < 600))
etc...

Je vous laisse décider.

Code:

#include <Servo.h>
#include "LiquidCrystal.h"

void incrementVT();
void incrementFR();


float tempsI;
float tempsR;


const int IE = 10;
int ie = 1.2;
int marche = 9;
int valeurvt = 500;
int valeurfr = 10;
int course = 67;
LiquidCrystal lcd(6, 7, 12, 11, 4, 5);

Servo moteur;

void setup() {
  attachInterrupt( 0, incrementVT, RISING );
  attachInterrupt( 1, incrementFR, RISING );
  moteur.attach(8);
  pinMode (IE, INPUT);

  lcd.begin(16, 2);
  lcd.print("****TDS****");
  delay(1000);
  Serial.begin(9600);
}

void loop() {
  float cycle;
  float i;
  float e;
  int position ;

  if (digitalRead(marche) == 0) // départ cycle
  {
    lcd.clear();
    lcd.setCursor(0, 0);
    lcd.print("on");
    //
    if (digitalRead(IE) == 0) IE0();
    else   IE1() ;

    lcd.setCursor(4, 0);
    lcd.print("VT=");
    lcd.setCursor(7, 0);
    lcd.print(valeurvt / 1000);
    lcd.print("L");
    lcd.setCursor(0, 1);
    lcd.print("IE=");
    lcd.setCursor(3, 1);
    lcd.print(ie);
    lcd.print("%");
    lcd.setCursor(7, 1);
    lcd.print("Fr=");
    lcd.setCursor(11, 1);
    lcd.print(valeurfr);

    // ouvre
    for (position = 0; position <= course; position++) {
      moteur.write(position);
      delay(tempsI);
    }
    // ferme
    for (position = course; position >= 0; position--) {
      moteur.write(position);
      delay(tempsR);
    }

  }
  else { // Mode OFF
    lcd.clear();
    lcd.setCursor(0, 0);
    lcd.print("off");
    delay(100);
  }
}



void IE1() {
  int ie;
  ie = 1.2;
  if (valeurvt == 500)
  {
    course = 67;
    cycle = 60000 / valeurfr;
    i = cycle / (1 + ie);
    e = cycle - i;
    tempsI = i / course;
    tempsE = e / course;
  }

  if (valeurvt == 600)
  {
    course = 130;
    cycle = 60000 / valeurfr;
    i = cycle / (1 + ie);
    e = cycle - i;
    tempsI = i / course;
    tempsE = e / course;
  }
  if (valeurvt == 700)
  {
    course = 180;
    cycle = 60000 / valeurfr;
    i = cycle / (1 + ie);
    e = cycle - i;
    tempsI = i / course;
    tempsE = e / course;
  }

}
void IE0()
{
  int ie;

  ie = 1.3;
  if (valeurvt == 500) {
    course = 67;
    cycle = 60000 / valeurfr;
    i = cycle / (1 + ie);
    e = cycle - i;
    tempsI = i / course;
    tempsE = e / course;
  }
  if (valeurvt == 600) {
    course = 130;
    cycle = 60000 / valeurfr;
    i = cycle / (1 + ie);
    e = cycle - i;
    tempsI = i / course;
    tempsE = e / course;
  }
  if (valeurvt == 700)
  { course = 180;
    cycle = 60000 / valeurfr;
    i = cycle / (1 + ie);
    e = cycle - i;
    tempsI = i / course;
    tempsE = e / course;
  }
}

void incrementVT() {
  valeurvt = valeurvt + 500;
  if ( valeurvt > 700 ) {
    valeurvt = 500;
  }
}
void incrementFR() {
  valeurfr = valeurfr + 1;
  if ( valeurfr > 10 ) {
    valeurfr = 0;
  }
}

**freepicbasic** · 02/04/2020, 18h30

Il y a eu un raté dans la copie du code , ça a copié une version intermédiaire.

je remet le bon !

Code:

#include <Servo.h>
#include "LiquidCrystal.h"

void incrementVT();
void incrementFR();


float   cycle;
float   i;
float   e;
float tempsI;
float tempsE;


const int IE = 10;
int ie = 1.2;
int marche = 9;
int valeurvt = 500;
int valeurfr = 10;
int course = 67;
LiquidCrystal lcd(6, 7, 12, 11, 4, 5);

Servo moteur;

void setup() {
  attachInterrupt( 0, incrementVT, RISING );
  attachInterrupt( 1, incrementFR, RISING );
  moteur.attach(8);
  pinMode (IE, INPUT);

  lcd.begin(16, 2);
  lcd.print("****TDS****");
  delay(1000);
  Serial.begin(9600);
}

void loop() {
  float cycle;
  float i;
  float e;
  int position ;

  if (digitalRead(marche) == 0) // départ cycle
  {
    lcd.clear();
    lcd.setCursor(0, 0);
    lcd.print("on");
    //
    if (digitalRead(IE) == 0) IE0();
    else   IE1() ;

    lcd.setCursor(4, 0);
    lcd.print("VT=");
    lcd.setCursor(7, 0);
    lcd.print(valeurvt / 1000);
    lcd.print("L");
    lcd.setCursor(0, 1);
    lcd.print("IE=");
    lcd.setCursor(3, 1);
    lcd.print(ie);
    lcd.print("%");
    lcd.setCursor(7, 1);
    lcd.print("Fr=");
    lcd.setCursor(11, 1);
    lcd.print(valeurfr);

    // ouvre
    for (position = 0; position <= course; position++) {
      moteur.write(position);
      delay(tempsI);
    }
    // ferme
    for (position = course; position >= 0; position--) {
      moteur.write(position);
      delay(tempsE);
    }

  }
  else { // Mode OFF
    lcd.clear();
    lcd.setCursor(0, 0);
    lcd.print("off");
    delay(100);
  }
}



void IE1() {
  int ie;
  ie = 1.2;
  if (valeurvt == 500)
  {
    course = 67;
    cycle = 60000 / valeurfr;
    i = cycle / (1 + ie);
    e = cycle - i;
    tempsI = i / course;
    tempsE = e / course;
  }

  if (valeurvt == 600)
  {
    course = 130;
    cycle = 60000 / valeurfr;
    i = cycle / (1 + ie);
    e = cycle - i;
    tempsI = i / course;
    tempsE = e / course;
  }
  if (valeurvt == 700)
  {
    course = 180;
    cycle = 60000 / valeurfr;
    i = cycle / (1 + ie);
    e = cycle - i;
    tempsI = i / course;
    tempsE = e / course;
  }

}
void IE0()
{
  int ie;

  ie = 1.3;
  if (valeurvt == 500) {
    course = 67;
    cycle = 60000 / valeurfr;
    i = cycle / (1 + ie);
    e = cycle - i;
    tempsI = i / course;
    tempsE = e / course;
  }
  if (valeurvt == 600) {
    course = 130;
    cycle = 60000 / valeurfr;
    i = cycle / (1 + ie);
    e = cycle - i;
    tempsI = i / course;
    tempsE = e / course;
  }
  if (valeurvt == 700)
  { course = 180;
    cycle = 60000 / valeurfr;
    i = cycle / (1 + ie);
    e = cycle - i;
    tempsI = i / course;
    tempsE = e / course;
  }
}

void incrementVT() {
  valeurvt = valeurvt + 500;
  if ( valeurvt > 700 ) {
    valeurvt = 500;
  }
}
void incrementFR() {
  valeurfr = valeurfr + 1;
  if ( valeurfr > 10 ) {
    valeurfr = 0;
  }
}

**polo974** · 04/04/2020, 18h36

Envoyé par Seb.26

je ne pense pas, non ...

Regarde bien le code (de la même fonction et l'usage de la variable).

**Seb.26** · 06/04/2020, 08h43

Envoyé par HULK28

Je n'ai pas regardé la pertinence du contenu du code mais ce qui est certain c'est que ... [...] ...

Et pourtant ...

... moi j'en suis pas sûr du tout ...

**Seb.26** · 06/04/2020, 08h45

Envoyé par polo974

Regarde bien le code (de la même fonction et l'usage de la variable).

-> je pensais à une erreur de frappe ( 500 -> 100 ) et que tu voulais juste lui faire mettre un x+=n plutôt qu'un x=x+n ...

... j'ai pas regardé si le 100 est plus pertinent que le 500 ...

( my bad si c'est le cas )

invite03481543 · 06/04/2020, 09h01

Envoyé par Seb.26

Et pourtant ...

... moi j'en suis pas sûr du tout ...

Oui mais comme je te l'ai dit je n'ai pas regardé, s'agissant d'arduino...

**freepicbasic** · 06/04/2020, 14h08

Notre ami sakloub faisant le mort ...
L œuvre de bienfaisance semble tombé à l'eau.

ça me parait faux;
valeurvt = valeurvt + 500;
Parce qu'on passe jamais par 600 ou 700 !

C est surement ça;
valeurvt = valeurvt + 100;

Pourquoi incrémenter de 100 ?
Mystère...

invite03481543 · 06/04/2020, 16h54

Envoyé par freepicbasic

Pourquoi incrémenter de 100 ?
Mystère...

Faut dire vu la tonne de commentaires dans le code...

Du grand art.

j'ai développer un code pour une machine de réanimation

J'espère ne pas avoir à être réanimé un jour par un arduino...

problème erreur arduino " exit status 1 expected '}' before 'else' "

problème erreur arduino " exit status 1 expected '}' before 'else' "

Re : problème erreur arduino " exit status 1 expected '}' before 'else' "

Re : problème erreur arduino " exit status 1 expected '}' before 'else' "

Re : problème erreur arduino " exit status 1 expected '}' before 'else' "

Re : problème erreur arduino " exit status 1 expected '}' before 'else' "

Re : problème erreur arduino " exit status 1 expected '}' before 'else' "

Re : problème erreur arduino " exit status 1 expected '}' before 'else' "

Re : problème erreur arduino " exit status 1 expected '}' before 'else' "

Re : problème erreur arduino " exit status 1 expected '}' before 'else' "

Re : problème erreur arduino " exit status 1 expected '}' before 'else' "

Re : problème erreur arduino " exit status 1 expected '}' before 'else' "

Re : problème erreur arduino " exit status 1 expected '}' before 'else' "

Re : problème erreur arduino " exit status 1 expected '}' before 'else' "

Re : problème erreur arduino " exit status 1 expected '}' before 'else' "

Re : problème erreur arduino " exit status 1 expected '}' before 'else' "

Re : problème erreur arduino " exit status 1 expected '}' before 'else' "

Re : problème erreur arduino " exit status 1 expected '}' before 'else' "

Re : problème erreur arduino " exit status 1 expected '}' before 'else' "

Re : problème erreur arduino " exit status 1 expected '}' before 'else' "

Discussions similaires

error: ld returned 1 exit status

Où sont passés les nombres "entiers" "EXIT"? et autres "bizarreries"

comment se fournir en graines de variétés authentiques dites "anciennes" (donc exit les F1)

programme "status.msi"

programme "status.msi"