Erreur dans mon programme divide par 0

Version imprimable

Bonjour ,j'ai un gros probleme avec mon programme alors le but du programme est donner les nombre premiers allant de 1 jusqu'a N , n étant la valeur entrée par l'utilisateur et ce en utilisant l'algo d'eratosphere . Voici l'enonce
Le crible d’Eratosthène permet de déterminer les nombres premiers inférieurs à une certaine valeur
N. On place dans un tableau uni-dimensionnel Tab les entiers compris entre 1 et N. L’algorithme
consiste, pour chaque élément Tab[i] du tableau, à rechercher parmi tous les suivants (d’indices i+1
à N), ceux qui en sont des multiples et à les éliminer (à les remplacer par 0, par exemple). Lorsque
l’ensemble du tableau a subi ce traitement, seuls les nombres premiers du tableau n’ont pas été
éliminés. Ecrire en pseudo-langage l’algorithme de ce crible et le traduire en Java, en recherchant
par exemple les nombres premiers inférieurs à 500.

Voici mon code :
Code:

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 45 46 47 public class Eratosphere { public static void main (String args[]) { int n ; System.out.println("Quelle est la valeur de n ? " ) ; n = Lire.i() ; int[] tableau = new int[n] ; int i ; int j ; i = 1 ; j= 0 ; int intermediaire ; int intermediaire2 ; for (i=0; i<n; i+=1){ // on remplie le tableau d'entiers allant de 1 a n tableau[i] = i+1 ; } for (i=0; i<n; i+=1){ //Traitement de eratosphere intermediaire = tableau[i] ; for (j=i+1; j<n ; j+=1){ intermediaire2 = tableau[j] ; if ( intermediaire2%intermediaire == 0 ) { tableau[j] = 0 ; } } } System.out.println(" Les nombres premiers sont : " ) ; for (i=0; i<n; i+=1){ if ( tableau[i] != 0 ) { System.out.println(tableau[i]) ; } } } }
Le probleme c'est que lorsque je met a n une valeur , j'ai l'erreur suivante :

Exception in thread "main" java.lang.ArithmeticException : / by zero at Eratosphere.main(Eratosphere.java:31 )

Or je vois pas comment je pourrais divisé par zéro sachant que mon tableau a des valeurs allant de 1 a n et non de 0 a n-1 .... HELP ME !

Au fait pour n = Lire.i() ce n'est pas une erreur j'ai utilisé une classe qui facilite la lecture ^^

Merci pour votre aide

17/02/2014, 22h45
thelvin

Citation:

Envoyé par omar6995

Or je vois pas comment je pourrais divisé par zéro sachant que mon tableau a des valeurs allant de 1 a n et non de 0 a n-1

Dis-moi, le but de ton programme, c'est pas de remplacer des nombres par zéro pour se rappeler qu'ils sont pas premiers, par hasard ?

Citation:

Envoyé par omar6995

Au fait pour n = Lire.i() ce n'est pas une erreur j'ai utilisé une classe qui facilite la lecture ^^

Je vois pas ce que ça a de plus simple que Scanner, personnellement.
17/02/2014, 22h47
joel.drigo

Salut,

Citation:

Envoyé par omar6995

Or je vois pas comment je pourrais divisé par zéro sachant que mon tableau a des valeurs allant de 1 a n et non de 0 a n-1 ....

Bah, lors de la première passe tu mets des zéros dans ton tableau, sur la condition intermediaire2%intermediaire == 0, donc lors de la deuxième passe, tu peux avoir des 0 dans ton tableau.
17/02/2014, 23h25
omar6995

Bonsoir et merci de vos réponses
@Thelvin , je suis totalement d'accord avec toi mais je ne sais pas pourquoi dans mon école on nous impose d'utiliser cette classe .... [ Sachant qu'on a commencé a faire du java depuis 2 semaines je pense que c'est pour simplifier un peu ]

@Joel.drigo Que veux tu dire par la 1ere passe et la 2eme passe :p?
17/02/2014, 23h31
joel.drigo
Citation:

Envoyé par omar6995

@Joel.drigo Que veux tu dire par la 1ere passe et la 2eme passe :p?
Code:

1 2 for (i=0; i<n; i+=1){ //Traitement de eratosphere intermediaire = tableau[i] ;
i=0 : première passe
i=1 : deuxième passe
i=2 : troisième passe...

PS à moins que je me trompe, mais c'est Ératosthène, pas eratosphere.
17/02/2014, 23h34
omar6995
Citation:
Envoyé par joel.drigo

Code:

1 2 for (i=0; i<n; i+=1){ //Traitement de eratosphere intermediaire = tableau[i] ;

i=0 : première passe
i=1 : deuxième passe
i=2 : troisième passe...

PS à moins que je me trompe, mais c'est Ératosthène, pas eratosphere.
Aha peut etre que c'est Eratosthene :p

Sinon, pour la premiere passe ça devrait pas donner tableau[0] = 0+1 d'apres la boucle for d'avant?
17/02/2014, 23h48
thelvin

Si... Continue, ça donne quoi pour tableau[1] ?
17/02/2014, 23h54
joel.drigo

Il faut toujours déjà faire marcher un algorithme sur papier avant de l'encoder : mais, tu peux te faciliter ce travail, en affichant le tableau à la fin de chaque passe, par exemple. Ce qu'on cherche à te faire comprendre te sautera alors aux yeux.
18/02/2014, 00h14
omar6995

Citation:

Envoyé par joel.drigo

Il faut toujours déjà faire marcher un algorithme sur papier avant de l'encoder : mais, tu peux te faciliter ce travail, en affichant le tableau à la fin de chaque passe, par exemple. Ce qu'on cherche à te faire comprendre te sautera alors aux yeux.

http://image.noelshack.com/fichiers/...06-aucun-0.jpg

EDIT : C'est bon je crois savoir d'ou viens le probleme :p

C'est sûr qu'en enlevant la boucle qui met des zéros dans le tableau, y'a plus de zéros dans le tableau :roll:

Code:

1
2
3
4
5
6
7
8
9
10
11
for (i = 0; i < n; i += 1) {  
   intermediaire = tableau[i]; // vaut 1 lors de la première passe
   for (j = i + 1; j < n; j += 1) {
       intermediaire2 = tableau[j];
       if (intermediaire2 % intermediaire == 0) { // le reste de la division de n'importe quel entier par 1 vaut toujours 0, donc test toujours vrai !
 
          tableau[j] = 0;
      }
   }
   System.out.println(Arrays.toString(tableau)); // affichera que des 0, sauf pour tableau[0]
}

Merci les gars probleme résolu voici le code final :

Code:

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
45
46
47
48
49
50
51
52
 
 
public class Eratosphere {
 
    public static void main (String args[]) {
        int n ;
        System.out.println("Quelle est la valeur de n ? " ) ;
        n = Lire.i() ;
        int[] tableau = new int[n] ;
        int i ; 
        int j ;
        i = 1 ;
        j= 0 ;
        int intermediaire ;
        int intermediaire2 ;
 
 
 
 
 
 
 
        for (i=0; i<n-1; i+=1){ // on remplie le tableau d'entiers allant de 1 a n
        tableau[i] = i+2 ;
 
    }
    for (i=0; i<n; i+=1){ //Traitement de eratosphere
        intermediaire = tableau[i] ;
        if ( intermediaire != 0 ) {
 
        for (j=i+1; j<n ; j+=1){
            intermediaire2 = tableau[j] ; 
 
            if ( intermediaire2%intermediaire == 0 && intermediaire !=1) {
                tableau[j] = 0  ; 
 
 
            }
        }
        }
        }
        System.out.println(" Les nombres premiers sont : " ) ;
        for (i=0; i<n; i+=1){
            if ( tableau[i] != 0 ) {
                System.out.println(tableau[i]) ;
 
        }
 
 
    }
}
}

18/02/2014, 09h10
Oussapik

Bonjour
Peux-tu tagger le fil de discussion comme résolu dans ce cas ?
Merci