einlesen, abfrage

[Uzi88]

hallo ich habe folgende Aufgabe:

http://puu.sh/5Igxc.png



ich hab das eigentlich fast fertig geschrieben nur verstehe ich nicht wie ich diese abfrage machen soll ob er ein stamm kunde ist oder nicht?

(ich bin ziemlich neu in java, bzw. programmieren)

 

[rme]

Hm. Vermutlich sollst du einlesen lassen, ob der Kunde ein Stammkunde ist und das Ergebnis in einer boolean-Variable speichern, die zum Beispiel [c]stammkunde[/c] heißt. Dann kannst du sie so abfragen:

if(stammkunde) {
  // setze Rabatt auf Stammkundenrabatt
} else {
  // setze Rabatt auf normalen Rabatt
}

 

[Uzi88]

genau aber wie mache ich diese abfrage? also Stammkunde: ja oder nein

 

[Uzi88]

import java.util.Scanner;


public class Aufgabe37 {

	public static void main(String[] args) {
		// TODO Auto-generated method stub
		Scanner scan = new Scanner(System.in);
		
		
		System.out.println("Einzelpreis? ");
		double epreis = scan.nextInt();
		
		System.out.println("Anzahl? ");
		double anzahl = scan.nextDouble();
		
		System.out.println("Stammkunde?");
		String stammkunde = scan.next();
				
		double skunde0 = 1;
		double skunde1 = 0.97;
		double skunde2 = 0.95;
		double skunde3 = 0.95;
		
		
		double gesamtpreis = anzahl * epreis; 
		
		if( anzahl < 10)
		{
			gesamtpreis = gesamtpreis * skunde1;
		}
		else if(anzahl >= 10 && anzahl < 200)
		{
			gesamtpreis = gesamtpreis * skunde2;
		}

		
	

			System.out.println(+gesamtpreis+"");
	} 
}

so sieht mein Code aus nur verstehe ich eben das mit dem stammkunden nicht

 

[rme]

Du könntest prüfen, ob "ja" eingegeben wurde:

if(stammkunde.equals("ja")) {
 ...
}

 

[Uzi88]

oh gott es klappt endlich haha fühlt sich das gut an XD
viel viel dank :applaus::toll:

import java.util.Scanner;


public class Aufgabe37 {

	public static void main(String[] args) {
		// TODO Auto-generated method stub
		Scanner scan = new Scanner(System.in);
		
		
		System.out.println("Einzelpreis? ");
		double epreis = scan.nextInt();
		
		System.out.println("Anzahl? ");
		double anzahl = scan.nextDouble();
		
		System.out.println("Stammkunde?");
		String stammkunde = scan.next(); 
		
		double skunde0 = 1;
		double skunde1 = 0.97;
		double skunde2 = 0.95;
		double skunde3 = 0.95;
		
		
		double gesamtpreis = anzahl * epreis; 
		
		if( anzahl < 10 && stammkunde.equals("ja"))
		{
			gesamtpreis = gesamtpreis * skunde1;
		}
		else if(anzahl >= 10 && anzahl < 200)
		{
			gesamtpreis = gesamtpreis * skunde2;
		}
		else if (anzahl >= 200)
			gesamtpreis = gesamtpreis * skunde3;
		else{
			gesamtpreis = gesamtpreis *  skunde0;
		}

		
	

			System.out.println(+gesamtpreis+"");
	} 
}

 

[rme]

Genieß den kognitiven Höhepunkt Er wird so lange anhalten, bis der erste Stammkunde mehr als 10 Artikel kauft und sich wütend beschwert, dass er den Normalo-Rabatt bekommt..

 

[Uzi88]

ich hab es grad getestet funkt doch alles oder siehst du was was ich nicht sehe ^^

 

[rme]

Stimmt, es ist umgekehrt: Wenn es mehr als 10 und weniger 200 Artikel sind, wird immer der Stammkundenrabatt genommen

 

[Uzi88]

du musstest ja diese blase zerplatzen -.-
naja ich bin immer noch dran, aber denke ich habs gleich :bloed:

 

[Uzi88]

ich hab das jetzt so geändert, aber irgendwie stimmt das nicht und ich komm nicht drauf warum.. kann jemand helfen? ;(

import java.util.Scanner;


public class Aufgabe37 {

	public static void main(String[] args) {
		// TODO Auto-generated method stub
		Scanner scan = new Scanner(System.in);
		
		
		System.out.println("Einzelpreis? ");
		double epreis = scan.nextInt();
		
		System.out.println("Anzahl? ");
		double anzahl = scan.nextDouble();
		
		System.out.println("Stammkunde?");
		String stammkunde = scan.next(); 
		
		double skunde0 = 1;
		double skunde1 = 0.97;
		double skunde2 = 0.95;

		
		
		double gesamtpreis = anzahl * epreis; 
		
		if( anzahl < 10 && stammkunde.equals("ja"))
		{
			gesamtpreis = gesamtpreis * skunde1;
		}
			else
			{
				gesamtpreis = gesamtpreis *skunde0;
			}
		if(anzahl >= 10 && anzahl < 200 && stammkunde.equals("ja"))
		{
			gesamtpreis = gesamtpreis * skunde2;
		}
			else
			{
				gesamtpreis = gesamtpreis * skunde1;
			}
		if (anzahl >= 200)
			gesamtpreis = gesamtpreis * skunde2;
	

			System.out.println(+gesamtpreis+"");
	} 
}

 

[rme]

Du hast zwei dafür relevante if-Abfragen: Eine in Zeile 28 und eine in Zeile 36. Die in Zeile 28 hat aber keine Auswirkung, denn es wird ja auf jeden Fall eins von beiden wahr - der if-Teil in Zeile 38 oder der else-Teil 42. Dadurch wird der Gesamtpreis also auf jeden Fall überschrieben, weil einer dieser Zweige genommen werden muss.

 

[Uzi88]

hmm und wie kläre ich das? wie gesagt ich bin neu in java

 

[rme]

Ich möchte dir nicht zu viel verraten, um die Ausbildung deiner Frustrationstoleranz nicht zu gefährden

Es gibt da mehrere Varianten. Du darfst eine if-Abfrage auch innerhalb des Blocks (ein Block ist das, was mit {geschweiften} Klammern abgetrennt ist) machen. Es wäre zum Beispiel möglich, erstmal die if-Abfrage zu machen, ob es ein Stammkunde ist - im if-Fall dann die Abfrage nach der Menge, um den Preis für Stammkunden zu berechnen. Im Else-Fall der if-Abfrage für den Stammkunden dann die if-Abfragen für die Mengen der Nicht-Stammkunden-Preise.

 

[Uzi88]

hä? ???:L
ich bin mir nicht sicher ob ich dich richtig verstanden habe aber meinst du das so?

		if(stammkunde.equals("ja"))
		{
			
			if(anzahl <10)
			{
				gesamtpreis = gesamtpreis * skunde1;
			}
			else{
				gesamtpreis = gesamtpreis * skunde0;
			}
		if(anzahl >= 10 && anzahl < 200 && stammkunde.equals("ja"))
		{
			gesamtpreis = gesamtpreis * skunde2;
		}
			else
			{
				gesamtpreis = gesamtpreis * skunde1;

 

[rme]

So ähnlich, der Anfang sieht gut aus.

Alles, was ab Zeile 2 passiert, betrifft nun nur noch Stammkunden. Du musst da also nur noch die Prüfungen machen, um Preise für Stammkunden zu ermitteln. Die erneute Abfrage in Zeile 11, ob der Kunde ein Stammkunde ist, fällt also weg.

Irgendwo schließt sich die Klammer für das if, das in Zeile 2 geöffnet wurde. Da kommt dann ein else hin, in dem du die gleichen Abfragen nochmal machst, aber andere Preise ermittelst.

 

[Uzi88]

ich checks echt nicht.. das will einfach nicht klappen, ich hab es jetzt so umgeschrieben kp was da wieder los ist

import java.util.Scanner;


public class Aufgabe37 {

	public static void main(String[] args) {
		// TODO Auto-generated method stub
		Scanner scan = new Scanner(System.in);
		
		
		System.out.println("Einzelpreis? ");
		double epreis = scan.nextInt();
		
		System.out.println("Anzahl? ");
		double anzahl = scan.nextDouble();
		
		System.out.println("Stammkunde?");
		String stammkunde = scan.next(); 
		
		double skunde0 = 1;
		double skunde1 = 0.97;
		double skunde2 = 0.95;

		
		
		double gesamtpreis = anzahl * epreis; 
		
		if(stammkunde.equals("ja"))
		{
			
			if(anzahl <10)
			{
				gesamtpreis = gesamtpreis * skunde1;
			
				if (anzahl >= 10 && anzahl < 200)
				{
					gesamtpreis = gesamtpreis *skunde2;
					
				}
					if(anzahl >= 200)
					{
						gesamtpreis = gesamtpreis *skunde2;
					}
			{
				gesamtpreis = gesamtpreis * skunde2; 
			}
		} 

	}
		else {gesamtpreis = gesamtpreis *skunde0;
		
		}System.out.println(+gesamtpreis+"");
}
}

 

[rme]

Jetzt hast du ein bisschen zu viel verschachtest - in Zeile 35 schaust du z.B., ob die anzahl >= 10 ist, dabei befindet sich die Abfrage noch innerhalb einer anderen if-Abfrage, die sicherstellt, dass die anzahl < 10 ist. Das in Zeile 35 kann also nie wahr werden.

Vielleicht könntest du dir einen Programmablaufplan oder so zeichnen, um das erstmal graphisch zu lösen?

 

[Uzi88]

ok ich hab es soweit gelöst aber das ist so nicht komplett richtig

import java.util.Scanner;


public class Aufgabe37 {

	public static void main(String[] args) {
		// TODO Auto-generated method stub
		Scanner scan = new Scanner(System.in);
		
		
		System.out.println("Einzelpreis? ");
		double epreis = scan.nextInt();
		
		System.out.println("Anzahl? ");
		double anzahl = scan.nextDouble();
		
		System.out.println("Stammkunde?");
		String stammkunde = scan.next(); 
		
		double skunde0 = 1;
		double skunde1 = 0.97;
		double skunde2 = 0.95;

		
		
		double gesamtpreis = anzahl * epreis; 
		
		if(stammkunde.equals("ja"))
		{
			
			if(anzahl <10)
			{
				gesamtpreis = gesamtpreis * skunde1;
			}
			else if(anzahl >= 10 && anzahl <200)
				{
				gesamtpreis = gesamtpreis * skunde2;
				}
			else if(anzahl >200)
			{
				gesamtpreis = gesamtpreis * skunde2;
			}
	}
		else {gesamtpreis = gesamtpreis *skunde0;
		
		}System.out.println(+gesamtpreis+"");
}
}

mir fehlt das "wenn kein stammkunde aber mehr als 10 artikel"

 

[rme]

Der Teill "wenn kein Stammkunde" ist bei dir Zeile 44. Da setzt du unabhängig von der Anzahl den Rabatt auf skunde0. Beheben kannst du das, indem du die if-Abfragen aus Zeile 31 - 42 dort wiederholst, aber eben andere Preise vergibst

 

[Uzi88]

rat mal wer endlich die lösung hat -.-

import java.util.Scanner;


public class Aufgabe37 {

	public static void main(String[] args) {
		// TODO Auto-generated method stub
		Scanner scan = new Scanner(System.in);
		
		
		System.out.println("Einzelpreis? ");
		double epreis = scan.nextInt();
		
		System.out.println("Anzahl? ");
		double anzahl = scan.nextDouble();
		
		System.out.println("Stammkunde?");
		String stammkunde = scan.next(); 
		
		double skunde0 = 1;
		double skunde1 = 0.97;
		double skunde2 = 0.95;

		
		
		double gesamtpreis = anzahl * epreis; 
		
		if(stammkunde.equals("ja"))
		{
			
			if(anzahl <10)
			{
				gesamtpreis = gesamtpreis * skunde1;
			}
			else if(anzahl >= 10 && anzahl <200)
				{
				gesamtpreis = gesamtpreis * skunde2;
				}
			else if(anzahl >=200)
			{
				gesamtpreis = gesamtpreis * skunde2;
			}
	}
		else {
			if(anzahl <10)
			{
				gesamtpreis = gesamtpreis * skunde0;
			}
			else if(anzahl >= 10 && anzahl <200)
				{
				gesamtpreis = gesamtpreis * skunde1;
				}
			else if(anzahl >=200)
			{
				gesamtpreis = gesamtpreis * skunde2;
			}
		}
		System.out.println(+gesamtpreis+"");
}
}

keine widersprüche bitte :bae:

 

[rme]

Die Klammern in Zeilen 43 und 59 sind falsch eingerückt.

So ist's gut und erfüllt die Aufgabe

 

[Uzi88]

hätte man das irgendwie besser und einfacher lösen können? vielleicht mit ner schleife oder so?

 

[Uzi88]

ich bin eigentlich total tot und sowas von keine lust aber ich hätte noch ne aufgabe

lust und zeit mit mir zu lösen?

 

[rme]

Eine Schleife würde hier nicht helfen, da ja keine Ausführung von irgendwas wiederholt werden soll.

Man könnte höchstens sehen, dass der Fall anzahl >= 200 bei beiden gleich behandelt wird und ihn getrennt abfragen.. aber ob es das übersichtlicher macht, ist eine andere Frage - ich denke nicht, denn wenn man das Programm später mal anpassen möchte, wäre es seltsam, dass einige Preise innerhalb und andere außerhalb der Stammkunden-Abfrage geändert werden.

Ansonsten: Da sind 6 unterschiedliche Fälle, das ergibt eine Entropie von 2.5sh, was nichts anderes bedeutet, als dass du mindestens 3 if-Abfragen benötigen wirst, um das Problem zu lösen. Falls du interessiert bist, wie das mit 3 Abfragen geht, könnte ich dir den Code zeigen - aber er wäre dann sehr seltsam und unwartbar. In der Praxis würde man es mit deinen Mitteln so machen, wie du es gemacht hast.

Später könnte man das mit Datenstrukturen lösen, damit der Code schöner aussieht, aber da kommst du noch hin

Edit: Die Fragen zur neuen Aufgabe in einen neuen Thread

 

[Uzi88]

ok ich nenne das them "primfaktorzerlegung" hoffe du bist wieder dabei

und ja vielen dank nochmal