gøre min kode lidt mere smart

Lad P være ovenstående klasse.

Benyt

public class P1 extends P {}

til at erklære de andre klasser.

Alt hvad der er fælles for alle klasserne kan du lægge i P.

Lav en:
abstract class AbstractP{
protected int identifier = 10;

protected int getIdentifier()
    {
        return identifier;
    }
   
    protected void SetIdentifier(int id)
    {
        this.identifier = id;
    }
}

class P1 extends AbstractP{
...
}

noget i den stil?

Når nu din variabel 'identifier' er public er der vel egentlig slet ikke nogen grund til at lave get- og set- metoder til at få adgang til den med.
dvs istedetfor
    int hansIdentifyer = instansAfP1.getIdentifyer();
kan du blot skrive
    int hansIdentifyer = instansAfP1.identifyer;

Kort of til at forstå.

mvh JakobA

jakoba -> korrekt, men det er nu pænere design hvis den bliver private (protected)

JakobA: Det er vel ikke pænt at give direkte adgang til variablerne i en klasse. Den pæne løsning er vel at gøre variablen private, og lade public metoder der kun tillader de operationer du øsnker på variablen.

repsak>

Jeg forstår ikke helt pointen i at lade metoderne være protected - det
giver da ikke samme funktionalitet.

Polymorfi vil sige at have flere metoder med samme navn, men med forskellige argumenter. Det burde ikke være nødvendigt hvis du bare vil dele fælles metoder.

jjust -> mener du ikke overloading?

P.java

public class P {
  private int identifier = 10;
    public int getIdentifier() {
        return identifier;
    }
    public void SetIdentifier(int id) {
        this.identifier = id;
    }
}

P1.java:

public class P1 extends P {
  ...
}

må være det rigtige.

Evt. kan member være protected, hvis den skal bruges direkte af metoder i Px.

jjust>

Polymorfi er ikke det samme som operator overloading.

arne_v -> hvad vil du ellers gøre med metoderne?

repsak: Jo, jeg gør :)

public selvfølgelig, når de er public i før synes jeg at det er mest logisk
at de også er public bagefter - ellers har man da ændret funktionaliteten
voldsomt.

arne_v du har heller ikke lavet class P abstract?

Der er ikke nogen abstrakte metoder, så det er ikke strengt nødvendigt.

Så spørgsmålet om den skal erklæres abstrakt eller ej afhænger jo af om det
giver mening at instantiere et objekt af typen eller ej.

Og jeg ved ikke om man kan have en P.

ja - jeg antog bare at P stod for Person eller sådan noget (og en Person er ofte abstract)  :-)

Ok jeg havde ikke lavet metoderne private, men jo det skal de være.

Hvorfor have private metoder der laver så lidt ?

repsac >> når han nu skriver den er public så må man vel tro ham :-))

Hvis det er mang forskellige variable der skal tilgås med samme sæt funktioner i hver klasse kan du lave lidt smart snavs:

    String variableIDenneKlasse = "|identifier1    |tanteOda        |identifier2    |";

Og så en fælles lille klasse de andre extendes udaf:

class TilgaaVariable {

    String variableIDenneKlasse;

    private int findVariabelIndex ( String variabelNavn ) {
        String temp = "|" + variabelNavn + "                ";
        temp = temp.substring( 0, 17 ) +"|";
        int iTemp = variableIDenneKlasse.indexOf( temp );
        if ( iTemp < 0 ) return iTemp;
        return (iTemp/17);
    }

} // endclass TilgaaVariable.

i dine klasser sætter du så en konstantværdi i 'variableIDenneKlasse' med nabnene på de variabler der skal kunne nås i den klasse.
og naver en enkelt get- og set- funktion med en svitch statement i:

class EnAfDineKlasser extends TilgaaVariable {

    private int identifier1;
    private int tanteOda;
    private int identifier2;  // disse 3 skal kunne tilgås

    public EnAfDineKlasser()
        String variableIDenneKlasse =
            "|identifier1    |tanteOda        |identifier2    |";
        // plus hvad ellers constructoren skal lave.
    }

    public int getInt( String name ) {
        switch ( findVariabelIndex( navn ) ) {
            case 0: return this.identifier1;
            case 1: return this.tanteOda;
            case 2: return this.identifier2;
            default: {
                // kast en exeption eller
                System.out.println( "Der er ingen variabel med navnet '"
                                    +navn +" i denne klasse." );
                return 0;
            }
    }

    og ligesådan laver du en set metode.

}

udefra kan du så finde værdien at 'tanteOda' med
        int hansIdentifyer = instansAfP1.getInt("tanteOda");

og andre klasser kan så lave med de samme eller andre variable som du vil.

ret ubrugeligt. men grumme 'smart'

mvh JakobA

jakoba>> jeg kan godt se hvor du vil hen, men synes nu godt koden er en smule grum. Jeg kan ikke engang extende en super klasser, hvor hver klasse P1 .. P4, har hver deres egen variabel som skal skrives til. Hvis jeg extendede vil der jo kun være 1 variabel og ikke 4 som jeg skal bruge. Nå men vil ligge rode lidt med din kode.

øhh mener vil lige og ligge

Med al respekt for Jakob så synes jeg ikke at du skal vælge den kode
at arbejde videre på.

arne v>> hmm men hvad gør jeg så. Jeg kan jo ikke extende for hver klasse har jo sin egne variabler. Har du nogle forslag ?

Du kan da godt extende en basis klasse og så lade de afledte klasser
have sine egne variable.

arne V>> ja det er rigtigt, men getIndentifier() og setIdentifier(), læser og skriver variabler i 4 forskellige klasser. Men metoderne jeg har implementeret i de 4 klasser er ens. Den eneste forskel , at hver klasse har en variabel som skal skrives i selve klassen

Øh.

Er det ikke det som sker når man extender ?

arne>> jo men. Lad os nu antage at jeg først kalder metoden setIdentifier() fra min klasse P1 Så skriver jeg i variablen private int identifier. Senere kalder min klasse p2 metoden setIdentifier() som også skriver i variablen identifier og det dur jo ikke for så overskrives variablen identifier

Sådan fungerer extend ikke.

En instans variabel eksisterer for hvert objekt.

D.v.s. at o1, o2 og o3 har hver sin identifier (uasnset om det er P1, P2 og P3
klasser eller 3 gange P1 klasse).

arne V>> hmm er ikke helt med hvad mener. Kan du ikke smide lidt kode.

public class Arv {
    public static void main(String[] args) {
        P o1 = new P1();
        o1.setIdentifier(1);
        P o2 = new P2();
        o2.setIdentifier(2);
        P o3 = new P1();
        o3.setIdentifier(3);
        System.out.println(o1.getIdentifier());
        System.out.println(o2.getIdentifier());
        System.out.println(o3.getIdentifier());
    }
}

class P {
    private int identifier;
    public int getIdentifier() {
        return identifier;
    }
    public void setIdentifier(int id) {
        identifier = id;
    }
}

class P1 extends P {
}

class P2 extends P {
}

Output:

1
2
3

Men måske er den her mere realistisk:

public class Arv {
    public static void main(String[] args) {
        P o1 = new P1();
        P o2 = new P2();
        P o3 = new P1();
        System.out.println(o1.getIdentifier());
        System.out.println(o2.getIdentifier());
        System.out.println(o3.getIdentifier());
    }
}

class P {
    private int identifier;
    public P(int id) {
        identifier = id;
    }
    public int getIdentifier() {
        return identifier;
    }
}

class P1 extends P {
    public P1() {
        super(1);
    }
}

class P2 extends P {
    public P2() {
        super(2);
    }
}

Output:

1
2
1

arne v>> du har da så ganske ret. Har været lidt væk fra kodning i et stykke tid. Smider du ikke lige et svar. Takker for hjælpen

svar