Affichage "propre" d'une matrice

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:

---

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:

---

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:

---

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:

---

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 ;)

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

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:

---

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:

---

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:

---

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++

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