Methoden addString Methode endet bei Fehleingabe

bes01

Mitglied
Hi,

ich hätte gedacht ich krieg das hin aber irgendwie kann ich das Problem nicht lösen.

Ich habe eine ArrayList, der bis zu 3 Elemente (string) hinzugefügt werden können. Diese Elemente dürfen nicht identisch sein. Sofern letzteres gegeben ist, funktioniert die Methode wunderbar. Sobald aber dann tatsächlich das selbe Element hinzugefügt wird, kommt zwar die entsprechende Fehlermeldung, aber danach lassen sich garkeine Elemente mehr hinzufügen.

Code:
   private int progress;	
   private String titel;
   private boolean enthalten = false;
   private ArrayList<String> liste = new ArrayList<String>(2);

public String add() {

	titel = Bean.getTitle();
	int size = liste.size();
	FacesContext context = FacesContext.getCurrentInstance(); 
			
	if (size >= 0) {
			
		for (int i=0; i<liste.size(); i++)	 {
			if (titel == liste.get(i)) {
				enthalten = true;
				context.addMessage("warnMsg", new FacesMessage("Fehler", "ist bereits enthalten!"));	
					
			}				
		}
	}	
		if (enthalten == false) {
			if (size == 0) {												
		            progress = 34;	            	    
		            liste.add(0, titel);
			    context.addMessage("successMsg", new FacesMessage("Erfolgreich", "wurde der Liste hinzugefügt"));
		    	    return titel;
				} else if (size == 1) {
		            progress = 67;				
		            liste.add(1, titel);	
			    context.addMessage("successMsg", new FacesMessage("Erfolgreich", "wurde der Liste hinzugefügt"));
		    	    return titel;
				} else if (size == 2) {
		            progress = 100;					
		            liste.add(2, titel); 
			    context.addMessage("successMsg", new FacesMessage(""Erfolgreich", "wurde der Liste hinzugefügt"));	            
		    	    return titel;
			}
		}  				
	return titel;	
}


Ich habe eine Vermutung: sobald ein identisches Element hinzugefügt wird, setze ich erhalten ja auf "true", sodass beim erneuten Methodenaufruf die 3 If-Else garnicht erreicht werden. Wenn dies nicht geschieht, ist enthalten ja per default auf "false", sodass er diese normal ausführt.

Lieg ich damit richtig? Wie umgehe ich das Problem?
(ja ich weiss elegant ist das nicht aber das ist ja auch das Anfängerforum ^^)

ach ja, getMethode:

Code:
		public boolean isEnthalten() {
			return true;
		}
 
Zuletzt bearbeitet:

sirkarpfen

Mitglied
Also zuerst einmal solltest du 2 Strings niemals mit == vergleichen. Richtig wäre das hier:

Java:
title.equals(liste.get(i))

zweitens gibt deine isEnthalten() Methode immer true zurück, was ja so nicht gewollt sein kann oder? Außerdem sehe ich nicht wirklich wo diese Methode aufgerufen wird.

Drittens ist deine Formatierung grauenvoll^^. Du solltest dich mal mit den Java Code Conventions auseinander setzen. Hier eine gute Referenz zum nachschlagen: Java Code Conventions

Natürlich könnte ich dir eine elegantere Variante hier posten, aber ich bin der Meinung, wenn du deinen Code mal ordentlich ordnest und nochmal Schritt für Schritt durchgehst, wirst du auf eine für dich passende Lösung stoßen.

Drei Tipps geb ich dir aber noch.

1. Du solltest dich mal mit Exceptions auseinander setzen. Eine selbstgeschriebene Exception ist wohl die elegantere Lösung um einen Error abzufangen/auszugeben.

2. Eventuell solltest du dich mal fragen ob so viele If/Else Abfragen wirklich von nöten sind.

3. Ganz genau solltest du dir die erste If-Abfrage ansehen: size >= 0. Eventuell liegt der Fehler ja da ;-)
 
Zuletzt bearbeitet:

bes01

Mitglied
Danke schonmal für dein Feedback.

Es handelt sich um eine JSF-Anwendung, die Methode wird in der index.html aufgerufen:

Code:
<h:commandButton value="hinzufügen" actionListener="#{liste.add()}" update="messages, :mainForm:Panel" />

Also zuerst einmal solltest du 2 Strings niemals mit == vergleichen. Richtig wäre das hier:

Java:
title.equals(liste.get(i))

gefixt, danke.

zweitens gibt deine isEnthalten() Methode immer true zurück, was ja so nicht gewollt sein kann oder? Außerdem sehe ich nicht wirklich wo diese Methode aufgerufen wird.

auch gefixt, löst leider nicht das Problem

Drittens ist deine Formatierung grauenvoll^^. Du solltest dich mal mit den Java Code Conventions auseinander setzen. Hier eine gute Referenz zum nachschlagen: Java Code Conventions

Sorry ich wollte es zunächst richtig einrücken aber war dann doch zu faul - ich gelobe Besserung!
(ich formatiere den Code in der Regel fast immer entsprechend Convention)


1. Du solltest dich mal mit Exceptions auseinander setzen. Eine selbstgeschriebene Exception ist wohl die elegantere Lösung um einen Error abzufangen/auszugeben.

2. Eventuell solltest du dich mal fragen ob so viele If/Else Abfragen wirklich von nöten sind.

3. Ganz genau solltest du dir die erste If-Abfrage ansehen: size >= 0. Eventuell liegt der Fehler ja da ;-)

Ich kann mir das
Code:
if (enthalten == false)
sicher sparen da ich es ja mit false initialisiere; mehr kann ich mMn nicht kürzen. size >= 0 habe ich probeweise auf 0 gestellt, von der Logik her ist size >= 1 richtig da ein vermeintlich doppeltes Element ja schon in der Liste ist, also size ist mindestens 1 (immer)

übrigens: ich erhalte noch nicht einmal eine exception, die console bleibt leer. Die Anwendung funktioniert weiterhin ordnungsgemäß, außer dass die Methode add nicht mehr funktioniert (nach Hinzufügen des doppelten Elements
 
Zuletzt bearbeitet:

sirkarpfen

Mitglied
Also die size >= 0 Abfrage gibt immer true zurück. Entweder ist size 0 oder size ist größer 0 und da size niemals negativ werden kann ist die abfrage obsolet.

Ich denke du gehst das alles falsch an. Ein TreeSet zb lässt keine doppelten Werte zu. Schau dir die Klasse einmal an: java.util.TreeSet. Die add(E e) Methode gibt true zurück wenn das Element noch nicht vorhanden war und false falls es schon vorhanden ist.

Hier liegt übrigens dein Problem:

Java:
if (size == 0) {												
		            progress = 34;	            	    
		            liste.add(0, titel);
			    context.addMessage("successMsg", new FacesMessage("Erfolgreich", "wurde der Liste hinzugefügt"));
		    	    return titel;
				} else if (size == 1) {
		            progress = 67;				
		            liste.add(1, titel);	
			    context.addMessage("successMsg", new FacesMessage("Erfolgreich", "wurde der Liste hinzugefügt"));
		    	    return titel;
				} else if (size == 2) {
		            progress = 100;					
		            liste.add(2, titel); 
			    context.addMessage("successMsg", new FacesMessage(""Erfolgreich", "wurde der Liste hinzugefügt"));	            
		    	    return titel;
			}

Du packst alle liste.add() Aufrufe in eine If-Abfrage ob size == 0 ist. size ist aber sofort > 0 wenn du 1 Element in die liste gepackt hast :)

Achja und mit Exceptions meinte ich das du dir deine eigene Exception schreibst die du, falls das Element schon in der liste ist, werfen kannst um deinen Error auszugeben :)
 
Zuletzt bearbeitet:

bes01

Mitglied
gelöst!

Das Problem war: die Boolean-Variable enthalten wurde in der Methode nicht mit false initialisiert. Das hat dazu geführt, dass sie nach dem Versuch ein doppeltes Element hinzuzufügen, nicht mehr funktioniert hat.
 
Ähnliche Java Themen
  Titel Forum Antworten Datum
T Rekursive Methode Java Basics - Anfänger-Themen 13
Ü Methode soll Quadrat aus der Summer zurückgeben Java Basics - Anfänger-Themen 10
P Objekt einer Methode eines anderen Objektes übergeben Java Basics - Anfänger-Themen 5
Leyla Spezifischte Methode Java Basics - Anfänger-Themen 16
M Methode zielnah zeigt das gewünschte Ausgabe nicht an Java Basics - Anfänger-Themen 3
L Variablenwerte aus einer Methode übergeben Java Basics - Anfänger-Themen 2
T Methode soll etwas ausrechnen und zurückgeben (klappt nd) hat wer eine Idee? Java Basics - Anfänger-Themen 11
P Main Methode scheint Constructor aufzurufen, ohne dass es so gecoded ist Java Basics - Anfänger-Themen 2
T Aufruf der Methode einer Oberklasse, wenn sie in der Unterklasse überschrieben ist. Polymorphie. Java Basics - Anfänger-Themen 2
C Zugriff auf Methode Java Basics - Anfänger-Themen 2
M Datentypen While-Schleife eine Java Methode erstellen Java Basics - Anfänger-Themen 3
T Methode akzeptiert String nicht Java Basics - Anfänger-Themen 18
M Methode sperren bis ein Kriterium erfüllt wurde Java Basics - Anfänger-Themen 3
D Switch Case Methode aufrufen Java Basics - Anfänger-Themen 3
C Unbekannte Methode add bei Klasse die JTree erweitert Java Basics - Anfänger-Themen 14
M methode aufrufen ohne parameter Java Basics - Anfänger-Themen 1
marcelnedza Finde meinen Fehler in einer Methode nicht, Java Karol Java Basics - Anfänger-Themen 15
monsterherz einfache Methode mit Fehler den ich nicht finde Java Basics - Anfänger-Themen 21
Ostkreuz Wieso wird die Methode nochmal aufgerufen? Java Basics - Anfänger-Themen 5
G Variable aktualisiert sich nicht in rekursiver Methode Java Basics - Anfänger-Themen 4
MoxMorris Wie macht man String[] = String[] aus einer anderer Methode? Java Basics - Anfänger-Themen 18
Say super.methode / super.variable und super(variable) Java Basics - Anfänger-Themen 2
B Wie kann ich folgende Klasse/Methode per Button ausführen? Java Basics - Anfänger-Themen 1
D Interface Methode wird ungewollt in der Subklasse überschrieben Java Basics - Anfänger-Themen 5
L Methoden Eine Methode um zu testen ob es ein Nachbar gibt Java Basics - Anfänger-Themen 10
til237 Iterative Methode in rekursive Methode umschreiben Java Basics - Anfänger-Themen 4
M Daten aus errechneter Methode in Datenbank(SQLite) schreiben Java Basics - Anfänger-Themen 60
D next() Methode mehrfach verwenden Java Basics - Anfänger-Themen 1
Ostkreuz Methoden Von Dezimal zu Hexadezimal Methode toHex Java Basics - Anfänger-Themen 2
I Entity Objekt nicht gefunden -> Webhook empfangen in der gleichen Methode (Transaktion) Java Basics - Anfänger-Themen 37
N Throw an Main Methode übergeben Java Basics - Anfänger-Themen 7
M Methoden Methode 'wiederhole' nicht gefunden (Uebersetzungsfehler) Java Basics - Anfänger-Themen 1
H Zu langen String aufteilen - bequeme Methode? Java Basics - Anfänger-Themen 14
_user_q Wie eine Methode/Funktion aus einer Klasse mit Constructor aufrufen? Java Basics - Anfänger-Themen 20
S Array mit Methode löschen Java Basics - Anfänger-Themen 2
J Java To String Methode, Array mit For-Schleife Java Basics - Anfänger-Themen 2
T Variable von Objekten in einer Methode überprüfen Java Basics - Anfänger-Themen 26
M Anzahl Kommandozeilenparamter mittels Methode Java Basics - Anfänger-Themen 11
D Methode: Array Reihenfolge tauschen Java Basics - Anfänger-Themen 3
julian0507 Array aus Methode in anderer Methode sichtbar machen Java Basics - Anfänger-Themen 10
frager2345 Problem mit Methode Java Basics - Anfänger-Themen 4
J Die statische Main-Methode ändert Instanzvariable? Java Basics - Anfänger-Themen 10
D Methode aus dem Aufrufer aufrufen Java Basics - Anfänger-Themen 1
T IOStreams read(byte[]b) methode Java Basics - Anfänger-Themen 2
frager2345 Java Singleton Muster -> Methode für Konstruktor mit Parametern Java Basics - Anfänger-Themen 3
U Beispiel Methode size() vom "Collection"-interface... Wie kann man sichtbar machen, was die Methode unter der Haube macht? Java Basics - Anfänger-Themen 8
D Warum kann ich hier nicht auf die Methode zugreifen? Java Basics - Anfänger-Themen 5
M generate Methode für Streams Java Basics - Anfänger-Themen 6
M Methoden Zweidimensionaler Array mit Setter Methode ändern Java Basics - Anfänger-Themen 4
I Optionaler Parameter bei Methode, der nur optional ist? Java Basics - Anfänger-Themen 6
berserkerdq2 Wozu benötigt man den BiPredicate, kann ich nicht einfach eine normale Methode nutzen, statt BiPredicate? Java Basics - Anfänger-Themen 3
T Linked List set-Methode Java Basics - Anfänger-Themen 2
D Arrays an replaceAll-Methode übergeben Java Basics - Anfänger-Themen 12
B Attribute eines Objekts einer Klasse durch statische Methode einer 2. Klasse ändern? Java Basics - Anfänger-Themen 32
berserkerdq2 Habe eine Klasse, welche public ist, diese hat eine public Methode, die nicht static ist. Wenn ich nun versuche aufzurufen Probleme? Java Basics - Anfänger-Themen 8
viktor1 Methoden Methode schreiben static void readText (String filename) {...} zu WordHistogramSample.java Java Basics - Anfänger-Themen 13
W Equals-Methode überschreiben bei composition Java Basics - Anfänger-Themen 20
V Hilfe bei Implementierung einer boolean Methode Java Basics - Anfänger-Themen 6
T Methode, die prüft ob in einem Int-Array maximal 2 Zahlen enthalten sind, die größer als ihr Vorgänger sind Java Basics - Anfänger-Themen 5
V Methoden printChar Methode mit Rückgabetyp void Java Basics - Anfänger-Themen 26
F Graph Tiefensuche Methode Java Basics - Anfänger-Themen 7
Jambolo Methode, welche die 3 letzten Parameter Werte speichert Java Basics - Anfänger-Themen 20
berserkerdq2 wie funktioniert contenthandler, was muss ich bei der Methode startElement und endElement tun? Java Basics - Anfänger-Themen 11
M Warum return die Methode den Wert nicht Java Basics - Anfänger-Themen 5
berserkerdq2 Wann soll ich den Stream schließen, wenn ich das in einer Methode habe? Java Basics - Anfänger-Themen 8
berserkerdq2 Ich gebe eine ArrayList als List zurück per MEthode, wie kann ich nun aber die ArrayList speichern? Java Basics - Anfänger-Themen 46
S Methode Java Basics - Anfänger-Themen 4
M Eine Methode die erkennt ob die ein gegebene zahl größer oder kleiner sein muss Java Basics - Anfänger-Themen 2
U Methode wird genutzt, ohne dass ich die aufrufe? Java Basics - Anfänger-Themen 4
F nach Methode Programm nicht beenden Java Basics - Anfänger-Themen 9
Liroyd Methode mit Objektvariabel rechnen? Java Basics - Anfänger-Themen 6
H Mit setter-Methode JLabel in einer andern Klasse ändern. Java Basics - Anfänger-Themen 40
D Methode um mögliche Rezepte auszugeben Java Basics - Anfänger-Themen 12
U Warum kann ich die Methode in der ENUM Klasse nicht aufrufen? Und warum geht die Switch nicht? Java Basics - Anfänger-Themen 8
J Hallo zusammen , was macht diese Methode hier genau? Java Basics - Anfänger-Themen 3
D Array in Main Methode aus anderer Klasse aufrufen Java Basics - Anfänger-Themen 3
H Eine Methode über Actionlistener beenden Java Basics - Anfänger-Themen 8
G jButton führt Main Methode nicht richtig aus Java Basics - Anfänger-Themen 3
G Main Methode wird beim ersten Aufruf nicht richtig ausgeführt Java Basics - Anfänger-Themen 1
C60 Methoden Main-Methode erkennt meine Arrays nicht. Java Basics - Anfänger-Themen 7
A Ein Array bearbeiten und in einer anderen Methode nutzen Java Basics - Anfänger-Themen 6
A Ergebnis einer Methode bei einer anderen verwenden Java Basics - Anfänger-Themen 13
L Iteratorform und Methode mit variabler Parameterzahl Java Basics - Anfänger-Themen 31
F Methode ArrayList mit Eingabewert Java Basics - Anfänger-Themen 2
M Wie kann eine Methode für ein vorhandenes "Array von char" einen Index-Wert zurückliefern? Java Basics - Anfänger-Themen 3
M Wie kann die Implementation einer Methode den Wert eines Attributs vermindern? Java Basics - Anfänger-Themen 3
Csircc Rekursive Methode Stack Overflow Java Basics - Anfänger-Themen 10
M Wie kann eine Methode (string) eine andere Methode (void) mit zufälligen int-Werten aufrufen? Java Basics - Anfänger-Themen 4
M Wie verknüpfe ich eine Bedingung mit einer Methode ohne if-Verzweigung & Bedingungsoperator? Java Basics - Anfänger-Themen 2
M Wie kann eine Methode eine andere Methode um Werte wie z.B. 1 erhöhen? Java Basics - Anfänger-Themen 6
schredder Strings und reguläre Ausdrücke - Methode mit return string.matches Java Basics - Anfänger-Themen 5
D mehrere Berechnungen in einer Methode Java Basics - Anfänger-Themen 9
H String Repräsentation eines Rechtecks mit Instanz-Methode Java Basics - Anfänger-Themen 8
M Wie kann ich eine Methode aus einem Interface in eine Klasse implementieren, so dass sie ihre Funktion ausführt? Java Basics - Anfänger-Themen 7
J ArrayList add methode selbst programmieren Java Basics - Anfänger-Themen 10
X Methoden Methode zur Punktezählung in Blackjack Java Basics - Anfänger-Themen 2
W Methode ändern Java Basics - Anfänger-Themen 65
M Wie kann ich in einem Konstruktor die Methode eines anderen Interfaces mit den jeweiligen Parametern aufrufen? Java Basics - Anfänger-Themen 8
W Methode, die mit einem Datum arbeitet? Java Basics - Anfänger-Themen 22
E Einrücken inner der toString()-Methode Java Basics - Anfänger-Themen 34

Ähnliche Java Themen

Neue Themen


Oben