for Schleife mit merkwürdigen Ergebnisen

[Mr.Java]

Habe ein kleines Problem und weiß nicht weiter. Eine Vorschleife in einem Programm von mir spielt volkommen verückt. Ich habe sie Programmiert dass eine Int Variable von null bis zu einer bestimmten anzahl ansteigen soll. Doch plötzlich ist die Variable negativ.

for(int i = 0;ABC.length > (i);i++){
			
			System.out.println(i);

Dann kommt da raus:0
1
2
3
4
5
6
7
8
Exception in thread "main" 9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
-1

 

[stg]

Das Problem muss woanders liegen, nicht in dem geposteten Code-Schnipsel. Ohne weitere Informationen wird dir so niemand helfen können.

 

[Joose]

Ich nehme an das "-1" gehört zu dieser Exception die fliegt und stellt den "RC" des Programms dar. Dieser ist normal "-1" wenn man keinen spezifischen angegeben hat.

 

[Mr.Java]

Hier der Code vor der Klasse

package Entschlüsseln;
/*
 * Verschlüsselt eine Nachricht mit einem Schlüsselwort
 */

import java.util.ArrayList;


public class Verschlüsseln {

	// Variablen
	// Abc als Char Array
	
	String abc = new String("abcdefghijklmnopqrstuvwxyz");
	char[] ABC = abc.toCharArray();
	ArrayList Abc = new ArrayList();
	
	// Variable in der die verschlüsselte Nachricht gespeichert wird.
	
	String v_Nachricht;
	
	
	// Methode zum erzeugen der verschlüsselten Nachricht - gibt einen String zurück
	
	String V_nachricht(String Klartext, ArrayList Verschlüsselte_abc){
		
		for(int i = 0;ABC.length > (i);i++){
			//Abc.add(ABC[i]);
			System.out.println(i);

So ein Mist, da wil man einmal ein einfaches Programm schreiben und schon kommen solche Fehler.

 

[stg]

post doch mal bitte einen vollständigen compilierbaren Code, bei dem das Problem auftritt. Dast hier ist jedenfalls wieder nur Murgx anhand dessen man nix erkennen kann.

 

[Mr.Java]

Hier der Code der Klasse mir der for Schleife, der Code der Main und der Code von einer Abfrage. - Ja das Programm ist noch nicht fertig und die Kommentare nicht so gut.

Die Klasse mit der Main Methode:

package Entschlüsseln;
/*
 * Verschlüsselt eine Nachricht mit einem Schlüsselwort
 */

import java.util.ArrayList;


public class Verschlüsseln {

	// Variablen
	// Abc als Char Array
	
	String abc = new String("abcdefghijklmnopqrstuvwxyz");
	char[] ABC = abc.toCharArray();
	ArrayList Abc = new ArrayList();
	
	// Variable in der die verschlüsselte Nachricht gespeichert wird.
	
	String v_Nachricht;
	
	
	// Methode zum erzeugen der verschlüsselten Nachricht - gibt einen String zurück
	
	String V_nachricht(String Klartext, ArrayList Verschlüsselte_abc){
		
		for(int i = 0;ABC.length > (i);i++){
			//Abc.add(ABC[i]);
			System.out.println(i);
		}
		
		
		// Schleife die solange läuft bis alle Buchstaben verschlüsselt sind
		
		for(int zähler = 0;zähler < Klartext.length();zähler++){
			
			
			char[] Hilfe = Klartext.toCharArray();
				char test = Hilfe[zähler];
			String Test = String.valueOf(test);
			
			
			// Verhindert dass Leerzeichen als Fehler erkannt werden
			
			if(Test.equals(" ")){
				v_Nachricht = v_Nachricht + " ";
			}
			else{
				
				// lokale Variablen zum Vergleich wichtig
				
				
				int h1 = Abc.indexOf(test);
				System.out.println(h1);
				v_Nachricht = v_Nachricht + Verschlüsselte_abc.get(h1);
			}
			
		}
		System.out.println(v_Nachricht);
		
		return null;
	}
}

Die Main:

package Entschlüsseln;
import java.util.ArrayList;

import javax.swing.JFrame;


public class main {

	/**
	 * @param args
	 */
	public static void main(String[] args) {
		//Abfrage nach Schlüsselwort und verschlüsselter Nachricht
		
		Abfrage Abfrage = new Abfrage();
		String[]Daten = Abfrage.Eingabe();
		String Schlüsselwort = Daten[0];
		
		// Buchstaben des Schlüsselwortes werden so verarbeitet dass die nächste Methode diese nutzen kann
		
		Schlüsselwort_Buchstaben_Trennung Buchstaben1 = new Schlüsselwort_Buchstaben_Trennung();
		ArrayList<Character> Buchstaben = Buchstaben1.Buchstaben(Schlüsselwort);
		
		Verschlüsseln t = new Verschlüsseln();
		t.V_nachricht(Daten[1], Buchstaben);
		/*
		// Code wird entschlüsselt
		Entschlüsseln en = new Entschlüsseln();
		String nachricht = en.Nachricht(Buchstaben, Daten[1]);
		
		// Ausgabe der Nachricht
		Augabe ausgabe = new Augabe();
		ausgabe.augabe(nachricht);
		*/
	}

}

Und die Abfrage

package Entschlüsseln;

import java.awt.BorderLayout;
import java.awt.Dimension;
import java.awt.Toolkit;

import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JOptionPane;


public class Abfrage {

	String[] Rückgabe = new String[2];
	String[] Eingabe(){
		
		
		
		
			Rückgabe[0] = JOptionPane.showInputDialog("Geben Sie das Schlüsselwort ein");
			Rückgabe[1] = JOptionPane.showInputDialog("Geben Sie die verschlüsselte Nachricht ein");
			
		return Rückgabe;
	}
}

Bitte mir einfach nur sagen wie ich es verbessern soll.

 

[Zet]

Gibt es noch eine Klasse Schlüsselwort_Buchstaben_Trennung?

 

[Harry Kane]

Ich habe folgenden Anmerkungen und Kritikpunkte zu deinem Code:
Beachte bitte die Java-Namenskonventionen für Klassen (UpperCamelCase), Variablen (lowerCamelCase), und packages (Kleinbuchstaben).
Das ganze Hin und Her mit String abc, char[] ABC und ArrayList Abc ist extrem verwirrend (verschiedenen Objekttypen, aber sehr ähnliche Variablennamen, die sich nur durch Groß- und Kleinschreibung unterscheiden) und teilweise überflüssig. ABC wird zwar mit chars gefüllt, aber abgesehen davon, dass du mit einer for-Schleife über das Array drübergehst, wird es nirgends verwendet. Abc (die ArrayList) wird nie gefüllt (die entsprechende Zeile ist auskommentiert), aber in deiner V_nachricht Methode verwendet. Das Abc leer ist, sollte Abc.indexOf(char) stets -1 ergeben, was 2 Zeilen später zu einer ArrayIndexOutOfBoundsException führen sollte.
In der 2. for-Schleife der V_nachricht Methode legst du bei jedem Schleifendurchlauf die Hilfsvariable char[] Hilfe neu an. Da sie sich bei den ganzen Schleifendurchläufen nicht ändert, könntest du sie auch einmal vor der Schleife anlegen. Besser noch: du verzichtest auf diesen ganzen redundanten Kram und holst dir den char test mit Klartext.charAt(zähler).
Deine V_nachricht Methode gibt immer null zurück, daher ist dein Kommentar der Methode nicht nur „nicht so gut“, sondern einfach falsch und irreführend.
Wo ist die Klasse Schlüsselwort_Buchstaben_Trennung?
Zu guter Letzt: Selbst wenn die ArrayList Abc gefüllt werden würde, könnte dein Programm in der aktuellen Form nur mit Kleinbuchstaben umgehen, ohne das eine Exception fliegt. Wenn du was anderes eingibst (was wir nicht überprüfen können), knallt es trotzdem.

 

[Mr.Java]

Sorry mit dem Kommentaren aber das war mein erstes richtiges Javaprogramm und mein Lehrer will dass ich Kommentare schreibe.

Warum ich das mit der for Schleife gemacht habe:
Weiter unten in Zeile 53 muss ich Stelle eines bestimmten Buchstaben wissen aber da ich dies nicht mit einem normalen Char Array oder einem String machen kann (nach meinem Wissen) habe ich versucht dies in eine Array List umzuwandeln.

Die Klasse Schlüsselwort_Buchstaben_Trennung:

package Entschlüsseln;
import java.util.*;

public class Schlüsselwort_Buchstaben_Trennung {

	//Variablen
	
	int Anzahl_Buchstaben;
	
	ArrayList<Character> buchstaben = new ArrayList<Character>();
	
	
	//Methode gibt Buchstaben einer ArrayList weiter
	
	ArrayList<Character> Buchstaben(String Schlüsselwort){
	
	
	//Buchstaben hinzufügen um diese danach (falls doppelt) zu löschen
		
	Schlüsselwort = Schlüsselwort + "abcdefghijklmnopqrstuvwxyz";
		//Schlüsselwort = "abc";
	//in Kleinbuchstaben umwandeln
	
	Schlüsselwort = Schlüsselwort.toLowerCase();
	char[] c = Schlüsselwort.toCharArray();
	
	Anzahl_Buchstaben = c.length;
	
	// in eine ArrayList packen
	for(int i = 0;i < Anzahl_Buchstaben;i++){
		
		char Zeichen = c[i];
		
		buchstaben.add(Zeichen);
		
	}
	Anzahl_Buchstaben = buchstaben.size();
	
	//Doppelte aus der ArrayList löschen
	for(int i = 0;i < Anzahl_Buchstaben;i++){
		
		char testzeichen = buchstaben.get(i);
		String Testzeichen = String.valueOf(testzeichen);
		
		for(int a = Anzahl_Buchstaben -1;a >= 0;a--){
			
			char testbuchstabe = buchstaben.get(a);
			
			String Testbuchstabe = String.valueOf(testbuchstabe);
			
			if(i==a){}
			else if(Testbuchstabe.equals(Testzeichen)){
				
				buchstaben.remove(a);
				
			}
			
			
			
		}//System.out.println("Hallo");
		Anzahl_Buchstaben = buchstaben.size();
	}
	return buchstaben;
	}
}

PS Das Programm sollte eigentlich nur entschlüsseln können musste es aber dann verbessern

 

[Mr.Java]

Sorry jetzt sehe ich meinen Fehler ein. - ein Dummheitsfehler:
1.Habe ausversehen Großbuchstaben verwenden führt zu Fehlermeldung
2.Um den Fehler zu endecken etwas zu einem Kommantar umwandeln obwohl dies später noch gebraucht wird.

 

[Harry Kane]

Die Klasse String hat die sehr nützlichen Methoden:

string.indexOf(char c);
string.charAt(int pos);

Diese liefern dir entweder den char an einer bestimmten Position oder die Position bzw. das erste Auftreten eines bestimmten chars.
Und dieser Spaghetti hier

if(i==a){}
else if(Testbuchstabe.equals(Testzeichen)){
buchstaben.remove(a);
}

sollte äquivalent sein zu:

if(i!=a && Testbuchstabe.equals(Testzeichen)){
buchstaben.remove(a);
 }