Umfrage wo ist der fehler?

[newbie2009]

Hallo Leute bräuchte mal Hilfe:
Problem:
Ich fordere den Benutzer auf Fragen , die ich davor in einem Array gespeichert habe, mit ja oder nein zu antworten.In einer for Schleife liest er die Antworten ein, und nun wollte ich, dass wenn der Benutzer ein "ja " eingibt er die eine Variable erhöht, und bei nein die andere Variable .
Aber irgendwie bekomme ich die erhöhung der Variablen mit der If-Anweisung nicht hin, er zählt die Variablen nicht hoch.Wo könnte der Fehler sein ?

class FrageStellen extends Frage {
	

	
	
	
	static int jaAnt;
	static int neinAnt;
	String[]jaNein= {"ja","nein"};
	String[] Antworten = new String [Quiz.length];

	
	FrageStellen(int k){
		super(k);
		
		
	}			 
		

	
	
	
	void notierenDerAntworten(){
		IO.println(jaNein[1]);
					for(int u=0;u<Quiz.length;u++){
							IO.println(Quiz[u]);
								Antworten[u]=IO.readString("Stimmen sie zu , ja oder nein ein");
									if(Antworten[u]==jaNein[0]){
											jaAnt++;
											IO.println(jaAnt);
				   
										} else if(Antworten[u]==jaNein[1]){
											neinAnt++;
											IO.println(neinAnt);
										}
		
		}
		
		
		
	}
	
	
	
	
	
	
}

 

[eRaaaa]

Strings solltest du mit equals vergleichen, nicht mit ==

Antworten == jaNein[0]

==>

Antworten.equals(jaNein[0])

Daran wird`s liegen !

 

[newbie2009]

aber dann müsste ich doch java.util importieren oder nicht?
geht das nicht ohne? im prinzip vergleicht er ja die einzelnen Felder, es ist ja eine Umschreibung von equals.

thx^^

 

[Spin]

Nein musst du nicht

String bietet die Methode equals um Strings vom Content her zu vergleichen.
Du vergleichst nun mihilfe von Indexen und kannst nicht die equals Methode innerhalb der if Bedingung benutzen.

Denn wenn du Antwort u an Position 0 mit dem StrinArray an Position 0 vergleichst , dann bekommst du ein true oder false , wegen des Contents bezogen auf den index.

Modifiziert:

void notierenDerAntworten(){
        IO.println(jaNein[1]); 
                    for(int u=0;u<Quiz.length;u++){
                            IO.println(Quiz[u]);
                                Antworten[u]=IO.readString("Stimmen sie zu , ja oder nein ein");
                                    String temp = Antworten[u];
                                    if(temp.equals("ja")){
                                            jaAnt++;
                                            IO.println(jaAnt);
                   
                                        } 
                                            else {
                                            neinAnt++;
                                            IO.println(neinAnt);
                                        }
        
        }

Man braucht keine zweite Bedingung
Denn wenn nicht ja , dann eh nein


grüße

 

[newbie2009]

ok danke das funktioniert ja doch
ja die if, else bedingungen die brauche ich , weil ich nun auch einen Fall habe wo der Benutzer ein unerlaubtes Wort einfügt.

	void  vergleichen (){
		IO.println(jaNein[1]);
		for(int u=0;u<Quiz.length;u++){
				IO.println(Quiz[u]);
					Antworten[u]=IO.readString("Stimmen sie zu , ja oder nein ein");
						if(Antworten[u].equals(jaNein[0])){
								jaAnt++;
								IO.println("Anzahl der ja antworten");
								IO.println(jaAnt);
								 
	   
							} else if(Antworten[u].equals(jaNein[1])){
								IO.println("Anzahl der nein Antworten");
								neinAnt++;
								
								
								
							}else{
								IO.println("diese Antwort ist nich zulässig");
								u--;
							}

}
	}

Nun habe ich eigentlich soweit alle Klassen und methoden fertig.Eigentlich ....
In der Aufgabenstellung, ist es dem Benutzer erlaubt, zu entscheiden ob an der Umfrage auch weitere Teilnehmer teilnehmen sollen oder nicht.
Ich habe irgendwie keine Idee wie ich das jetzt lösen soll.
Ich hatte mir überlegt eine neue Klasse zu definieren sprich class Teilnehmer die mit den bereits definierten Klassen und Methoden arbeitet, aber an der Umsetzung happerts irgendwie noch

 

[faetzminator]

im prinzip vergleicht er ja die einzelnen Felder, es ist ja eine Umschreibung von equals.

Nein ist es eben nicht! [c]==[/c] vergleicht auf die gleiche Referenz. [c]equals()[/c] ist eine Methode, um ein Objekt mit dem aktuellen zu vergleichen.
Beispiel:

public class Foo {
    private String bar;
    public Foo(String bar) {
        this.bar = bar;
    }
    public boolean equals(Object o) {
        if (this == o) {
            return true;
        }
        if (!(o instanceof Foo)) {
            return false;
        }
        Foo foo = (Foo) o;
        return this.bar.equals(foo.bar); // hier werden alle Objektvariablen auf ihre Gleichheit überprüft
    }
}

Dann folgende Beispiele:

Foo a = new Foo("foo");
Foo b = a;
a == b; // true, da es das gleiche Objekt ist
a.equals(b); // true, da b ein Foo ist und bar.equals(foo.bar) true ergibt (gleicher String)

Nun, wie ist es mit zwei Objekten?

Foo a = new Foo("test");
Foo b = new Foo("test");
a == b; // false!!!
a.equals(b); // true

 

[nrg]

würde dir empfehlen dieses Kapitel mal durchzugehen. Da werden Objektreferenzen sehr schön erklärt (mit Beispielen und kleinen Fragen - im Inhaltsverzeichnis findest du bei diesem Kapitel auch noch Programmieraufgaben und ein Quiz).

Kapitel 26 — Objektreferenzen

grüße
nrg

 

[newbie2009]

ok danke für eure Mühe.
So sieht mein Quellcode nun aus, und es funktioniert eigentlich auch alles soweit.
Aber passt immer noch nicht .

class Frage {
	
	static int AnzahlFragen;
	static String [] Quiz =new String[0];
	
	
	
	Frage(int k){
		AnzahlFragen=k;
		Quiz=new String[k];
			
		
	
	
	}
	
	 void frageEintragen(){
		
	
	for(int j=0;j<Quiz.length;j++){
	
	
		Quiz[j]=IO.readString("Geben sie eine Frage ein ");
		
		
	}
}
	
	
	
	
	
}




class FrageStellen extends Frage {
	

	
	
	
	static int jaAnt;
	static int neinAnt;
	String[]jaNein= {"ja","nein"};
	String[] Antworten = new String [Quiz.length];

	
	FrageStellen(int k){
		super(k);
		
		
	}
	
	
		void einfachvergleichen(){
			for(int u=0;u<Quiz.length;u++){
				IO.println(Quiz[u]);
			char a=IO.readChar("ja oder nein?");
			switch(a){
			case 'j':jaAnt++;
			break;
			case'n':neinAnt++;
			break;
			default:IO.println("Sie haben eine unzulässige Antwort gegeben");
			}
			}
			IO.println(jaAnt);
		}
	
	
void  vergleichen (){
		IO.println(jaNein[1]);
		for(int u=0;u<Quiz.length;u++){
				IO.println(Quiz[u]);
					Antworten[u]=IO.readString("Stimmen sie zu , ja oder nein ein");
						if(Antworten[u].equals(jaNein[0])){
								jaAnt++;
								IO.println("Anzahl der ja antworten");
								IO.println(jaAnt);
								 
	   
							} else if(Antworten[u].equals(jaNein[1])){
								IO.println("Anzahl der nein Antworten");
								neinAnt++;
								
								
								
							}else{
								IO.println("diese Antwort ist nich zulässig");
								u--;
							}

}
	}
	
	
	
	
	
	
}
	class Statistik {
		
		
			
		
		static void drucken(){
			IO.println("anzahl der ja antworten ");
			IO.println(" betraget "+ FrageStellen.jaAnt);
			IO.print("in prozent sind das ");
			IO.println((double)FrageStellen.jaAnt/FrageStellen.Quiz.length*100+ " % ");
			IO.println("anzahl der nein antworten ");
			IO.println(" betraegt "+ FrageStellen.neinAnt);
			IO.print("in prozent sind das ");
			IO.println((double)FrageStellen.neinAnt/FrageStellen.Quiz.length*100 + " % ");
			
		}
		
		
	}
	
class Teilnehmer {
	
	
char a=	IO.readChar("wollen sie eine weitern teilnehmer?"); // Syntax error on token ";", { expected after this token

while(a!='j'){
FrageStellen.vergleichen();        // identifier expected
}


public class Umfrage {
	
	
	
	
	
	
	
	
	
	
	
	
	
	public static void main(String[] args) {
		
		
		IO.println("Fragen eingeben  ");
		IO.println("<------------------------>");
		
		FrageStellen k = new FrageStellen(IO.readInt("geben sie die Anzahl der Fragen ein "));
		
	k.frageEintragen();
	k.einfachvergleichen();
		
		
	
		
		
	
	}

}

So nun habe ich die 2 Fehlermeldungen in der neuen Klasse Teilnehmer, der erste kp , was da nich stimmt, und beim zweiten erwartet er einen identifier, aber dazu müsste ich die methode in der Klasse FrageStellen ändern und dann würden neue Fehler entstehen.
Weiß einer wie ich dieses Problem am besten löse?


Habe das Problem jetzt ohne die Klasse Teilnehmer gelöst, einfach mit paar anweisungen in der main-methode,aber ich würde gerne rumexperementieren und lieber die Klassen untereinander arbeiten lassen,

 

[Painii]

So nun habe ich die 2 Fehlermeldungen in der neuen Klasse Teilnehmer, der erste kp , was da nich stimmt, und beim zweiten erwartet er einen identifier, aber dazu müsste ich die methode in der Klasse FrageStellen ändern und dann würden neue Fehler entstehen.
Weiß einer wie ich dieses Problem am besten löse?

Du darfst keine Methode ausserhalb einer Methode aufrufen, und du darfst kein while ausserhalb einer Methode stehen haben.
Deine Klasse Teilnehmer ist einfach falsch.

 

[newbie2009]

ok hast recht doofer fehler die while schleife außerhalb einer methode , bin wohl schon zu müde

aber wie kann ich denn die Methode vergleichen aus der klasse FrageStellen nutzen? ohne die methode in eine static zu ändern.
eigentlich geht das doch immer mit Klassennamme.methode .aber hier klappt es irgendwie nicht.

class Teilnehmer {


char a= IO.readChar("wollen sie eine weitern teilnehmer?");

void rechnen(){
while(a!='j'){
FrageStellen.vergleichen(); // ---> problem
}}

 

[Schumi]

ok hast recht doofer fehler die while schleife außerhalb einer methode , bin wohl schon zu müde

aber wie kann ich denn die Methode vergleichen aus der klasse FrageStellen nutzen? ohne die methode in eine static zu ändern.
eigentlich geht das doch immer mit Klassennamme.methode .aber hier klappt es irgendwie nicht.

Klassenname.methode geht eben nur bei statischen Methoden, ist Deine Methode nicht statisch brauchst Du eine Instanz der Klasse und an dieser kannst Du die Methode dann ausführen.

 

[newbie2009]

meinst du so ?

class Teilnehmer {

Teilnehmer joe = new Teilnehmer;

char a= IO.readChar("wollen sie eine weitern teilnehmer?");

void rechnen(){
while(a!='j'){
joe.FrageStellen.vergleichen(); // ---> problem
}}

aber der Compiler meckert rum, der meint ich soll "new" löschen warum?
und

joe.FrageStellen.vergleichen(); // ---> problem

an dieser Stelle muss der Klassennamme sprich FrageStellen aufgeführt werde? oder reicht auch

joe.vergleichen ();

 

[Schumi]

Eher so:

class Teilnehmer {
 
char a= IO.readChar("wollen sie eine weitern teilnehmer?");
 
void rechnen(){
  FrageStellen frageStellen = new FrageStellen(5); //z.B. 5 für die Anzahl der fragen, die will der Konstruktor von FrageStellen ja haben.
  while(a!='j'){
    frageStellen.vergleichen(); 
  }
}

vergleichen ist ja eine Methode aus Fragestellen, da brauchst Du dementsprechend ein FrageStellen Objekt. Und dann rufst Du die Methode direkt am Objekt auf, da brauchst Du keinen Klassennamen mehr.

 

[newbie2009]

class Teilnehmer {
 
FrageStellen peter= new FrageStellen;

char a= IO.readChar("wollen sie eine weitern teilnehmer?");
 
void rechnen(){
  FrageStellen frageStellen = new FrageStellen(5); //z.B. 5 für die Anzahl der fragen, die will der Konstruktor von FrageStellen ja haben.
  while(a!='j'){
    peter.vergleichen(); 
  }
}

Klingt logisch danke, aber irgendwie lässt sich die methode vergleichen immer noch nich ausführen.

 

[Schumi]

Es gibt nun verschiedene Möglichkeiten:
Entweder Du löschst die 3. Zeile FrageStellen peter= new FrageStellen; und änderst Zeile 10 auf frageStellen. anstatt peter. (das hieße, dass Du die Instanz der FrageStellen Klasse nur in der Methode rechnen nutzt.

Oder Du lässt doch den peter in der 3. Zeile aber lässt das = new FrageStellen weg (das ist syntaktisch auch nicht ok). Schreibst dafür dann in der Zeile 8 peter = new FrageStellen(5). Dann hättest Du eine globale variable vom Typ FrageStellen, die Du in Zeile 8 instanziierst (wobei es warscheinlich glücklicher wäre, das im Konstruktor zu machen) und dann in Zeile 10 nutzt.

3. könntest Du den Peter auch wirklich dort instanziieren wo Du es jetzt versuchst (also in Zeile 3), dann aber syntaktisch korrekt mit FrageStellen peter = new FrageStellen(5); Dann könntest Du Dir das frageStellen in Zeile 8 sparen. (Aber die Variante willst Du glaube ich nicht)

So wie es jetzt ist hast Du 2 Variablen vom Typ FrageStellen, die eine instanziierst Du, die andere willst Du aber nutzen.

Vielleicht solltest Du Dir aber nochmal Galileo Computing :: Java ist auch eine Insel (8. Auflage) – 3.4 Neue Objekte erzeugen und ein paar andere Kapitel durchlesen, damit Du auch verstehst, warum das nicht klappt, was Du versucht hast.

 

[newbie2009]

hehe ok dankeschön
ist dann wohl erledigt endlich , ach übrigens habe jetzt eine frage zum array aber die stelle ich im neuen thread aber du hast mir echt weiter geholfen, vll kannst dir das ja auch anschauen :toll::toll: