Brauche RIESEN Hilfe zu OOP Projekt!

[Penelope]

Hi! Ich hoffe ihr seid bereit mir zu helfen. Befinde mich in ner ziemlich verzweifelten Lage, weil ich Montag ein OOP Projekt abgeben muss, nachdem ich 2 Wochen krank war und leider so gar keine Ahnung habe.
Zum Programm: Es soll die typischen Bankschaltervörgänge können. Heisst: Kreditlimit festsetzen, Evt noch Besitzername, Abheben, Einzahlen etc.
Und natürlich sollten wir so viele Specialeffects wie möglich drin haben, nur dummerweise kann ich nichtmal gescheid programmieren, weil ich zu viel verpasst habe. Ich habe jetzt einen Anfang, bei dem wahrlich nicht alles funktioniert und hoffe, dass ihr mir evt ein paar Tipps geben könnt, was ich da so reinprogrammieren könnte. (ganz toll wär was in richtung benutzeroberfläche, wo man sobald man ne "2" eingibt zB zum Abheben kommt oder so, wenn sowas auf dem low-level überhaupt geht...

Erstmal Klasse Konto mit den Befehlen etc :

 import java.io.*;

public class Konto
{
private static double kontostand=0.0;
private static double limit=0.0;

	public static void setLimit(double klimit)
	{
		if (klimit>=0.0)
		
		{	System.out.println("Bitte negative Zahl eingeben!");}
		
		else limit=klimit;

	}
	
	

	public double getLimit()
	{
		return(this.limit);
	}



	public double getKontostand()
	{
		return(this.kontostand);
	}

	
	
	public void Einzahlung(double einzahlung)
	{
		 if (einzahlung<0.0)
		
 		{	System.out.println("Der von Ihnen eingebene Betrag ist negativ!");}
		
		else this.kontostand=einzahlung+kontostand;
	}



	public void Auszahlung(double ausz)
	{
		if (ausz<=0.0)
	
		{System.out.println("Der von Ihnen eingebene Betrag ist negativ!");}
	
		else if (limit>kontostand-ausz)
		{System.out.println("Ihr Kreditlimit :"+limit+" wird überschritten. Bitte neuen Wert eingeben!");}
		
		else this.kontostand=kontostand-ausz;

	}

}

Dann Hauptklasse in der das ganze funktionieren soll (aber nicht tut) :

import java.io.*;

public class Kontosteuerung
{
	public static void main(String argv[])throws IOException
	{
	
	Konto eins=new Konto();
	Konto zwei=new Konto();

	BufferedReader eingabe=new BufferedReader(new InputStreamReader(System.in));

	System.out.println("Kreditlimit fuer Konto 1 eingeben: ");
	eins.setLimit(Double.parseDouble(eingabe.readLine()));

	System.out.println("Kreditlimit fuer Konto 1: "+eins.getLimit());
	
	System.out.println("");

	System.out.println("Kreditlimit fuer Konto 2 eingeben: ");
	eins.setLimit(Double.parseDouble(eingabe.readLine()));

	System.out.println("");

	System.out.println("Kreditlimit - Konto 2: "+zwei.getLimit());
	System.out.println("");

	System.out.println("aktueller Kontostand - Konto 1: "+eins.getKontostand());
	System.out.println("aktueller Kontostand - Konto 2: "+zwei.getKontostand());
	System.out.println("");

	System.out.println("Einzahlung - Konto 1: ");
	eins.Einzahlung(Double.parseDouble(eingabe.readLine()));

	System.out.println("Einzahlung - Konto 2: ");
	zwei.Einzahlung(Double.parseDouble(eingabe.readLine()));
	
	System.out.println("");

	System.out.println("Abbuchungsbetrag - Konto 1: ");


}
}

wär echt superlieb wenn sich da jmd mal reindenken würde und mir helfen würde :/[/code]

 

[SebiB90]

also pack erstmal die statics aus der klasse Konto raus, weil du die für jedes konto brauchst und nicht nur einmal, auch bei setlimit mach das static weg, außer du willst das limit für alle konten gilt, dann vor setLimit und limit das static dar lassen, aber bei kontostand auf jeden fall weg machen.
außerdem fehlt ein default konstruktor also das in die klasse Konto rein:

public Konto() {
}

wenn limit static bleiben soll, sollte auch getLimit() static sein, außerdem macht man den zugriff auf statische variablen nicht mit this sondern mit Klassenname.variablen name, also bei dir Konto.limit, oder wenn du in dieser klasse drin bist und es keine gleichnamige lokale variable gibt kannst auch nur limit hin schreiben.

 

[bygones]

1. einen default Konstruktor brauchst du nicht, der ist immer da, solange du keine parametrisierten Konstruktoren definierst.

2. static ist im OOP immer gefährlich. static bezieht sich auf eine Klasse und nicht auf einzelne Instanzen. D.h. deine beiden "Instanzvariablen" kontostand und limit gelten für alle instanzen gleich die du erzeugst und nicht für jede instanz einzeln, ergo alle statics raus.

Ansonsten für das posten hier: Bitte IMMER schreiben, was genau nicht geht, was sagt java usw. Für uns ist es dann wesentlich einfacher zu antworten und zu helfen. da wir so nicht den code einzeln durchforsten müssen....

 

[Penelope]

ja danke erstmal

ist mir jetzt auch aufgefallen, sorry. ähm ich glaube, wenn ich ne negative zahl eingebe, kann ich nicht nochmal erneut eine eingeben. fehler an sich zeigts nicht an bei jcreator.

wegen static: ich glaube mein lehrer will unbedingt statics. gibts nicht irgendne variable in der ich das lassen kann? :wink:

liebe grüsse

 

[bygones]

static ist wenn dann nur bei limit sinnvoll. wie gesagt eine static variable ist für alle instanzen gleich. Daher ist kontostand schonmal unlogisch als static (das würde heißen, alle konten haben immer den gleichen kontostand). Wenn alle konten das gleiche limit haben, so kann man limit static machen. da du aber für beide konten ein limit abfrägst ist static unsinnig.....

 

[Penelope]

ah okay danke, dann werd ich das mal wegmachen

habt ihr irgendwelche tipps für mich was ich noch so supertolles dazu machen könnte um zumindest ne 2 zu kriegen?

 

[bygones]

methoden immer mit Kleinbuchstaben beginnen lassen

ich würde in der main methode eine schleife einbauen, damit der user solange er will was mit dem Konto machen kann....