Erstellen eines Steuerprogramms mittels if-anweisungen

[TeddyderBär]

Brauche Hilfe in Bezug auf Aufgabe unten: Ist der Anfangsgedanke richtig, was muss ich beachten?
Wäre echt froh wenn ich mal einen vernünftigen Anfang hinbekommen würde. bzw einen groben Aufbau des Programms.


Aufgabe
Die Einkommensteuer in den USA wird nach einem recht einfachen Verfahren berechnet, das auf Steuerklassen
und zu versteuerndem Einkommen beruht. Es gibt vier Steuerklassen: Ledige, Verheiratete mit gemeinsamer
Veranlagung sowie Witwe/r, Verheiratete mit getrennter Veranlagung und Haushaltsvorstand. Die Steuersätze
für das Jahr 2005 für jede dieser Klassen sind in folgender Tabelle dargestellt: Tabelle bekomme ich net kopiert, vielleicht wird das Problem trotzdem klar!? (Es gibt Stufungen von 10 bis 35% bei den Einkommen.)
Bei einem Ledigen mit einem zu versteuernden Einkommen von $10000 würden also die ersten $7300 mit 10%
versteuert und die verbleibenden $2700 mit 15%. Insgesamt wären hier also $730 + $405 = $1135 Steuern zu
zahlen. Schreiben Sie ein Java-Programm, das nach Eingabe der Steuerklasse und des zu versteuernden
Einkommens die abzuführenden Steuern ermittelt.


[/img]

public class EKST  {

    public static void main(String[] args)	{
	
	int zahl1 = Integer.parseInt (args[0]);		//zahl1 = Steuerklasse 1,2,3,4
	double zahl2 = Integer.parseInt (args[1]);	//zahl2 = Einkommen
	double Steuer1 = 0;

	if (zahl1 = 1);
	if (zahl2 <=7300);
		Steuer1 = zahl2*10/100;

	
	
	System.out.println("Steuer: "+Steuer1 +"Dollar");

}

}

 

[DocRandom]

..wäre aber nett, wenn Du die Tabelle dazu auch posten könntest, dann würden wir* uns mit der Hilfestellung etwas leichter tun!

mfg
DocRandom

* ich zumindest!

 

[SlaterB]

if (zahl2 <=7300);

Vorsicht mit dem Semikolon, das macht hinter einem if nie Sinn,
derzeit wird Steuer1 = zahl2*10/100; immer ausgeführt

wenn du Double parsen willst benutze vielleicht Double.parseDouble,

allgemein:
arbeite auf jeden Fall auch mit Listen/ Arrays und ähnlichem,
if (zahl2 <=7300)
ist zu spezifisch,
eher
if (zahl2 <=grenze[0][steuertyp]) {
Steuer1 = zahl2*steuersatz[0][steuertyp];
}

so dass diese Zeile allgemein für die untereren Grenzen aller Steuerklassen funktioniert,
(falls bei allen Klassen so eine Unterscheidung da ist)

und natürlich von Variablen wie zahl2 wegkommen hin zu 'einkommen'
oder was auch immer

 

[TeddyderBär]

Weiß leider keine andere Möglichkeit die Tabelle hochzuladen, sorry!

fotos.mtb-news.de/fotos/showphoto.php?photo=315008&cat=509

Wie würd ich den meinen Anfang zum laufen bringen? (Damit ich wenigstens ein kleines Erfolgserlebnis habe :bahnhof: )

 

[DocRandom]

..nun lt. Deiner Tabelle hast Du 4 verschiedene Steuerklassen und je 6 dazu gehörende Steuersätze!
Entweder packst Du dies in ein Array oder eine Klasse.
Die Berechnung der Steuer kannst du per Schleife lösen, da Du ja immer wieder die selben Schritte durchführst!

Für eine weitere Hilfestellung, müsste ich schon etwas mehr Code Deinerseits sehen.

mfg
DocRandom

 

[Guest]

class EKST {

	public static void main (String [] args) {

	double Steuerklasse = Integer.parseInt (args[0]);
	double Einkommen = Integer.parseInt (args [1]);
	
	double Steuern = 0 ;

	if(Steuerklasse == 1 && Einkommen <=7300)
	Steuern = Einkommen*0.1;

	if(Steuerklasse == 1 && Einkommen >=7301 && Einkommen <=29700)
	Steuern = ((7300*0.1) + ((Einkommen - 7300)*0.15));

	if(Steuerklasse == 1 && Einkommen >=29701 && Einkommen <=71950)
	Steuern = ((7300* 0.1) + (22399*0.15) + ((Einkommen - 29700)*0.25));

	if(Steuerklasse == 1 && Einkommen >=71951 && Einkommen <=150150)
	Steuern = ((7300* 0.1) + (22399*0.15) + (42249*0.25) + ((Einkommen - 71950)*0.28));

	if(Steuerklasse == 1 && Einkommen >=150151 && Einkommen <=326450)
	Steuern = ((7300* 0.1) + (22399*0.15) + (42249*0.25) + (71950*0.28) +((Einkommen - 150151)*0.33));

	if(Steuerklasse == 1 && Einkommen >=326451)
	Steuern = ((7300* 0.1) + (22399*0.15) + (42249*0.25) + (71950*0.28) + (150151*0.33) + ((Einkommen - 326450)*0.35));

	
	if(Steuerklasse == 2 && Einkommen <=14600)
	Steuern = (Einkommen*0.1);
	
	if(Steuerklasse == 2 && Einkommen >=14601 && Einkommen <=59400)
	Steuern = ((14600*0.1) + ((Einkommen - 14600)*0.15));
	
	if(Steuerklasse == 2 && Einkommen >=59401 && Einkommen <=119950)
	Steuern = ((14600*0.1) + (44799*0.15) + ((Einkommen - 59401)*0.25));
	
	if(Steuerklasse == 2 && Einkommen >=119951 && Einkommen <=182800)
	Steuern = ((14600*0.1) + (44799*0.15) + (60549*0.25) + ((Einkommen - 119951)*0.28));

	if(Steuerklasse == 2 && Einkommen >=182801 && Einkommen <=326450)
	Steuern = ((14600*0.1) + (44799*0.15) + (60549*0.25) + (62849*0.28) + ((Einkommen - 182801)*0.33));

	if(Steuerklasse == 2 && Einkommen >=326450)
	Steuern = ((14600*0.1) + (44799*0.15) + (60549*0.25) + (62849*0.28) + (143649*0.33) + ((Einkommen - 326450)*0.35)); 



	System.out.println("Steuern: "+Steuern);


	}
}

Ok, das funzt schon ganz gut, obwohl man das bestimmt auch kürzer machen kann?! Arrays und Schleifen hatten wir in der Vorlesung noch nicht. Mich würde jetzt interessieren was man schöner und kürzer machen kann? Kommentare werde ich hinzufügen wenn ich fertig bin.

 

[TeddyderBär]

Das sind jetzt nur die ersten beiden Steuerklassen, ist ne ganz schöne Schreibarbeit. Oben hab ich double noch nicht ausgetauscht...

 

[DocRandom]

..auf alle fälle würde ich das mal andern:
von:

double Steuerklasse = Integer.parseInt (args[0]);
double Einkommen = Integer.parseInt (args [1]);

zu:

int Steuerklasse = Integer.parseInt (args[0]);
double Einkommen = Double.parseDouble(args [1]);

mfg
DocRandom

 

[DocRandom]

..nun, da Ihr noch keine Arrays und Schleifen kennt, wird Dir nix anderes übrig bleiben als alles in dieser Manier runter zu klopfen!
Ich habe mir allerdings erlaubt Deinen bisherigen Code etwas zu bereinigen.
Immerhin werden Variablen in Java klein geschrieben.
Außerdem habe ich etwas Struktur in Dienen Code gebracht.

class EKST {
		
	public static void main (String [] args) {
		int steuerklasse = Integer.parseInt (args[0]); 		// Steuerklasse
		double einkommen = Double.parseDouble(args [1]); 	// Einkommen
	   
		double steuern = 0 ;

		if (steuerklasse == 1) {
			if (einkommen <= 7300)
				steuern = (einkommen * 0.1);
		   
			if (einkommen > 7300 && einkommen <= 29700)
				steuern = ((7300 * 0.1) + ((einkommen - 7300) * 0.15));
		   
			if ( einkommen > 29700 && einkommen <= 71950)
				steuern = ((7300 * 0.1) +((29700 - 7300)* 0.15)+ ((einkommen - 29700) * 0.25));
			
			if (einkommen > 71950 && einkommen <= 150150) 
				steuern = ((7300 * 0.1) + ((29700 - 7300)* 0.15) + ((71950 - 29700) * 0.25)+ ((einkommen - 71950)* 0.28));
				
			if (einkommen > 15150 && einkommen <= 326450)
				steuern = ((7300 * 0.1) + ((29700 - 7300)* 0.15) + ((71950 - 29700) * 0.25)+ ((150150 - 71950)* 0.28)+ ((einkommen - 150150) * 0.33 ));
			
			if ( einkommen > 326450)
				steuern = ((7300 * 0.1) + ((29700 - 7300)* 0.15) + ((71950 - 29700) * 0.25)+ ((150150 - 71950)* 0.28)+ ((326450 - 150150) * 0.33 ) + ((einkommen - 326450) * 0.35));
		}
		if ( steuerklasse == 2 ) {
			if ( einkommen <= 14600)
				steuern = (einkommen * 0.1);
			
			if ( einkommen > 14600 && einkommen <= 59400 )
				steuern = ((14600 * 0.1) + ((einkommen - 14600) * 0.15));
			
			if ( einkommen > 59400 && einkommen <= 119950)
				steuern = ((14600 * 0.1) + ((59400 - 14600) * 0.15)+ ((einkommen - 59400) * 0.25));
			
			if ( einkommen > 119950 && einkommen <= 182800)
				steuern = ((14600 * 0.1) + ((59400 - 14600) * 0.15)+ ((11990 - 59400) * 0.25) +((einkommen - 11990) * 0.28));
			
			if ( einkommen > 182800 && einkommen <= 326450)
				steuern = ((14600 * 0.1) + ((59400 - 14600) * 0.15)+ ((11990 - 59400) * 0.25) +((182800 - 11990) * 0.28) +((einkommen - 188200) * 0.33));
			
			if ( einkommen > 326450)
				steuern = ((14600 * 0.1) + ((59400 - 14600) * 0.15)+ ((11990 - 59400) * 0.25) +((182800 - 11990) * 0.28) +((326450 - 188200) * 0.33) + ((einkommen - 326450) * 0.35));
		}
		if ( steuerklasse == 3) {
			
		}
		if ( steuerklasse == 4) {
			
		}
		System.out.println("Zu zahlende Steuern: "+steuern);
	}
}

mfg
DocRandom

 

[SlaterB]

         if (einkommen > 15150 && einkommen <= 326450)

fetten Fehler eingebaut!
15.000 stat 150.000


ohne Arrays kann man zumindest mit einem anderen Aufbau noch ein wenig Wiederholung sparen,
Ansichstsache was schöner ist,

double resteinkommen = einkommen
if (steuerklasse == 1) {
  if ( resteinkommen > 326450) {
      steuer += (resteinkommen -326450)*0.35;
      resteinkommen = 326450;
  }
  if ( resteinkommen > 150150) {
      steuer += (resteinkommen -150150)*0.33;
      resteinkommen = 150150;
  }
  usw. am Ende 

  steuer += resteinkommen * 0.1;
}

andererseits könnte man Werte wie
(14600*0.1) + (44799*0.15) + (60549*0.25) + (62849*0.28) + (143649*0.33)
fertig ausrechnen und dann 45436 oder so hinschreiben, wäre auch kurz und insgesamt schneller,

drei Varianten mit unterschiedlichen Blickwinkeln

 

[TeddyderBär]

Erstmal vielen Dank für die Tipps. Werde es mal ein wenig umändern.

Hab dasselbe jetzt auch mit IO-Anweisungen hinbekommen.

So langsam macht es auch Spass. Werde bestimmt mit den nächsten Aufgaben wieder ein paar Fragen haben, die ich dann hier poste