Werte in Stacks zusammenrechnen

[Gamma]

Hallo,
Ich hab mal ne Frage:
Wenn ich einen Stack mit 2 belibigen werten drinnen hab, z.B. 0,31 und 0,49, wie lautet dann der Code um beide zusammenzurechnen und z.B. als String "Ergebnis" abzuspeichern.

Mfg Gamma

 

[Schandro]

(Annahme das du mit Stack die Klasse java.util.Stack meinst)
Du addierst das Ergebnis vom 1. pop()-Aufruf zum 2. pop()-Aufruf und rufst darauf String.valueOf(...) auf. Wo ist das Problem?

 

[Gamma]

Bin noch ein totaler Anfänger wäre nett wenn du mal den kompletten Code schickst.

 

[eRaaaa]

also wenn du wirklich java.util.Stack meinst, dann z.b. so

        Stack<Double> stack = new Stack<Double>();
		stack.push(0.31);
		stack.push(0.49);
		String ergebnis = String.valueOf(stack.pop() + stack.pop());

oder wie sieht dein stack aus?

 

[Gamma]

ja ich meine java.util.stack, aber das problem ist ich weiß vorher nicht wie viele Werte es gibt. Und was soll <Double>?

 

[Ark]

Kann es sein, dass du gerade so ... mittendrin anfängst? Ich meine nicht nur die Analyse des Problems, sondern auch, was deine Erfahrungen im Umgang mit den benutzten Strukturen angeht!?

Ark

 

[Schandro]

Und was soll <Double>

Les dir was über "Generics" durch, wär zu umständlich das hier zu erklären.

aber das problem ist ich weiß vorher nicht wie viele Werte es gibt

Warum hast du das nicht am Anfang gesagt?^^
Benutz ne Schleife: (im Gegensatz nur Methode pop bleiben hier die Elemente übrigens noch im Stack)

		java.util.Stack<Double> stack = new java.util.Stack<Double>();

		stack.push(0.51);
		stack.push(0.27);
		stack.push(0.02);

		double sum = 0;
		for(double d : stack){
			sum += d;
		}
		System.out.println(sum);

 

[eRaaaa]

mit pop evtl. so : ?

int size = stack.size();
		double sum = 0;
		for (int i = 0; i < size; i++) {
			sum += stack.pop();
		}
		System.out.println(sum);

 

[Schandro]

oder so:

        double sum = 0;
        while(!stack.empty()){
            sum += stack.pop();
        }
        System.out.println(sum);

 

[eRaaaa]

hast gewonnen

 

[Wortraum]

Und was soll <Double>?

Das gibt an, welche Objekte auf den Stapel gelegt werden können, hier also nur Objekte vom Typ Double. Ein anderes Beispiel wäre eine Liste für Strings:

ArrayList<String> myList = new ArrayList<String>();

Wenn man dann versucht, etwas anderes als String der Liste hinzuzufügen, gibt es eine Fehlermeldung. Man kann also sagen, daß es angibt, was in einem anderen Objekt, also beispielsweise einem Stapel oder einer Liste, enthalten sein kann. Das sollte Dir als Information reichen, alles andere ist vorerst zu kompliziert. Der Vollständigkeit halber sei hier dennoch ein Link zur Sun-Dokumentation über allgemeine (generische) Typen erwähnt: Lesson: Generics

 

[Gamma]

also ich hab das jetzt so gemacht:

int size = warenkorb.size();
        String sum = "0";
        for (int i = 0; i < size; i++) {
            sum += warenkorb.pop();
        }
           
			JOptionPane.showMessageDialog (null, "Zu zahlen:"+ sum + "€");
			}
			}

Damit kriege ich Ergebnisse wie z.B. 0,490,49 wenn der wert 0,49 ist und ich in zwei mal in den stack tue. Ich will aber gerne 0,98 da stehen haben.

 

[Leroy42]

gelöscht
Edit: Huch! Ich habe die meisten Posts überlesen!

 

[Gamma]

Also hier mal das ganze

package stack;

import java.util.Stack;
import java.lang.NullPointerException;
import javax.swing.*;
import java.util.EmptyStackException;

public class stacks1 {

	
	

	public static void main(String[] args) {
		 		 
		
		
		String Apfel = "0,49";
		
		String Kaugummi = "0,29";
	
		String Pflaume = "0,29";
		
		String Banane = "0,39";
		
		String Orange = "0,51";
		
		
	    Stack warenkorb = new Stack();

	    
		try{
			
			for(int Nr = 1; Nr <= 1000000000;Nr++)
			{
		String Produkt = JOptionPane.showInputDialog("Wählen sie ihr Produkt. Zurzeit auf Lager sind:\nApfel\nKaugummi\nPflaume\nBanane\nOrange\nBitte Groß- und Kleinschreibung beachten!\nUm zur Kasse zu gehen drücken sie auf Abbrechen"); 
			
		if (Produkt.equals("Apfel"))
		warenkorb.push(Apfel);
		if (Produkt.equals("Kaugummi"))
		warenkorb.push(Kaugummi);
		if (Produkt.equals("Pflaume"))
		warenkorb.push(Pflaume);
		if (Produkt.equals("Banane"))
		warenkorb.push(Banane);
		if (Produkt.equals("Orange"))
		warenkorb.push(Orange);
			}
		}
		catch(NullPointerException n){
			
			
		
		int size = warenkorb.size();
        String sum = "0";
        for (int i = 0; i < size; i++) {
            
       	sum += warenkorb.pop();
        }
           
			JOptionPane.showMessageDialog (null, "Zu zahlen:"+ sum + "€");
			}
			}}

 

[Leroy42]

Damit kriege ich Ergebnisse wie z.B. 0,490,49 wenn der wert 0,49 ist und ich in zwei mal in den stack tue. Ich will aber gerne 0,98 da stehen haben.

int size = warenkorb.size();
        Double sum = 0.0;
        for (int i = 0; i < size; i++) {
            sum += warenkorb.pop();
        }
           
			JOptionPane.showMessageDialog (null, "Zu zahlen:"+ sum + "€");
			}
			}

 

[Leroy42]

Ach deine Inhalte sind ja Strings. Dann

int size = warenkorb.size();
        Double sum = 0.0;
        for (int i = 0; i < size; i++) {
            sum += Double.parseDouble(warenkorb.pop());
        }
           
			JOptionPane.showMessageDialog (null, "Zu zahlen:"+ sum + "€");
			}
			}

 

[Gamma]

er zeigt mir an
The operator += is undefined for the argument type(s) double, Object

 

[Noctarius]

Double sum = 0.0;
for (int i = 0; i < size; i++) {
    try {
        double value = Double.parseDouble(warenkorb.pop());
        sum += Double.parseDouble(value);
    } catch (NumberFormatException e) { ... }
}

 

[Gamma]

Also was ist falsch
wäre nett wenn jemand den ganzen code verbessert zurückschickt

package stack;

import java.util.Stack;
import java.lang.NullPointerException;
import javax.swing.*;
import java.util.EmptyStackException;

public class stacks1 {

	
	

	public static void main(String[] args) {
		 		 
		
		
		String Apfel = "0,49";
		
		String Kaugummi = "0,29";
	
		String Pflaume = "0,29";
		
		String Banane = "0,39";
		
		String Orange = "0,51";
		
		
	    Stack warenkorb = new Stack();

	    
		try{
			
			for(int Nr = 1; Nr <= 1000000000;Nr++)
			{
		String Produkt = JOptionPane.showInputDialog("Wählen sie ihr Produkt. Zurzeit auf Lager sind:\nApfel\nKaugummi\nPflaume\nBanane\nOrange\nBitte Groß- und Kleinschreibung beachten!\nUm zur Kasse zu gehen drücken sie auf Abbrechen"); 
			
		if (Produkt.equals("Apfel"))
		warenkorb.push(Apfel);
		if (Produkt.equals("Kaugummi"))
		warenkorb.push(Kaugummi);
		if (Produkt.equals("Pflaume"))
		warenkorb.push(Pflaume);
		if (Produkt.equals("Banane"))
		warenkorb.push(Banane);
		if (Produkt.equals("Orange"))
		warenkorb.push(Orange);
			}
		}
		catch(NullPointerException n){
			
			
			Double sum = 0.0;
			for (int i = 0; i < size; i++) {
			    try {
			        double value = Double.parseDouble(warenkorb.pop());
			        sum += Double.parseDouble(value);
			    } catch (NumberFormatException e) { 
			     	JOptionPane.showMessageDialog (null, "Ein Fehler ist aufgetreten");		
			    
			    }
			    JOptionPane.showMessageDialog (null, "Zu zahlen:"+ sum + "€");
			}
		 
			    	}}}

MfG Gamma

 

[eRaaaa]

ohweia ;D
geändert u.a.:
-stack mit generics !!!
-Strings (String Apfel = "0.49" )
- sum += ....value ist doch schon ein double
-JOptionPane.showMessageDialog ausserhalb der schleife !

so läufts zumindestens glaub ich so, wie du dir das vorstellst, allerdings ist der gesamte aufbau ein wenig komisch (u.a. die berechnung/abbruch in dem auffang der nullpointerexception zu machen ???:L)

import java.util.Stack;
import java.lang.NullPointerException;
import javax.swing.*;


public class stacks1 {

	public static void main(String[] args) {

		String Apfel = "0.49";

		String Kaugummi = "0.29";

		String Pflaume = "0.29";

		String Banane = "0.39";

		String Orange = "0.51";

		Stack<String> warenkorb = new Stack<String>();

		try {

			for (int Nr = 1; Nr <= 1000000000; Nr++) {
				String Produkt = JOptionPane
						.showInputDialog("Wählen sie ihr Produkt. Zurzeit auf Lager sind:\nApfel\nKaugummi\nPflaume\nBanane\nOrange\nBitte Groß- und Kleinschreibung beachten!\nUm zur Kasse zu gehen drücken sie auf Abbrechen");

				if (Produkt.equals("Apfel"))
					warenkorb.push(Apfel);
				if (Produkt.equals("Kaugummi"))
					warenkorb.push(Kaugummi);
				if (Produkt.equals("Pflaume"))
					warenkorb.push(Pflaume);
				if (Produkt.equals("Banane"))
					warenkorb.push(Banane);
				if (Produkt.equals("Orange"))
					warenkorb.push(Orange);
			}
		} catch (NullPointerException n) {

			Double sum = 0.0;
			while (!warenkorb.isEmpty()) {
				try {
					double value = Double.parseDouble(warenkorb.pop());
					sum += value;
				} catch (NumberFormatException e) {
					JOptionPane.showMessageDialog(null,
							"Ein Fehler ist aufgetreten");

				}

			}
			JOptionPane.showMessageDialog(null, "Zu zahlen:" + sum + "€");

		}
	}
}

/edit: klein bisschen refactoring?! ;D :

import java.util.Stack;
import javax.swing.*;

public class stacks1 {

	static Stack<String> warenkorb;

	public static void main(String[] args) {

		String Apfel = "0.49";

		String Kaugummi = "0.29";

		String Pflaume = "0.29";

		String Banane = "0.39";

		String Orange = "0.51";

		warenkorb = new Stack<String>();

		while(true) {
			String Produkt = JOptionPane
					.showInputDialog("Wählen sie ihr Produkt. Zurzeit auf Lager sind:\nApfel\nKaugummi\nPflaume\nBanane\nOrange\nBitte Groß- und Kleinschreibung beachten!\nUm zur Kasse zu gehen drücken sie auf Abbrechen");
			if (Produkt == null)
				break;
			if (Produkt.equals("Apfel"))
				warenkorb.push(Apfel);
			if (Produkt.equals("Kaugummi"))
				warenkorb.push(Kaugummi);
			if (Produkt.equals("Pflaume"))
				warenkorb.push(Pflaume);
			if (Produkt.equals("Banane"))
				warenkorb.push(Banane);
			if (Produkt.equals("Orange"))
				warenkorb.push(Orange);
		}

		JOptionPane.showMessageDialog(null, "Zu zahlen:" + getBill() + "€");

	}

	public static double getBill() {
		Double sum = 0.0;
		while (!warenkorb.isEmpty()) {
			try {
				sum += Double.parseDouble(warenkorb.pop());
			} catch (NumberFormatException e) {
				JOptionPane.showMessageDialog(null,
						"Ein Fehler ist aufgetreten");

			}

		}
		return sum;
	}
}

 

[Landei]

Ein bisschen mehr Refactoring (aus'm Kopp - keine Gewähr):

import java.util.Stack;
import java.util.Map;
import java.util.HashMap;
import javax.swing.*;
 
public class Stacks1 {
 
    private static Stack<String> warenkorb;
    private static Map<String, Double> preise = new HashMap<String,Double>();
  
    static {
       preise.put("Apfel",0.49);
       preise.put("Kaugummi",0.29);
       preise.put("Pflaume",0.29);
       preise.put("Banane",0.39);
       preise.put("Orange",0.51);
    }
 
    public static void main(String[] args) {
 
        warenkorb = new Stack<String>();
 
        while(true) {
            String produkt = JOptionPane
                    .showInputDialog("Wählen sie ihr Produkt. Zurzeit auf Lager sind:\nApfel\nKaugummi\nPflaume\nBanane\nOrange\nBitte Groß- und Kleinschreibung beachten!\nUm zur Kasse zu gehen drücken sie auf Abbrechen");
            if (produkt == null)
                break;
            if (preise.containsKey(produkt)) {
                warenkorb.push(produkt);
            } else {
               JOptionPane.showMessageDialog(null, "Wir führen kein Produkt " + produkt);
            }
        }
 
        JOptionPane.showMessageDialog(null, "Zu zahlen:" + getBill() + "€");
 
    }
 
    public static double getBill() {
        double sum = 0.0;
        while (!warenkorb.isEmpty()) {
                sum += preise.get(warenkorb.pop());
        }
        return sum;
    }
}

 

[madboy]

for (int nr = 0; nr <= Integer.MAX_VALUE; nr++) {

Endlosschleife? Macht man normalerweise so (wenn kein Zähler benötigt wird natürlich nur):

while (true) {

 

[Landei]

Stimmt, das ist extrem ungeschickt. Hatte ich nur mitkopiert und überlesen. Ist korrigiert.

 

[madboy]

Stimmt, das ist extrem ungeschickt. Hatte ich nur mitkopiert und überlesen. Ist korrigiert.

Dachte ich mir schon Wollte nur nicht dass der Threadersteller denkt, das macht man so...