Config int Problem!

[Masterfran99]

Hallo Leute,

			}else if(args[0].equalsIgnoreCase("setslot")){
				if(args[0].length() == 0){
					p.sendMessage("§cGebe ein Argument an!");
					return true;
				}
				String name = args[1].toLowerCase();
				String slotx = args[2].toLowerCase();
				Integer i;
				try {
					int slot = Integer.parseInt(slotx);
					invsetslot(p, name, slotx);
					cs.sendMessage("§aErfolgreich Slots gesetzt!");
				} catch (Exception e) {
				    p.sendMessage("§cDu musst eine Zahl angeben!");
				}
				return true;

Damit setze ich in meiner Config die Slot Anzahl eines Inventars jedoch setzt es immer einen String also "5" aber eigendlich sollte dort nur 5 stehen als Beispiel.

..

 

[MiCrJonas]

Wie genau sieht es in deiner Config aus? Steht da wirklich "5" mit "? Eigentlich sollte dort nur 5 ohne "" stehen. Außerdem solltest du etwas nur mit einer Exception catchen, die du auch erwartest, also in dem Fall "NumberFormatException", da du, falls es noch andere (unabsichtiche) Fehler gibt, diese sonst schlecht finden kannst.

args[0].length() == 0

^ Ist falsch. Die Anzahl an Objekten in einem Array werden abgefragt mit: "args.length". So wie du es machst, fragts du die Länge des ersten Objektes im Array ab.

invsetslot(p, name, slot);

^ Damit übergibst du den Integer (int) an die Konfigdatei und nicht den String aus dem Befehl.

 

[Masterfran99]

Hä?

Das mit args[0].length usw.. ist nur damit keine fehlermeldung kommt das hat hier jetzt nciht viel damit zutun.

Ich frag nur warum immer "zahl" statt zahl in der Config steht?

 

[MiCrJonas]

Hä?

Das mit args[0].length usw.. ist nur damit keine fehlermeldung kommt das hat hier jetzt nciht viel damit zutun.

Um die Zahl der Argumente, die bei dem Befehl benutzt wurden, musst du "args.length" benutzen.

Bei dem letzten Code im obrigen Beitrag sollte das Problem nicht mehr auftreten.

 

[Masterfran99]

nein nein du verstehst das falsch ich will doch nicht die args länge haben.
Lass alles mit args mal weg das hat hier gar nix damit zu tun.

 

[MiCrJonas]

Kannst du bitte ein Bild deiner Config schicken, ich verstehe dein Problem nicht ganz. Und bitte poste einmal deine Methode, mit der du in der Config schreibst.

//Bitte Chat lesen

 

[Masterfran99]

Danke aber ich habe es nun selbst geschafft.

Nur eine Frage ist es möglich mit einer for-Schleife für jeden Wert z.B. in einer Config etwas auszuführen?
z.B. sieht jetzt die Config so aus:

Items:
Item 1:
slot: 0
id: 2
Item 2:
slot: 1
id: 155

Es soll für Item 1 & 2 etwas passieren das macht man normaler Weise mit einer for-Schleife diese verlangt aber eine ArrayLists oder Ähnliches nun geht es auch anders und wenn nicht wie kann ich die Items beim onEnable in eine ArrayList packen?

Danke

 

[MiCrJonas]

Ja, ist möglich. Bitte beim nächsten mal die Config als Code mit entsprechend richtigen Leerzeichen posten.

for (String path : getConfig().getConfiguration***tion("Items").getKeys(false)) {
  int slot = getConfig.getString("Items." path + ".slot");
}

"path" sind die Unterpunkte von "Items", die Nacheinander in der for-Schleife benutzt werden.

 

[Masterfran99]

Folgendes:

Code:

for(String path : invc.getConfigurationSection("Items.").getKeys(false)){
			String iname = invc.getString("Items.");
			int id = invc.getInt("Items." + iname + ".ItemID");
			int islot = invc.getInt("Items." + name + ".Slot");
			ItemStack istack = new ItemStack(id);
			ItemMeta istackMeta = istack.getItemMeta();
			//istackMeta.setDisplayName(invc.getString("Items."));
			istack.setItemMeta(istackMeta);
			
			inv.setItem(islot, istack);
		}

Config:

Items:
  mega:
    Slot: 0
    ItemID: 3
    Command: dd

Für das Item mega sollte eigendlich 1 Item im Inventar erstellt werden jedoch passiert nix aber auch keine Fehler.

 

[MiCrJonas]

Bei "getConfigurationSection("Items.").getKeys(false)) {" Darfst du keinen Punkt hiter "Items" machen. Benutze:

getConfigurationSection("Items").getKeys(false)) {

 

[Masterfran99]

hatte ich auch schon probier ging trotzdem nicht D:

 

[MiCrJonas]

Ich vestehe nicht ganz, was du da gemacht hast. "path" ist immer der aktuelle Unterpunkt von "Items", der im Moment in der For-Schleife benutzt wird. Diesen musst du auch benutzen!
Also mache das hier weg: "String iname = invc.getString("Items.");" und benutze überall, wo du "iname" benutzt "path".

 

[Masterfran99]

Ok es hat geklappt aber sobald mehr als nur 1 Item hinzugefügt wurden:

Items:
  mega:
    Slot: 0
    ItemID: 3
    Command: dd
  *****:
    Slot: 1
    ItemID: 55
    Command: c

ist das inventar wieder leer.

 

[MiCrJonas]

Benutze nach dem ganzen mal "p.updateInventory()". Das ist zwar deprecated (und dadurch gelb unterstrichen), aber es sollte damit funktionieren.

 

[Masterfran99]

Das Inventar hat aber keinen Holder ein Öffentliches Inventar.

 

[MiCrJonas]

Kann es sein, dass du jedes Item in den selben Slot setzt? Du musst zu der Schleife noch einen Counter hinzufügen:

int counter = 0;
for (Bedingung) {
  //Code
  inv.setItem(counter, ItemStack);
  counter++;
}