Discussion :

[Boss38400]

java erreur de code

Bonjour,
je dois faire un code de java pour mon td
mais il marche pas quand je compile

je dois pendre un entier dans le cas de mon programme l'entier "a "qui est égale à 7
vérifier si il est pair ou impair
dans le cas pair diviser par 2
dans le cas impair *3+1
jusqu'à que l'entier soit égale à 1
et tous afficher
exemple
7 22 11 34 17 52 26 13 40 20 10 5 16 8 4 2 1

voila mon programme

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
 
 
           {
           double [] monTableau;
           monTableau = new double[20];
           double a;
           int i;
           i=0;
           a=7;
          while (a!=1)
          {
     if (a%2 == 0) 
      {
       a=a/2;
       monTableau[i] = 'a';
      }
     else
      {
       a=(a*3)+1;
       monTableau[i] = 'a';
      }
     i++;
      }
     System.out.println(monTableau); 
}

[Tux++]

Bonjour,

1° tu nous parle d'entiers et tu utilises des doubles?
2° pas de définitions de classe ni de méthodes?
3° tu essayes d'afficher un tableau par: System.out.println(letableau)...Il faut boucler sur chaque élément d'un tableau pour l'afficher
4° tu assignes des 'a' à des doubles? autrement dit des chaines de caractères à des doubles?

bref, revois-tes bases, ca ne saurait compiler tel quel




EDIT: les infos que le compilateur te donne comme erreurs/warnings ne sont en général pas très difficile à trouver (sauf exceptions )

[MrCba]

juste comme ca pour le fun

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
public void static test(int y)
{
   while ( y != 1 )
   {
        if ( y%2 == 0 )
           y=y/2;
        else
           y=3*y-1;
        System.out.print(y+ " ");
   }
}

Je me demande si on peut pas améliorer les performances. Par exemple au lieu de faire y%2, n'y a t'il pas moyen de tester simplement le dernier binaire du nombre ?

Sinon je suis d'accord avec Tux++

[Boss38400]

le problème de ton code c'est que c'est une boucle sans fin
essaye le sur ce site
lol


http://www.zamples.com/JspExplorer/i...format=jdk16cl

[Boss38400]

j'ai modifier pour les double merci
sinon voila j'ai pas compris pour l'affichage du tableau
et pour la méthode je l'ai défini c'est juste que j'ai pas ajouté la partie code
car j'utilise un site pour tester donc il en a pas besoin

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
{
           int [] monTableau;
           monTableau = new int[20];
           int a;
           int i;
           i=0;
           a=7;
          while (a!=1)
          {
     if (a%2 == 0) 
      {
       a=a/2;
       monTableau[i] = 'a';
      }
     else
      {
       a=(a*3)+1;
       monTableau[i] = 'a';
      }
     i++;
      }
     System.out.println(monTableau); 
}

[MrCba]

Je ne fait qu'implémenté ton algorithme en supposant qu'elle se termine forcément. Si c'est pas le cas il faut surement ajouter d'autre protection.

D'un autre coté, ton code ne dois pas compiler et tu dois avoir des erreurs évidentes à corriger comme
monTableau[i] = 'a';
est faux.

Par contre, je ne sais pas si je dois donner la solution ou te laisser chercher.

[Boss38400]

en fait il se répète ton code il va jusqu'à 1 .ensuite il recommence
j'ai donné la valeur 7 a ton programme il est arrivé a 1 ensuite il est reparti de nouveau de 7 il me donne la suite que je cherche mais sans arrete

donc qsq je dois corriger pour montableau elle est ou l'erreur

[Invité]

Salut,
MrCba, ton programme boucle car tu fais "-1" au lieu de "+1" si y est impair...
Sinon ta version est beaucoup plus propre je trouve et marche parfaitement...

[Boss38400]

bien vu il y a bien un -1

et une fois corrigé sa marche
mais
je vois toujour pas l'erreur mon erreur a moi

[MrCba]

Ok autant pour moi c'est vraiment dommage de se tromper si bettement

[Boss38400]

si tu peux juste me guider sur mon erreur

[MrCba]

bien vu il y a bien un -1

et une fois corrigé sa marche
mais
je vois toujour pas l'erreur mon erreur a moi

monTableau[i] = 'a';
à remplacer par
monTableau[i] = a;

et le System.out.println(monTableau); ne fonctionne pas non plus si je me trompe pas tu dois boucler sur les éléments de ton tableau et les écrires un à un.

Il faut aussi que tu eneleve ton compteur i qui ne sert aps.

Tu peus ausis ne pas passer par un tableau et écrire directement dans la boucle

[Deadpool]

et le System.out.println(monTableau); ne fonctionne pas non plus si je me trompe pas tu dois boucler sur les éléments de ton tableau et les écrire un à un.

Pour l'affichage d'un tableau sans boucle, si tu travaille avec une version de Java supérieure à 1.5 tu peux utiliser le code suivant :

Code :

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

1
2
 
Arrays.toString(monTableau);

[Boss38400]

MrCba

je te remercie c bon j'ai pu le finir le code

juste un truc dans ton code je viens de le voir
c'est plutôt public static void
a la place de public void static
lol
merci encore pour le coup de main

[Redger]

Je me demande si on peut pas améliorer les performances. Par exemple au lieu de faire y%2, n'y a t'il pas moyen de tester simplement le dernier binaire du nombre ?

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
 
public class Bit {
 
	public static boolean estPair(int nb) {
		return 	(nb & 1) == 0;
	}
 
	public static void main(String[] args) {
 
		for (int i = -10; i <= 10; i++) {
			System.out.println( i + " est pair ? " + Bit.estPair(i) );
		}
 
	}
 
}