StringToDouble Methode: Vorkommazahlen nicht gerechnet

[conspectumortis]

Hallo,

neues noch unbekanntest Forum in dem ich mich hoffentlich bald zurechtfinde =)

Ich habe eine Problem und komme nicht wirklich weiter:

Unsere Aufgabe bestand darin ein Java Programm zu schreiben (ohne Hilfsbibliotheken etc. ) mit dem man Strings die man über console mit java Arraystring 232.323 3.233 etc. eingibt ,an die StringToDouble Methode schickt ,den String auseinandernimmt (siehe meine Idee im Programm im Anhang) und den als double Wert an ein Array zurückschickt.

Ich würde gerne an meiner Lösung oder zumindest die Idee bleiben,nur...:

Es wird bei dem Programm die Vorkomma Zahlsumme nicht ausgerechnet bzw. ned korrekt zurückgegeben und ich frag mich wieso. Rein von der Logik her finde ich keine grossartigen Fehler (kann mich auch irren),daher dachte ich es steckt eher an der Programmierung und ich frag mal nach.



Wäre zumindest für einen Hinweis ,dann kann ichs selber nachlesen,dankbar woran es wirklich hakt.

//Öffentliche Klasse Arraystring
public class Arraystring {


	//öffentliche statische Methode swaprows
	public static double [] [] swaprows (double [] [] a,int i,int j){
		int k=a.length;
		double [] [] c= new double [k] [k];
		if (i>=k || j>=k|| j==i ) return a;
		for(int d=0; d<k ; d++){
				c [i] [d] = a [i] [d];
				a [i] [d] = a [j] [d];
				a [j] [d] = c [i] [d];
					}
		return a;
	}


	
	//öffentliche statische Methode addrows
	public static double [] [] addrows(double [] [] a, int i,int j,double f){
		int k=a.length;
		if(i>=k || j>=k) return a;
		if(j==i){
			for (int d=0;d<k;d++){
				a [i][d] = a [i] [d] + (f* a [i] [d]);
							}
			return a;
			}
		else
		for (int d=0;d<k;d++){
			a[i] [d] = a [i] [d] + (f * a [j] [d]);
		}
		return a;
	}



	
	
	//öffentliche statische Methode StringToDouble
	public static double StringToDouble (String s) {
		double t=0,r=0,a=0,g=0.1,h=0.1,b=0.0;
		double fa=10, n=s.length();
		char c=0;

		for (int i=0; i<n;i++){
			c = s.charAt(i);
			a = c - '0';
			t *=fa + a;
			if (a<=9 && a>=0 || c=='.' || c==','){

			if (c=='.' || c==','){
				for (i+=1;i<n;i++){
					c = s.charAt(i);
					b = c - '0';
					r+=(b*g);
					g=g*h;
		
					}
				}
			}
		}
			
	return r+t;
	}


	
	//Hauptmethode.
	public static void main (String [] args) {
	    	   int q=args.length/3,l=0;
	    	   double c [] []  = new double [q] [q];
	    	   for(int u=0; u<q; u++){
	    		   for(int v=0; v<q ; v++){
	    			   c [u] [v]= StringToDouble(args [l++]);	
	    			   
	    		   } 
	    	   }


	    	   for (int p=0;p<q;p++){
	    		   for (int o=0;o<q;o++){
	   	    		System.out.println(c [p] [o]);
	    		   }
	    	   }

			
	     
      	}
}

So siehts dann aus wenn ich es ausführe:

 

[SlaterB]

soviel Text und doch nicht auf den Punkt gekommen,

was genau geht denn nun nicht?
aus
> Es wird bei dem Programm die Vorkomma Zahlsumme nicht ausgerechnet bzw. ned korrekt zurückgegeben

lese ich ein Problem mit Vorkommastellen,
aber was zeigt dein Screnshot?:
es werden nur Zahlen wie 0,213 eingegeben und es kommt 0.213 raus,
was stimmt denn nun nicht?

 

[conspectumortis]

edit: war doch das richtige Bild,hier aber nochmal eins zur Verdeutlichung:



Die Summe vor der Kommazahl wird nicht berechnet und/oder zurückgegeben ,obwohl ich das in der Methode StringToDouble in der Zeile 50/51 mache und bei Zeile 66 zurückgebe.

Beispiel auf dem Bild: 3233.2332 wird nur als 0.2332 ausgegeben.

 

[SlaterB]

ok, ein Ansatz ist damit vorhanden,
aber schau dir mal folgendes Test-Programm an, ist das nicht unendlich leichter, einfacher, schöner usw?:

public class Test {

	public static double StringToDouble(String s) {
		double t = 0, r = 0, a = 0, g = 0.1, h = 0.1, b = 0.0;
		double fa = 10, n = s.length();
		char c = 0;

		for (int i = 0; i < n; i++) {
			c = s.charAt(i);
			a = c - '0';
			t *= fa + a;
			if (a <= 9 && a >= 0 || c == '.' || c == ',') {

				if (c == '.' || c == ',') {
					for (i += 1; i < n; i++) {
						c = s.charAt(i);
						b = c - '0';
						r += (b * g);
						g = g * h;

					}
				}
			}
		}

		return r + t;
	}

	public static void main(String[] args) throws Exception {

		System.out.println(StringToDouble("3.3"));
	}
}


-------
Ausgabe: 0.30000000000000004

ganz ohne Screenshot

------

egal, schaue es mir jetzt an, gleich kommt hoffentlich ein edit mit der Korrektur:


-------

edit:

1.
t ist anfangs 0, wird danach nur multipliziert, wird also immer 0 bleiben, wenn das a addiert werden soll, dann verwende nicht *=...

t = t * fa + a;
funktioniert weitaus besser, ausprobieren??

2.
der Punkt/ das Komma wird auch in t reingerechnet, du musst erst auf ./ Komma testen und nur im else-Fall t ändern

mit Debuggen oder simplen Ausgaben a la
System.out.println("ich bin dein Programm und mache mit Buchstabe .. folgendes: ..");
ganz leicht herauszufinden


---------

die doppelte ineinandergeschachtelte i-Schleife ist natürlich ganz schlimm,
müsste noch anders programmiert werden

 

[conspectumortis]

Danke dir SlaterB,funktioniert nun 1a


Nachdem du mich auf else aufmerksam gemacht hast und ich das korrigiert habe,funkioniert alles einwandfrei =)

Hatte es wohl immer beim durchgehen überlesen das da noch n else fehlt an der richtigen position :/
Der Teufel liegt doch im Detail


Das mit der inneren Schleife werde ich nochmal verbessern. mal schauen wie ich das hinkriege.

 

[conspectumortis]

Leider hab ich gemerkt das ich immer noch ein Problem mit dem Programm habe:


bei 0.11 bsw. kommt 0.1100000000007 oder 0.1100000000001 usw. heraus.

Erkennt jemand hier einen Umwandlungsfehler im code ?

 

[SlaterB]

double rechnen generell ungenau,
verwende zur Ausgabe DecimalFormat

oder rechne genau mit BigDecimal statt double (dauert dann 100x so lange, aber bei einfachen Rechnungen auch nur wenige Nanosekunden)