Discussion :

[ikuzar]

Bonjour, .
Dans le code ci-dessous, dès que err < 1 je dois retourner -1. Est ce que l'écriture correcte ? En effet, je travaille sous emacs et return -1 est aligné avec print_error() automatiquement.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
if ( err < 1 ){
     switch(err){
          case ERR1:
              goto fail;
          case ERR2:
             goto fail:
          ...
     }
fail;
    get_error( );
    print_error();
 
    return -1;
}

Merci d'avance.

[adrienmgs]

Je passe peut être à coté de quelque chose mais pourquoi tu ne fais pas tout simplement :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
if (err<1) return -1;

Après les fonctions get_error() et print_error() n'existent pas à ma connaissance en C donc j'imagine que ce sont des fonctions de ton programme, auquel cas tu fais

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
 
if (err<1){
    get_error();
    print_error();
    return -1;
}

Edit : sinon par rapport à ta question sur l'utilisation de goto et return,

"goto etiquette" fait un saut jusqu'à l'emplacement de cette etiquette dans ton code.

"return variable" permet de sortir d'une fonction de type autre que void, par exemple :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
 
int mafonction(){
    int a;
    scanf("%d", &a);
    return a;
}

[souviron34]

Bonjour, .
Dans le code ci-dessous, dès que err < 1 je dois retourner -1. Est ce que l'écriture correcte ? En effet, je travaille sous emacs et return -1 est aligné avec print_error() automatiquement.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
if ( err < 1 ){
     switch(err){
          case ERR1:
              goto fail;
          case ERR2:
             goto fail:
          ...
     }
fail;
    get_error( );
    print_error();
 
    return -1;
}

Merci d'avance.

Bonjour

Plusieurs points ...

• Utilise la balise "code" (#) pour mettre ton code dans un cadre..
  
  
  
• utiliser GoTo c'est mal
  
  Le même code sans GoTo :
  
  

  Code :

  Sélectionner tout - Visualiser dans une fenêtre à part

  1
  2
  3
  4
  5
  6
  7
  8
  9
  10
  11
  12
  13
  14
  15
  16
  17
  18
  19
  if ( err < 1 ){
      Fail = 0 ;
       switch(err){
            case ERR1:
                Fail = 1 ;
                break ;
            case ERR2:
               Fail = 1 ;
               break ;
            ...
       }
  if ( Fail != 0 )
  {
      get_error( );
      print_error();
  }
   
      return -1;
  }

  L'utilisation du GoTo est à proscrire.... Il n'y a que de très rares cas où c'est nécessaire, mais dans 99.999999999999% des cas, c'est à éviter comme la peste... Cela amène invariablement à du code "spaghetti"...
  
  D'autre part, je ne vois pas pourquoi rajouter un if (fail) alors que tu es dans le cas où err < 1....
  
  
  
  
• Il manque une accolade (du if)

[matafan]

Un label se termine par deux points, pas par un point-virgule :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

fail:

[ManusDei]

Regarde également dans ton switch, tu finis un cas par un : et pas par un ;

Bref attention aux caractères qui se ressemblent mais ne s'assemblent pas