[Java] Arraylist gegevens in een andere classe

[barakuda]

Hallo, ik ben voor mijn eindopdracht voor Java bezig met een taxiritten registratiesysteem...
Ik heb een class Rit aangemaakt waarin de ritten in een arraylist opgeslagen worden.
In de tweede class Taxibedrijf wil ik overzichten maken zoals totaal inkomen, gemiddelde afstand en langste afstand..

Ik krijg de arraylist van class Rit niet in de class Taxibedrijf!!

Dit is mijn class Rit

Java:

 
import java.util.ArrayList;
/**
 * Eindopdracht Inleiding programmeren Java
 * Opdracht 1 - Taxikosten
 * @version v1
 */

public class Rit
{
    // Prijs per gereden kilometer
    private double prijsGeredenKM;
    // Toeslag per gereden minuut
    private double prijsMinuutToeslag;
    // Weekend toelsag percentage van vrijdag 22.00 tot maandag 07.00
    private double prijsWeekendToeslag;
    // ArrayList voor taxirit
    public ArrayList <Integer> begintijden = new ArrayList<Integer>();
    public ArrayList <Integer> eindtijden = new ArrayList<Integer>();
    public ArrayList <String> dagen = new ArrayList<String>();
    public ArrayList <Integer> afstanden = new ArrayList<Integer>();

    /**
     * Constructor for objects of class Taxirit
     */
    public Rit()
    {
        prijsGeredenKM = 0.50;
        prijsMinuutToeslag = 0.17;
        prijsWeekendToeslag = 0.15;
    }
    
    /**
     * Method newTaxirit
     *
     * @param begintijd bv. voor 9:15, 915 invullen
     * @param eindtijd bv. voor 9:15, 915 invullen
     * @param dag "Maandag", "Dinsdag", "Woensdag", "Donderdag",
     * @param dag "Vrijdag", "Zaterdag", "Zondag"
     * @param afstand in kilometers opgeven
     */
    public void newTaxirit(int begintijd, int eindtijd, String dag, int afstand)
    {
        begintijden.add(begintijd);
        eindtijden.add(eindtijd);
        dagen.add(dag);
        afstanden.add(afstand);
    }
    
    /**
     * Method berekenVerschuldigdBedrag
     *
     * @param taxiritNummer parameter
     * @return The return value
     */
    public double berekenVerschuldigdBedrag(int index)
    {
        int beginUurInt = begintijden.get(index);
        int beginUur = beginUurInt / 100 * 60;
        int beginMinuut = beginUurInt % 100;
        int begin = beginUur + beginMinuut;
        
        int eindUurInt = eindtijden.get(index);
        int eindUur = eindUurInt / 100 * 60;
        int eindMinuut = eindUurInt % 100;
        int eind = eindUur + eindMinuut;
        
        int minuten = eind - begin;
        
        double minuutToeslag = prijsMinuutToeslag * minuten;
        
        double prijs = prijsGeredenKM * afstanden.get(index) + minuutToeslag;
        
        if((dagen.get(index) == "Vrijdag" && beginUurInt >= 2200) || dagen.get(index) == "Zaterdag" || dagen.get(index) == "Zondag" || (dagen.get(index) == "Maandag" && beginUurInt <= 700))
        {
            prijs = prijs * prijsWeekendToeslag + prijs;
            prijs = prijs * 1.0;
            prijs = Math.round(prijs * 100.0)/100.0;
            
            return prijs;
        }
        
        prijs = Math.round(prijs * 100.0)/100.0;

        return prijs;
    }
    
    /**
     * Method showVerschuldigdBedrag
     *
     * @param taxiritNummer A parameter
     */
    public void showVerschuldigdBedrag()
    {
        //Clear terminal screen
        System.out.print("\f");
        
        int index = 0;
        for(Integer begintijd : begintijden)
        {
            System.out.println("***** ***** ***** ***** ****** *****");
            System.out.println("Ritnummer..........: " + (index + 1));
            System.out.println("Begintijd..........: " + begintijd);
            System.out.println("Eindtijd...........: " + eindtijden.get(index));
            System.out.println("Dag................: " + dagen.get(index));
            System.out.println("Afstand............: " + afstanden.get(index) + " km");
            System.out.println("Verschuldigd bedrag: € " + berekenVerschuldigdBedrag(index));
            index++;
        }       
    }
}

[Herko_ter_Horst]

Je aanpak klopt niet. Rit moet de informatie bevatten van 1 rit. Taxibedrijf moet dan een ArrayList (of andere collection) hebben waarin de Ritten zitten.

Java:

class TaxiBedrijf {
  private List<Rit> ritten = new ArrayList<Rit>();

  public void rijRitten() {
    Rit rit1 = new Rit(....);
    ritten.add(rit1);
    Rit rit2 = new Rit(....);
    ritten.add(rit2);
  }

  public static void main(String[] args) {
    TaxiBedrijf tb = new TaxiBedrijf();
    tb.rijRitten();
  }
}

class Rit {
  private int beginTijd;
  private int eindTijd;
  private String dag;
  private int afstand;

  public Rit(.....) {
   etc..
  }
}

[ Voor 62% gewijzigd door Herko_ter_Horst op 02-04-2011 15:03 ]

[barakuda]

Herko_ter_Horst schreef op zaterdag 02 april 2011 @ 14:58:
Je aanpak klopt niet. Rit moet de informatie bevatten van 1 rit. Taxibedrijf moet dan een ArrayList (of andere collection) hebben waarin de Ritten zitten.

Oke bedankt, ik probeer het aan te passen..

[Jaap-Jan]

Zou je de begin- en einddtijd niet beter op kunnen slaan als een datum?

[Herko_ter_Horst]

Nog een paar tips:

• voor datum/tijd gerelateerde zaken, heeft Java de klasse java.util.Date. Mogelijk kun je hier iets mee, maar mogelijk gaat dit nog te ver voor deze opdracht
• voor een lijstje vaste waarden heeft Java enums. Als je geen Date gaat gebruiken, zou je kunnen overwegen de dagen als enum te representeren
• ik zou de waarden die gebruikt worden voor de berekening van de prijs één keer opslaan in de TaxiBedrijf klassen en ze doorgeven bij het bereken van de prijs van een Rit (de prijsGeredenKilometer, prijsMinuutToeslag en prijsWeekendToeslag)

offtopic:
Ik heb het topic even gerapporteerd zodat een modje het naar het juiste sub-forum (Programming) kan verplaatsen.

[ Voor 65% gewijzigd door Herko_ter_Horst op 02-04-2011 15:15 ]

[MueR]

Herko_ter_Horst schreef op zaterdag 02 april 2011 @ 15:04:

offtopic:
Ik heb het topic even gerapporteerd zodat een modje het naar het juiste sub-forum (Programming) kan verplaatsen.

Dat heb ik dan maar even gedaan

Ik mis wel een beetje de "waarom werkt $x niet" vraag, de daaropvolgende uitleg wat er geprobeerd is om het zelf op te lossen en enkel de relevante code. Ik vind 110 regels code wat aan de ruimte kant, maar ik doe niets met Java, dus misschien kan het niet anders in die gekke taal Dus slaven84, wil je misschien wat duidelijker zijn waar je vast zit?

[ Voor 5% gewijzigd door MueR op 02-04-2011 15:50 ]

[barakuda]

Dit heb ik nu gemaakt... hoe moet ik Rit rit1 = new Rit(); ritten.add(rit1); aanpassen zodat de gegevens erin komen??

Java:

class Rit {
  private int beginTijd;
  private int eindTijd;
  private String ritDag;
  private int ritAfstand;

    public void Rit(int begintijd, int eindtijd, String dag, int afstand)
    {
        beginTijd = begintijd;
        eindTijd = eindtijd;
        ritDag = dag;
        ritAfstand = afstand;
    }
}

import java.util.ArrayList;

class TaxiBedrijf 
{
  private ArrayList<Rit> ritten = new ArrayList<Rit>();

  public void rijRitten() 
  {
    Rit rit1 = new Rit();
    ritten.add(rit1);
  }

  public static void main(String[] args) 
  {
    TaxiBedrijf tb = new TaxiBedrijf();
    tb.rijRitten();
  }
} 

[Jaap-Jan]

Iedere rit toevoegen, zoals je nu doet in rijRitten(), lijkt me. Maar dan wel ervoor zorgen dat je alle argumenten die je wilt hebben in je constructor ook meegeeft als argument (op regel 24 moet je de argumenten toevoegen).

Overigens kun je TaxiBedrijf beter een method ritToevoegen() geven, waarbij je de mogelijkheid geeft om een rit toe te voegen aan de lijst.

[barakuda]

Jaap-Jan schreef op zaterdag 02 april 2011 @ 16:14:
Iedere rit toevoegen, zoals je nu doet in rijRitten(), lijkt me. Maar dan wel ervoor zorgen dat je alle argumenten die je wilt hebben in je constructor ook meegeeft als argument (op regel 24 moet je de argumenten toevoegen).

Overigens kun je TaxiBedrijf beter een method ritToevoegen() geven, waarbij je de mogelijkheid geeft om een rit toe te voegen aan de lijst.

Ik begrijp het niet meer.. elke keer als ik rijRitten() uitvoer wordt de arraylist ritten met 1 verhoogt maar de gegevens van beginTijd, eindTijd, ritDag en ritAfstand blijven leeg...

[Jaap-Jan]

Ik zit niet op te letten (en jij ook niet ):

Java:

    public void Rit(int begintijd, int eindtijd, String dag, int afstand)

Hier doe je iets fout, dit zou een constructor moeten zijn, maar dat is het niet. Als je dit fixt, dan kun je weer verder.

[ Voor 3% gewijzigd door Jaap-Jan op 02-04-2011 17:22 ]

[Verwijderd]

Als je dat fixt, zie je ook direct waarom begintijd e.d. niet gezet worden, want je geeft ze niet mee aan de rit.
Dat had een beetje debugger je ook kunnen vertellen

[barakuda]

Klopt dit nu?

Java:

class Rit {
  private int beginTijd;
  private int eindTijd;
  private String ritDag;
  private int ritAfstand;
  
  public Rit()
    {
        beginTijd = 0;
        eindTijd = 0;
        ritDag = "";
        ritAfstand = 0;
    }

    public void newRit(int begintijd, int eindtijd, String dag, int afstand)
    {
        beginTijd = begintijd;
        eindTijd = eindtijd;
        ritDag = dag;
        ritAfstand = afstand;
    }
}

[apokalypse]

http://en.wikibooks.org/w...gramming/Defining_Classes

[Verwijderd]

En http://download.oracle.co.../javaOO/constructors.html

En doe jezelf een plezier zoals hierboven gesuggereerd. Als je toch bij een "dag" wil blijven ipv een datum, maak dan een Enum aan met daarin de 7 weekdagen

[barakuda]

Verwijderd schreef op zaterdag 02 april 2011 @ 18:45:
En http://download.oracle.co.../javaOO/constructors.html

En doe jezelf een plezier zoals hierboven gesuggereerd. Als je toch bij een "dag" wil blijven ipv een datum, maak dan een Enum aan met daarin de 7 weekdagen

In mijn opdracht staat dat er van een correcte invoer wordt uit gegaan en zoals je ziet ben ik nog niet zo ver met Java....
Volgen die link van jou zou mijn code dan zo moeten uitzien of?? want elk rit heeft maar één begintijd, eindtijd....

Java:

class Rit {
  private int beginTijd;
  private int eindTijd;
  private String ritDag;
  private int ritAfstand;
  
  public Rit(int begintijd, int eindtijd, String dag, int afstand)
    {
        beginTijd = begintijd;
        eindTijd = eindtijd;
        ritDag = dag;
        ritAfstand = afstand;
    }
}

[woutertje]

aangezien je instantievariabelen dezelfde naam hebben als de variabelen die je in je constructor gebruikt moet je in de constructor "this." voor de variabelen aan de linkerkant van het '=' teken plaatsen. Nu gebeurt er namelijk niets.

Edit: ik zit niet op te letten, je doet iets met de hoofdletters. Dat is sowieso een manier die ik je afraad te gebruiken.

Edit2: de code ziet er nu wel goed uit, je hebt alleen geen getters en setters dus je kunt er verder nog niets mee.

[ Voor 35% gewijzigd door woutertje op 02-04-2011 19:03 ]

[barakuda]

woutertje schreef op zaterdag 02 april 2011 @ 19:02:
aangezien je instantievariabelen dezelfde naam hebben als de variabelen die je in je constructor gebruikt moet je in de constructor "this." voor de variabelen aan de linkerkant van het '=' teken plaatsen. Nu gebeurt er namelijk niets.

Edit: ik zit niet op te letten, je doet iets met de hoofdletters. Dat is sowieso een manier die ik je afraad te gebruiken.

Edit2: de code ziet er nu wel goed uit, je hebt alleen geen getters en setters dus je kunt er verder nog niets mee.

Ik heb het aangepast. Nu wil ik elk rit in een classe TaxiBedrijf aanroepen en daar in een arraylist zetten...

Java:

class Rit {
  private int beginTijd;
  private int eindTijd;
  private String ritDag;
  private int ritAfstand;
  
  public Rit(int begintijd, int eindtijd, String dag, int afstand)
    {
        this.beginTijd = begintijd;
        this.eindTijd = eindtijd;
        this.ritDag = dag;
        this.ritAfstand = afstand;
    }
}

Java:

import java.util.ArrayList;

class TaxiBedrijf 
{
  private ArrayList<Rit> ritten = new ArrayList<Rit>();

  public void rijRitten() 
  {
    Rit rit1 = new Rit();                    // cannot find symbol constructor Rit()
    ritten.add(rit1);
  }

  public static void main(String[] args) 
  {
    TaxiBedrijf tb = new TaxiBedrijf();
    tb.rijRitten();
  }
} 

[woutertje]

Lijkt me eerder zoiets:

Java:

class Rit {
  private int beginTijd;
  private int eindTijd;
  private String ritDag;
  private int ritAfstand;
  
  public Rit(int beginTijd, int eindTijd, String ritDag, int ritAfstand)
    {
        this.beginTijd = beginTijd;
        this.eindTijd = eindTijd;
        this.ritDag = ritDag;
        this.ritAfstand = ritAfstand;
    }
  public int getBeginTijd() {
   return beginTijd;
  }
  public int getEindTijd() {
   return eindTijd;
  }
  public int getRitDag() {
   return ritDag;
  }
  public getRitAfstand() {
   return ritAfstand
  }
}

Java:

import java.util.ArrayList;

class TaxiBedrijf 
{
  private ArrayList<Rit> ritten = new ArrayList<Rit>();

  public void rijRitten() 
  {
    Rit rit1 = new Rit(100, 200, "maandag", 25);                    // cannot find symbol constructor Rit()
    ritten.add(rit1);
  }

  public static void main(String[] args) 
  {
    TaxiBedrijf tb = new TaxiBedrijf();
    tb.rijRitten();
  }
} 

Ik weet trouwens niet met wat voor'n opleiding of iets dergelijks je bezig bent maar ik raad je aan om het boek wat je voor dat vak gekocht hebt eens te lezen.

[ Voor 7% gewijzigd door woutertje op 02-04-2011 19:19 ]

[Herko_ter_Horst]

Volgens mij wordt het nu inderdaad tijd om je studiemateriaal erbij te pakken, want als dit je eindopdracht is en je weet niet hoe je een object maakt met een constructor en hoe parameters werken, dan vallen we iets te ver terug in "maak mijn huiswerk".

Iets meer eigen inzet dus. Wat "cannot find symbol" betekent, had je makkelijk even kunnen opzoeken.

[ Voor 19% gewijzigd door Herko_ter_Horst op 02-04-2011 19:23 ]

[LinuX-TUX]

slaven84 schreef op zaterdag 02 april 2011 @ 19:13:
[...]


Ik heb het aangepast. Nu wil ik elk rit in een classe TaxiBedrijf aanroepen en daar in een arraylist zetten...

Rit rit1 = new Rit(); // cannot find symbol constructor Rit()

Aanvullende tip:
- Ga in het engels coden / documenteren. Nu zie je het nut er nog niet van in, maar uiteindelijk is het vele malen leesbaarder

Daarnaast lijkt me dat de foutmelding redelijk vanzelfsprekend is. Kan geen constructor vinden die er zo uit ziet: Rit() <> met andere woorden, waar is de constructor "public Rit()" gebleven. We weten dat je een "public Rit(met heel veel argumenten)" constructor hebt gemaakt, maar die initialiseer je niet. Je initialiseerd "Rit rit1 = new Rit()" ... en die bestaat niet.

Nog een tip:
- Pak een IDE erbij. Dat scheelt een hoop klopwerk met platte pojo's en voorkomt dat je deze tijdrovende foutjes maakt. (anderzijds FOEI natuurlijk dat je dit niet netjes onder de knie hebt)
Zou bijvoorbeeld Netbeans pakken. Dan maak je een java classe rit, daar definieer je alle private properties. Vervolgens druk je op Alt + Insert en zeg je "Getters & Setters". Selecteer je allemaal, heb je binnen 30 seconden een compleet werkend objectje

edit:

@hieronder: thnx voor de tip, klasse van je

[Verwijderd]

Het is class of klasse, niet classe.

[barakuda]

Het is gelukt

Java:

class Rit {
  private int beginTijd;
  private int eindTijd;
  private String ritDag;
  private int ritAfstand;
  
    public Rit(int beginTijd, int eindTijd, String ritDag, int ritAfstand)
    {
        this.beginTijd = beginTijd;
        this.eindTijd = eindTijd;
        this.ritDag = ritDag;
        this.ritAfstand = ritAfstand;
    }
    
    public int getBeginTijd() 
    {
        return beginTijd;
    }
    public int getEindTijd() 
    {
        return eindTijd;
    }
    public String getRitDag() 
    {
        return ritDag;
    }
    public int getRitAfstand() 
    {
        return ritAfstand;
    }
}

Java:

import java.util.ArrayList;

class TaxiBedrijf 
{
  private ArrayList<Rit> ritten;
  
  public TaxiBedrijf()
  {
      ritten = new ArrayList<Rit>();
  }

  public void ritToevoegen(Rit taxirit) 
  {
    ritten.add(taxirit);
  }
}

[barakuda]

Weet iemand hoe ik nu de begintijd, eindtijd... uit de ArrayList ritten krijg??

[woutertje]

code:

ritten.get(0).getEindTijd();

Zoals gezegd is dit de absolute basis van Java en dit zou je dus gewoon moeten weten.

[barakuda]

woutertje schreef op zondag 03 april 2011 @ 14:02:

code:

ritten.get(0).getEindTijd();

Zoals gezegd is dit de absolute basis van Java en dit zou je dus gewoon moeten weten.

Bedankt.. ben ook absolute beginner wat java betreft

[Hydra]

slaven84 schreef op zondag 03 april 2011 @ 14:04:
Bedankt.. ben ook absolute beginner wat java betreft

Hoe kan het dan dat je je "eindwerk" in Java moet doen? Ik denk dat het verstandig is dat je de eerste hoofdstukken van je Java boek gewoon ff opnieuw leest, want tweakers is niet echt de plaats voor complete basale vragen. Je komt nu een beetje over alsof je liever hier op antwoord wacht dan zelf op zoek gaat.

[Creepy]

Dat wat Hydra zegt + het feit dat je vraag na vraag blijft dumpen zonder zelf aan te geven wat je a hebt geprobeerd heeft ervoor gezorgd dat ik je topic nu ga sluiten. Een beginner zijn is niet erg, maar je lijkt er zelf nu 0,0 moeite in te stoppen en direct tegen elk probeem dat je aanloopt maar direct hier om een oplossing te vragen. Dat is hier niet de bedoeling op het forum, en al helemaal niet als het om schoolopdrachten gaat. Je zou genoeg informatie van je docent gekregen moeten hebben om dit goed af te kunnen ronden. Natuurlijk kan je dan nog wel eens tegen wat aan loopen, maar dan zou je veel meer informatie moeten kunnen geven over wat je a hebt geprobeerd, wat daar niet mee lukte etc. Nu kom je een paar keer niet verder dan: "dit is mijn code, hoe moet dit en dit?" en dat is echt niet de bedoeling.