Java Problem

[[GS]_WinDowS_]

Hallöchen..
Also für eine Schulaufgabe soll ich die Klasse Vertreter erstellen & anschließend einen Vertreter erstellen in der Startklasse..

Nun möchte ich mit der Methode "berechneProvision"
die Provision eines Vertreters berechnen.

private double provision;
		
		public double berechneProvision(double provision){
		return this.umsatz * this.newProvisionssatz / 100;
			}

		public double getProvision(){
				return this.provision;
			}

ich habe einen ausgangs Provisionssatz, der sich allerdings abhängig vom Umsatz ändert. Und den "neuen" Provisionssatz (newProvisionssatz) bestimmte ich so:

private double newProvisionssatz;
			public void ermittleProvisionssatz(double newProvisionssatz) 
			{
				if (this.umsatz >= 20000)
					{
					newProvisionssatz = Vertretter.provisionssatz + 10;
					}
				
				else 
					{
						if (this.umsatz >= 10000) {
							newProvisionssatz = Vertretter.provisionssatz + 5;
							}
						else 
					
						{newProvisionssatz = Vertretter.provisionssatz;
						}
					}
				}

Problem: Wenn ich wie oben erwähnt, die Provision so ausrechne:

return this.umsatz * this.newProvisionssatz / 100;

zeigt er mir 0 an...

Muss ich newProvisionssatz als einen Parameter oder irgendwie so zwischenzeitlich abspeichern?

Lg... ._.

[snow]

Du hast in ermittleProvisionssatz newProvisionssatz als Parameter obwohl es nie verwendet wird. Dadurch wird in diesem Kontext deine Instanzvariable newProvisionssatz überschrieben. Lösen kannst du das, indem du dort den Parameter komplett entfernst oder die Zuweisung explizit mit this.newProvisionssatz = /* ... */ machst.

Kleiner Tipp: Schau dir nochmal an was Parameter sind und wann man sie verwendet.

[[GS]_WinDowS_]

Also den newProvisionssatz bestimmte ich ja so, während ich in der Startklasse dem neuen Objekt den Umsatz zuvor zuweise

p1.setUmsatz(50000);

Vertretter.setProvisionssatz(10);

 Spoiler

private double newProvisionssatz;
		
				public double getNewProv()
					{
						return this.newProvisionssatz;
					}
		
				public void ermittleNewProv(double newProvisionssatz) 
					{
						if (this.umsatz >= 20000)
						{
							this.newProvisionssatz = Vertretter.provisionssatz + 10;
						}
			
						else 
						{
							if (this.umsatz >= 10000) 
								{
									this.newProvisionssatz = Vertretter.provisionssatz + 5;
								}
							else 
								{
									this.newProvisionssatz = Vertretter.provisionssatz;
								}
						}
					}

nur speichert er mir den newProvisionssatz nicht richtig ab..

hab mir nämlich denn ma printen lassen und er sagt mir 0.. obwohl umsatz>20000 ist.. sollte er mir ja 20 anzeigen (alter Provisionssatz + 10= 10+10 = 20)

Wo steckt hier der Fehler? :/

[Warnuk3r]

Hab mal dein Code von oben genommen und es funktioniert, denke deine Parameter versauen das ganze. Wie snow schon sagte, du solltest dir dringend anschauen wie das mit den Parametern funktioniert.

class Vertreter {
	
	private double provision;
	private double umsatz;
	private double newProvisionssatz;
	private double provisionssatz;
	
	public Vertreter() {
		provisionssatz = 10;
	}
	
	public void setUmsatz(double umsatz) {
		this.umsatz = umsatz;
	}
	
	public double berechneProvision(){
		return this.umsatz * this.newProvisionssatz / 100;
	}

	public double getProvision(){
		return this.provision;
	}
	
	public void ermittleProvisionssatz() 
	{
		if (this.umsatz >= 20000) {
			newProvisionssatz = provisionssatz + 10;
		} else {
			if (this.umsatz >= 10000) {
				newProvisionssatz = provisionssatz + 5;
			} else {
				newProvisionssatz = provisionssatz;
			}
		}
	}
	
	public static void main(String[] param) {
		Vertreter v = new Vertreter();
		v.setUmsatz(25000);
		v.ermittleProvisionssatz();
		System.out.println(v.berechneProvision());
	}
}

Umsatz 25000
Provisionssatz = 10+10= 20
Umsatz * Provisionsssatz / 100 = 5000
Konsole gibt aus: 5000.0

Also ist alles so wie gewollt?

[[GS]_WinDowS_]

So habs jetzt...

Also mein Fehler war, dass ich "erwartet" habe, dass die Startklasse den neuen Provisionssatz automatisch ausrechnet ohne das ich es extra angeben muss, da sie den neuen ja bei der Provision&Gesamtgehalt braucht..

Naja alles neu geschrieben, mir eine Struktur beigebracht..

Danke dir vielmals Warnuk3r

Noch eine letzte Frage:
Wie kann ich bei der erstellung der neuen Klasse die Attribute
gleich reinschreiben ohne v1.setName...

Also so:
Vertreter v = new Vertreter("Udo", 2000);

[Zunft]

Originally Posted by [GS]_WinDowS_

Noch eine letzte Frage:
Wie kann ich bei der erstellung der neuen Klasse die Attribute
gleich reinschreiben ohne v1.setName...

Also so:
Vertreter v = new Vertreter("Udo", 2000);

Dann übergibst du die Daten an den Konstruktor der aufgerufen wird:

public Vertreter(String name, int number) 
{
	provisionssatz = 10;
}

[[GS]_WinDowS_]

Wo genau?

 Spoiler

Hab es bei 1 & 2 eingefügt..
also:

public Vertretter(String name, double umsatz)
		{
			festgehalt = 2000;
			provisionssatz = 10;
		}

und

public class Vertretter(String name, double umsatz) {

	private String name;

Geht aber beides nicht.. :/

[Zunft]

Am Besten liest du dir mal das hier bzw. das komplette Buch durch.

[[GS]_WinDowS_]

Hat perfekt funktioniert.. hab das this.name = name vergessen.

Vielen herzlichen Dank an alle