Parametre

Er det hele programmet ?
Hvis det er, så mangler du en 'main' klasse.

----------
public static void main(String[] args) {

}
----------

for at starte hele programmet.

Nej. Main classen ser sådan ud:

import java.io.BufferedReader;
import java.io.InputStreamReader;
import java.io.IOException;

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

    {    P340BTank benzinTank = new P340BTank();
        P340KmTæller kørteKilometer = new P340KmTæller();
        P340Bil test1 = new P340Bil();
       
        InputStreamReader read = new InputStreamReader(System.in);
        BufferedReader console = new BufferedReader(read);
   
   
        { try   
            {
   
            System.out.println("Hvor meget benzin fylder du på? ");
            String read1 = console.readLine();
            int liter = Integer.parseInt(read1);
            benzinTank.påfyldning(liter);
                       
            System.out.println("Hvor har du kørt? ");
            String read2 = console.readLine();
            int km = Integer.parseInt(read2);
            kørteKilometer.tripTæller(km);
            System.out.println("\nSe er der  " + benzinTank.getTankNiveau() + " liter benzin tilbage i tanken");
            }       
       
            catch(IOException e)
            {    System.out.println(e);
                System.exit(1);
            }
       
        }
           
    }
}

Herforuden er der:


public class P340BTank
{           
    int niveau;
    int påfyldning;
    int forbrug;
   
    P340Bil bForbrug = new P340Bil();
   
    public P340BTank()
    { niveau =0;
    }

    public void påfyldning(int påfyldning)
    { niveau = påfyldning + niveau;
    }
   
    public void tømning()
    { niveau =  - niveau - bForbrug.getBenzinForbrug();
    }
   
    public int getTankNiveau()
    {    return niveau;
    }
   
}

Hvad får du af runtimefejl-hvis der er nogen?

Lav P340Bil om til:
class P340Bil{ 
   
    private int forbrug;
    private int kmPrL;
    private P340KmTæller tTæller = null;
   
    public P340Bil(){
    tTæller = new P340KmTæller();
    forbrug =0;
    kmPrL();
    }
Ellers resten som det er.

Og
public class P340BTank
{           
    private int niveau;
    private int påfyldning;
    private int forbrug;
    private P340Bil bForbrug = null;
   
    public P340BTank(){
    bForbrug = new P340Bil();
    niveau =0;
    }
Ellers resten som det er..

Og i din main-metode behøver du nu kun at lave en reference til P340Bil:
P340Bil test1 = new P340Bil();
Den opretter nu reference til P340KmTæller og P340KmTæller objekter.

Om igen: P340BTank skal oprettes i main.
P340BTank benzinTank = new P340BTank();
Den opretter de to andre.

Jeg får stadig denne udskrift:

Hvor meget benzin fylder du på?
[TEST]  int liter = Integer.parseInt(read1);  34

Hvor meget har du kørt?
[TEST]  int km = Integer.parseInt(read2);  3

[TEST]kørteKilometer.getTripTæller() 0

[TEST]P340Bil.getBenzinForbrug() 0

Så er der  34 liter benzin tilbage i tanken

Det er ikke rigtig til gennemskue

Så er jeg her igen.
Prøv det her (de kommer lidt efter lidt!)
public class P340KmTæller{           

    private int tripTæller;
   

    public P340KmTæller(){
    tripTæller = 0;
    }

    public void setTripTæller(int tripTæller){
    this.tripTæller += tripTæller;
    }

    public int getTripTæller(){
      return tripTæller;
}
}

Ups
public void setTripTæller(int tripTæller){
    this.tripTæller = tripTæller;
    }
og

public void TælTripTællerOp(int tripTæller){
    this.tripTæller += tripTæller;
    }

Hvorfor er der både en forbrug-attribut på Bil og Tank?

public class P340Bil{ 
   
    private int forbrug;
    private int kmPrL;
    private P340KmTæller tTæller=null;
   
    public P340Bil(){ 
      tTæller = new P340KmTæller();
    forbrug = 0;
        kmPrL();
    }
 
    public void kmPrL(){   
        kmPrL = 10;
    }   
     
    public void setForbrug(int forbrug){
    this.forbrug=forbrug;
    }

    public int getForbrug(){
    return forbrug;
    }

    public void setKmPrL(kmPrL){
    this.kmPrL=kmPrL;
    }

    public int getKmPrL(kmPrL){
    return kmPrL;
    }
 
    public int getBenzinForbrug(){ 
        return tTæller.getTripTæller()/kmPrL;
    } 
}

brug float istedet for int, der hvor du har kommatal som følge af dividering
2/3 = 0 , hvorimod
2.0/3.0 = 0.66666

public class P340BTank{
           
    private int niveau;
    private int påfyldning;
    private int forbrug;
    private P340Bil bForbrug = null;
   
public P340BTank(){
    bForbrug = new P340Bil();
    niveau = 0;
}

public void påfyldning(int påfyldning){
    niveau = påfyldning + niveau;
}
   
public void tømning(){
    niveau =  - niveau - bForbrug.getBenzinForbrug();
} //Er den her rigtig??

public int getTankNiveau(){   
    return bForbrug.getBenzinForbrug(){ 
} 

}

Og din triptæller skal jo også sættes - Ellers vil din division altid give nul.

public void tømning(){
    niveau = 0;
}

...burde kunne gøre det, eller hvad ?


Overvej at omdøbe dine metoder til f.eks.
getNiveau, setNiveau, getForbrug, setForbrug osv.

..ligesom magoo20000 har gjort  :-)

Så er jeg her igen.
Jeg havde et problem med et udendørsstik med for meget vand i....
Jeg ser og vender tilbage...

Surt!
Man kunne også overveje, om det ikke ville være bedre design at løse denne opgave vha. arv. Bil som Superklasse og de to andre som sub-klasser.

Et skridt længere men ...

P340Bil.java:32: <identifier> expected
  public void setKmPrL(kmPrL)
                              ^
Der er noget(meget ;-)) som jeg ikke har forstået.

Hvorfor forlanger den en identifier efter (kmPrL)?



public class P340Bil
{ 

    private int forbrug;
  private int kmPrL;
  private P340KmTæller tTæller=null;

  public P340Bil()
    { 
        tTæller = new P340KmTæller();
        forbrug = 0;
      kmPrL();
    }
 
    public void kmPrL()
    {   
        kmPrL = 10;
    }   
     
    public void setForbrug(int forbrug)
    {
        this.forbrug=forbrug;
    }

    public int getForbrug()
    {
        return forbrug;
    }

    public void setKmPrL(kmPrL)
    {    this.kmPrL=kmPrL;
    }

    public int getKmPrL(kmPrL)
    {
        return kmPrL;
    }
 
    public int getBenzinForbrug()
    { 
        return tTæller.getTripTæller()/kmPrL;
    } 

}

/*
P340Bil.java:32: <identifier> expected
    public void setKmPrL(kmPrL)
                              ^
P340Bil.java:44: ')' expected
    } 
    ^
P340Bil.java:32: cannot resolve symbol
symbol  : class kmPrL 
location: class P340Bil
    public void setKmPrL(kmPrL)
                        ^
P340Bil.java:32: missing method body, or declare abstract
    public void setKmPrL(kmPrL)
                ^
4 errors
*/

...Den er jo defineret med " private int kmPrL " såvidt jeg ved.
Men... Jeg bruger måske ikke den identifier i parameteren...

I p340Tank burde du have:
public void setTripTæller(){   
    bForbrug.setTripTæller(){ 
} 
Og i p340Bil:
public void setBenzinForbrug(){ 
        tTæller.setTripTæller();
    } 
Nu kan du sætte, hvor mange L benzin, der skal på dyret og hvad triptælleren er(=antal kørte km).

I din main-metode skal du nun kun oprette et objekt af typen P340BTank.
P340BTank tank = new P340BTank():
getTankNiveau() burde nu returnere noget korrekt.
Som det tidligere blev nævnt, er det en god ide at vælge en anden primitiv datatype til dine attributter.
En float er langt mere fornuftig, end en int.
Hvis du vælger det, så husk at ret Integer.parseInt(read1) til Integet.parseFloat(read1)...

Husk at angive, at det skal være en int.
Altså (int kmPrL), som parameter!

Det var vist mig, der glemte det.
Jeg sidder ved en pc uden SDK'en installeret, så jeg kan ikke teste det :-((, så det er frit efter hovedet!

...Er det dit hoved eller er det et sprog kursus som har bidraget til den beherskelse ;-)

(ps. jeg mener at jeg er i tvivl ved blot en "hello world" syntax)

Hehe! Øvelse gør mester!
Man lærer kun at kode ved at øve sig!
Jeg har lige hentet den nødvendig software her, hvor jeg sidder, så hæng lige på..Har fundet et par fejl mere!

Ok så er den løst!

import java.io.BufferedReader;
import java.io.InputStreamReader;
import java.io.IOException;

class Main{
       
      public static void main(String[] args){
          P340BTank benzinTank = new P340BTank();
           
      InputStreamReader read = new InputStreamReader(System.in);
      BufferedReader console = new BufferedReader(read);
        {
       
        try   
            {
   
            System.out.println("Hvor meget benzin fylder du på? ");
            String read1 = console.readLine();
            float liter = Float.parseFloat(read1);
            benzinTank.påfyldning(liter);
                       
            System.out.println("Hvor har du kørt? ");
            String read2 = console.readLine();
            float km = Float.parseFloat(read2);
            benzinTank.setTripTæller(km);
            System.out.println("\nSå er der  " + benzinTank.getTankNiveau() + " liter benzin tilbage i tanken");
            }       
       
            catch(IOException e)
            {    System.out.println(e);
                System.exit(1);
            }
       
        }
           
    }
}

public class P340BTank{
           
    private float niveau;
    private float påfyldning;
    private float forbrug;
    private P340Bil bForbrug = null;
   
public P340BTank(){
    bForbrug = new P340Bil();
    niveau = 0;
}

public void påfyldning(float påfyldning){
    niveau = påfyldning + niveau;
}
   
public void tømning(){
    niveau = 0;
}

public float getTankNiveau(){   
    return niveau-bForbrug.getBenzinForbrug();
     
} 
public void setTripTæller(float antalkm){ 
  bForbrug.setTripTaeller(antalkm);
} 

}

public class P340Bil{ 
   
    private float forbrug;
    private float kmPrL;
    private P340Taeller tTaeller=null;
   
    public P340Bil(){ 
      tTaeller = new P340Taeller();
      forbrug = 0;
      kmPrL();
    }
 
    public void kmPrL(){   
        kmPrL = 10;
    }   
     
    public void setForbrug(float forbrug){
    this.forbrug=forbrug;
    }

    public float getForbrug(){
    return forbrug;
    }

    public void setKmPrL(float kmPrL){
    this.kmPrL=kmPrL;
    }

    public float getKmPrL(float kmPrL){
    return kmPrL;
    }
 
    public float getBenzinForbrug(){ 
      return (tTaeller.getTripTaeller()/kmPrL);
       
    } 
    public void setTripTaeller(float antalKm){ 
        tTaeller.setTripTaeller(antalKm);
    } 

}

public class P340Taeller{           

    private float tripTaeller;
   

    public P340Taeller(){
    tripTaeller = 0;
    }

    public void setTripTaeller(float tripTaeller){
    this.tripTaeller += tripTaeller;
    }
     
    public float getTripTaeller(){
      return tripTaeller;
}
}

Så sætter jeg mig i en krog og læser på det jeg ikke forstår og skulle blive klogere af det (lidt af et paradoks).

Mange tak for hjælpen begge to.
Magoo20000 får pointene. Men føler du dig forbigået blackadder så sig til.

Mange tak! Bare sig til en anden gang!
Og god fornøjelse her i Javaverdenen!