[Tableau] problème de clonage

un exemple qui reproduit mon problème :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22

public class Bidon {   private int[][] tableau = new int [2][2];   public Bidon (int [][] newTableau) { tableau = newTableau; }   public void modifyTab() { tableau[0][0] += 1; }   public int[][] getTab() { return tableau; }   public void test() { Bidon temp = new Bidon(this.tableau.clone()); temp.modifyTab(); System.out.println(tableau[0][0]+" "+temp.getTab()[0][0]); } }

---

et la classe test

Code:

---

1 2 3 4 5 6 7 8 9

public class TestBidon { public static void main(String[] args){ int[][] newtab = {{1,2},{3,7}}; Bidon tab = new Bidon(newtab); tab.test(); } }

---

j'ai epluché plusieurs article sur clone(), mais aucun ne resoud vraiment le problème de cette maniere (et on comprend pourquoi vu que ca ne marche po) . pourtant ca me semble logique , mais avec tout ces pointeurs caché je ne sais plus vraiment ce que je manipule.
merci d'avance

[ Modéré par SEMPERE Benjamin ]
Ajout des balises code
:arrow: Les Règles du Forum

Avant que quelqu'un lise :
* balise code :tagcode:
* explique ton probleme, s'il y a des erreurs de compilations, met les.

Citation:

---

Envoyé par TabrisLeFol

Avant que quelqu'un lise :
* balise code :tagcode:
* explique ton probleme, s'il y a des erreurs de compilations, met les.

---

+1

Celà dit pour cloner un tableau il y a System.arraycopy()

En fait, tu clones un tableau, et ça c'est pas forcément le top....

je ne sais pas comment est implémentée la méthode clone sur les tableaux (jamais utilisée), mais ce que je sais c'est que ton problème vient de là... les valeurs contenues ne sont pas forcément copiées, il est possible que cela ne clone que la structure du tableau en fait - donc du coup, il faut passer par un "System.arraycopy"...


Conseil: pour le clonage d'objets, on utilise volontier les constructeurs de recopie en POO ;)

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19

class Foo() { private int[][] array = new int[2][2];   // constructeur normal Foo(){super();}   // constructeur de clonage Foo(Foo foo) { super(); this.array = new int[foo.array.length][foo.array[0].length];   for(int i=0; i<array.length; i++) { for(int j=0; j<array[0].length; j++) { array[i][j] = foo.array[i][j]; } } } }

---

et ça s'utilise:

Code:

---

1 2 3

Foo f = new Foo(); Foo clone = new Foo(f);

---

:wink:

Oui, le problème est fréquent. La méthode clone fonctionne très bien sur des objets dont les attributs sont primitifs. Si ce n'est pas le cas, la méthode clone est appelée récursivement sur les attributs de type objet (non primitif).
Ton tableau est un objet particulier certes, mais c'est un objet. Il est fort probable que la méthode clone ne soit pas implémentée. A la place on te propose effectivement d'utiliser "System.arraycopy".

Ce qui veut dire en clair que tu disposes de 2 solutions quasi-équivalente :
Redéfinir toi même la méthode clone de ton objet
Définir un constructeur par recopie (Cf. Pill_S)

Citation:

---

Envoyé par Hephaistos007

La méthode clone fonctionne très bien sur des objets dont les attributs sont primitifs. Si ce n'est pas le cas, la méthode clone est appelée récursivement sur les attributs de type objet (non primitif).

---

:nono: je ne suis pas d'accord, ce comportement est indéfini et est laissé au bon vouloir des classes redéfinissant la méthode clone. Par exemple, la plupart des listes (LinkedList, ArrayList, ...), lors du clonage, ne clonent pas leurs éléments (et oui... une simple copie de référence)
tu confond peut-être avec la sérialisation, qui, effectivement, stocke tous les attributs non-transient des objets récursivement

Citation:

---

Ce qui veut dire en clair que tu disposes de 2 solutions quasi-équivalente :

---

qui peuvent facilement s'utiliser en même temps

Code:

---

1 2 3 4 5 6 7 8 9 10

class Foo { Foo(){} Foo(Foo foo){...}   public Object clone() { return new Foo(this); }   }

---

;)

Citation:

---

Envoyé par mabdylon

content de voir que ce n'était pas un problème bidon et des solutions apporté. je testerai ca demain et je n'oublierai pas le "resolu". un peu d'eau a mon moulin:
exemple

Code:

---

1 2 3 4

int x[]={2,4}; int y [] = x.clone(); y[1]+=1;

---

ici la methode clone fonctionne très bien. petite curiosité : pourquoi?
merci encore

---

Comme explique precedement, un tableau est un objet, la methode clone() est donc definie dessus. Attention cependant la, le tableau contient des literaux, donc le clone aura des copies des valeurs. Ca ne fonctionnera pas de la meme maniere si les tableau contient des objets (ex: Integer[] ou int[][]), dans ce cas seules les references seront copiees.

ca marche nickel avec le constructeur de copie. j'essaierai la methode clone s'il me reste du temps avant de rendre le devoir.
merci
ps : ou se trouve le bouton resolu?

En bas à gauche normalement. :roll: