Discussion :

[Booniak]

Bonjour peuple de la Terre

.
J'ai écrit un programme qui:
remplit un tableau à deux dimensions avec les coordonnées [colonnes,lignes]
l'affiche
Inverse la valeur des coordonnées colonnes horizontalement
le ré-affiche.
Alors ça marche mais ce code est autant optimisé qu'un piano est aérodynamique.
Et pis j'voulais faire :

Code :

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

tableau_2D[compteur_col][compteur_lig]="["+compteur_col-compteur_col*2+7+"]["+compteur_lig+"]";

Mais Java il dit ça: bad operand types for binary operator, pourtant compteur_col c'est du int en plus quand je fais compteur_col+compteur_col ça marche sans soucis!
Si vous m'aidez vous irez au paradis, si vous ne m'aidez pas vous serez transformé en friteuse.

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
45
46
47
48
49
50
51
import javax.swing.*;
 import java.util.ArrayList;
 
public class exo_12 {
 
    public static void main(String[] args) 
    	{
    		String[][] tableau_2D=new String [8][6];
    		int compteur_col=0,compteur_lig=0;
 
 
    		//On rempli les emplacements du tableau avec les coordonées correspondantes.
    		for(compteur_col=0;compteur_col<8;compteur_col++)
    		{
    			for(compteur_lig=0;compteur_lig<6;compteur_lig++)
    			{
    				tableau_2D[compteur_col][compteur_lig]="["+compteur_col+"]["+compteur_lig+"]";
    			}
    		}
 
    		//On affiche le tableau.
    		for(compteur_col=0;compteur_col<8;compteur_col++)
    		{
    			for(compteur_lig=0;compteur_lig<6;compteur_lig++)
    		    {
    		      System.out.println(tableau_2D[compteur_col][compteur_lig]);
    		    }
    		}
 
    		for(compteur_col=0;compteur_col<8;compteur_col++)
    		{
    			for(compteur_lig=0;compteur_lig<6;compteur_lig++)
    			{
    				//On crée une variable qui par un calcul digne des plus grand astro-celésto-mathématiciens renvoi la valeur en symétrie horizontale de la case.
    				int clodo_strat=compteur_col-compteur_col*2+7;
    			tableau_2D[compteur_col][compteur_lig]="["+clodo_strat+"]["+compteur_lig+"]";
    		    }
    		}
 
    		//On affiche le tableau avec les valeurs des colonnes inversé horizontalement.
    		for(compteur_col=0;compteur_col<8;compteur_col++)
    		{
    			for(compteur_lig=0;compteur_lig<6;compteur_lig++)
    			{
    		System.out.println(tableau_2D[compteur_col][compteur_lig]);
    			}
 
            }
 
                           }
                              }

[joel.drigo]

Salut,

Quand tu écris ça :

Code :

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

tableau_2D[compteur_col][compteur_lig]="["+compteur_col-compteur_col*2+7+"]["+compteur_lig+"]";

Le compilateur se dit : tiens, on calcule un truc. je dois traiter un calcul en tenant compte de la priorité des opérateurs,

• donc j'ai le * qui est prioritaire par rapport à tous les autres opérateurs (+ et -), je le traite d'abord : ça fait du int * int, le compilo sait traiter et sait que ça fait du int
• ensuite il traite le +, donc il regarde "["+compteur_col, et voit une concaténation de chaînes (parce que pour lui String + int ça fait du String)
• ensuite il vois le -, et là le compilateur ne sait pas quoi en faire, parce que ça n'existe pas le signe - dans une opération de String avec un int.

Je ne sais pas trop pourquoi tu aurais le message dont tu parles, mais j'obtiens bien un message qui dit que - n'est pas un opérateur valide pour les opérandes de type String et int.

Tu as contourné le problème en faisant une variable intermédiaire, mais tu aurais pu utiliser des parenthèses pour forcer la priorité.

[Booniak]

Ah ouais je savais pas que la concaténation était prioritaire ! Tout s'explique, comme à la fin d'un épisode de scoobydoo.
Mercy Jamy!

[nchal]

Bonjour

En même temps, utiliser Java pour faire un exercice bidon et coder le tout en impératif, c'est moche, très moche. Autant faire du C ...
Si tu fais des exercices simples, essaie au moins de respecter le principe objet le plus possible.

Par exemple, évite de tout mettre dans un gros static main tout dégueu. Amuses toi à faire plusieurs objets (pas 10 non plus pour un truc comme ça), factorise au maximum ton code. Par exemple, tu utilises plusieurs fois la même boucle, pourquoi ne pas faire une méthode pour factoriser tout ça ? Évite aussi les magic number, y a rien pire pour foutre la merde à un moment donné.
Et je vois que tu importes swing et ArrayList, mais tu ne les utilises pas. Déjà que Java c'est pas folichon niveau perf, mais si en plus, tu lui mets des bâtons dans les roues

[joel.drigo]

En fait, ici, c'est plutôt par la règle d'associativité que le compilateur choisi le + (utilisé quand les opérateurs ont la même priorité, l'associativité left-to-right fait que la concaténation est faite d'abord).

Code :

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

 System.out.println( 100 - 42 + "42" );

compilera et affichera 5842.

[Booniak]

nchal effectivement j'aurais du factoriser tu fais bien de me le rappeler par contre c'est quoi un "magic number"?
Au passage fais pas trop l'ancien, j'ai posté sur le forum débutant, répond avec humilité.
Allez merci quand même t'as pas l'air d'être un mauvais bougre.

[nchal]

C'est bien tu m'as fait rire.
Sinon un nombre magique, ça regroupe plusieurs choses mais en gros dans ton appli, au lieu de mettre 8 et 6 en taille de tableau, tu déclares des constantes et tu n'utilises que ces constantes dans ton code, ainsi si ce nombre doit changer pour une raison ou une autre, pas besoin d'aller fouiller dans le code, juste à changer une variable