erfoefening

[Verwijderd]

Hoi iedereen,

Vandaag was ik een beetje aan het oefenen met erven en ik liep hier vast. Ik heb een fulltime werknemer die pensioenpremie moet betalen over zijn loon en ik heb al een klasse werknemer gemaakt met de private properties naam en functie. Dat is me helemaal gelukt. Vervolgens heb ik de volgende klasse gemaakt:

code:

public class Fulltime extends Werknemer {
private double Weeksalaris;

public void setWeeksalaris() {
Weeksalaris = WeeksalarisIn
}

public String getWeeksalaris() {
return Weeksalaris;
}

public double Pensioenpremie() {
return Weeksalaris * 0.05;
}

public double Totaal() {
return Weeksalaris - Pensioenpremie;
}
}

Ook deze wordt nog goed opgepakt door de compiler. Dan maak ik vervolgens dit:

code:

import java.util.Scanner;

public class Salaris {
public static void main(String args[]) {

Fulltime Fwerknemer = new Fulltime();
Scanner Toetsenbord = new Scanner(System.in);

Fwerknemer.setNaam(Toetsenbord.next());
Fwerknemer.setFunctie(Toetsenbord.next());
Fwerknemer.setWeeksalaris(Toetsenbord.nextDouble());
Fwerknemer.Pensioenpremie(Fwerknemer.Pensioenpremie());
Fwerknemer.Totaal();

Partime Pwerknemer = new Partime();

Pwerknemer.setNaam(Toetsenbord.next());
Pwerknemer.setFunctie(Toetsenbord.next());
Pwerknemer.setUurloon(Toetsenbord.nextDouble());
Pwerknemer.setGewerkteUren(Toetsenbord.nextInt());
Pwerknemer.Totaal();

Toetsenbord.close();

}
}

Nu gaat het fout. De foutmelding die ik krijg is:

code:

E:\mijn hobbypagina\java\javabestanden>javac Salaris.java
Salaris.java:12: error: cannot find symbol
Fwerknemer.Pensioenpremie(Fwerknemer.Pensioenpremie());
                                    ^
  symbol:   method Pensioenpremie()
  location: variable Fwerknemer of type Fulltime
1 error

Wie of wie helpt mij uit de brand.

[Philip Ross]

Waarom doe je:
Fwerknemer.Pensioenpremie(Fwerknemer.Pensioenpremie());

terwijl je daarna dit doet:
Fwerknemer.Totaal();

Dat zijn toch twee vergelijkbare funcites in je klasse? Beide hebben geen input argumenten nodig maar bij de een gaaf je die wel en bij de andere niet.

[martyw]

Lees bijvoorbeeld dit artikel over encapsulation https://www.w3schools.com/java/java_encapsulation.asp

[Hydra]

Je code klopt voor geen meter, sorry:

Ten eerste:

code:

public void setWeeksalaris() {
Weeksalaris = WeeksalarisIn
}

Dit doet dus helemaal niks.

Daarnaast:

code:

public double Totaal() {
return Weeksalaris - Pensioenpremie;
}
}

Dit werkt natuurlijk niet. Er is geen Pensioenpremie member variabele. Je wil hier waarschijnlijk de Pensioenpremie() methode hier aanroepen.

En dan tenslotte je main is helemaal een rommeltje. Wat zou dit in hemelsnaam doen:

code:

Fwerknemer.Pensioenpremie(Fwerknemer.Pensioenpremie());

Je mist volledig de basisbeginselen van (member)variabelen en methodes. Focus je je daar ajb eerst op voordat je in inheritance duikt.

[BertS]

Waar je compiler over struikelt is je Main, regel 12:

Java:

Fwerknemer.Pensioenpremie(Fwerknemer.Pensioenpremie());

Wat probeer je met die regel te bereiken?


_{@Hydra TS is volgens mij hard haar best aan het doen om te leren, een beetje meer empathie kan geen kwaad}

[.oisyn]

Verwijderd schreef op woensdag 3 april 2019 @ 10:26:
Ook deze wordt nog goed opgepakt door de compiler.

Geloof ik niets van, het zit vol met fouten (zie post Hydra) . Ik ben dan wel benieuwd hoe je tot deze conclusie komt. Hoe heb je gecompileerd?

[ Voor 18% gewijzigd door .oisyn op 03-04-2019 17:16 ]

[Hydra]

BertS schreef op woensdag 3 april 2019 @ 16:55:
_{@Hydra TS is volgens mij hard haar best aan het doen om te leren, een beetje meer empathie kan geen kwaad}

Je leest er te veel in. Ik geef alleen aan dat 'ie eerst aan de basics moet werken, da's alles. Dit soort dingen leren en alles maar op een hoop gooien en dan om hulp gaan vragen als 't niet werkt is extreem inefficient.

[ Voor 15% gewijzigd door Hydra op 03-04-2019 17:36 ]

[BertS]

Hydra schreef op woensdag 3 april 2019 @ 17:35:
[...]


Je leest er te veel in. Ik geef alleen aan dat 'ie eerst aan de basics moet werken, da's alles. Dit soort dingen leren en alles maar op een hoop gooien en dan om hulp gaan vragen als 't niet werkt is extreem inefficient.

_{ik doelde op de toon van je bericht, met 'Je code klopt voor geen meter, sorry:' en 'helemaal een rommeltje'. Dat is niet zo moedgevend voor een beginner. /offtopic}

[Hydra]

BertS schreef op woensdag 3 april 2019 @ 18:34:
_{ik doelde op de toon van je bericht, met 'Je code klopt voor geen meter, sorry:' en 'helemaal een rommeltje'. Dat is niet zo moedgevend voor een beginner. /offtopic}

Tja. Als je bij je docent hiermee aankomt met de melding "maar het compiled heus wel" terwijl het dat absoluut niet doet krijg je een reactie die een stuk onvriendelijker is hoor. Ik neem je commentaar mee, maar ik vind het zelf wel meevallen

[BertS]

Hydra schreef op donderdag 4 april 2019 @ 09:20:
Tja. Als je bij je docent hiermee aankomt met de melding "maar het compiled heus wel" terwijl het dat absoluut niet doet krijg je een reactie die een stuk onvriendelijker is hoor.

_{[laatste offtopic]Maar dat is de melding niet. Als je komt met 'ik begrijp niet waarom de compiler dit niet lust' mag ik hopen dat de docent in de behulpzame modus gaat. Anders kan hij beter een ander vak gaan uitoefenen[/offtopic]}

@Verwijderd Er zijn wat vragen aan je gesteld, dus de hulptroepen zitten blijkbaar voor je klaar

[Hydra]

BertS schreef op donderdag 4 april 2019 @ 09:44:
[...]

[sub][laatste offtopic]Maar dat is de melding niet.

Uhm, jawel. Zie OP:

Verwijderd schreef op woensdag 3 april 2019 @ 10:26:
Ook deze wordt nog goed opgepakt door de compiler.

Die class compiled absoluut niet.

[BertS]

Hydra schreef op donderdag 4 april 2019 @ 10:21:
[...]

[...]


Die class compiled absoluut niet.

Vanwege die WeeksalarisIn? Die kan toch in bovenliggende werknemer zitten? Of schiet mijn kennis van Java nu tekort (is 15 jaar geleden...)?

[Marco1994]

BertS schreef op donderdag 4 april 2019 @ 12:49:
[...]

Vanwege die WeeksalarisIn? Die kan toch in bovenliggende werknemer zitten? Of schiet mijn kennis van Java nu tekort (is 15 jaar geleden...)?

Nee vanwege regel 17

[BertS]

Marco1994 schreef op donderdag 4 april 2019 @ 12:51:
[...]

Nee vanwege regel 17

O ja, nu zie ik hem. Dat wordt wat lastig ja.

[Verwijderd]

Even alle reacties doorgenomen hoor. Wat fijn dat er zo veel mensen willen helpen. Momenteel heb ik de code gecompileerd in command prompt Ik heb iig de code nu zo:

code:

public class Fulltime extends Werknemer {
private double Weeksalaris;
private double Pensioenpremie;

public void setWeeksalaris(double WeeksalarisIn) {
Weeksalaris = WeeksalarisIn;
}

public double getWeeksalaris() {
return Weeksalaris;
}

public double Pensioenpremie() {
return Weeksalaris * 0.05;
}

public double Totaal() {
return Weeksalaris - Pensioenpremie;
}
}

En de volgende klasse is nu:

code:

import java.util.Scanner;

public class Salaris {
public static void main(String args[]) {

System.out.println("Fulltime werknemer: ");
Fulltime Fwerknemer = new Fulltime();
Scanner Toetsenbord = new Scanner(System.in);

System.out.print("Naam: ");
Fwerknemer.setNaam(Toetsenbord.next());
System.out.print("Functie: ");
Fwerknemer.setFunctie(Toetsenbord.next());
System.out.print("Weeksalaris: ");
Fwerknemer.setWeeksalaris(Toetsenbord.nextDouble());
System.out.print("Pensioenpremie: ");
System.out.println(Fwerknemer.Pensioenpremie());
System.out.print("Brutoloon: ");
System.out.print(Fwerknemer.Totaal());

System.out.println("Partime werknemer: ");
Partime Pwerknemer = new Partime();

System.out.print("Naam: ");
Pwerknemer.setNaam(Toetsenbord.next());
System.out.print("Functie: ");
Pwerknemer.setFunctie(Toetsenbord.next());
System.out.print("Uurloon: ");
Pwerknemer.setUurloon(Toetsenbord.nextDouble());
System.out.print("Gewerkte uren: ");
Pwerknemer.setGewerkteUren(Toetsenbord.nextInt());
System.out.print("Brutoloon: ");
System.out.print(Pwerknemer.Totaal());

Toetsenbord.close();

}
}

En hij werkt. Ik ben heel blij. Superbedankt iedereen.

[Marco1994]

Verwijderd schreef op donderdag 4 april 2019 @ 18:12:
Even alle reacties doorgenomen hoor. Wat fijn dat er zo veel mensen willen helpen. Momenteel heb ik de code gecompileerd in command prompt Ik heb iig de code nu zo:

code:

public class Fulltime extends Werknemer {
private double Weeksalaris;
private double Pensioenpremie;

public void setWeeksalaris(double WeeksalarisIn) {
Weeksalaris = WeeksalarisIn;
}

public double getWeeksalaris() {
return Weeksalaris;
}

public double Pensioenpremie() {
return Weeksalaris * 0.05;
}

public double Totaal() {
return Weeksalaris - Pensioenpremie;
}
}

En de volgende klasse is nu:

code:

import java.util.Scanner;

public class Salaris {
public static void main(String args[]) {

System.out.println("Fulltime werknemer: ");
Fulltime Fwerknemer = new Fulltime();
Scanner Toetsenbord = new Scanner(System.in);

System.out.print("Naam: ");
Fwerknemer.setNaam(Toetsenbord.next());
System.out.print("Functie: ");
Fwerknemer.setFunctie(Toetsenbord.next());
System.out.print("Weeksalaris: ");
Fwerknemer.setWeeksalaris(Toetsenbord.nextDouble());
System.out.print("Pensioenpremie: ");
System.out.println(Fwerknemer.Pensioenpremie());
System.out.print("Brutoloon: ");
System.out.print(Fwerknemer.Totaal());

System.out.println("Partime werknemer: ");
Partime Pwerknemer = new Partime();

System.out.print("Naam: ");
Pwerknemer.setNaam(Toetsenbord.next());
System.out.print("Functie: ");
Pwerknemer.setFunctie(Toetsenbord.next());
System.out.print("Uurloon: ");
Pwerknemer.setUurloon(Toetsenbord.nextDouble());
System.out.print("Gewerkte uren: ");
Pwerknemer.setGewerkteUren(Toetsenbord.nextInt());
System.out.print("Brutoloon: ");
System.out.print(Pwerknemer.Totaal());

Toetsenbord.close();

}
}

En hij werkt. Ik ben heel blij. Superbedankt iedereen.

Hij compileert maar dat is het dan ook. Je doet niets met de functie Pensioenpremie. Zet eens een breakpoint op regel 17 en kijk dan wat er gebeurd.

[.oisyn]

Ik geloof er nog steeds niets van, regel 18 is gewoon fout.
.edit: oh wacht er is nu ook een variabele Pensioenpremie. Mag dat?

[ Voor 36% gewijzigd door .oisyn op 04-04-2019 20:04 ]

[Hydra]

.oisyn schreef op donderdag 4 april 2019 @ 20:03:
Ik geloof er nog steeds niets van, regel 18 is gewoon fout.
.edit: oh wacht er is nu ook een variabele Pensioenpremie. Mag dat?

Ja hoor. De code klopt overigens nog steeds voor geen meter maarja, 'het werkt'.

[.oisyn]

Hydra schreef op donderdag 4 april 2019 @ 20:08:
[...]


Ja hoor.

Welke krijgt dan voorrang?

[Verwijderd]

.oisyn schreef op donderdag 4 april 2019 @ 20:03:
Ik geloof er nog steeds niets van, regel 18 is gewoon fout.
.edit: oh wacht er is nu ook een variabele Pensioenpremie. Mag dat?

O je hebt gelijk. Ik ben die haakjes vergeten. Mijn code werkt, maar de pensioenpremie wordt niet afgetrokken. Thanks. Ik heb nu die haakjes erbij gezet en nu werkt het wel.

Zet eens een brakepoint op regel 17 en kijk dan wat er gebeurd.

Brakepoint??

Ja hoor. De code klopt overigens nog steeds voor geen meter maarja, 'het werkt'.

Tel me more. Ik heb nu 4 klassen gebruikt, maar slechts twee getoond. Is dat het punt of is er iets anders aan de hand.

[Zeehond]

Verwijderd schreef op donderdag 4 april 2019 @ 20:49:
[...]

Brakepoint??

[...]

Door een breakpoint te zetten kun je code debuggen en makkelijker naar fouten zoeken. De code pauzeert op de regel waar de breakpoint staat. Je kunt dan waardes van variabeles bekijken op dat moment, de code verder later uitvoeren of een stap verder springen. Handig om je dus hiermee bekend te maken.

Tel me more. Ik heb nu 4 klassen gebruikt, maar slechts twee getoond. Is dat het punt of is er iets anders aan de hand.

Als ik de code van de Fulltime class bekijk vraag ik mij af waar je de waarde van de berekende pensioenpremie opslaat in de pensioenpremie variabele.

[Hydra]

.oisyn schreef op donderdag 4 april 2019 @ 20:34:
Welke krijgt dan voorrang?

Het is geen kwestie van voorrang. Als ik de methode aanroep via foo() dan wordt de methode aangeroepen, als ik een member foo aanspreek (zonder haakjes) wordt de member gebruikt. Je kunt geen methode aanroepen zonder haakjes in Java.

[.oisyn]

Ah je kunt natuurlijk ook geen methode referencen, of een variabele "aanroepen", dus het is nooit ambigu.

[ Voor 36% gewijzigd door .oisyn op 05-04-2019 08:41 ]

[Woy]

Maar sowieso, waarom zou je een class Fulltime maken die erft van Werknemer? Dat is IMHO helemaal geen IS-A relatie, maar ik zou een class Dienstverband maken en een werknemer heeft een Dienstverband ( HAS-A )

[martyw]

Maak voor je member variabele Pensioenpremie een setter en een getter, zoals in de link die ik hierboven gepost heb.

[Hydra]

Verwijderd schreef op donderdag 4 april 2019 @ 20:49:
Tel me more. Ik heb nu 4 klassen gebruikt, maar slechts twee getoond. Is dat het punt of is er iets anders aan de hand.

Dat is een issue ja, want als je gaat erven zit er dus 'gedrag' in die base class die de class die inherit overneemt. Als jij ons die code niet toont weten wij niet waar fouten zitten.

Waar je eerst eens mee moet beginnen is gewoon netjes de Java naamgevingen te volgen. Het verbaast me dat ze er op je school niet op hameren (bij mij was destijds die code de prullenbak ingegaan). In Java PascalCase je alleen de naam van de class. Methodes en variabelen camelCase je. Dat door elkaar gebruiken maakt je code lastig leesbaar. Ook is het wel zo netjes je code netjes te indenten; maakt het een stuk leesbaarder voor ons. Vergeet niet dat je ons vraagt tijd te investeren in je. Dat doen we natuurlijk met plezier, maar dan moet je het ons wel zo makkelijk mogelijk maken.

Waar je code een rommeltje wordt is dat je setters/getters gebruikt, maar ook methodes met de naam van de onderliggende member vars. Dus Pensioenpremie() methode en de Pensioenpremie member variabele.

Waar het dan echt misgaat is bijvoorbeeld hier:

Java:

public double Pensioenpremie() {
    return Weeksalaris * 0.05;
}

public double Totaal() {
    return Weeksalaris - Pensioenpremie;
}

In plaats van dat je in Totaal() (zou getTotaal moeten zijn bijv.) de Pensioenpremie() method (zou getPensioenPremie() moeten zijn) aanroept, gebruik je hier een Pensioenpremie (-> pensioenPremie) member var die gewoon 0.0 bevat.

Je code werkt dus (als ik het zo check) helemaal niet correct. Je Totaal() geeft een verkeerde waarde. Het compiled, maar dat betekent absoluut niet dat de code ook correct is. Je zult hier ook geen voldoende voor krijgen. Indien dit geaccepteerd wordt zou ik een andere opleiding kiezen

Nog een kleine p.s.: als je een scanner gebruikt om input te lezen (dus van System.in) moet je 'em nooit .close()-en. Het closed namelijk ook de onderliggende System.in en die kun je daarna niet meer openen.

martyw schreef op vrijdag 5 april 2019 @ 09:50:
Maak voor je member variabele Pensioenpremie een setter en een getter, zoals in de link die ik hierboven gepost heb.

Ze heeft al een getter, Pensioenpremie(), die alleen de verkeerde naam heeft. Het percentage ervan is een constante en volgens de logica heeft ze ook geen setter nodig. Haar fix om een member var (die default gewoon 0.0 is) toe te voegen zorgt misschien dat het compiled, maar nu is de uitkomt gewoon fout.

Woy schreef op vrijdag 5 april 2019 @ 08:45:
Maar sowieso, waarom zou je een class Fulltime maken die erft van Werknemer? Dat is IMHO helemaal geen IS-A relatie, maar ik zou een class Dienstverband maken en een werknemer heeft een Dienstverband ( HAS-A )

Waarschijnlijk komt dit van haar leraar. Die bedenken wel vaker rare inheritance structuren om dit soort zaken uit te leggen, vaak omdat ze er zelf ook weinig kaas van gegeten hebben. Dus het is goed mogelijk dat ze hier niet van af mag wijken.

[ Voor 25% gewijzigd door Hydra op 05-04-2019 10:12 ]

[Verwijderd]

O ik ben met mijn werkgever in gesprek om een opleiding te gaan doen. Momenteel leer ik eigenlijk alleen van online cursussen en uit boeken om alvast een voorsprongetje te nemen. Echt nog een groentje dus. Ik zit hier met Programmeren met Java voor dummies op schoot. Ik probeer alleen de dingen die ik leer te combineren ipv letterlijk de codes over te schrijven, want anders ben ik bang dat ik de logica uit de ene situatie niet kan toepassen in de andere situatie. Als ik momenteel compileer krijg ik dit:

code:

E:\mijn hobbypagina\java\javabestanden>java Salaris
Fulltime werknemer:
Naam: Marianne
Functie: recruiter
Weeksalaris: 500
Pensioenpremie: 25.0
Brutoloon: 475.0
Partime werknemer:
Naam: Pieter
Functie: tester
Uurloon: 20
Gewerkte uren: 32
Brutoloon: 640.0

Ik heb hier natuurlijk aan de gebruikerskant wel even fictieve gegevens moeten invoeren met werknemers die ik zelf bedacht heb en dit systeem werkt natuurlijk alleen maar voor een bedrijf met één fulltimer en één parttimer.

Ook is het wel zo netjes je code netjes te indenten; maakt het een stuk leesbaarder voor ons. Vergeet niet dat je ons vraagt tijd te investeren in je. Dat doen we natuurlijk met plezier, maar dan moet je het ons wel zo makkelijk mogelijk maken.

Je hebt helemaal gelijk.

In plaats van dat je in Totaal() (zou getTotaal moeten zijn bijv.) de Pensioenpremie() method (zou getPensioenPremie() moeten zijn) aanroept, gebruik je hier een Pensioenpremie (-> pensioenPremie) member var die gewoon 0.0 bevat.

Moet dit ook bij het aanmaken van een method. Ik dacht dat dit alleen moet als je een variabele die je private had gedeclareerd toch kon gebruiken, maar hier zeg ik waarschijnlijk iets heel doms. Ik zeg het echter toch, want anders leer ik er niet van.

Hier is trouwens nog de werknemer class:

code:

public class Werknemer {
private String Naam;
private String Functie;

    public void setNaam(String NaamIn) {
        Naam = NaamIn;
    }

    public String getNaam() {
        return Naam;
    }

    public void setFunctie(String FunctieIn) {
        Functie = FunctieIn;
    }

    public String getFunctie() {
        return Functie;
    }
}

Dit is de partime class:

code:

public class Partime extends Werknemer {
private double Uurloon;
private int GewerkteUren;

    public void setUurloon(double UurloonIn) {
        Uurloon = UurloonIn;
    }

    public double getUurloon() {
        return Uurloon;
    }

    public void setGewerkteUren(int GewerkteUrenIn) {
        GewerkteUren = GewerkteUrenIn;
    }

    public int getGewerkteUren() {
        return GewerkteUren;
    }

    public double Totaal() {
        return Uurloon * GewerkteUren;
    }
}

[ Voor 83% gewijzigd door Verwijderd op 09-04-2019 21:31 ]

[BertS]

Geen Javaan hier, maar even voor de wil die je toont. Komt linksom of rechtsom wel goed.

[Verwijderd]

BertS schreef op dinsdag 9 april 2019 @ 22:55:
Geen Javaan hier, maar even voor de wil die je toont. Komt linksom of rechtsom wel goed.

Dankjewel .