If abfragen!

[BigPun]

Hallo, ich habe folgendes Problem:



public class PreferencesExecute {
public static AndereKlasse config;

public static void main(String[] args) throws IOException {
config = new MilkRunPrefConfig();

System.out.println("Program Arguments:");
for (String s : args) {
System.out.println("\t" + s);
}
// Execute(null);

String command = args[0];
String key = args[1]; http://www.java-forum.org/images/smilies/noe.gif
String value = args[2];

// check args
if (command.equalsIgnoreCase("list")) {
config.showKeys();
} else if (command.equalsIgnoreCase("get")) {
config.getKey();
}
if (command.equalsIgnoreCase("set")) {
config.setKeys(key, value);
}
if (command.equalsIgnoreCase("remove")) {
config.removeNode(null);
}

}

void printUsage() {
System.out.println("call programm with command ");
System.out
.println("commands are list, get key, set key value, remove key");

}

--> In der Zeile wo ich meinen String key = args[1]; setzte gibts logischerweise einen Array.IndexOutOfBoundsException!!
Ich denke ich muss eine If Abfrage davor setzen, tue mich da aber schwer!! Vielleicht hat jemand einen Vorschlag?!


Das Programm soll bei einem Aufruf im DOS die methoden starten. Beispiel, wenn ich im DOS java -jar PreferencesExecute list eingebe soll dann showKeys(); starten, diese gibt mir dann eine Liste von gesetzten Keys aus! Wenn ich statt list get eingebe, startet er die getKeys() methode, mit set soll ich dann einen key eingeben und den wert setzen können usw....

Preferences ist eine Importierte bzw. vordefinierte Java-Klasse die unter Windows mit dem befehl put werte in die Registrie schreibt mit den dazugehörigen schlüsseln!

 

[nillehammer]

Der Wert, den Du kontrollieren musst, ist args.length, entweder mit if/else oder mit einem switch/case.

P.S. Und bitte den Java-Tag für Code benutzen. So liest es sich viel leichter.

 

[Landei]

Und beherzige bitte die große, rote Überschrift über dem Antworten-Feld:

Java™ Quelltext ist bitte in

-Tags zu schreiben:...[/Size][/COLOR]

 

[Gast2]

Es scheint noch zu klein geschrieben zu sein ...

 

[HimBromBeere]

Am einfachsten wäre wohl, entweder mit

args.length

die Länge des Parameterfeldes zu prüfen, oder einen

Try-Catch-Block

drumherumzubauen, der im Falle eines Fehlers aus der Funktion

main

herausspringt.

if (args.length < 3) return; // DANKE für den Hinweis, muss natrülich < 3 heißen
// oder eben mittels try-catch (würde aber trotzdem die erste Variante preferieren)
try {
    command = args[0];
    key = args[1];
    value = args[2];
} catch (IndexOutOfBoundsException e) {
    return;
}

 

[Wishmaster51]

Am einfachsten wäre wohl, entweder mit

args.length

die Länge des Parameterfeldes zu prüfen, oder einen

Try-Catch-Block

drumherumzubauen, der im Falle eines Fehlers aus der Funktion

main

herausspringt.

Wenn schon, dann

if (args.length < 3)  return;

(wäre IMHO auch die schönere Variante)

Man könnte/sollte dann auch eine entsprechende Ausgabe auf der Konsole machen

 

[BigPun]

danke schonmal für die bisherigen antworten, aber ich glaube ich bin einfach zu blind um zu sehen wo die if anweisung am besten hin passt!

if (args.length < 3) {
			String command = args[0];
			String key = args[1];
			String value = args[2];
			if (command.equalsIgnoreCase("list")) {
				config.showKeys();
			} else if (command.equalsIgnoreCase("get")) {
				config.getKey(key);
			} else if (command.equalsIgnoreCase("set")) {
				config.setKeys(key, value);
			} else if (command.equalsIgnoreCase("remove")) {
				config.removeNode(null);
			}
		}return;

In Zeile 3 zgibt er mit ein ArrayIndexOutOfBoundsEx was muss ich hier beachten

http://www.java-forum.org/images/smilies/rtfm.gif

 

[Gast2]

dann übergibst du nicht genügend Argumente.

Dein if Abfrage ist zudem auch falsch. Du fragst ab ob weniger als 3 Argumentes übergeben werden, wenn das wahr ist willst du auf die ersten drei zugreifen, das knallt

 

[BigPun]

if (args.length <= 1 && command.equalsIgnoreCase("list")) {
			config.showKeys();

		} else if (args.length <= 2 && command.equalsIgnoreCase("get")) {
			String key = args[1];
			config.getKey(key);

		} else if (args.length <= 3 && command.equalsIgnoreCase("set")) {
			String key = args[1];
			String value = args[2];			
			config.setKeys(key, value);
		}

es klappt, danke vielmals für die tipps

 

[Landei]

Fast richtig

if (args.length == 1 && command.equalsIgnoreCase("list")) {
            config.showKeys();
 
        } else if (args.length == 2 && command.equalsIgnoreCase("get")) {
            String key = args[1];
            config.getKey(key);
 
        } else if (args.length == 3 && command.equalsIgnoreCase("set")) {
            String key = args[1];
            String value = args[2];         
            config.setKeys(key, value);
        } else {
            System.out.println("...Dem Nutzer die Kommandozeilenargumente erklären..."); 
        }

Ansonsten würde z.B. ein Aufruf mit "set" ohne etwas dahinter in die Hose gehen.