Saved Modules Toggle bug [REQ]

[MasterOf13FPS]

Heey, ich habe etwas gemacht, dass meine modules beim Restart des Clients gesaved werden, aber wenn ich meinen Client nun neustarte, sind die modules zwar getoggled aber sie funktionieren nicht...

hier mein loadStatus:
[hide]
public void loadStatus() {
File file = new File(Xerion.instance.path, "activeModules.xerion");
if (file.exists()) {
try (Scanner scanner = new Scanner(file)) {
while (scanner.hasNextLine()) {
String[] part = scanner.nextLine().split(separator);
if (part[0].equals(versionID)) {
if (!part[1].equals(Xerion.path)) {
scanner.close();
saveStatus();
return;
}
}
for (Module mod : getMods()) {
if (part[0].equals(mod.getName())) {
mod.setActive(Boolean.parseBoolean(part[1]));
}
}
}
scanner.close();
} catch (FileNotFoundException e) {
e.printStackTrace();
}
} else {
this.saveStatus();
}
}
[/hide]

schonmal danke im vorraus c:

lg, gnmodz

 

[MasterOf13FPS]

In dem Code kann ich leider keinen Grund dafür finden, allerdings kann man diesen auch verbessern.

Xerion.instance.path

Xerion.path

Scheinbar ist 'path' statisch, ich würde den 'static'-Modifier weglassen.

try (Scanner scanner = new Scanner(file)) {
[...]
scanner.close();
}

In einem try-with-resources Block ist es nicht nötig, die jeweiligen Closeables zu schließen

if (part[0].equals(versionID)) {
 if (!part[1].equals(Xerion.path)) {
  scanner.close();
  saveStatus();
  return;
 }
}

Das kann man so zusammenfassen:

if (part[0].equals(versionID) && !part[1].equals(Xerion.path)) {
 saveStatus();
 return;
}

Ich würde außerdem nicht mal schauen, ob die Version die gleiche ist, dann können die Benutzer die Konfiguration auch in einer späteren Version verwenden.

Du kannst auch mal JSON testen, das ist meiner Meinung nach eine bessere Möglichkeit eine Konfiguration zu speichern.