Kopie eines Objektes

[MyD]

Also irgendwo habe ich noch eines kleinen Denkfehler - vielleicht könnt ihr mir da weiterhelfen.

d) Erweitern Sie die Klasse Combination um eine Methode
Combination copy(Combination c), um eine Kopie des Combination–Objektes zu erhalten

public class CombinationTester {

	public static void main(String[] args) {
		
		Combination comb= new Combination();
		Combination combKopie= comb.copy( comb);
}

public class Combination {

	public Combination copy (Combination c) {
			return new Combination();
	}
}

ich weiss leider nicht weiter wie ich in der public class Combination mit dem Objek Combination c umgehen soll.

hoffe ihr könnt mir da weiterhelfen.

vielen dank schonmal im voraus.

 

[André Uhres]

Object#clone() ?

 

[MyD]

habe ich mir auch angeschaut - aber zumal gibt es ja
* Flaches Kopieren
* Tiefes Kopieren
http://www.dpunkt.de/java/Die_Sprache_Java/Objektorientierte_Programmierung_mit_Java/45.html

und ich weiss nicht ganz wie ich mit dem clone objekt umzugehen habe.

habe das ganze jetzt mal so gelöst:

		public Combination copy (Combination c) {
			Combination combKopie = new Combination();
			combKopie.zahlenfolge = c.zahlenfolge;
			return combKopie;
		}

ist das akzeptabel ?

 

[stev.glasow]

Einfaches Klonen kannst du so machen

public class Combination implements Cloneable{ 
   // implements Cloneable nicht vergessen sonst wirft er dir eine java.lang.CloneNotSupportedException

    public static void main(String[] args) {
            
            Combination comb= new Combination();
            System.out.println(comb.getIrgendwas());
            System.out.println(comb.getNochwas());
            
            Combination combKopie= comb.copy( comb);
            System.out.println(combKopie.getIrgendwas());
            System.out.println(combKopie.getNochwas());

    }
    
    private Object irgendwas = new Object();
    private Object nochwas = new Object();
    
    public Object getIrgendwas() {
        return irgendwas;
    }

    public void setIrgendwas(Object irgendwas) {
        this.irgendwas = irgendwas;
    }
    

    public Object getNochwas() {
        return nochwas;
    }
    
    public Combination copy (Combination c) {
        try{
            return (Combination)c.clone();
        }
        catch (CloneNotSupportedException err){ 
            // CloneNotSupportedException kann eh nicht geworfen werden  
            throw new RuntimeException(err); 
            
        }
    }
  
}

Das ist aber wie im Chat bereits gesagt keine Tiefe kopie und irgendwie eine schwachsinnige Hausaufgabe wenn ihr das über clone() lösen sollt.
[edit]
jo das mit dem

public Combination copy (Combination c) {
     Combination combKopie = new Combination();
     combKopie.zahlenfolge = c.zahlenfolge;
     return combKopie;
}

wäre auch eine Lösungsmöglichkeit, kommt halt drauf an wie ihr was kopieren sollt.

 

[Guest]

Habe mal die Lösung ganz unten ausprobiert. Die funktioniert aber nicht! Mein Objekt enthält nur Nullen.
Ist die Lösung wirklich richtig?

 

[stev.glasow]

Habe mal die Lösung ganz unten ausprobiert. Die funktioniert aber nicht! Mein Objekt enthält nur Nullen.

Aha. Fein.

Ist die Lösung wirklich richtig?

Sollte eigentlich.

 

[MyD]

also bei mir funktioniert sie - ist deine instanzvariable auch gesetzt - sonst ist sie automatisch null

 

[Guest]

Also das macht mich noch wahnsinnig. Ich kriegs nicht mit der Methode zum laufen. Wie gesagt, c.clone(); würde funktionieren, aber meine Methode funzt nicht.
Ich poste ich mal meinen Code:

Klasse:

public class Combination implements Cloneable   // Klasse klonbar machen
{
   private int arraysize;
   int alter;
   private int [] ifeld=new int[arraysize];     //Hier stehen die Felder
   
//*****************************************************************************
   public int getArraySize()
   {
      return arraysize;
   }
   
   public void setArraySize(int i)
   {
      arraysize=i;
   }
   
   public void init(int[] a)			//Hier stehen die Methoden
   {
      ifeld=a;
   }
   
   public void print()
   {
      int i=0;
      do{
      System.out.print(ifeld[i]+" ");
      i++;
      }
      while(i<arraysize);
      System.out.println();
   }
//******************************************************************************  
   protected Object clone() {
      try  {
         Combination combi = (Combination) super.clone();
         return combi;
      
      }  catch(CloneNotSupportedException e)  {       //Bei klonen überschreiben
         return null;
      }
   }


//******************************************************************************

   public Combination copy (Combination c) {
         Combination cnew = new Combination();	      //Funktioniert nicht???
         cnew.alter = c.alter;
         return cnew;
      }      
}

Main Klassse:

public class CombinationTester
{
   public static void main(String[] args)
   {				
//************************************************************************
   Combination c=new Combination();     //Objekt erzeugen
   Combination c2=new Combination();     //Objekt erzeugen
   c.setArraySize(20);                  //Arraygroesse festlegen
   c2.setArraySize(20);                  //Arraygroesse festlegen
   int [] intArray=new int[c.getArraySize()];
   System.out.println("Zahlenfolge: ");
   for(int i=0;i<intArray.length;i++)
   {
      intArray[i]=i+1;                //Bsp. einer Zahlenfolge 
   }  
   c.init(intArray);                  //Array übergeben
   c.print();                         //Folge darstellen
   //c2=(Combination)c.clone();          // das würde funktionieren
   c.alter=5;                         //alter nur zum Testen des clonens
   System.out.println(c.alter);       
   c2.copy(c);
   System.out.println(c2.alter);
   }
}

 

[stev.glasow]

Also:
wenn hier c2.copy(c); werte von c in c2 übernommen werden sollen müsste das so aussehen:

public void copy (Combination c) {
         this.alter = c.alter;
}

Wenn die Methode copy() eine kopie des Objectes zurückgeben soll:

public Combination copy () {
      Combination cnew = new Combination();
      cnew.alter = this.alter;
      return cnew;
}
// Aufruf wäre : Combination c2 = c1.copy();

Und wenn die Methode eine kopie des übergebendes Objektes zurück geben soll

public static Combination copy (Combination c) {
      Combination cnew = new Combination();
      cnew.alter = c.alter;
      return cnew;
}
// Aufruf wäre : Combination c2 = Combination.copy(c1);

Sowas

public Combination copy (Combination c) { 
...
}

ist eigentlich aus meiner Sicht sinnfrei, denn du hast 3 Combination Objekte mit denen du arbeiten sollst:
c,
das Objekt dass zurück gegebn werden soll
und this

Das dort oben sind nur einige Möglichkeiten wie die Methode aussehen könnten. Aber eure Vorgabe ist irgendwie hohl.
Oder du sagst mal genau was ihr machen sollt.