Methoden Switch wählt das Falsche

[rooonny]

Guten Tag,

ich stell mich erst einmal vor: Ich bin der Ronny bin 18 Jahre und programmiere ein bisschen rum
Darauf gekommen bin ich eigentlich durch die Berufsschule, weil da haben wir JAVA-Programmierung und das hat mir halt Spaß gmehacht und seitdem mach ich so kleine Projekte wie: Taschrechner, BMI-Rechner und so weiter. Als letzen Schritt haben ich mit meinem Lehrer das Speichern besprochen und das klappt auch alles ganz gut. (mit BuffredWriter)

Als ich das alles ganz gut verstanden habe, habe ich mich an mein erstes "großes" Projekt getraut.

Und da bin ich jetzt leider an ein Problem gestoßen mit switch und case. Das ist ja eigentlich eine ganz einfach Sache, aber leider klappt es bei mir nicht

Ich möchte für jeden Tisch eine eigene Datei erstellen, deshalb habe ich diese Anweisung in eine eigene Methode geschrieben: void schutzmittelEintrag(final int nr)
Es wird der button: schutzeintragen gedrückt damit das geschieht.
Der Übergabeparameter ist da, damit bei jeder Datei die Nummer hinzugefügt wird.
Mit einem chioce (hier zahlEins) wähle ich den Tisch aus
und mit switch welche nummer in den Dateinamen geschrieben wird, damit ich es zuordnen kann.

ich füge mal den Code ein:

void schutzmittelEintrag(final int nr){
		
		schutzeintragen.addActionListener(new ActionListener(){
			public void actionPerformed(ActionEvent e){
				String mittel;
				String datum;
				String ftext;
				mittel = cschutzm.getSelectedItem();
				datum = schutzdatum.getText();
				ftext = mittel + " am " + datum + " gegeben\n";
				schutzhistorie.appendText(ftext);
				
				
				try
				{
				BufferedWriter schutzspeicher;
				schutzspeicher = new BufferedWriter(new FileWriter(nr+ "schutzdatei.txt", true));
				schutzspeicher.write(ftext);
				schutzspeicher.close();
				}
			catch(IOException fehler)
			{}
			
			}
		});}
		
			
			
			
		void schutzmittelladen(final int nr){
			bfuellen.addActionListener(new ActionListener(){
				public void actionPerformed(ActionEvent e){
					
					try
					{	
						BufferedReader texteingabe;
						texteingabe = new BufferedReader(new FileReader("schutzdatei" + nr + ".txt"));
						
						String text = "";
						String textzeile;
						textzeile = texteingabe.readLine();
						while(textzeile != null)
						{
							
							text = text + textzeile + "\n";
							textzeile = texteingabe.readLine();
						}
						texteingabe.close();
						schutzhistorie.setText(text);
					}
					catch(IOException f)
					{}
					}});
			}

		
			

		void auswahlTisch(){
			
			if(kob.getSelectedItem() == "K")
			{ int i = Integer.parseInt(zahlEins.getSelectedItem());
				switch(i)
				{	case 1:
					schutzmittelEintrag(1);
					break;
				 	case 2:
				 	schutzmittelEintrag(2);
				 	
					break;
				 	case 3:
				 	schutzmittelEintrag(3);	
						
					break;
					case 4:
					schutzmittelEintrag(4);	 
						 
					break;
					case 5:
					schutzmittelEintrag(5);
							
					break;
					case 6:
					schutzmittelEintrag(6);		 
							 
					break;
					case 7:
					schutzmittelEintrag(7);			
								
					break;
					case 8:
					schutzmittelEintrag(8);			 
								 
					break;
					case 9:
					schutzmittelEintrag(9);				
									
					break;
					case 10:
					schutzmittelEintrag(10);				 
									 
					break;
					case 11:
					schutzmittelEintrag(11);					
										
					break;
					case 12:
					schutzmittelEintrag(12);					 
										 
					break;
					default:
						}}
			else
			{
				{switch(zahlEins.getSelectedItem())
				{	case "1":
					schutzmittelEintrag(1);	
					
					break;
				 	case "2":
				 	schutzmittelEintrag(2);	
					 
					break;
				 	case "3":
				 	schutzmittelEintrag(3);	
						
					break;
					case "4":
					schutzmittelEintrag(4);	
						 
					break;
					case "5":
					schutzmittelEintrag(5);	
							
					break;
					case "6":
					schutzmittelEintrag(6);	
							 
					break;
					case "7":
					schutzmittelEintrag(7);	
								
					break;
					case "8":
					schutzmittelEintrag(8);	
								 
					break;
					case "9":
					schutzmittelEintrag(9);	
									
					break;
					case "10":
					schutzmittelEintrag(10);	
									 
					break;
					case "11":
					schutzmittelEintrag(11);	
										
					break;
					case "12":
					schutzmittelEintrag(12);	
										 
					break;
					default:
						}}
				
				
				
				
				
			}
		}
}

Der Code ist nicht fertig.
Zum Verständnis: Es gibt zwei Gruppen von Tischen K und B. Bei if-Auswahl wird K gewählt.

Mein Problem ist jetzt, dass switch IMMER den ersten nimmt, also "case 1:" EGAL welche zahl ich bei zahlEins übergebe. Ich habe es auch schon mit einem String versucht, aber leider das gleiche Ergebnis.
Also kompalieren lässt sich alles und das Programm starten auch.

Ich weiß nicht mehr weiter, wäre nett wenn ihr mir sagen könntet wo das Problem ist. DANKE!!

 

[njans]

kob.getSelectedItem() == "K"

Strings vergleicht man mittels equals ->

kob.getSelectedItem().equals("K")

Außerdem frage ich mich nach dem Sinn deiner Switche:

if (i > 0 && i < 13)
    schutzmittelEintrag(i);

Das erfüllt die selbe Funktion, wie dein erster Switch. Analog zum zweiten kann man da natürlich auch etwas bauen.

 

[rooonny]

Hey Danke!! Für die schnell Antwort :toll:

Ich werde es gleich testen.

 

[rooonny]

Leider geht´s jetzt gar nicht mehr. Ich habe die Switch-Sachen rausgenommen....
Er reagiert jetzt nicht mahr aus den Button.

Warum? Kann ich mir echt nicht erklären, weil in den ersten beiden Methoden habe ich gar nicht verändert.

void schutzmittelEintrag(final int nr){
		
		schutzeintragen.addActionListener(new ActionListener(){
			public void actionPerformed(ActionEvent e){
				String mittel;
				String datum;
				String ftext;
				mittel = cschutzm.getSelectedItem();
				datum = schutzdatum.getText();
				ftext = mittel + " am " + datum + " gegeben\n";
				schutzhistorie.appendText(ftext);
				
				
				try
				{
				BufferedWriter schutzspeicher;
				schutzspeicher = new BufferedWriter(new FileWriter(nr+ "schutzdatei.txt", true));
				schutzspeicher.write(ftext);
				schutzspeicher.close();
				}
			catch(IOException fehler)
			{}
			
			}
		});}
		
			
			
			
		void schutzmittelladen(final int nr){
			bfuellen.addActionListener(new ActionListener(){
				public void actionPerformed(ActionEvent e){
					
					try
					{	
						BufferedReader texteingabe;
						texteingabe = new BufferedReader(new FileReader(nr+"schutzdatei.txt"));
						
						String text = "";
						String textzeile;
						textzeile = texteingabe.readLine();
						while(textzeile != null)
						{
							
							text = text + textzeile + "\n";
							textzeile = texteingabe.readLine();
						}
						texteingabe.close();
						schutzhistorie.setText(text);
					}
					catch(IOException f)
					{}
					}});
			}

		
			

		void auswahlTisch(){
			
			if(kob.getSelectedItem().equals("K"))
			{ int i = Integer.parseInt(zahlEins.getSelectedItem());
				{if ( i > 0 && i > 13){
					schutzmittelEintrag(i);
				}
				else{
					
					//...
					
					
				};
						}}
			else
			{
				{switch(zahlEins.getSelectedItem())
				{	case "1":
					schutzmittelEintrag(1);

Unten ist es noch mit Switch... ändere ich dann ab wenn ich weiß warum das nicht mehr geht.

 

[njans]

if ( i > 0 && i > 13)

Wenn du das if auch falsch machst ist es kein Wunder.

if ( i > 0 && i < 13)

 

[rooonny]

Danke, aber das Problem besteht leider weiterhin. Das Programm erstellt immer die Datei: "1schutzdatei.txt" egal was ich bei
zahlEins auswähle.

Schön langsam ist mur das peinlich. Aber Alles klappt außer das.

Wenn ich es allerdings so schreibe...

void auswahlTisch(){
			
			if(kob.getSelectedItem().equals("K"))
			{ int i = 8;
				{if ( i > 0 && i < 13){
					schutzmittelEintrag(i);
				}
				else{
					
					//...
					
					
				};

Erstellt er die Datei "8schutzdatei.txt" So wie es sein soll. Dann stimmt doch was mit dem zahlEins.getSelectedItem() nicht. Oder?

 

[njans]

ja.
Aber ohne den passenden Code kann man dazu nichts sagen.

 

[rooonny]

kann man einen Beitrag nicht komplett löschen?

 

[njans]

Nein, das können nur Moderatoren.

 

[rooonny]

Hi,

ich habe jetzt mal bisschen weiter geforscht, warum das nicht gehen könnte. Ich habe mal die Ausgabe von

zahlEins.getSelectedItem()

in einen String schreiben lassen. Und da kaommt dann "null".

Das heißt ja eigentlich "gar nix" oder so

Aber da ist was ausgewählt und blau hinterlegt ist es auch.

Vielleicht hilft das ja weiter. DANKE!

 

[njans]

Dann würde ich vermuten, dass du in der Component
a) nichts ausgewählt hast
b) die falsche Component abfragst

Aber was genau kann man ohne mehr Code nicht sagen.

 

[rooonny]

Hallo ich bin es nochmal Ich habe jetzt eine andere Möglichkeit gefunden die Auswahl zutreffen. Nur leider komme ich jetzt beim laden mit BufferedReader nicht weiter und da wollte ich jetzt keine extra Thema anfangen.

Nämlich:

int laden(String was){
	
	try{
		BufferedReader la;
		String be1 = kob.getSelectedItem();
		String be2 = zahlEins.getSelectedItem();
		String wert;
		int werti = 0;
		la = new BufferedReader(new FileReader(be1+be2+was+"rtb"));
		wert = la.readLine();
		werti = Integer.parseInt(wert);
	    la.close();
	    return werti;
	    
	}
	
	catch(IOException fehler){}
	}

Ich wollte da eine Methode schreiben die den geladenen Wert als int ausgibt, da die Datei als String gespeichert ist parse ich sie um, aber leider meint er trotzdem immer noch das, das die Methode kein Int ausgibt.

Wäre um jede Hilfe dankbar!

Da Ronny

 

[Thallius]

Tj,a was gibst du denn zurück wenn eine Exception auftritt?

Gruß

Claus

 

[rooonny]

Gar nix , weil wenn ich vorher nichts gespeichert habe dann soll er ja auch nichts machen, dann soll das Blanko (ich habe eine GUi mit Zeilen) einfach da bleiben...

Ist das ein Problem? Beim speichern habe ich das übrigens genau so und das geht super.

Da Ronny

 

[Thallius]

Die Method laden erwartet einen Rückgabewert vom typ int. Das hast du selber so definiert. Also muss diese auch in jedem Fall ein int zurück geben auch wenn sie eine Exception wirft.

Gruß

Claus

 

[rooonny]

Das habe ich nicht gewusst, was aber logisch ist. DANKE hast mir echt weitergeholfen.

Da Ronny

 

[husox123]

Hi, Ich habe leider nicht so richtig verstanden, was man programmieren sollte. Das könnte an mir liegen sorry.
Wenn du es gründlich erklärst, kann ich vielleicht dazu etwas sagen.

 

[rooonny]

Hi husox123 ich wollte nur eine methode schreiben die den eingelesenen wert als int zurüch gibt. da die gespeicherte datei allerdings ein String ist war das alles nicht ganz so einfach. Aber jetzt hat es geklappt, nochmal danke an Thallius!!

Da Ronny