Discussion :

[Compassio]

Bonjour à tous.

je travail actuellement sur une classe représentant les matrices, je travail en console uniqument, pas d'interface graphique.
Le problème intervient lorsque je veux afficher ma matrice en sortie sur la console :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
public String toString() {
		String s = "";
 
		for (int i = 0; i < this.M; i++) {
			for(int j = 0; j < this.N; j++)
				s += this.data[i][j] + "  ";
			s += "\n";
		}
 
		return s;
	}

Pour une matrice comme ceci :

Code :

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

1
2
3
4
5
6
7
8
 
double[][] d = {
			{4, 6, 9},
			{3, 5, 2}
		};
 
		Matrix m = new Matrix(d);
		System.out.println(m);

J'ai un affichage propre :

Code :

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

1
2
3
 
4.0  6.0  9.0  
3.0  5.0  2.0

Mais si je j'ai des plus gros nombres pas endroit, j'ai un affichage décalé :

Code :

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

1
2
3
 
4.0  690.0  9.0  
32334.0  5.0  2.0

J'aimerai trouver un moyen pour que l'affichage soit bien aligné, peut importe les nombres dans ma matrice. Avez vous une astuce ?
Merci

[wax78]

Si tu connais la syntaxe "a la printf", tu as par exemple String.format(); Mais il y'a d'autres moyens.

https://docs.oracle.com/javase/tutor...berformat.html

[adiGuba]

Salut,


Déjà attention car ta méthode toString() est très dangereuse : il ne faut pas modifier une String avec += dans une boucle !!
En effet ceci génère plusieurs objets temporaires, ce qui rend le code très gourmand en mémoire, et donc très lent.
Et ceci de manière exponentielle plus on augmente le nombre d'itération du tableau.

Par exemple le code suivant prend plus de 38 secondes sur ma machine :

Code :

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

1
2
3
4
5
6
7
8
		double[][] d = new double[400][400];
 
		Matrix m = new Matrix(d);
 
		long t0 = System.currentTimeMillis();
		String s= m.toString();
		long t1 = System.currentTimeMillis();
		System.out.println((t1-t0)+"ms");

La bonne manière de faire consiste à utiliser un StringBuilder :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
	@Override
	public String toString() {
		StringBuilder sb = new StringBuilder();
 
		for (int i = 0; i < this.M; i++) {
			for(int j = 0; j < this.N; j++) {
				sb.append(this.data[i][j]).append("  ");
			}
			sb.append("\n");
		}
 
		return sb.toString();
	}

Et là le code précédent passe de 38 secondes à 34... milli-secondes !

Avec StringBuilder j'ai dû monter à un tableau de 5000x5000 pour dépasser la seconde d'exécution...
Dans ce cas la limite vient plus de la mémoire disponible qu'autre chose.





Maintenant pour en revenir à ton problème et comme le dit wax78 la solution la plus simple consiste à utiliser String.format() pour formater le nombre.
Par exemple pour l'afficher sur 10 caractères maximum avec 2 décimales tu peux utiliser String.format("%10.2f", this.data[i][j]) ...
Reste juste à déterminer la longueur. Soit tu mets une valeur fixe suffisamment grande, soit tu dois parcourir tes données pour la déterminer à l'avance...





Dernière remarque si tu dois afficher de très grosse matrice, System.out.println(m) est inefficace.
En effet tu vas créer une String énorme avant de l'afficher. Il est préférable d'afficher tout directement dans le flux.
Tu peux faire cela simplement via l'interface Appendable afin de ne pas dupliquer le code :

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
	public void appendTo(Appendable output) throws IOException {
		for (int i = 0; i < this.M; i++) {
			for(int j = 0; j < this.N; j++) {
				String value = String.format("%10.2f", this.data[i][j]);
				output.append(value);
				output.append("  ");
			}
			output.append("\n");
		}
	}
 
	@Override
	public String toString() {
		StringBuilder sb = new StringBuilder();
		try {
			appendTo(sb);
		} catch (IOException e) {
			// Ceci ne devrait pas arriver :
			throw new IllegalStateException(e);
		}
		return sb.toString();
	}

Désormais tu peux remplacer System.out.println(m) par m.appendTo(System.out) pour plus d'efficacité.


a++

[Compassio]

Merci à vous, je regarde ça et je vous donne des nouvelles