Fehler beim erstellen einer API

noci · Dec 31st 2017

Hallo,

ich versuche gerade eine kleine API zu bauen und bekomme volgende Fehlermeldung beim Starten von RW.
möchte mit einer protected void INI()" Funktion Arbeiten, in NetBeans gibt es keine Probleme beim einbinden der Class.
Was Fehlt damit er instantiate kann?

RW SERVER: Fatal error occurred! pluginapi.PluginException: SpracheAPI: Cannot instantiate plugin main class: "de.chaoswg.SprachAPI" at pluginapi.PluginLoader.loadPlugin(SourceFile:274) at pluginapi.PluginLoader.loadPluginFromJar(SourceFile:194) at pluginapi.PluginLoader.loadPlugins(SourceFile:45) at pluginapi.PluginManager.initialize(SourceFile:67) at server.Main.e(SourceFile:517) at server.Main.initApplication(SourceFile:282) at commons.JIWApplication.initialize(SourceFile:236) at com.jme3.system.NullContext.initInThread(NullContext.java:87) at com.jme3.system.NullContext.run(NullContext.java:130) at java.lang.Thread.run(Unknown Source)

red51 · Dec 31st 2017

Poste am besten einmal den Code der Klasse hier Der Fehler, dass die Klasse nicht instantiiert werden kann, tritt meist entweder auf, wenn der Konstruktor private oder protected ist, oder wenn die Klasse abstrakt ist.

PS: Beim Posten von Source-Code oder auch Fehlermeldungen am besten einen Code Block </> verwenden Denn mit dem [tt] ist Mehrzeiligkeit nicht möglich, was das ganze dann unlesbar macht^^

noci · Dec 31st 2017

Java

/*
 * To change this license header, choose License Headers in Project Properties.
 * To change this template file, choose Tools | Templates
 * and open the template in the editor.
 */
package de.chaoswg;


import java.io.BufferedWriter;
import java.io.File;
import java.io.FileInputStream;
import java.io.FileOutputStream;
import java.io.FileWriter;
import java.io.IOException;
import java.io.PrintWriter;
import java.util.ArrayList;
import java.util.Collections;
import java.util.Enumeration;
import java.util.List;
import java.util.Properties;
import java.util.Vector;
import net.risingworld.api.Plugin;
import net.risingworld.api.objects.Player;


public class SprachAPI extends Plugin {
    private Plugin plugin;
    private int debug = 1;
    private ArrayList<String> VariablenName;
    private ArrayList<String> inputText;
    private List<List<String>> Text;

    protected ArrayList<String> Sprache;
    protected String[ ][ ] Daten;

    private SortedProperties textConfig = new SortedProperties();
    String sFile;
    String sDir = "/locale";
    String sFileRest =".lang";
    File pfile;
    File pdir; 

    @Override
    public void onEnable() {
    }


    @Override
    public void onDisable() {
    }

    public SprachAPI (Plugin plugin) {
        this.plugin = plugin;
        VariablenName = new ArrayList();
        Text = new ArrayList<>();


        inputText = new ArrayList();
        //### Standart Sprachen Setzen
        Sprache = new ArrayList();
        //Sprache.add("en");
        //Sprache.add("de");
        //Sprache=Sprachen;
        INI();


        //### Convertieren in das Arbeistformat
        for (String[] arr : Daten){
            VariablenName.add(arr[0]);
            inputText = new ArrayList();
            inputText.add(arr[1]);
            inputText.add(arr[2]);
            Text.add(inputText);
        }
        //### Verzeichnis Erstellen, wenn vorhanden Fehler egal
        pdir = new File(plugin.getPath() + sDir);
        if (pdir.mkdir()) { if(debug>0){System.out.println("[" + plugin.getDescription("name") + "]"+" Verzeichnis erstellt.");} }
        //### Laden und Prüfen oder Erstellen der Voreinstellungen
        for (int lang=0; lang< Sprache.size(); lang++) {
            if(debug>2){System.out.println("\n[" + plugin.getDescription("name") + "] "+"Sprache "+Sprache.get(lang));}
            sFile = sDir+""+Sprache.get(lang)+sFileRest;
            pfile = new File(plugin.getPath() + sFile);
            if(debug>2){System.out.println("[" + plugin.getDescription("name") + "] "+plugin.getPath() +  sFile);}
            if(debug>2){System.out.println("[" + plugin.getDescription("name") + "] "+"File: "+pfile.toString());}

            if (pfile.exists()) {
                if(debug>0){System.out.println("[" + plugin.getDescription("name") + "] "+"Lade["+Sprache.get(lang)+"]");}
                try {
                    textConfig.clear();
                    textConfig.load(new FileInputStream(plugin.getPath() + sFile));
                    for (int varName=0;varName<VariablenName.size();varName++){
                        if(debug>1){System.out.print("[" + plugin.getDescription("name") + "] ");}
                        if (textConfig.getProperty(VariablenName.get(varName))==null){
                            textConfig.setProperty(VariablenName.get(varName), Text.get(varName).get(lang));
                            textConfig.store(new FileOutputStream(plugin.getPath() + sFile), null);
                            if(debug>1){System.out.print("neu ");}
                        }else{
                            if(debug>1){System.out.print("Lade ");}
                        }
                        Text.get(varName).set(lang, textConfig.getProperty(VariablenName.get(varName)));                        
                        if(debug>1){System.out.println("   "+VariablenName.get(varName)+" = "+Text.get(varName).get(lang));}
                    }
                }catch(IOException e){
                    System.err.println("[" + plugin.getDescription("name") + "] "+e.getMessage());
                }                
            } else {
                if(debug>0){System.out.println("[" + plugin.getDescription("name") + "] "+"Erstelle["+Sprache.get(lang)+"]");}
                try{
                    //### Datei vorbereiten
                    try (PrintWriter pWriter = new PrintWriter(new BufferedWriter(new FileWriter(plugin.getPath() + sFile)))) {
                        //### Schreibt die Config! (Es werden die Standartwerte gesetzt!)
                        if (Sprache.get(lang).equals("de")){
                            pWriter.println("UmlauteNorm: |  Ae  |  ae  |  Oe  |  oe  |  Ue  |  ue  |  ss  |");
                            pWriter.println("UmlauteCode: |Ä|ä|Ö|ö|Ü|ü|ß|");
                        }
                        pWriter.close(); //Datei muss erste erstellt werden, befor sie abgerufen werden kann, sonst NULL!
                        textConfig.clear();
                        textConfig.load(new FileInputStream(plugin.getPath() + sFile));
                        for (int varName=0;varName<VariablenName.size();varName++){
                            if(debug>1){System.out.println("[" + plugin.getDescription("name") + "] "+"Erstelle "+VariablenName.get(varName)+" = "+Text.get(varName).get(lang));}
                            textConfig.load(new FileInputStream(plugin.getPath() + sFile));
                            textConfig.setProperty(VariablenName.get(varName), Text.get(varName).get(lang));
                            textConfig.store(new FileOutputStream(plugin.getPath() + sFile), null);
                        }
                    }
                }catch(IOException e){
                    System.err.println("[" + plugin.getDescription("name") + "] "+"Fehler "+e.getMessage());
                }
            }
            textConfigRewrite(plugin.getPath() + sFile,lang);
        } 
        if(debug>1){System.out.println();}
        File dirLange = new File(plugin.getPath() + sDir);
        File[] fileArray = dirLange.listFiles();
        if(debug>1){System.out.println("[" + plugin.getDescription("name") + "] "+fileArray.length+" von "+Sprache.size());}
        if (fileArray.length>Sprache.size()){
            int iLang = -1;
            for (File fFind:fileArray){
                if(debug>1){System.out.println("[" + plugin.getDescription("name") + "] "+fFind.getName()+" "+fFind.getName().substring(0, 2)+" "+Sprache.indexOf(fFind.getName().substring(0, 2)));}
                String lang = fFind.getName().substring(0, 2);
                if(debug>2){System.out.println("[" + plugin.getDescription("name") + "] "+lang+" "+Sprache.indexOf(lang));}
                if(Sprache.indexOf(lang)<0){
                    Sprache.add(lang);
                    sFile = sDir+"/"+lang+sFileRest;
                    pfile = new File(plugin.getPath() + sFile);
                    pdir = new File(plugin.getPath() + sDir);
                    iLang = Sprache.indexOf(lang);
                    if(debug>0){System.out.println("[" + plugin.getDescription("name") + "] "+"Lade["+lang+"]");}
                    try {
                        textConfig.clear();
                        textConfig.load(new FileInputStream(plugin.getPath() + sFile));
                        for (int varName=0;varName<VariablenName.size();varName++){
                            if(debug>1){System.out.print("[" + plugin.getDescription("name") + "] ");}
                            if (textConfig.getProperty(VariablenName.get(varName))==null){
                                textConfig.setProperty(VariablenName.get(varName), Text.get(varName).get(0));
                                textConfig.store(new FileOutputStream(plugin.getPath() + sFile), null);
                                if(debug>1){System.out.print("neu ");}
                            }
                            Text.get(varName).add(textConfig.getProperty(VariablenName.get(varName)));                        
                            if(debug>1){System.out.println("dazu "+Text.get(varName).get(iLang));}
                        }
                        textConfigRewrite(plugin.getPath() + sFile,iLang);
                    }catch(IOException e){
                        System.err.println("[" + plugin.getDescription("name") + "] "+e.getMessage());
                    }
                }
            }
        }
        if(debug>1){System.out.println();}
    }

    private void textConfigRewrite(String file, int iLang){
        //## Datei leeren und textConfig reinschreiben
        if(debug>2){System.out.println("[" + plugin.getDescription("name") + "] "+"Aktualiesieren "+(VariablenName.size()+(Sprache.get(iLang).equals("de")?2:0))+" | "+textConfig.size());}
(Sprache.get(iLang).equals("de")&&VariablenName.size()+2<textConfig.size()) ){
        if ( (VariablenName.size()+(Sprache.get(iLang).equals("de")?2:0))<textConfig.size() ){
            if(debug>0){System.out.println("[" + plugin.getDescription("name") + "] "+"Aktualiesieren "+"");}
            try (PrintWriter pWriter = new PrintWriter(new BufferedWriter(new FileWriter(file)))) {
                if (!Sprache.get(iLang).equals("de")){
                    pWriter.write("");
                }
                pWriter.close(); //Datei muss erste erstellt werden, befor sie abgerufen werden kann, sonst NULL!
                textConfig.clear();
                if (Sprache.get(iLang).equals("de")){
                    textConfig.setProperty("UmlauteNorm", "|  Ae  |  ae  |  Oe  |  oe  |  Ue  |  ue  |  ss  |");
                    textConfig.store(new FileOutputStream(file), null);
                    textConfig.setProperty("UmlauteCode", "|Ä|ä|Ö|ö|Ü|ü|ß|");
                    textConfig.store(new FileOutputStream(file), null);
                }
                for (int varName=VariablenName.size()-1;varName>0;varName--){
                    //System.out.println("[" + plugin.getDescription("name") + "] "+varName);
                    if(debug>1){System.out.println("[" + plugin.getDescription("name") + "] "+"Aktualiesieren "+Text.get(varName).get(iLang));}
                    textConfig.setProperty(VariablenName.get(varName), Text.get(varName).get(iLang));
                    textConfig.store(new FileOutputStream(file), "-_-");
                }
            }catch(IOException e){
                System.err.println("[" + plugin.getDescription("name") + "] "+e.getMessage());
            }
        }
    }


    public String getText(Player player,String varName){
        String strLang="";
        //### Sprache erkennen
        if (player==null){strLang = player.getLanguage();}else{strLang="en";}
        //### Sprache Finden
        int iSprache = Sprache.indexOf(strLang);
        if (iSprache<0){iSprache=1;}
        if (iVariable>=0){return Text.get(iVariable).get(iSprache);}else{return null;}
    }


    /**
     *
     */
    protected void INI() {
        System.err.println("Keine Text Daten, Initialiesiert.");
    }


  
    class SortedProperties extends Properties {
        public Enumeration keys() {
            Enumeration keysEnum = super.keys();
            Vector<String> keyList = new Vector<String>();
            while(keysEnum.hasMoreElements()){
               keyList.add((String)keysEnum.nextElement());
            }
            Collections.sort(keyList);
            return keyList.elements();
        }    
    }
}

Display More

red51 · Dec 31st 2017

Danke für den Code Das Problem liegt darin, dass unbedingt ein leerer Konstruktor vorhanden sein muss, oder genauer ausgedrückt, ein Konstruktor, welcher keine Parameter entgegennimmt. Wenn kein Konstruktor definiert ist, ist so ein Konstruktor in Java immer automatisch vorhanden.
Da du bei dir aber den Konstruktor mit einem "plugin" Parameter erstellt hast, verschwindet der Standardkonstruktor automatisch - es sei denn, du deklarierst den ebenfalls.

Die "direkte Lösung" (wenngleich falsche Lösung) wäre, einen zweiten Konstruktor ohne Parameter hinzuzufügen:

Java

public SprachAPI(){
}

Das ist allerdings sicherlich nicht das, was deiner Intention entspricht. Dein Konstruktor enthält den "Plugin" Parameter, falls du da eine Referenz auf das SprachAPI Objekt erwartest, dann lass diesen Parameter ganz weg, da SprachAPI ja direkt auf sich selbst zugreifen kann (du kannst also innerhalb der Klasse SprachAPI sowieso direkt auf die eigenen Plugin-Methoden wie zB "getDescription()" oder "getPath()" zugreifen). In dem Fall ist es ggf. besser, nicht mit einem Konstruktor zu arbeiten (da dieses Plugin-Objekt zu diesem Zeitpunkt noch nicht ganz initialisiert ist), sondern den Code, der jetzt im Konstruktor ist, lieber in die "onEnable()" Funktion zu packen (diese wird dann aufgerufen, wenn das Plugin voll initialisiert und geladen ist).

Falls da als Parameter jedoch ein ganz anderes Plugin erwartet wird, würde ich das lieber auslagern in eine separate Klasse o.ä., denn SprachAPI ist ja bereits ein Objekt vom Typ Plugin (denn auch hier ist wieder die Sache, dass andere Plugins zum Zeitpunkt des Konstruktor-Aufrufs u.U. noch nicht initialisiert sind).

noci · Dec 31st 2017

Erstmal Danke fürs promte Antworten.

Da hast du recht der Code soll schon sauber sein und nach den Regeln der Kunst.

Also das mit dem Leeren Konstruktor hat wunderbar Funktioniert.
Nur kann ich natürlich nicht alles einfach in die onEnable reinschieben ohne NullPointerException zu bekommen. Das strukturiere ich nachher um wenn die Augen wieder weiter aufgehen und das Hirn mehr als zwei Zeilen Springen kann.

noci · Dec 31st 2017

Frage geht das überhaupt?

mann soll nachher einer Class

Code

class wgClassText extends SprachAPI {
    @Override
    protected void setDaten(){
        this.Sprache.add("en");
        this.Sprache.add("de");
        this.Daten = new String[][] {
            {"Variable 1","en 1","de 1"},
            {"Variable 2","en 2","de 2"}
        }
}

Die Variablen Setzen können.
habe sie erstmal in eine INI-Funktion gesetzt die nach der initialiesierung in der Haupt Class aufgerufen wird.
textDaten = new wgClassText();
textDaten.INI();
Bekomme ich immer noch einen NullPointerException wenn ich danach auf die Variablen zugreifen möchte.

noci · Dec 31st 2017

GETer/SETer X/

red51 · Jan 1st 2018

Quote from noci

Nur kann ich natürlich nicht alles einfach in die onEnable reinschieben ohne NullPointerException zu bekommen.

Dann liegt das aber am Code, denn das ist so der vorgesehene Weg Denn es ist nicht vorgesehen, dass eigene Instanzen einer Plugin-Klasse erzeugt werden. Das Spiel bzw. der Server sind die einzigen, welche neue Plugin-Instanzen erzeugen dürfen, denn von selbsterzeugten Instanzen weiß das Spiel bzw. der Server nichts (somit würden auch die meisten der Plugin-Methoden für diese selbsterzeugten Instanzen nicht funktionieren). Natürlich kannst du trotzdem mit Klassen und eigenen Objekten arbeiten, OOP ist ja weiterhin möglich (und auch zu bevorzugen), aber diese sollten dann am besten nicht von Plugin abgeleitet sein.

Im Zweifelsfall kannst du die Version mit dem Code in "onEnable()" einmal hier posten mit ggf. dem Hinweis, in welcher Zeile die NPE auftritt

Participate now!