Vererbung

[nihilfire]

Hallo!
Habe die Aufgabe bekommen eine Unterklasse Schieberegister zu erstellen, welche Eigenschaften von einer Oberklasse Register erbt.
Der Konstruktor der Klasse Schieberegister bekommt ein Array mit den Registereinträgen z.B. {1,0,1,0,1,0} übergeben und soll prüfen, ob die einträge im Register "0" oder "1" sind. Wenn das der fall ist wir das Register gespeichert, wenn nicht soll ein Register derselben Länge wie das übergebene Array erzeugt und mit Nullen vorbesetzt werden. Dabei soll der Konstruktor der Oberklasse benutzt werden.

Wie kann man das realisieren?

Konstruktor der Klasse Register:

public class Register {

  private short[] feld;
  private short index;

  Register(short laenge){
    feld = new short[laenge];
    for (index = 0; index < feld.length; index++){
	feld[index] = 0;
    }
  }
}

Konstruktor der Klasse Schieberegister:

public class Schieberegister extends Register {

  private short[] speicherRegister;
  private boolean anfErr = false;


  Schieberegister(short[] speicherRegister){
	for (int i = 0; i < speicherRegister.length; i++){
		if(!(speicherRegister[i] == 0 || speicherRegister[i] == 1)){
			anfErr = true;
		}
		else{
			this.speicherRegister = speicherRegister;
		}
	}
  }
}

 

[byte]

Den Konstruktor der Oberklasse kannst Du mit super(); aufrufen. Du musst natürlich auch die richtigen Parameter gemäß des Konstruktors übergeben. In Deinem Fall dann wohl:

super(speicherRegister.length);

 

[nihilfire]

Den Konstruktor der Oberklasse kannst Du mit super(); aufrufen. Du musst natürlich auch die richtigen Parameter gemäß des Konstruktors übergeben. In Deinem Fall dann wohl:

super(speicherRegister.length);

ja, aber das soll ja nur dann aufgerufen werden, wenn ein Registereintrag nicht "0" oder "1" ist, also wenn

anfErr = true

ist und ich muss "speicherRegister.length" erst in short umwandeln also:

laenge = (short) speicherRegister.length

und dann müsste

super(laenge);

irgendwo hin da es aber nur zu beginn des Konsturktor stehen darf weiß ich nicht mehr weiter...

Gruß, nihilfire

 

[SnooP]

Also schwierig... das einzige das du da machen kannst, ist erst super(laenge) zu machen und erst danach auf prüfen ob das erstelle Schieberegister überhaupt passt, wenn ja dann gut... wenn nein erstelle neues register und speichere es anstatt dem vorher gespeicherten... - ist etwas unsinnig und hier auch seltsam, dass man unbedingt auf den super-Konstruktor zugreifen soll, wenn man doch was ganz anderes machen will

 

[nihilfire]

Also schwierig... das einzige das du da machen kannst, ist erst super(laenge) zu machen und erst danach auf prüfen ob das erstelle Schieberegister überhaupt passt, wenn ja dann gut... wenn nein erstelle neues register und speichere es anstatt dem vorher gespeicherten... - ist etwas unsinnig und hier auch seltsam, dass man unbedingt auf den super-Konstruktor zugreifen soll, wenn man doch was ganz anderes machen will

Aufgebenstellung:
Der Konstruktor soll ein Array mit short-Einträgen als Parameter übergeben bekommen. Er soll prüfen, ob alle diese Einträge 0 oder 1 sind und (falls das der Fall ist) sie bei der Erzeugunug der neuen Registers als Anfangswerte benutzen. Ist das nicht der Fall, so soll ein Register mit derselben Länge wie der Übergebene Array erzeugt und mit Nullen vorbesetzt werden. Dabei soll der Konstruktor der Oberklasse benutzt werden.

ohne den letzten Satz zu beachten sieht meine Lösung so aus:

Schieberegister(short[] speicherRegister){
  for (int i = 0; i < speicherRegister.length; i++){
 if(!(speicherRegister[i] == 0 || speicherRegister[i] == 1)){
   anfErr = true;
 }
 else{
   this.speicherRegister = speicherRegister;
 }
  }
  if(anfErr){
    for (int j = 0; j < speicherRegister.length; j++){
      speicherRegister[j] = 0;
    }
  }
}

Gruß, nihilfire

 

[Murray]

Den super-Konstruktor muss man ja ohnehin aufrufen, da die Basisklasse ja keinen Default-Konstruktor hat. Vor dem Aufruf des super-Konstruktor darf man zwar nicht auf Instanzmethoden, wohl aber auf statische Methoden zugreifen.

Also z.B. so:

public class Schieberegister extends Register {

  private short[] speicherRegister;
  private boolean anfErr = false;


  Schieberegister(short[] speicherRegister){
  
	super( calcLength( speicherRegister));
	
	//-- Da feld in der Basisklasse private ist, müssen wir calcLength leider
	//-- erneut aufrufen, um zu wissen, ob das übergebene Array nur aus Nullen
	//-- und Einsen bestand
	anfErr  = calcLength( speicherRegister) < speicherRegister.length;
	
	if ( !anfErr) {
	   this.speicherRegister = speicherRegister;
	}
  }
  
  static short calcLength( short[] speicherRegister) {
  
	boolean ok = true; 
	for (int i = 0; i < speicherRegister.length; i++){
		if(!(speicherRegister[i] == 0 || speicherRegister[i] == 1)) {
			ok = false;
			break;
		}
	}
	return (short) ( ok ? 0 : speicherRegister.length); 
  }
}

 

[nihilfire]

Danke, habe noch was ergänzt/verbessert:

	if (anfErr) {
		this.speicherRegister = speicherRegister;
	}
	else{
		this.speicherRegister = super.getFeld();
	}

jetzt funktioiert es!

Gruß, nihilfire