Hilfe bei nem Taschenrechner

[Peter1988]

Also Leute, ich bin neu hier im Forum

Bin aber auch ein sogenannter Anfänger in der Programmierung mit Java. Mein sozusagen erstes Projekt soll ein Taschenrechner sein, der Punkt vor Strichrechnung beherrscht. Habe dazu schon ein Fenster erstellt, in dem dann die verschiedenen Button generiert werden, sowie das FormattedField. Ich hoffe ihr könnt mir folgen.

Hier folgender Programmcode, der euch da weiterhelfen könnte

private Vector<myValues> vecSpeicher = new Vector<myValues>();

private class myValues {

		private BigDecimal bigDecimal;
		private int iInt;
		
		public myValues(BigDecimal bigDecimal, int iInt){
			this.bigDecimal = bigDecimal;
			this.iInt = iInt;
		}
		
		private BigDecimal getZahl(){
			this.bigDecimal = bigDecimal;
			return new BigDecimal("0");
		}
		private int getOperator(){
			this.iInt = iInt;
			return 0;
		}

	}

private void addValue(myValues myvalues) {
			vecSpeicher.addElement(myvalues);



//				Rechenoperatoren
				if(sName.equalsIgnoreCase("+")){				
					this.addValue(new myValues(getDisplay(),ADDIEREN));
					fldDisplay.setText("");
				return;
				}
				if(sName.equalsIgnoreCase("-")){
					this.addValue(new myValues(getDisplay(),SUBTRAHIEREN));
					fldDisplay.setText("");

					return;
				}
				if(sName.equalsIgnoreCase("*")){
					this.addValue(new myValues(getDisplay(),MULTIPLIZIEREN));
					fldDisplay.setText("");

				return;
				}		
				if(sName.equalsIgnoreCase("/")){
					this.addValue(new myValues(getDisplay(),DIVIDIEREN));
					fldDisplay.setText("");

					return;
				}

		private BigDecimal getResult(){
			vecSpeicher.trimToSize();
...

So weit so gut, bis jetzt hat alles funktioniert. Ich hoff ich hab hier jetzt auch nichts relevantes vergessen ...
Jetzt will ich eigentlich gerne die Daten aus dem Speicher des Vektors quasi rauslesen.

Man muss sich das so vorstellen, dass der Vektor 2 Spalten hat, einmal den Inhalt, die Zahl mit der er arbeiten soll, und einmal den Operator. Der Inhalt, ist vom Typ BigDecimal, der Operator ist vom Typ integer.

Jetzt weiß ich nur nicht, wie ich genau weitermachen soll, sodass ich die Daten aus dem Vektor lese (das ist wohl mein größtes Problem ), und das der dann weiterrechnet, und die Punkt vor Strichrechnung beachtet ...

Könnt ihr mir da weiterhelfen, ist bestimmt ganz simpel, aber ich komm da wohl nicht drauf ...[/code]

 

[jPat]

Was meinst du mit "rauslesen?"
->
myValues value = vecSpeicher.get(int index)

 

[Der_Dude]

Hiho Peter,

schau mal hier im FAQ unter Pattern ich glaube da wird dir geholfen.

mfg Dude

 

[Peter1988]

Mit rauslesen meine ich folgendes:

Und zwar schreibt mein Programm ja nachdem man eine Zahl eingetippt hat, und danach auf einen Operator klickt, die jeweiligen Daten in den Vektor. Müsste ja so aussehen, zum Beispiel

ich klick auf 7 und dann auf + (für Plus hab ich die Variable 0 definiert)

nun müsste im Vektor ja die 7 und das + gespeicher sein.



Ich würd jetzt gerne durch den Vektor gehen, mit ner for Schleife und kontrollieren, if mein integer wert (die Zahl, die für den Operator steht) ist beispielsweise 2, dann addiere mir die bigdecimal Zahl auf dem Vektor index 0 und index 1.

Ist schwer zu erklären, ich brauch halt sozusagen den inhalt des Vektors, auch wenn ich mich noch so schwer damit tu, das zu erklären, wisst ihr doch hoffentlich was damit gemeint ist ^^

 

[jPat]

MyValues value;
for (int i = 0 ; i<vec.size();i++){
value = (MyValue) vec.get(i); // Cast nach MyValue ... evtl lag hier deine Schwierigkeit?
if ( Hier dein value prüfen) { machWas();}
}

 

[jPat]

addiere mir die bigdecimal Zahl auf dem Vektor index 0 und index 1.

MyValue value  = (MyValue) vec.remove(0);Index "0" ist aus dem Vector Drausen

value.add(otherVal);

vec.add(value,0);

 

[Peter1988]

Ja danke, das bringt mich schonmal ein Stück weiter, das casten.

Jetzt wird's meiner Meinung nach aber kompliziert.

Und zwar sieht der Vektor so aus, zumindest sollte er das ^^

BigDecimal|Integer

2.5|0
3|0
1.6|3


Soweit so gut, das sind nur beispielwerte.

	private final int MULTIPLIZIEREN 	= 0;
	private final int DIVIDIEREN 		= 1;
	private final int ADDIEREN 			= 2;
	private final int SUBTRAHIEREN 		= 3;

Das sind die Integerwerte.

Jetzt würde ich meinem Taschenrechner gerne sagen, kontrolliere, ob die Zahl im Vektor zum Beispiel 0 ist. Dabei soll er aber nur den Integer Wert des Vektors überprüfen. Und wenn das dann gegeben ist, soll er denn BigDecimal Wert 1 mit dem zweiten multiplizieren. In dem Beispiel wäre das dann Weil der erste Integer Wert 0 ist, sollte der Taschenrechner 2,5*3,0 rechnen.

Ich glaub wenn ich jetzt noch weiß, wie ich das hinkrieg, dann weiß ich wie ich weitermachen kann, wäre noch Klasse, wenn du /ihr mir dabei helfen könnt.[/code]

 

[Peter1988]

Das ist übrigens der Quellcode der Klasse myValues

	private class myValues {

		private BigDecimal bigDecimal;
		private int iInt;
		
		public myValues(BigDecimal bigDecimal, int iInt){
			this.bigDecimal = bigDecimal;
			this.iInt = iInt;
		}
		
		private BigDecimal getZahl(){
			this.bigDecimal = bigDecimal;
			return new BigDecimal("");
		}
		private int getOperator(){
			this.iInt = iInt;
			return 0;
		}

	}

 

[mikachu]

	private class myValues {

		private BigDecimal bigDecimal;
		private int iInt;
		
		public myValues(BigDecimal bigDecimal, int iInt){
			this.bigDecimal = bigDecimal;
			this.iInt = iInt;
		}
		
		private BigDecimal getZahl(){
			this.bigDecimal = bigDecimal; // <- warum ?!?
			return new BigDecimal(""); // <- warum ?!?
		}
		private int getOperator(){
			this.iInt = iInt; // <- warum ?!?
			return 0; // <- warum ?!?
		}

	}

bitte beantworte die fragen, die als kommentar hinzugefügt wurden

 

[Peter1988]

Hmmm ...

this.bigDecimal = bigDecimal; // <- warum ?!?
this.iInt = iInt; // <- warum ?!?

die beiden versteh ich, die gehören da nicht hin, hab ich auch gerade selbst entdeckt ^^

Aber
return new BigDecimal(""); // <- warum ?!?
return 0; // <- warum ?!?

die müssen doch dahin, denn wenn ich die weglasse, zeigt eclipse mir ne Fehlermeldung an, und zwar, dass ich ne Rückkehranweisung hinzufügen soll. Dies ist mit return getan.

Oder willst du mich damit auf den Pfad bringen, dass ich darauf private void Befehle mache?

 

[mikachu]

ne, aber sind die nicht gedacht, um die objekte, die als private in der class gekennzeichnet wurden, zurückzugeben?
aber warum sind die dann private?

 

[Peter1988]

Gute Frage, jedoch kann ich doch auch, wenn sie als private gekennzeichnet sind, weiterhin drauf zugreifen, in meiner class Taschenrechner, oder täusche ich mich da?

Die waren jedenfalls dafür gedacht, die Werte der Values zurückzugeben, daher auch getZahl und getOperator ...

Hatte mir das vorhin schon überlegt, ob ich mir die Werte nicht wie folgt holen kann:

vecSpeicher.get(i).getZahl();

Jedoch hat der mir, wenn ich mir die Werte über sysout ausgeben hab lassen, immer nur 0 ausgegeben ...[/code]

 

[Der_Dude]

Hiho Peter,

mhhh vecSpeicher ist dein Vektor?

mit vecSpeicher.add(7);

fügst du am ende des Vektors eine 7 hinzu.

mit vecSpeicher.get(i);

gibt dir dann einen Wert aus deinem Vektor an dem Index i (also an dem i-tem Feld des Vektors sozusagen).

 

[jPat]

Ich glaube, dein Algorithmus, funktioniert so nicht wirklich. "Normalerweise" bzw es gibt einen Ansatz mit 2 "Kellerspeichern", wobei in den Ersten du die Zahlen speicherst, in den Zweiten die Operanten, wenn alles fertig ist, dann kannst du die Zahlen aus dem Ersten vektor nehmen, eine nach der anderen, dann in dem Zweiten nachschauen, ob ein *,+,-,/ drin steht, und dann die gewünschte operation ausführen. Wie du die Punkt vor Strichrechnung hinbekommst, überlasse ich dir. (Man kann ja nicht alles veraten!) Aber mit den Kellerspeichern lässt es sich auf jeden fall lösen .... Mal im Internet nachschauen

 

[Peter1988]

hehe, das mit der punkt und strichrechnung hab ich mir schon überlegt, das sollte ich schon hinkriegen. ich denke ich hab jetzt auch den richtigen ansatz hier gefunden, werd's weiterhin mit dem vektor versuchen, wenn ich's nicht schaffe, versuche ich's mit dem kellerspeicher.

werde mich dann morgen mal melden, ob's dann letzendlich geklappt hat, trotzdem schonmal vielen dank für die durchaus große hilfe