[java] Maakt ergens een foutje bij een class, alleen waar?

[Neptunus]

Onderstaande code heb ik geschreven, alleen nu is de uitvoer van:
System.out.println("Eigenschappen van Plank: " + p1.toString());
null voor de string en de getal waarden zijn 0.

Ik heb volgens mij ergens een foutje bij een kopeling tussen de classes. Ik ben er nu 2 uur mee bezich geweest, ik krijg het er niet uit. Wie kan mij vertellen waar ik de fout maak?

Volgens mij is de bedoeling van het programma wel duidelijk, maar als ik het even moet toelichten dan doe ik dat graag!

Java:

public class Apl{

    public static void main(String[] args) {
        Plank p1 = new Plank();
        Plank p2 = new Plank();
        Plank p3 = new Plank();

        // Voeg de verschillende planken toe
        p1.voegToe(2,3.5,5,"grenen");
        p2.voegToe(180,250,400,"grenen");
        p3.voegToe(5,8,12,"beuken");

        // Druk de waarden van p1 af
        System.out.println("Eigenschappen van Plank: " + p1.toString());
        // Druk de waarden van p2 af
        System.out.println("Eigenschappen van Plank: " + p2.toString());
        // Druk de waarden van p3 af
        System.out.println("Eigenschappen van Plank: " + p3.toString());
    }
}

class Plank {
    private double Lengte;
    private double Breedte;
    private double Dikte;
    private double Volume;
    private String Houtsoort;

    public void voegToe(double lengte, double breedte, double dikte, String houtsoort) {
        lengte = Lengte;
        breedte = Breedte;
        dikte = Dikte;
        houtsoort = Houtsoort;
    }


    public void Volume(){
        Volume = Lengte * Breedte * Dikte;
    }

    public String toString() {
        String result = "Houtsoort:" + Houtsoort + "Lengte" + Lengte + ",Breedte:" + Breedte + ",Dikte:" + Dikte + ", Volume:" + Volume + "";
        return result;
    }
}

[ Voor 6% gewijzigd door Neptunus op 16-09-2003 13:57 ]

[whoami]

Moet je in die toString() method, de members lengte, breedte, etc.... niet naar een string converten vooraleer je ze concateneert tot de string result?

[Janoz]

toekenning is verkeerd om

Het is trouwens ook minder done om je variabelen namen met een hoofdletter te laten beginnen. Dit wordt normaliteir voor class name gebruikt. Instanties hoor je met een kleine letter te schrijven.

Om toch onderscheid temaken kun je 'this.lengte = lengte;' oid gebruiken.

[ Voor 85% gewijzigd door Janoz op 16-09-2003 13:54 ]

[Gerco]

In je voegToe() method assign je de waarden van de class variabelen aan de parameters zelf, doe dat eens andersom. Waarom gebruik je trouwens geen constructor om dat te doen? Dan kun je gewoon dit schrijven:

Java:

  p1 = new Plank(100, 20, 30, "Grenen");

En als je toch bezig bent, geef de method Volume een return type en return ook echt iets. Compileert je code wel? Probeer het zo:

Java:

public double Volume() {
  return Lengte * Breedte * Hoogte;
}

Veel mensen vinden het ook een goed idee om je class variabelen en/of je parameters te prefixen met een scope prefix (_ voor class bijv) en het type van de variabele (i voor int, s voor String, etc).

[ Voor 122% gewijzigd door Gerco op 16-09-2003 13:57 ]

[Erkens]

whoami schreef op 16 September 2003 @ 13:51:
Moet je in die toString() method, de members lengte, breedte, etc.... niet naar een string converten vooraleer je ze concateneert tot de string result?

hoeft niet, doet java automagisch voor je.

[Neptunus]

Gerco schreef op 16 september 2003 @ 13:52:

Ik denk dat ik het nut van de constructor snap. Alleen krijg ik het nog niet goed. Komt denk ik omdat ik iets in het hoofd had, en nu is alles een beetje omgegooit.

Bij mijn eerste code greeg ik geen foutjes bij het compileren, bij de onderstaande code wel.

Ik heb nu het volgend;

Java:

class Plank {
    private double Lengte;
    private double Breedte;
    private double Dikte;
    private double Volume;
    private String Houtsoort;


    public void voegToe(double lengte, double breedte, double dikte, String houtsoort) {
        Lengte = lengte;
        Breedte = breedte;
        Dikte = dikte;
        Houtsoort = houtsoort;
    }


    public double Volume() {
        return Lengte * Breedte * Dikte;
    }

    public String toString() {
        String result = "Houtsoort:" + Houtsoort + "Lengte" + Lengte + ",Breedte:" + Breedte + ",Dikte:" + Dikte + ", Volume:" + Volume + "";
        return result;
    }

    public void maakPlank() {
        // Druk de waarden van p1 af
        System.out.println("Eigenschappen van Plank: " + p1.toString());
    }
}

public class Apl{

    public static void main(String[] args) {
        // Voeg de verschillende planken toe
        p1 = new Plank(2, 3.5, 5, "grenen");
        p1.maakPlank();

        p2 = new Plank(180, 250, 400, "grenen");
        p2.maakPlank();

        p3 = new Plank(5, 8, 12, "beuken");
        p3.maakPlank();
    }
}

[Eskimootje]

public void maakPlank() {
// Druk de waarden van p1 af
System.out.println("Eigenschappen van Plank: " + p1.toString());
}


p1 is helemaal niet beschikbaar daar, kun je dus niet daar aanroepen.

[Neptunus]

Eskimootje schreef op 16 September 2003 @ 14:21:
public void maakPlank() {
// Druk de waarden van p1 af
System.out.println("Eigenschappen van Plank: " + p1.toString());
}


p1 is helemaal niet beschikbaar daar, kun je dus niet daar aanroepen.

Als ik die p1 weg haal dan ben ik nog steeds iets niet goed in de constructor. Nu rest mij de vraag hoe kan ik het beste de constructor opzetten?

[Woy]

Je roept nu ook wel een constructor aan met 4 parameters maar je hebt helemaal geen constructor gedefineerd. Dit doe je op de volgende manier

Java:

public Plank( Type parameter1 [ , Type parameterN .... ] )
{
}
//hier dus
public Plank(double lengte, double breedte, double dikte, String houtsoort)
{
    //assign hier je parameters aan je class vars
}

[ Voor 25% gewijzigd door Woy op 16-09-2003 14:30 ]

[RayNbow]

Java:

    public void maakPlank() { 
        // Druk de waarden van p1 af 
        System.out.println("Eigenschappen van Plank: " + p1.toString()); 
    } 

Moet dus worden:

Java:

    public void maakPlank() { 
        // Druk de waarden van p1 af 
        System.out.println("Eigenschappen van Plank: " + this.toString()); 
    } 

Edit: ik zie dus dat ik een beetje traag ben met typen

[ Voor 12% gewijzigd door RayNbow op 16-09-2003 14:29 ]

[vogeltje]

De constructor moet je gewoon de naam van class geven, dus hierbij gewoon:

Java:

public Plank(double lengte, double breedte,  double dikte, String houtsoort){
  this.Lengte = lengte
  this.Breedte = breedte
  this.Dikte = dikte
  this.Houtsoort = houtsoort
}

en dan maak je gewoon nog de benodigde methods aan, (evt extra GET/SET) en de 'Print'-method:

Java:

public printPlank(){
  System.out.println("lengte is: " +this.Lengte+ " breedte is:" +this.Breedte+ "   dikte is: "+this.Dikte+ " houtsoort is: "+this.Houtsoort);
}

En dank kan je vanuit een andere klasse netjes je instanties aanmaken:

Java:

public static void main(String []args){
  Plank p1 = new Plank(2.0,1.0,3.0,"grenen");
  Plank p2 = new Plank(5.0,0.1,2.0,"grenen");
  Plank p3 = new Plank(9.0,1.3,7.0,"grenen");

  // en nu kan je hier vrolijk de printPlank aanroepen, er bestaan nu 3 instanties met elk hun eigen gegevens (lengte, breedte, dikte en houtsoort)
  p1.printPlank();
  p2.printPlank();
  p3.printPlank();

}

En probeer nog wat aan je naamgeving van de attributen & methods te doen, niet String Houtsoort maar iets van sHoutsoort ofzo (prefix)

Tenminste, dit is wat ik zou doen. En lijkt mij een stuk efficienter dan die van jou

[ Voor 102% gewijzigd door vogeltje op 16-09-2003 14:49 ]

[Verwijderd]

public String toString() gaat ook nog niet goed.

je hebt de variabele Volume nog geen waarde gegeven

[Neptunus]

Ben er bijna heb ik het idee. Krijg nog een fout bij het compileren.

D:\zooi\Apl.java:35: invalid method declaration; return type required
public printPlank(){
^
1 error

Java:

public class Apl{

    public static void main(String[] args) {
        // Voeg de verschillende planken toe
        Plank p1 = new Plank(2, 3.5, 5, "grenen");
        Plank p2 = new Plank(180, 250, 400, "grenen");
        Plank p3 = new Plank(5, 8, 12, "beuken");

        //ptint de planken
        p1.printPlank();
        p2.printPlank();
        p3.printPlank();
        }
}

public class Plank {
    private double dLengte;
    private double dBreedte;
    private double dDikte;
    private String sHoutsoort;

    private double dVolume;

    public void Plank(double dlengte, double dbreedte, double ddikte, String shoutsoort) {
        this.dLengte = dlengte;
        this.dBreedte = dbreedte;
        this.dDikte = ddikte;
        this.sHoutsoort = shoutsoort;
    }

    public double dVolume() {
         return dLengte * dBreedte * dDikte;
    }

    public printPlank(){
        System.out.println("houtsoort is: " +this.Houtsoort+ " lengte is: " +this.Lengte+ " breedte is: " +this.Breedte+ " dikte is: "+this.Dikte+ " volume is: "+dVolume);
    }
}

[ Voor 5% gewijzigd door Neptunus op 16-09-2003 14:55 ]

[vogeltje]

Een constructor heeft nooit een return type, je moet het zelfs niet eens neerzetten dus niet:

Java:

public VOID Plank(.......)
 //.........
}
 // maar

public Plank(....)
  //.......
}

constructor zonder VOID dus !

En btw, bij printPlank dus: public void printPlank(...)
Daar kreeg je ook de foutmelding over

// edit: En dan zal die werken!

[ Voor 32% gewijzigd door vogeltje op 16-09-2003 15:01 ]

[Neptunus]

Het is gelukt! Mede tweakers bedankt. Het nut van een constructor is nu veel duidelijker.

Zijn er nog zo tips die bij jullie opkomen zodat ik de volgende keer niet meer zo hoef te vragen?

Java:

public class Apl{

    public static void main(String[] args) {
        // Voeg de verschillende planken toe
        Plank p1 = new Plank(2, 3.5, 5, "grenen1");
        Plank p2 = new Plank(180, 250, 400, "grenen2");
        Plank p3 = new Plank(5, 8, 12, "beuken1");

        //ptint de planken
        p1.printPlank();
        p2.printPlank();
        p3.printPlank();
        }
}

class Plank {
    private double dLengte;
    private double dBreedte;
    private double dDikte;
    private String sHoutsoort;

    private double dVolume;

    public Plank(double dlengte, double dbreedte, double ddikte, String shoutsoort) {
        this.dLengte = dlengte;
        this.dBreedte = dbreedte;
        this.dDikte = ddikte;
        this.sHoutsoort = shoutsoort;

        this.dVolume = dLengte * dBreedte * dDikte;
    }

    public void printPlank(){
    System.out.println("Houtsoort is: "+this.sHoutsoort);
    System.out.println("lengte is: " +this.dLengte);
    System.out.println("breedte is:" +this.dBreedte);
    System.out.println("dikte is: "+this.dDikte);
    System.out.println("volume is: "+this.dVolume);
    System.out.println("");
    }
}

[Voutloos]

Java:

    public void printPlank(){
    System.out.println("Houtsoort is: "+this.sHoutsoort);
    System.out.println("lengte is: " +this.dLengte);
    System.out.println("breedte is:" +this.dBreedte);
    System.out.println("dikte is: "+this.dDikte);
    System.out.println("volume is: "+this.dVolume);
    System.out.println("");
    }

Geen echt grote tip, maar die 'this.' is niet nodig binnen deze methode. Leerzaam om zelf te bedenken waarom.

Trouwens (maar dat is niet het antwoord dat je moet hebben op mijn vraag) ben je inconsequent met je namen voor Strings mbt hoofdletters. Is ook niet echt fout, maar wel slordig. Als je code uitgebreider wordt, wordt het ook lastiger als je naamgeving niet helemaal waterdicht is namelijk.

Ook ben je inconsequent met het inspringen bij printPlank().

[ Voor 10% gewijzigd door Voutloos op 16-09-2003 15:27 ]

[vogeltje]

Neptunus schreef op 16 September 2003 @ 15:17:

Zijn er nog zo tips die bij jullie opkomen zodat ik de volgende keer niet meer zo hoef te vragen?

Veel oefenen met Java, als je et eenmaal goed doorhebt dan ben je er aan verslaafd Was ik wel tenminste

En idd, mbt. this. in de methode kent ie maar 1 member breedte, lengte etc. Daar is het dus niet noodzakelijk om expliciet naar het huidige object te verwijzen

[ Voor 20% gewijzigd door vogeltje op 16-09-2003 15:27 ]

[Glimi]

Neptunus schreef op 16 september 2003 @ 15:17:
Het is gelukt! Mede tweakers bedankt. Het nut van een constructor is nu veel duidelijker.

Zijn er nog zo tips die bij jullie opkomen zodat ik de volgende keer niet meer zo hoef te vragen?

Ja.
Als je begint met programmeren zijn boeken vaak onmisbaar (voor enkele voorbeelden zie P&W FAQ - Java). Zo'n boek zal over het algemeen correcte code bevatten en goed duidelijk uitleggen waarom de constructie zo in elkaar zit.

Als ik jouw vraag zo lees, mis je gewoon nog wat kennis welke het best op te doen is uit verschillende boeken. Zo leer je het begrijpen voordat je echt gaat beginnen ipv je jezelf allemaal dingen aanleert die niet altijd correct hoeven te zijn en wat zit aan te prutsen

[Erkens]

Voutloos schreef op 16 september 2003 @ 15:25:
[...]

Geen echt grote tip, maar die 'this.' is niet nodig binnen deze methode. Leerzaam om zelf te bedenken waarom.

Zelf gebruik ik bijna altijd "this" juist om duidelijkheid te scheppen van waar dat "ding" vandaan komt.

[Gerco]

Erkens schreef op 16 September 2003 @ 15:27:
Zelf gebruik ik bijna altijd "this" juist om duidelijkheid te scheppen van waar dat "ding" vandaan komt.

Daar is nu die scope prefix voor, _ voor member variables bijv.

[Voutloos]

Erkens schreef op 16 September 2003 @ 15:27:
[...]

Zelf gebruik ik bijna altijd "this" juist om duidelijkheid te scheppen van waar dat "ding" vandaan komt.

OK, maar dan is het bij jou een bewuste keuze. Ik wil gewoon nagaan of TS begrijpt waarom.

[Jochem Knoops]

Neptunus schreef op 16 September 2003 @ 15:17:
Het is gelukt! Mede tweakers bedankt. Het nut van een constructor is nu veel duidelijker.

Zijn er nog zo tips die bij jullie opkomen zodat ik de volgende keer niet meer zo hoef te vragen?

Java:

public class Apl{

    public static void main(String[] args) {
        // Voeg de verschillende planken toe
        Plank p1 = new Plank(2, 3.5, 5, "grenen1");
        Plank p2 = new Plank(180, 250, 400, "grenen2");
        Plank p3 = new Plank(5, 8, 12, "beuken1");

        //ptint de planken
        p1.printPlank();
        p2.printPlank();
        p3.printPlank();
        }
}

class Plank {
    private double dLengte;
    private double dBreedte;
    private double dDikte;
    private String sHoutsoort;

    private double dVolume;

    public Plank(double dlengte, double dbreedte, double ddikte, String shoutsoort) {
        this.dLengte = dlengte;
        this.dBreedte = dbreedte;
        this.dDikte = ddikte;
        this.sHoutsoort = shoutsoort;

        this.dVolume = dLengte * dBreedte * dDikte;
    }

    public void printPlank(){
    System.out.println("Houtsoort is: "+this.sHoutsoort);
    System.out.println("lengte is: " +this.dLengte);
    System.out.println("breedte is:" +this.dBreedte);
    System.out.println("dikte is: "+this.dDikte);
    System.out.println("volume is: "+this.dVolume);
    System.out.println("");
    }
}

nou ja nog een java tip. probeer altijd een toString() functie te schrijven.

dus iets van

code:

class Plank {
/**
* KNIP
*/

public String toString(){

String temp = "";
temp += "Houtsoort is: "+this.sHoutsoort;
temp += "lengte is: " +this.dLengte;
temp += "breedte is:" +this.dBreedte;
temp +="dikte is: "+this.dDikte;
temp +="volume is: "+this.dVolume;
return temp;

}
}

[ Voor 7% gewijzigd door Jochem Knoops op 16-09-2003 15:35 ]

[Voutloos]

JochemKnoops schreef op 16 September 2003 @ 15:34:
[...]


nou ja nog een java tip. probeer altijd een toString() functie te schrijven.

Mee eens.

En die code zou ik, maar dat is subjectief zo schrijven:

Java:

public String toString(){
   return "Houtsoort is: " + this.sHoutsoort +
      "lengte is: " + this.dLengte + 
      "breedte is:" + this.dBreedte +
      "dikte is: " + this.dDikte +
      "volume is: " + this.dVolume;
}

Schijnt iets efficienter te zijn (en ik vind het leesbaarder).

_{en geen opmerkingen over this. graag}

[Erkens]

Gerco schreef op 16 September 2003 @ 15:30:
[...]

Daar is nu die scope prefix voor, _ voor member variables bijv.

kan ja, zelf ben ik daar geen voorstander van, maar hoe wil je dat doen bij methoden ed?
ja ik weet dat het niet nodig is, maar het maakt de code wel leesbaarder vind ik.

Voutloos schreef op 16 september 2003 @ 15:46:
[...]

Mee eens.

En die code zou ik, maar dat is subjectief zo schrijven:

Java:

..

Schijnt iets efficienter te zijn (en ik vind het leesbaarder).

_{en geen opmerkingen over this. graag}

ik zou op zijn minst spaties ertussen zetten
verder gebruik ik haakjes in al mijn "toString"-s

[ Voor 39% gewijzigd door Erkens op 16-09-2003 15:50 ]

[Glimi]

JochemKnoops schreef op 16 September 2003 @ 15:34:
nou ja nog een java tip. probeer altijd een toString() functie te schrijven.

toStrings() hebben vaak alleen een functie met debuggen. Als je je toString() werkelijk gaat gebruiken voor weergave van het object, dan leg je het imho te statisch vast.
Wil je de volgende keer een Persoon niet als "H. de Vries" afdrukken ipv "Henk de Vries"?

Erkens schreef op 16 September 2003 @ 15:48:
kan ja, zelf ben ik daar geen voorstander van, maar hoe wil je dat doen bij methoden ed?
ja ik weet dat het niet nodig is, maar het maakt de code wel leesbaarder vind ik.
(...)

Wat wil je doen bij methodes?
Verder ivind ik het expliciet vermelden van het impliciete eerste argument een beetje tegen de princiepes die C-syntax talen, waarbij de syntax vaak redelijk cryptisch te noemen is

[ Voor 37% gewijzigd door Glimi op 16-09-2003 16:05 ]

[Voutloos]

offtopic:

Glimi schreef op 16 September 2003 @ 16:01:
[...]
toStrings() hebben vaak alleen een functie met debuggen. Als je je toString() werkelijk gaat gebruiken voor weergave van het object, dan leg je het imho te statisch vast.
Wil je de volgende keer een Persoon niet als "H. de Vries" afdrukken ipv "Henk de Vries"?

Ja maar daarom moet je ook accessor methodes maken zoals getVoornaam en getAchternaam in jouw voorbeeld

offtopic:

[JeroenTheStig]

Erkens schreef op 16 September 2003 @ 13:52:
[...]

hoeft niet, doet java automagisch voor je.

offtopic:
is dat een bewuste typefout?

[Glimi]

Voutloos schreef op 16 September 2003 @ 16:04:

offtopic:
Ja maar daarom moet je ook accessor methodes maken zoals getVoornaam en getAchternaam in jouw voorbeeld

offtopic:
Maar dat stookt nogal met je toString(), daarmee geef je net aan wat de string representatie van het object zou moeten zijn

[Voutloos]

offtopic:
Ik paste alleen maar andermans code aan. Maar is dubbelop dan erg?

En ik hou nu op met de discussies over deze subjectieve dingetjes.

[Gerco]

Erkens schreef op 16 September 2003 @ 15:48:
kan ja, zelf ben ik daar geen voorstander van, maar hoe wil je dat doen bij methoden ed?

Dan doe je het niet, methodes zijn het laagste niveau waarop je variabelen declareert en die hebben als zodanig geen scope prefix nodig.