Compiler-Fehler cannot find symbol

[2Crazy4Nick]

Ich habe versucht,

int i = str.length;

zu setzen, wobei str der String ist, jedoch zeigt er mir beim compilen einen error an.
Das gleiche kommt später noch mal und zwar 3 mal in der Zeile

if (charAt(x) == ClosingClampRound || charAt(x) == ClosingClampCurved || charAt(x) == ClosingClampBoxed)

x ist "int x = 0"
ClosingClampRound und die anderen beiden sind durch

String ClosingClampRound = ")";
String ClosingClampCurved = "}";
String ClosingClampBoxed = "]";

deklariert.
Da es sich hierbei um meine erste Programmierarbeit in Rahmen eines Übungszettels für die Uni handelt bin ich dementsprechend Ahnungs- und Hilflos X_x

Ich verstehe einfach nicht, wo der Fehler liegt

 

[timbeau]

1. Fehler: Schau dir mal an wie man die charAt(int) Methode benutzt. Nämlich mit STRING.charAT(int)

2. Ein Fehler ist es charAt mit String per "==" vergleichen zu wollen. Bin mir ziemlich sicher, dass das immer falsch ist. dann lieber 2x charAt

 

[2Crazy4Nick]

1. Fehler: Schau dir mal an wie man die charAt(int) Methode benutzt. Nämlich mit STRING.charAT(int)

2. Ein Fehler ist es charAt mit String per "==" vergleichen zu wollen. Bin mir ziemlich sicher, dass das immer falsch ist. dann lieber 2x charAt

Den Tipp mit den == bekam ich von nem C#'ler, der sich auch unsicher war, ob das in Java auch so funktioniert. Aber was meinst du mit 2x charAT?

Ich habe die Klammern als Strings deklariert und wollte, dass er nun char für char überprüft, ob das Zeichen an dieser Stelle mit einer der 3 Klammern übereinstimmt :/

 

[timbeau]

String.charAt(x) == Klammer.charAt(0);

oder die Klammern statt String als char deklarieren.

Dann kannste auch String.charAt(x) = klammer testen.

Ansonsten mal mit JAVA-Tags deinen kompletten code posten.

 

[2Crazy4Nick]

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;

public class Wohlgeformt {
	
	// Ueberprueft den gegebenen String auf Wohlgeformtheit
	static void check (String str) {
		boolean stringIsClosingClamp = false;  	// Lässt die Suchschleife nach einer geschlossenen Klammer weiterlaufen, wird true gesetzt, wenn geschlossene Klammer gefunden und beendet die Schleife.
		String ClosingClampRound = ")";		   
		String ClosingClampCurved = "}";
		String ClosingClampBoxed = "]";			// Definitionen für die geschlossenen Klammern
		
		int i = str.length;						// Variablendeklaration der gesamten Stringlänge des Klammernausdrucks
		int x = 0;
		
		while (x <=i && stringIsClosingClamp == false); {		// Während der Index x der zu untersuchenden Klammer kleiner oder gleich i, also der gesamten Stringlänge ist und noch keine geschlossene Klammer gefunden wurde, tue...
	
		if (string.charAt(x) == ClosingClampRound || string.charAt(x) == ClosingClampCurved || string.charAt(x) == ClosingClampBoxed)	// Wenn die untersuchte Klammer ")","}" oder "]" ist, setze stringIsClosingClamp auf true
			stringIsClosingClamp = true;
			else x++;
		}
	}
	

	public static void main(String[] args) {
		// Konsoleneingabe vorbereiten
		// (die Tastatureingabe wird gelesen und in gepufferter Form gesichert)
		BufferedReader input = new BufferedReader(new InputStreamReader(System.in));
		System.out.println("Bitte geben Sie einen Klammerausdruck ein:");
		// Zum Lesen der Eingabe muss eine Ausnahmebehandlung (Exception) eingefuegt werden,
		// da waehrend des Einlesens unerwartete Fehler auftreten koennten
		try {
			// der gelesene Klammerausdruck wird an die Methode check uebergeben
			// wo er auf Wohlgeformtheit ueberprueft wird
			check(input.readLine());
		} catch (IOException ex) {
			ex.printStackTrace();
		}
	}
}

Das ganze ist ein vorgegebenes Grundgerüst, ab

static void check (String str) {

bis

else x++;

ist selbst hinzugefügt. Wir sollen einen Klammerausdruck auf Wohlgeformtheit überprüfen können und zwar nur mit charAt, substring und while/if-Anweisungen
Daher kam ich drauf, in dem Klammerausdruck so lange von vorne zu suchen, bis die erste schließende Klammer gefunden wird, dann sollte die Schleife beendet werden. An der Stelle bin ich.
Danach muss dann die gefundene Klammer mit dem Zeichen mit dem index x-1 verglichen werden, was dann eine passende sein muss, damit das ganze wohlgeformt ist.
Ist es der Fall, werden beide Zeichen aus dem String gelöscht und das ganze beginnt von vorn...
so hab ich mir das zumindest gedacht

 

[timbeau]

Jo, als Tipp für später immer kleine Tests durchzuführen, mal den Ausdruck

char klammer = '{';
String klammer2 = "{";
System.out.println(klammer == klammer2);

testen. glaube das müsste so klappen. Sitze hier nur bei meiner Freundin und hab keine IDE hier. Und meine Tipps von oben befolgen. Denke da liegt der Fehler.

Ansonsten kannst du auch die Klammern zählen. Wenn am Ende des Strings noch offene Klammern vorhanden sind, ist er nicht wohlgeformt. Und ich nehme an, ein String der über kreuz schließt ist auch nicht wohlgeformt oder? "({)}" Also musst du auch gucken welche Klammer zum schließen nötig ist.

 

[2Crazy4Nick]

public class Test{

static void check (String str) {
char klammer = '{';
String klammer2 = "{";
System.out.println(klammer == klammer2);
	}
}

ergibt einen
\Test.java:6: error: incomparable types: char and String
System.out.println(klammer == klammer2);

//edit: Über Kreuz geschlossen ist nicht wohlgeformt, genau. Deswegen soll er ja die erste schließende suchen -> die direkt davor MUSS das Gegenstück sein. Falls nicht -> Ausgabe.
Falls es stimmt: Beide Klammern löschen, alles von vorn mit dem neuen String. Und das, bis keine Klammern mehr da sind, oder n Fehler auftaucht

 

[bygones]

hehe - wohlgeformte Klammern sind anscheinend grade hoch in mode

 

[2Crazy4Nick]

Das Semester hat ja auch gerade erst angefangen
Nur irgendwie keine besonders pädagogische Methode
"Es gibt strings, es gibt if und while, jetzt macht ma hier".. in etwa so lief das x)

//edit: *sfz* Ich kriegs einfach nicht hin >x<

 

[yoro]

Tja, da besuchen wir wohl dieselbe Uni mit derselben Vorlesung
Folgende Fehler, die ich bei dir fand:
1. Du kannst Strings nicht mit dem Operator == vergleichen. Das dürfte false ergeben!

str1.equals(klammer1)

Das ist die bessere Lösung. In der else-Anweisung schreibst du dann

str1.equalsIgnoreCase(klammer1)

2. nicht

str.length

sondern

str.length()

Grüße
und viel Erfolg noch

 

[2Crazy4Nick]

Ich hab es inzwischen gelöst.. heute Nachmittag stell ich dann auch mal den QC rein, nach der Abgabefrist versteht sich.
Mich würde natürlich schon interessieren, ob wir tatsächlich auf die gleiche Uni gehen.
In meinem Fall wäre das Marburg, kann ich ja frei raus sagen

Gute Nacht erstmal ^^

 

[yoro]

Ja, bei mir ist es auch die Uni Marburg Der Quellcode wurde von dir ja 1zu1 übernommen
Ich stimme dir aber zu: Die Aufgabe war zu schwer für Anfänger_innen. In meiner Gruppe konnte das niemand außer mir. Ich habe aber schon etwas Erfahrung

Eine gute Nacht wird es aber nicht. Ich muss den Quellcode nämlich ausreichend kommentieren für meine Übungsgruppe

 

[Marcinek]

Also wenn das zählen von offenen und geschlossenen Klammern an einer UNI für Probleme sorgt, welche Aufgaben soll man dann erwachsenen Menschen geben?

---

Aber zumindest sollte man nicht nach dem ersten hinsehen sagen: Kann ich nicht. Will ich nicht mag ich nicht ^^

Und wie man sieht: Mit einer gesunden Portion Selbständigkeit und Mitarbeit wird hier bei den schweren Aufgaben geholfen ;D

 

[2Crazy4Nick]

Es gibt in der Hinsicht Probleme, dass uns zu dem Zeitpunkt noch nicht mal erklärt wurde, was Methoden, Klassen, Strings, etc. sind.

Nun gut, mein QC:

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;

public class Wohlgeformt {
	
	// Ueberprueft den gegebenen String auf Wohlgeformtheit
	static void check (String str) {
		boolean stringIsClosingClamp = false;  	// Lässt die Suchschleife nach einer geschlossenen Klammer weiterlaufen, wird true gesetzt, wenn geschlossene Klammer gefunden und beendet die Schleife.
		char ClosingClampRound = ')';		   
		char ClosingClampCurved = '}';
		char ClosingClampBoxed = ']';			// Definitionen für die geschlossenen Klammern
		
		int i = str.length();						// Variablendeklaration der gesamten Stringlänge des Klammernausdrucks
		int x = 0;									// "Leseposition"
		
		if (i == 0) {System.out.println("Na, Klammern sollten sie schon eingeben!");
		return;}
		if (i % 2 == 1) {System.out.println("Der Ausdruck kann nicht wohlgeformt sein, da die Zahl der Klammern ungerade ist!");
		return;}
		while (x < i && stringIsClosingClamp == false) {		// Während der Index x der zu untersuchenden Klammer kleiner i, also der gesamten Stringlänge ist und noch keine geschlossene Klammer gefunden wurde, tue...
	
		if (str.charAt(x) == ClosingClampRound || str.charAt(x) == ClosingClampCurved || str.charAt(x) == ClosingClampBoxed)// Wenn die untersuchte Klammer ")","}" oder "]" ist, setze stringIsClosingClamp auf true
			{stringIsClosingClamp = true;}
			else x = x+1;										// Wenn keine schließende Klammer kam, wechsle zur nächsten Stelle im String und Probiere von vorn
		
		
		char OpeningClampRound = '(';		// Deklaration weiterer Klammern
		char OpeningClampCurved = '{';
		char OpeningClampBoxed = '[';
		
		if (stringIsClosingClamp == true){						// Wenn eine Geschlossene Klammer kam, überprüfe, welche und prüfe, ob die dazu passende Klammer davor steht => Wenn ja entferne die beiden Klammern, setze x zurück und stringIsClosedClamp auf falsch, um die nächsten Klammern zu prüfen
			if (str.charAt(x) == ClosingClampRound){	
				if (str.charAt(x-1) == OpeningClampRound){
					str = removeCharAt(removeCharAt(str, x-1), x-1);	// entferne x-1 und das neue x-1 aus dem Substring, das vorher an der Stelle x war. Das ganze wird als neuer String definiert
					{stringIsClosingClamp = false;}						// Variable zurücksetzen und so while-schleife von vorn beginnen
					{x = 0;}											// x auf 0 setzen, damit von vorn gelesen wird
					{i = i-2;}											// i=i-2 setzen, da sich die Stringlänge um 2 verkleinert hat
				}	
					else {System.out.println("Der Ausdruck ist nicht wohlgeformt!");	// Falls die davor stehende Klammer nicht passt, ist der Ausdruck nicht wohlgeformt
					return;}															// verlasse die Funktion
				
			}	
			else if (str.charAt(x) == ClosingClampCurved){		// Überprüfe die andere Klammersorte
				if (str.charAt(x-1) == OpeningClampCurved){
					str = removeCharAt(removeCharAt(str, x-1), x-1);
					{stringIsClosingClamp = false;}
					{x = 0;}
					{i = i-2;}
				}	
					else {System.out.println("Der Ausdruck ist nicht wohlgeformt!");
					return;}
				
			}
			else if (str.charAt(x) == ClosingClampBoxed){		// Auch die 3. Klammersorte wird geprüft
				if (str.charAt(x-1) == OpeningClampBoxed){
					str = removeCharAt(removeCharAt(str, x-1), x-1);
					{stringIsClosingClamp = false;}
					{x = 0;}
					{i = i-2;}
				}	
					else {System.out.println("Der Ausdruck ist nicht wohlgeformt!");
					return;}
				
			}	
		
		}
	}
	System.out.println("Der Ausdruck ist wohlgeformt!"); 		// Ans Ende der Funktion gelangt man nur, wenn vorher kein Fehler auftrat => Wohlgeformtheit
		}
	public static String removeCharAt(String s, int pos) { 	// Entferne den Char an der Stelle "pos" und gebe den substring aus
   return s.substring(0,pos)+s.substring(pos+1); 
} 

	public static void main(String[] args) {
		// Konsoleneingabe vorbereiten
		// (die Tastatureingabe wird gelesen und in gepufferter Form gesichert)
		BufferedReader input = new BufferedReader(new InputStreamReader(System.in));
		System.out.println("Bitte geben Sie einen Klammerausdruck ein:");
		// Zum Lesen der Eingabe muss eine Ausnahmebehandlung (Exception) eingefuegt werden,
		// da waehrend des Einlesens unerwartete Fehler auftreten koennten
		try {
			// der gelesene Klammerausdruck wird an die Methode check uebergeben
			// wo er auf Wohlgeformtheit ueberprueft wird
			check(input.readLine());
		} catch (IOException ex) {
			ex.printStackTrace();
		}
	}
}

Ja ich weiß, Syntaxmäßig sieht da nich alles so einwandfrei aus, aber es funktioniert

Von daher, viel Spaß alles mit Java *hust*