kleiner fehler mit der "Zählvariable"

[Thaflasher]

Hi ich hab ein kleines Codeproblem bei dem ich gerade ein wenig auf dem Schlauch stehe.
Leider bin ich totaler Programmier-Noob, aber jeder fängt mal klein an. :rtfm:
Ich möchte Überprüfen ob ein Array in einem anderen Array vorkommt. Mein Ziel ist es die Anfangsstelle zu bekommen, in der die beiden Arrays gleich sind.

Dabei benutzte ich nach einer for-schleife if-anweisungen:

[JAVA=14]new CodeSnippet();

for(i=0; i < counter; i++)
if(array1 == array2[0]){
if(array1[i+1]==array2[1]{
if(array1[i+2]==array2[2]{
break;
}
}
}

j=i;

else
[/code]

Meine 2 Probleme:

1.
irgendwie kann ich nach den if anweisungen, dem Wert von i nicht auf das "j" zuweisen (i nicht initialisiert) ???:L . Ich brauche aber das "i" weil es der Anfangswert ist, wenn mein Array1 mit dem Array2 Übereinstimmt. Warum ist das "i" nicht initialisiert, wo es doch oben in der schleife durchläuft, die stopt wenn meine 3te if-anweisung zutrifft?

2. Das "else" mag leider auch nicht :autsch: wenn es nähmlich nicht überseinstimmt soll j = -1 sein, aber eclipse meint "token muss gelöscht werden wg. Syntaxfehler".

Ich hoffe dass wenigstens der

-Tag funktioniert in dem ich das reinschreiben soll :oops:

blubb!

Danke!

Grüße

 

[chri]

das else braucht ein if vorher. zb

if(i<counter) //schleife hat wegen übereinstimmung abgrbrochen
  j=i;
else //schleife hat wegen counter abgebrochen
  j=-1;

Fürs erste Problem müsste ich mehr von der Methode sehen.

 

[madboy]

davon abgesehen dass

j

, genau so wie

i

nicht deklariert sind, sollte das grob passen.
Am besten postest du deinen kompletten Code, so fern er nicht sehr lang ist. Ansonsten die komplette Klasse, die Probleme macht. Das was du gepostet hast sieht recht seltsam aus.

Im übrigen empfiehlt es sich, IMMER geschweifte Klammern bei if, else, for, ... zu setzen. Verbessert die Lesbarkeit und vermeidet Fehler.

 

[Thaflasher]

Danke für die Antworten.

Hier der komplette Code für diese Methode:

new CodeSnippet();

public static int teilArr(char arrCopy[], int Counter){
    
//Variable j entspricht der Anfangstelle an der das Array übereinstimmt
      int j;

//tal = teil array länge
      int tal=3;
      char tArr[]=new char[tal];

// Das Wort "ein" soll gesucht werden
      tArr[0]='e';
      tArr[1]='i';
      tArr[2]='n';

//"Main"-Array durchlaufen ob "e" drin vorkommt
      for(int=0; i < counter; i++)
          if(arrCopy[i]==tArr[0]{
            if(arrCopy[i+1]==tArr[1]){               //Wenn "e" drin vorkommt, gucken ob der nächste ein "i" ist
              if(arrCopy[i+2]==tArr[2]{              // wenns ein "i" war schauen ob der nächste ein "n" ist
              break;                                       // alles am start -> raus aus der schleife
}
}
}
j=i;                                                         // i ist die stelle an der das "e" war

else                                                        // war nix am start -> rückgabe soll -1 sein
j=-1;

return j;                                                  // j wird zurückgegeben
}

Das Problem an den Klammern ist (wenn ich es richtig verstanden habe), das ich dann einen Block bilde und die Informationen im Block nicht heraus können => die Methode findet meinen rückgabewert bei "return" nicht und das "i" wird trotzdem verschluckt.

Das ganze sieht komisch aus, weil ich zum ersten mal sowas mache. ;-) Alle bisherigen Programmierungen sahen komisch aus, aber beim 2. Mal wars immer kürzer und wesentlich einfacher geschrieben.

 

[madboy]

Du verwendest Eclipse? In deinem Code sollte noch ein "bisschen" mehr bemängelt werden...
- Was macht Zeile 1? Bzw. was sollte sie deiner Meinung nach tun?
- Jede öffnende Klammer sollte auch wieder geschlossen werden Siehe Zeilen 19 und 21
- Variablen müssen nicht i oder j heißen. Statt j würde ich z.B. "returnValue" oder "rueckgabeWert" schreiben, dann ist klarer was gemeint ist.

Das "else-Problem" löst sich dann so:

if(arrCopy[i+2]==tArr[2]){
   returnValue = i;
   break; 
}

 

[Painii]

Ein else muss immer direkt hinter dem if kommen (der schliessenden }-klammer vom if), deins kommt nach einer for-schleife.

Ähnlich beim i, das wird in der for-schleife aufgebaut, du willst es aber ausserhalb der for-schleife nutzen -> geht nicht.

Und als letztes, einrücken ist immer gut, dann siehst du wo dein if/for/sonstwie block aufhört.

 

[Thaflasher]

Danke vielmals Habe es schließlich mit einer Zählvariable gemacht, deren Anzahl sich pro Schleifendurchlauf um eines erhöht.