Discussion :

[mabdylon]

un exemple qui reproduit mon problème :

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

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

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
Les Règles du Forum

[TabrisLeFol]

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

[natha]

Avant que quelqu'un lise :
* balise code
* 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()

[Pill_S]

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 :

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

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

1
2
3
 
Foo f = new Foo();
Foo clone = new Foo(f);

[Hephaistos007]

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)

[Pill_S]

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

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

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

qui peuvent facilement s'utiliser en même temps

Code :

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

1
2
3
4
5
6
7
8
9
10
 
class Foo {
  Foo(){}
  Foo(Foo foo){...}
 
  public Object clone() {
    return new Foo(this);
  }
 
}

[bouye]

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 :

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

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.

[mabdylon]

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?

[TabrisLeFol]

En bas à gauche normalement.