methode erstellen

[latrellvie]

//funkt. noch nicht
import java.util.Scanner;

class Marmelade
{
  // Instanzvariablen
  private String inhalt;   // Art der Frucht im Glas
  private String datum;   // Datum des Abfüllens
  private int menge;       // Menge der Marmelade im Glas


  // Konstruktor
  public Marmelade( String inhalt, String datum, int menge )
  {
    this . inhalt = inhalt;
    this . datum  = datum;
    this . menge  = menge;
  }

  // Methoden
  public boolean istLeer()
  {
    return ( menge == 0 );
  }

  public void ausgeben()
  {
    System.out.println ( inhalt + "   " +  datum + "   " +  menge + " Unzen." );
  }

  public void auftragen ( int unzen)
  {
    if ( !istLeer() )
    {
      if ( unzen <= menge )
      {
        System.out.println("Auftragen von " + unzen + " Unzen "
            + inhalt );
        menge = menge - unzen;
      }
      else
      {
        System.out.println("Auftragen von  " + menge + " Unzen "
            + inhalt );
        menge =  0;
      }
     }
     else
       System.out.println("Keine Marmelade im Glas!");
  }

}

class Vorratskammer
{
  // Instanzvariablen
  private Marmelade  glas1;
  private Marmelade  glas2;
  private Marmelade  glas3;
  private Marmelade  glasAusgewaehlt;
  private String ausgabeAuswahl;

  // Konstruktor
  public Vorratskammer( Marmelade glas1, Marmelade glas2, Marmelade glas3 )
  {
    this . glas1    = glas1;
    this . glas2    = glas2;
    this . glas3    = glas3;
    glasAusgewaehlt = null;
    ausgabeAuswahl  = "\nTreffen Sie Ihre Auswahl (1, 2 oder 3). Exit -1:";
  }
  public Vorratskammer( Marmelade glas1,Marmelade glas2 )
  {
  	this.glas1 = glas1;
  	this.glas2 = glas2;
  	this.glas3 = null;
  	glasAusgewaehlt = null;
  	ausgabeAuswahl  = "\nTreffen Sie Ihre Auswahl (1,oder 2). Exit -1:";
  }
  public Vorratskammer( Marmelade glas1 )
  {
  	this.glas1 = glas1;
  	this.glas2 = null;
  	this.glas3 = null;
  	glasAusgewaehlt = null;
  	ausgabeAuswahl  = "\nLeider gibt es nur ein Glas (1 oder 1). Exit -1:";
  }
  

  // Methoden
  public void ausgeben()
  {
  	System.out.println("Die Vorratskammer enthaelt:");
  	
  	if (glas1!=null)
  	{
  		System.out.println("1:");
  		glas1.ausgeben();
  	}
  	if (glas2!=null)
  	{
  		System.out.println("2:");
  		glas2.ausgeben();
  	}
  	if (glas3!=null)
  	{
  		System.out.println("1:");
  		glas3.ausgeben();
  	}
  }
 

  // wir setzen voraus, dass der Anwender eine richtige Auswahl trifft, 1, 2, oder 3
  public boolean auswaehlen( int glasNummer )
  {
    if ( glasNummer == 1 && glas1 != null )
    {
      glasAusgewaehlt =  glas1;
      return true ;
	}
    else if ( glasNummer == 2&& glas2 != null )
    {
      glasAusgewaehlt =  glas2;
      return true ;
	}
  
      else if ( glasNummer == 3&& glas3 != null )
    {
      glasAusgewaehlt =  glas3;
      return true ;
	}
	else
	 return false;
      
  }

  // Auftragen der ausgewählten Marmelade
  public void auftragen( int unzen  )
  {
    glasAusgewaehlt . auftragen( unzen  );
  }
  public String getAuswahl()
  {
  	return ausgabeAuswahl;
  }
  public void ersetzen( Marmelade m , int slot )
  {
  	if (slot == 1)
  		return new Vorratskammer(Marmelade m, Marmelade glas2, Marmelade glas3);
  	else if (slot == 2)
  		return new Vorratskammer(Marmelade glas1, Marmelade m, Marmelade glas3);
  	else if (slot == 3)
  		return new Vorratskammer(Marmelade glas1, Marmelade glas2, Marmelade m);
  	else
  		System.out.println("Falsche Sloteingabe");
  }
}

class ZHU_Kap35_Auf3
{
  public static void main ( String[] args )
  {
    Marmelade stachelbeer = new Marmelade( "Stachelbeer", "04.07.09", 12 );
    Marmelade erdbeer     = new Marmelade( "Erdbeer", "31.09.10", 8 );
    Marmelade aprikosen   = new Marmelade( "Aprikosen", "31.10.10", 16 );

    Vorratskammer vorrat  = new Vorratskammer( stachelbeer, erdbeer, aprikosen );
    vorrat.ausgeben();

    Marmelade kirsch      = new Marmelade( "Kirsch", "01.07.09", 11 );

    vorrat.ersetzen( kirsch, 3 );
    vorrat.ausgeben();

  }
}

Hello !!! Hier gehts um die ersetzten Methode( Zeile 146 bis 155). Ich habe sie so formuliert, geht aber nicht ganz. Im Lösungsabschnitt steht da folgende Variante :

 public void ersetzen(Marmelade m, int slot)
  {
    if( slot == 1 )
      glas1 = m;
    else if( slot == 2 )
      glas2 = m;
    else
      glas3 = m;
  }

Die Lösung ist mir verständlich, aber warum funkt. meine Variante nicht ? Thx

 

[jgh]

weil du bei einer Methode, die ein Glas Marmelade ersetzen soll, eine ganz neue Vorratskammer returnst und somit den Rückgabetyp dieser Methode (

void

) nicht erfüllst.

 

[latrellvie]

Und wie könnte ich es dann machen. außerdem ist ja das was zurückgegeben wird "Vorratskammer( Marmelade glas1, Marmelade glas2, Marmelade glas3 )" eine Klasse also keine Zahlen und daher void. Ich geh schon davon aus, dass du recht hast aber ich viel natürlich mein Verständnis verbessern

 

[Gast2]

void bedeutet dass die Methode nichts zurückgibt. Wenn du ne Vorratskammer zurückgeben willst du dann musst das als Rückgabewert setzen:

public void ersetzen( Marmelade m , int slot )

=>

public Vorratskammer ersetzen( Marmelade m , int slot )

Dann musst du allerdings im else Fall auch was zurückgeben (oder ne Exception werfen).

 

[SlaterB]

eine Klasse zurückzugeben bzw. ein Objekt davon ist nicht void,
wenn String zurückgegeben wird, muss auch String der Rückgabetyp sein, siehe
> public String getAuswahl()

wenn Vorratskammer der Rückgabewert sein soll, dann muss das auch so sein, nicht void,
dann musst du übrigens auch bestimmen, was im else-Fall zurückgegeben wird, z.B. null,
auf Klammern bei if, else + Schleifen besser NIE verzichten, als Anfänger!


-----------

> new Vorratskammer(Marmelade m, Marmelade glas2, Marmelade glas3);
ist für sich nebenbei kein sinnvoller Code

in der main-Methode schreibst du ja auch
> Vorratskammer vorrat = new Vorratskammer( stachelbeer, erdbeer, aprikosen );
und nicht
> Vorratskammer vorrat = new Vorratskammer(Marmelade stachelbeer, Marmelade erdbeer, Marmelade aprikosen );

auf solchen strukturellen Unsinn sehr genau achten,
Typen schreibt man bei der Deklaration von Methoden dran, nicht beim Aufruf

 

[latrellvie]

Danke , danke , danke !!!! Seids großartig, werd mich hoffentlich heut abend noch mit den neuen Erkentnissen spielen .

 

[latrellvie]

import java.util.Scanner;

class Marmelade
{
  // Instanzvariablen
  private String inhalt;  // Art der Frucht im Glas
  private String datum;   // Datum des Abfüllens
  private int    menge;   // Menge der Marmelade im Glas
  
  // Konstruktoren
  public Marmelade(String inhalt, String datum, int menge)
  {
    this.inhalt = inhalt;
    this.datum  = datum;
    this.menge  = menge;
  }

  // Methoden
  public boolean istLeer()
  {
    return ( menge == 0 );
  }

  public void ausgeben()
  {
    System.out.println ( inhalt + "marmelade   " +
                         datum  + "   "          +  menge + " Unzen.");
  }

  public void auftragen(int unzen)
  {
    if ( !istLeer() )
    {
      if ( unzen <= menge )
      {
        System.out.println("Auftragen von " + unzen +
                           " Unzen von "    + inhalt);
        menge = menge - unzen;
      }
      else
      {
        System.out.println("Auftragen von  " + menge +
                           " Unzen von "     + inhalt );
        menge =  0;
      }
    }
    else
       System.out.println("Keine Marmelade im Glas!");
  }
}

class Vorratskammer
{
  // Instanzvariablen
  private Marmelade  glas1;
  private Marmelade  glas2;
  private Marmelade  glas3;
  private Marmelade  glasAusgewaehlt;
  private String ausgabeAuswahl;

  // Konstruktoren
  public Vorratskammer(Marmelade glas1, Marmelade glas2, Marmelade glas3)
  {
    this.glas1      = glas1;
    this.glas2      = glas2;
    this.glas3      = glas3;
    glasAusgewaehlt = null;
    ausgabeAuswahl  = "\nTreffen Sie Ihre Auswahl (1, 2 oder 3). Exit -1:";
  }

  public Vorratskammer(Marmelade glas1, Marmelade glas2)
  {
    this.glas1      = glas1;
    this.glas2      = glas2;
    this.glas3      = null;
    glasAusgewaehlt = null;
    ausgabeAuswahl  = "\nTreffen Sie Ihre Auswahl (1 oder 2). Exit -1:";
  }

  public Vorratskammer(Marmelade glas1)
  {
    this.glas1      = glas1;
    this.glas2      = null;
    this.glas3      = null;
    glasAusgewaehlt = null;
    ausgabeAuswahl  = "\nLeider gibt es nur ein Glas (1 oder 1). Exit -1:";
  }

  // Methoden
  public void ausgeben()
  {
  	System.out.println("\nDie Vorratskammer enthaelt:");
    if( glas1 != null )
    {
      System.out.print("1: ");
      glas1.ausgeben();
    }
    
    if( glas2 != null )
    {
      System.out.print("2: ");
      glas2.ausgeben();
    }
    
    if( glas3 != null )
    {
      System.out.print("3: ");
      glas3.ausgeben();
    }
  }

  // wir setzen voraus, dass der Anwender eine richtige Auswahl trifft, 1, 2, oder 3
  public boolean auswaehlen(int glasNummer)
  {
    if( glasNummer == 1 && glas1 != null )
    {
      glasAusgewaehlt = glas1;
      return true;
    }
    else if ( glasNummer == 2 && glas2 != null )
    {
      glasAusgewaehlt = glas2;
      return true;
    }
    else if ( glasNummer == 3 && glas3 != null )
    {
      glasAusgewaehlt = glas3;
      return true;
    }
    else
      return false;
  }

  // Auftragen der ausgewählten Marmelade
  public void auftragen(int unzen)
  {
     glasAusgewaehlt.auftragen(unzen);
  }
  
  public String getAuswahl()
  {
  	return ausgabeAuswahl;
  }
  
  public void ersetzen(Marmelade m, int slot)
  {
    if( slot == 1 )
      glas1 = m;
    else if( slot == 2 )
      glas2 = m;
    else
      glas3 = m;
  }

  
}

public class Chap35Ex3
{
  public static void main(String[] args)
  {
    Marmelade stachelbeer = new Marmelade("Stachelbeer", "04.07.09", 12);
    Marmelade erdbeer     = new Marmelade("Erdbeer", "30.09.10", 8);
    Marmelade aprikosen   = new Marmelade("Aprikosen", "31.10.10", 16);

    Vorratskammer vorrat  = new Vorratskammer(stachelbeer, erdbeer, aprikosen);
    int glasNr, unzen;

    Marmelade kirsch      = new Marmelade("Kirsch", "01.07.09", 11);

    vorrat.ersetzen(kirsch, 3);
    vorrat.ausgeben();
  }
}

Habs jetzt so probiert geht aber immer noch nicht, wird zwar kompiliert aber die die neuen Daten Zeile 171 vorrat.ersetzen(kirsch, 3); werden nicht eingesetzt, was muss ich noch verbessern ? Thx

 

[flopalko]

So, nachdem ich keinen Fehler durch's lesen gefunden habe, habe ich das Programm kompiliert und ausgeführt und bei mir ersetzt er es schon. Wieso denn auch nicht?

Die Ausgabe des Programms bei mir:

1: Stachelbeermarmelade   04.07.09   12 Unzen.
2: Erdbeermarmelade   30.09.10   8 Unzen.
3: Kirschmarmelade   01.07.09   11 Unzen.

 

[VfL_Freak]

Moin,

So, nachdem ich keinen Fehler durch's lesen gefunden habe, habe ich das Programm kompiliert und ausgeführt und bei mir ersetzt er es schon.

Wieso denn auch nicht?

Ist das jetzt eine Frage oder nur eine launige Bemerkung ???:L

Falls ersteres, dann formuliere sie bitte mal genauer .....

Gruß
Klaus

 

[SlaterB]

@VfL_Freak
ich kann die Aussage von flopalko in Form und Inhalt komplett bestätigen,
das Programm kompiliert, das angesprochene Ersetzen funktioniert, es ist unklar welches Problem noch bestehen soll,

hast du das Posting als eines vom Threadersteller interpretiert?
ansonsten fasse ich es jetzt auch als Gemecker mir gegenüber auf

Vorsicht mit solchen themenfremden Beschwerden anderen gegenüber,
lieber doppelt kontrollieren und genau überlegen, was deine Aussage dabei ist

 

[VfL_Freak]

Moin SlaterB,

hast du das Posting als eines vom Threadersteller interpretiert?

ja, genau das .... war wohl noch zu früh :autsch:
mea culpa

Gruß
Klaus

 

[latrellvie]

 // funkt noch nicht
import java.util.Scanner;

class Marmelade
{
  // Instanzvariablen
  private String inhalt;   // Art der Frucht im Glas
  private String datum;   // Datum des Abfüllens
  private int menge;       // Menge der Marmelade im Glas


  // Konstruktor
  public Marmelade( String inhalt, String datum, int menge )
  {
    this . inhalt = inhalt;
    this . datum  = datum;
    this . menge  = menge;
  }

  // Methoden
  public boolean istLeer()
  {
    return ( menge == 0 );
  }

  public void ausgeben()
  {
    System.out.println ( inhalt + "   " +  datum + "   " +  menge + " Unzen." );
  }

  public void auftragen ( int unzen)
  {
    if ( !istLeer() )
    {
      if ( unzen <= menge )
      {
        System.out.println("Auftragen von " + unzen + " Unzen "
            + inhalt );
        menge = menge - unzen;
      }
      else
      {
        System.out.println("Auftragen von  " + menge + " Unzen "
            + inhalt );
        menge =  0;
      }
     }
     else
       System.out.println("Keine Marmelade im Glas!");
  }

}

class Vorratskammer
{
  // Instanzvariablen
  private Marmelade  glas1;
  private Marmelade  glas2;
  private Marmelade  glas3;
  private Marmelade  glasAusgewaehlt;
  private String ausgabeAuswahl;

  // Konstruktor
  public Vorratskammer( Marmelade glas1, Marmelade glas2, Marmelade glas3 )
  {
    this . glas1    = glas1;
    this . glas2    = glas2;
    this . glas3    = glas3;
    glasAusgewaehlt = null;
    ausgabeAuswahl  = "\nTreffen Sie Ihre Auswahl (1, 2 oder 3). Exit -1:";
  }
  public Vorratskammer( Marmelade glas1,Marmelade glas2 )
  {
  	this.glas1 = glas1;
  	this.glas2 = glas2;
  	this.glas3 = null;
  	glasAusgewaehlt = null;
  	ausgabeAuswahl  = "\nTreffen Sie Ihre Auswahl (1,oder 2). Exit -1:";
  }
  public Vorratskammer( Marmelade glas1 )
  {
  	this.glas1 = glas1;
  	this.glas2 = null;
  	this.glas3 = null;
  	glasAusgewaehlt = null;
  	ausgabeAuswahl  = "\nLeider gibt es nur ein Glas (1 oder 1). Exit -1:";
  }
  

  // Methoden
  public void ausgeben()
  {
  	System.out.println("Die Vorratskammer enthaelt:");
  	
  	if (glas1!=null)
  	{
  		System.out.println("1:");
  		glas1.ausgeben();
  	}
  	if (glas2!=null)
  	{
  		System.out.println("2:");
  		glas2.ausgeben();
  	}
  	if (glas3!=null)
  	{
  		System.out.println("1:");
  		glas3.ausgeben();
  	}
  }
 

  // wir setzen voraus, dass der Anwender eine richtige Auswahl trifft, 1, 2, oder 3
  public boolean auswaehlen( int glasNummer )
  {
    if ( glasNummer == 1 && glas1 != null )
    {
      glasAusgewaehlt =  glas1;
      return true ;
	}
    else if ( glasNummer == 2&& glas2 != null )
    {
      glasAusgewaehlt =  glas2;
      return true ;
	}
  
      else if ( glasNummer == 3&& glas3 != null )
    {
      glasAusgewaehlt =  glas3;
      return true ;
	}
	else
	 return false;
      
  }

  // Auftragen der ausgewählten Marmelade
  public void auftragen( int unzen  )
  {
    glasAusgewaehlt . auftragen( unzen  );
  }
  public String getAuswahl()
  {
  	return ausgabeAuswahl;
  }
  public Vorratskammer ersetzen( Marmelade m , int slot )
  {
  	if (slot == 1)
  		return new Vorratskammer( m, glas2, glas3);
  	else if (slot == 2)
  		return new Vorratskammer( glas1, m, glas3);
  	else 
  		return new Vorratskammer(glas1, glas2, m);
  }
}

class ZHU_Kap35_Auf3_1
{
  public static void main ( String[] args )
  {
    Marmelade stachelbeer = new Marmelade( "Stachelbeer", "04.07.09", 12 );
    Marmelade erdbeer     = new Marmelade( "Erdbeer", "31.09.10", 8 );
    Marmelade aprikosen   = new Marmelade( "Aprikosen", "31.10.10", 16 );

    Vorratskammer vorrat  = new Vorratskammer( stachelbeer, erdbeer, aprikosen );
    vorrat.ausgeben();

    Marmelade kirsch      = new Marmelade( "Kirsch", "01.07.09", 11 );

    vorrat.ersetzen( kirsch, 3 );
    vorrat.ausgeben();

  }
}

Liebe Leute es tut mir sehr leid, ich habe vorher was anderes gepostet und zwar das Programm wie es laut Lösung funktioniert. Dies ist mein eigener Lösungsvorschlag, der zwar kompiliert wird, wo aber die ersetz-methode nicht funktioniert. (Zeile 146 bis 154 ). Ich versuche bei meinem Bsp. gleich die Vorratskammer zu ersetzen und nicht das glas in der Vorratskammer.

 

[flopalko]

Also erstens mal: ist dir beim Testen immer noch nicht aufgefallen, dass bei der Methode ausgeben in der Klasse Vorratskammer die Nummern "1, 2, 1" sind und nicht "1, 2, 3"?

Zweitens, die Methoden funktionieren an sich schon, nur musst du auch irgendwas mit der zurückgegebenen Vorratskammer der Methode ersetzen machen. Schreibe hierfür:

vorrat = vorrat.ersetzen( kirsch, 3 );

Dann funktioniert es einwandfrei, ist aber die schlechtere Variante. So, wie es in der Lösung ist, ist es weitaus besser. Ersetze einfach das Glas und nicht die gesamte Vorratskammer

 

[latrellvie]

Jetzt funktionierts, Danke !!!