Fehler anhand des log Files finden?

Hallo Kurt,

deine nodelist ist leer daher kommt der Fehler

@Override
protected void onPostExecute(Void args) {   //Methode zum eig. Parsen und Ausgeben auf Screen bzw. Schreiben in Stringarray
    for (int temp=0; temp<nodelist.getLength(); temp++) { 
        Node nNode = nodelist.item(temp); 

mach Bitte nen try/catch drum um das zu prüfen und du solltest ein Resultat sehen.

In Zeile 181 befindet sich der Fehler.

Was stehtn da ?

LG,
Stefan

— geändert am 30.12.2014, 20:04:19

Genau, aber ich würde direkt die "NullPointerException" nehmen, falls noch andere Errors kommen, da Du weist das es die NullPointer ist.
Ein Fehler fällt schneller auf, wenn die App abstürtzt

Poste mal bitte deinen aktuellen Code.
Warscheinlich hast du eine innerClass definiert und in dieser sind statische Methoden nicht zugelassen.

LG

Entweder su lässt das static Attribut weg, denn wozu brauchst du es?
Oder du machst für deinen AsyncTask Downlader in eine eigene Datei, wobei ich das mit der Datei unabhängig vom Fehler machen würde denn es ist übersichtlicher wenn eigene Klassen auch in eigenen Dateien sind.

Also Klasse in eigene Date -> Fehler weg.
Gleichzeitig kannst du noch das static Attribut entfernen.

LG

Du musst nur das "static" attribut löschen.
Und ja am besten eine eigene Date für den AsyncTask erstellen

Kurt

Gut. Wie mach is das dann mit der Übergabe zwischen den beiden Dateien?

Du musst dich nicht darum kümmern. Du kannst 10 Klassen in eine Datei tun oder auch in 10 einzelne. Das ist in deinem Fall egal.

Außerdem würde ich das static da lassen, weil dadurch klar wird, dass die Methode unabhängig von Klassenvariablen ist.

Hast du deinen AsyncTask jetzt in eine eigene Datei ausgelagert?

Poste bitte nochmal deinen Code.

LG

Eine eigene Activity brauchst du auf keinen Fall!
Du erstellst eine neue Java Datei und kopiersr deinen AsyncTask ein. mehr musst du nicht ändern denn die Daten kommen trotzdem an, denn eine Klasse ist etwas anderes als eine Activity.

Ja einfach in ne neue Datei.

Wieder das gleiche wie im anderen Thread
Einfach eine globale nodeList anlegen, da diese Variable beim Kopieren warscheinlich nicht mitkopiert wurde.

Du brauchst en Import:

import org.w3c.dom.NodeList;

Und die Variable

NodeList nodelist;

Dann sollte es gegen, falls nien poste deinen Code (mit Imports)

Du musst die Nodelist in der Klasse definieren, ausserhalb macht es keinen Sinn und funktioniert nicht:
import java.net.URL;
import java.util.ArrayList;

import javax.xml.parsers.DocumentBuilder;
import javax.xml.parsers.DocumentBuilderFactory;

import org.w3c.dom.Document;
import org.w3c.dom.Element;
import org.w3c.dom.Node;
import org.w3c.dom.NodeList;
import org.xml.sax.InputSource;

import android.os.AsyncTask;
import android.util.Log;

public class DownloadXML extends AsyncTask {
NodeList nodelist;

@Override
protected Void doInBackground(String... Url) {
    try {
        URL url = new URL(Url[0]);
        DocumentBuilderFactory dbf = DocumentBuilderFactory.newInstance();
        DocumentBuilder db = dbf.newDocumentBuilder();
        Document doc = db.parse(new InputSource(url.openStream())); 
        doc.getDocumentElement().normalize();
        nodelist = doc.getElementsByTagName("park");
    }
    catch (Exception e) {
        Log.e("Error", e.getMessage());
        e.printStackTrace();
    }
    return null;
}

@Override
protected void onPostExecute(Void args) {

    try {
    for (int temp=0; temp<nodelist.getLength(); temp++) {
        Node nNode = nodelist.item(temp);
        if (nNode.getNodeType() == Node.ELEMENT_NODE) {
            Element eElement = (Element) nNode;

            //Elemente
        }
    }
}
catch (Exception e) {
e.printStackTrace();
}

}

private static String getNode(String sTag, Element eElement) {
NodeList nList = eElement.getElementsByTagName(sTag).item(0).getChildNodes();
Node nValue = (Node) nList.item(0);
return nValue.getNodeValue();
}
}

Lies dir den Text bitte mal durch. (Oder ist das zu viel verlangt?)

Ein kleines bisschen muss man Englisch können, um das zu verstehen. Das sollte aber kein Problem sein.

Steht eh im Logfile ^^

Internet Permission fehlt im Manifest.

— geändert am 06.02.2015, 15:45:37

Stefan W

Steht eh im Logfile ^^ [...]

Das meinte ich ja 😉

Hallo,

OverviewActivity.java Zeile 53 ist der Fehler.

Poste doch bitte den Code dazu.

ListViewAdapter - getCount passt ned.

Gut gesehen Ludy 👍

@Kurt: Willst du das bei jedem Fehler jetzt so machen? :

1. Du sagst: Die App stützt ab
2. Wir fragen ob im LogCat ein Fehler ist
3. Du postest den LogCat
4. Wir sagen dir die Zeile, die im LogCat steht
5. Du postest die Zeile ausm Code und erwartest, dass wir das fixen

Zum Fehler: Nachdem du setAdapter aufgerufen hast, wird das ListView gerendert. Dafür muss sie wissen, wieviele Items es gibt. Das fragt sie mit der getItemsCount(). Da dein Adapter anscheinend von ArrayAdapter erbt und im LogCat steht, dass die super Methode aufgerufen wurde und du sie nicht überschrieben hast, versucht der Adapter(glaube ich, ich hab mir den Source Code nicht angeguckt) anhand der im super Konstruktor gegebenen Parameter die Anzahl der Items herauszufinden. Da scheint etwas schief zu gehen.
Mach dir am besten einen BreakPoint in der Zeile im ArrayAdapter.
Ich vermute, dass die Liste oder das Array, die/das im super Konstruktor übergeben wurde, null ist.

Korrigier mich, wenn ich bei meinen Vermutungen falsch liege.

Edit: Stefan hats in Kurzform geschrieben 😁

— geändert am 06.02.2015, 17:22:12

Dein Code läuft unweigerlich auf eine NullPointerException hinaus:

OnTaskCompleted otc = new OnTaskCompleted() {

    @Override
    public void onTaskCompleted(ArrayList<WP> list) {
        neue_liste = list; //<-- Hier setzst du zwar die Liste, da muss aber auch alles weitere nach dem download rein...
    }
};

new DownloadXML(otc).execute(URL);  //Aufruf des Parsers

//------- Ab hier muss alles in die onFinished, da ja erst auf den fertigen download gewartet werden muss
ListViewAdapter adapter = new ListViewAdapter(this, neue_liste);
setListAdapter(adapter);

this.getListView().setOnItemClickListener(new OnItemClickListener() {
     public void onItemClick(AdapterView<?> a, View v, int position, long id) {
        Intent i = new Intent(FirstActivity.this, SecondActivity.class);
        startActivity(i);
        }
     } 
});

Dein Fehler ist wahresceinlich eine Folge dieses Falschen Codes:

super((Context) onTaskCompleted, R.layout.rowlayout_layout, list);
this.context = (Activity) onTaskCompleted;

Du musst weiterhin den Contest übergeben:

public ListViewAdapter(Context ctx, ArrayList list) {
    super(ctx, R.layout.rowlayout_layout, list);
    this.context = ctx;
    this.list = list;
  }

Und beim aufrufen dann:

  ListViewAdapter adapter = new ListViewAdapter(FirstActivity.this, neue_liste); //Context der Activity übergeben
                setListAdapter(adapter);

Das geht nbur so, da du in einem Listener in einer anonymen Klasse bist, da hat das schlüsselwort this eine andere Bedeutung.

Dann must du wieder auf die Activity var umsteigen, wenn das mit einem Context nicht geht.

Irgend ein Fehler beim Parsen?
Was sagt der log, bzw. der debugger?

Es hast mit deinem Parser zutun.
Der sagt:

: java.lang.NumberFormatException: Invalid int: "4,62"

Was bedeutet das jetzt für dich?

Ja das gibt es:

float f = Float.parseFloat("0.0065");

Du darfst dann aber auch nicht vergessen den Typ der variable in deiner WP Klasse zu ändern.

Wegen dem Komma, da gehört ein Punkt hin. Also 4.62 und nicht 4,62