Discussion :

[Voldemort09]

Voilà mon code mais ça ne marche pas, le programme affiche seulement les printf du départ:

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
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
 
#include <stdio.h>
#include <stdlib.h>
 
int main(int argc, char *argv[])
{
	long Nombre[1000] = {2, 3, 5, 7, 11, 13, 17, 19, 23, 29, 31, 37, 41, 43, 47, 53, 59, 61, 67, 71, 73, 79, 83, 89, 97, 0};
	long Index = 0;
	long NbActuel = 0;
	int Temp = 0;
  printf("Ce programme calcule en temps r%cel les nombre premier:\n", 0x82);
  printf("2, 3, 5, 7, 11, 13, 17, 19, 23, 29, 31, 37, 41, 43, 47, 53, 59, 61, 67, 71, 73, 79, 83, 89, 97, ");
  while (1)
  {
  Prochaine:
  while (Nombre[Index] != 0)
  {
  Temp = NbActuel % Nombre[Index];
  if (Temp == 0)
  {
  goto Next;
  }
  Temp = 0;
  Index++;
  if ((Nombre[Index] == 0) || (Index < 1000))
  {
  goto Next;
  }
  }
  if (Nombre[Index] == 0)
  {
  goto Next;
  }
  printf("%ld, ", NbActuel);
  Nombre[Index] = NbActuel;
  Next:
  Temp = 0;
  NbActuel++;
  Index = 0;
  goto Prochaine;
  }
  system("PAUSE");
  return 0;
}

[Emmanuel Delahaye]

Voilà mon code mais ça ne marche pas, le programme affiche seulement les printf du départ:

<classement vertical >

Apprends à coder sans goto avant de poster ici.

ici, on fait C, pas du BASIC à l'ancienne...

[3DArchi]

Même réponse qu'Emmanuel: vire tous ces goto du code d'autant que t'avais déjà commencé à faire des boucles.
Je rajouterais: as-tu essayé de suivre le déroulement pas à pas avec un debugger. Avant de poser une question, c'est toujours bien de chercher soi même la réponse... La solution te sautera aux yeux.

[Voldemort09]

Ok, avant je codais sans goto mais depuis que j'ai commencer l'assembleur, je suis habitué au saut inconditionnel(ou conditionnel).
Ok je vais chercher, mais j'ai une question, pourquoi les goto sont a éviter ?

[Emmanuel Delahaye]

Ok, avant je codais sans goto mais depuis que j'ai commencer l'assembleur, je suis habitué au saut inconditionnel(ou conditionnel).
Ok je vais chercher, mais j'ai une question, pourquoi les goto sont a éviter ?

Ce sujet a été débattu maintes fois. En gros, ça rend le code illisible surtout si ça remonte (boucles). L'assembleur a ses contraintes que le C n'a pas. ce dernier dispose au contraire de tout ce qu'il faut pour coder correctement selon les principes de la programmation structurée (structures de code) :

1. décisions
   
   1. if
   2. if-else
   3. switch-case
2. itérations
   
   1. while
   2. do-while
   3. for

on raisonne en 'action conditionnelle' et non en 'saut conditionnel' comme en assembleur ou en BASIC primitif.

[Thierry Chappuis]

Ok, avant je codais sans goto mais depuis que j'ai commencer l'assembleur, je suis habitué au saut inconditionnel(ou conditionnel).
Ok je vais chercher, mais j'ai une question, pourquoi les goto sont a éviter ?

Pour une discussion complète, voir: http://www.developpez.net/forums/showthread.php?t=75764

Thierry

[Voldemort09]

Ok, je comprend, j'ai enlevé tout les goto et le désassembler/debugeur me coince dans une sorte de boucle infini .
Pouvez-vous m'aider svp, j'ai tendance a compliquer le C depuis que j'apprends l'ASM:

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
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
 
#include <stdio.h>
#include <stdlib.h>
 
int main(int argc, char *argv[])
{
	long Nombre[1000] = {2, 3, 5, 7, 11, 13, 17, 19, 23, 29, 31, 37, 41, 43, 47, 53, 59, 61, 67, 71, 73, 79, 83, 89, 97, 0};
	long i = 0;
	long NbActuel = 0;
	int Temp = 0;
  printf("Ce programme calcule en temps r%cel les nombre premier:\n", 0x82);
  printf("2, 3, 5, 7, 11, 13, 17, 19, 23, 29, 31, 37, 41, 43, 47, 53, 59, 61, 67, 71, 73, 79, 83, 89, 97, ");
  while (1)
  {
  while ((Nombre[i] != 0) && (i < 1000) && (Temp != 0))
  {
  Temp = NbActuel % Nombre[i];
  if (Temp == 0)
  {
  break;
  }
  i++;
  }
  if ((Nombre[i] != 0) && (i < 1000) && (Temp != 0))
  {
  printf("%ld, ", NbActuel);
  Nombre[i] = NbActuel;
  }
  Temp = 0;
  NbActuel++;
  i = 0;
  }
  system("PAUSE");
  return 0;
}