[Gelöst:] positive Null

[Localhope]

Hallo. Ich habe ein Kontoverwaltungsprogramm gecshrieben. Nun habe ich aber eine lästige Ausgabe. Wenn der abzuhebende Betrag größer als der Kontostand ist, soll man die Möglichkeit haben, die Methode kredit in Anspruch zu nehmen. Bei meiner Ausgabe wird dann aber gleich davon ausgegangen. Programm:

public class Konto{
	private float kontoStand;
	private String name;
	private float betrag;
	
	public Konto(){
		kontoStand=1200f;
		name="Tux";
	}
	
	public float getKontoStand(float kontoStand){
		return kontoStand;
	}
	
	public void gutschreiben(float kontoStand, float betrag){
		betrag=1f;					//Die Gutschrift ist hier von Hand einzutragen
		if(betrag<=0){
			System.out.println("Der angegebene Betrag ist zu klein. Bitte geben Sie einen positiven Betrag ein!");
		}
		else{
			this.kontoStand=kontoStand+betrag;                       
		}
	}
	
	public void gewaehreKredit(float kontoStand, float betrag){
		betrag=1f;                 //Der Betrag ist hier von Hand einzutragen
		if((kontoStand-betrag)<=-2000){
			System.out.println("Sie haben das Limit erreicht. Maximaler Dispo: 2000 EUR!");
		}
		else{
			System.out.println("Sie haben einen Kredit beantragt. Die Gebühr hierfür beträgt 10 EUR.");
			this.kontoStand=kontoStand-betrag;
		}
	}
	
	public void abheben(float kontoStand, float betrag){
		betrag=50000f;             //Der Betrag ist hier von Hand einzutragen
		if(betrag>=kontoStand){
			System.out.println("Angegebener Betrag zu hoch. Bitte nutzen Sie die Option \"gewaehreKredit\" !");
		}
		else{
			this.kontoStand=kontoStand-betrag;	
		}
	}
	
	public static void main(String[] args){
		Konto konto1 = new Konto();
		
		System.out.println("Name: " + konto1.name+ "\nKontostand: " +konto1.kontoStand);
		
		konto1.gutschreiben(konto1.kontoStand, konto1.betrag);
		konto1.abheben(konto1.kontoStand, konto1.betrag);
		konto1.gewaehreKredit(konto1.kontoStand, konto1.betrag);
		
		System.out.println("neuer Kontostand: " + konto1.kontoStand);
	}
}

Ich möchte gerne die Methode abheben dann auf null setzen, und die methode 'gewaehre kredit" in Anspruch nehmen, sonst macht er ja beides. Aber eine null erkennt er als negativ an und sagt mir dann: Zu kleiner Betrag, bitte höheren angeben.

Wie schaff ich es dass die null als positiv anerkannt wird? Sprich: Klappt es nicht, das Geld abzuheben, da es ein zu hoher Betrag ist, soll man bei 'abheben' den Betrag auf 0 stellen und dann die Option Kredit in Anspruch nehmen.

 

[madboy]

Hi. Ich schreibe meine Bemerkungen mal in den Quelltext als Kommentar rein.

public class Konto{ 
   private float kontoStand; 
   private String name; 
   private float betrag; //unnötig und verwirrend
    
   public Konto(){ //mach nen Konstruktor, dem man den kontoStand und name übergibt. public Konto(float kontoStand, name)
      kontoStand=1200f; 
      name="Tux"; 
   } 
    
   public float getKontoStand(float kontoStand){ //warum übergibst du hier den kontoStand? Lass das weg.
      return kontoStand; 
   } 
    
   public void gutschreiben(float kontoStand, float betrag){ //hier ebenfalls kontoStand weg lassen
      betrag=1f;               //Die Gutschrift ist hier von Hand einzutragen //nicht gut. zum Testen übergebe den Betrag in der main-Methode
      if(betrag<=0){ 
         System.out.println("Der angegebene Betrag ist zu klein. Bitte geben Sie einen positiven Betrag ein!"); 
      } 
      else{ 
         this.kontoStand=kontoStand+betrag;  //erstze this.kontoStand und kontoStand durch getKontoStand(). Dafür ist die get-Methode da.
      } 
   } 
    
   public void gewaehreKredit(float kontoStand, float betrag){ //kontoStand weg und get-Methode verwenden, wie oben
      betrag=1f;                 //Der Betrag ist hier von Hand einzutragen 
      if((kontoStand-betrag)<=-2000){ 
         System.out.println("Sie haben das Limit erreicht. Maximaler Dispo: 2000 EUR!"); 
      } 
      else{ 
         System.out.println("Sie haben einen Kredit beantragt. Die Gebühr hierfür beträgt 10 EUR."); 
         this.kontoStand=kontoStand-betrag; 
      } 
   } 
    
   public void abheben(float kontoStand, float betrag){ //kontoStand weg und get-Methode verwenden, wie oben
      betrag=50000f;             //Der Betrag ist hier von Hand einzutragen 
      if(betrag>=kontoStand){ 
         System.out.println("Angegebener Betrag zu hoch. Bitte nutzen Sie die Option \"gewaehreKredit\" !"); 
      } 
      else{ 
         this.kontoStand=kontoStand-betrag;    
      } 
   } 
    
   public static void main(String[] args){ 
      Konto konto1 = new Konto(); //entsprechend dem neuen Konstruktor aufrufen: new Konto(gewünschterKontostand, gewünschterName)
       
      System.out.println("Name: " + konto1.name+ "\nKontostand: " +konto1.kontoStand); 
       
      konto1.gutschreiben(konto1.kontoStand, konto1.betrag); //Was machst du da? konto1.gutschreiben(10f)
      konto1.abheben(konto1.kontoStand, konto1.betrag); //Was machst du da?
      konto1.gewaehreKredit(konto1.kontoStand, konto1.betrag); //Was machst du da?
       
      System.out.println("neuer Kontostand: " + konto1.kontoStand); 
   } 
}

Sehr wahrscheinlich ist dein Problem schon gelöst, wenn du obige Veränderungen vornimmst

 

[Localhope]

Nein, eigentlich nicht. Habe es so abgeändert. Es läuft nun und ist auch übersichtlicher

Es lag an der if-Methode. Es heißt nicht it(betrag<=0) sonder if(betrag<0). Das schließt die null mit ein

 

[Roar]

dein code ist trotzdem bisschen mistig und du solltest madboys änderungen einbauen.

 

[Localhope]

Habe ich schon. Danke für den Hinweis