Fehler beim parsen eines Bildlinks mit XmlPullParser

Ich habe folgenden Code der Super funktioniert

private List readFeed(XmlPullParser parser, String rssLink, String autorpic) throws XmlPullParserException, IOException {
parser.require(XmlPullParser.START_TAG, null, "rss");
String title = null;
String link = null;
String pubDate = null;
String autor = null;
String autorimage = null;
String imageUrlContent = null;
String imageUrlDescription = null;
String imageUrl = null;

    List<RssItem> items = new ArrayList<>();
    Boolean readItemContent = false;

    while (parser.next() != XmlPullParser.END_DOCUMENT) {
        if (parser.getEventType() != XmlPullParser.START_TAG) {
            continue;
        }
        String name = parser.getName();
        if (name.equals("item")) {
            readItemContent = true;
        }
        if (readItemContent) {
            if (name.equals("title")) {
                String dirtTitle = readValue(parser, "title");
                {
                    title = cleanTitle(dirtTitle);
                }
            } else if (name.equals("link")) {
                link = readValue(parser, "link");
                //  Log.d("RssParserLink", "url: " + link);
            } else if (name.equals("pubDate")) {
                pubDate = readDate(parser);


            } else if (name.equals("description")) {

                // Parse the html description to get the image url
                String html = readValue(parser, "description");
                org.jsoup.nodes.Document docHtml = Jsoup.parse(html);
                Log.d("RssParserLink", "Des: "+ autor  + docHtml);
                Element imgElement = docHtml.select("img").first();
                if (imgElement != null){

                    imageUrlDescription = imgElement.attr("src");
                    Log.d("RssParserLink", "Imagendes: "+ autor + imageUrl);

                } else {

                    Log.d ("RssParserLink", "Kein Link" + autor );
                }


            }   else if (name.equals("content:encoded")) {      // name.equals("description")||name.equals("content:encoded")) {

                // Parse the html description to get the image url
                String html = readValue(parser, "content:encoded");

                org.jsoup.nodes.Document docHtml = Jsoup.parse(html);
                Log.d("RssParserLink", "Con: " + docHtml);
                Element imgElement = docHtml.select("img").first();
                if (imgElement != null){

                    imageUrlContent = imgElement.attr("src");

                    if (imageUrlContent != null) {
                        //  String imagenUrlDes = imageUrlContent;
                        Log.d("RssParserLink", "Imagendes: " + imageUrl);
                    }
                    //Log.d("RssParser", "url: " + imageUrlDescription);


                }


            }


            if (imageUrlDescription != null) {
                imageUrl = imageUrlDescription;
            } else {
                if (imageUrlContent != null) {
                    imageUrl = imageUrlContent;
                }

            }
        autor = new String(rssLink);
        autorimage = new String(autorpic);

        Log.d("RssParserLink", "RssItem: " + autor + imageUrl);


        }


        Log.d("sturmboard", "url: " + autor + pubDate);
        //  Log.d("sturmboard", "image: " + autorimage);

        if (title != null && link != null && pubDate != null) {

            RssItem item = new RssItem(title, link, pubDate, autor, autorimage, imageUrl);
            items.add(item);
            title = null;
            link = null;
            pubDate = null;
            autor = null;
            autorimage = null;
            imageUrl = null;


        }


    }
    return items;


}

Leider werden die "content:encoded" Links dem Falschem Item zugeteilt (immer dem folgenden ... und sollte das leer sein ... dann auch dem nächsten)..... hat wer eine Idee ... was da falsch ist?

um es Richtig zu Formulieren ....

Alles Läuft perfekt .... nur das Ergebnis von else if (name.equals("content:encoded")) { wird immer dem falschem Item zugeteilt!

Wenn Ich alles Säubere und nur titel / Link / pubDate / description auslese funktioniert auch alles .... wenn ich aber nur description gegen content:encoded tausche ..... ergeht die imageUrl an Falsche Items........

und ja ich bin Ratlos .......

kommt per Nachricht

Auch sonst keiner eine Idee, woran es liegen könnte? Ich muss halt beide Tags "description" und "content:encoded" auslesen können ...

— geändert am 14.01.2015, 21:13:20

RSSHandler.class

import android.util.Log;
import android.util.Xml;

import org.jsoup.Jsoup;
import org.jsoup.nodes.Element;
import org.jsoup.select.Elements;
import org.xmlpull.v1.XmlPullParser;
import org.xmlpull.v1.XmlPullParserException;

import java.io.IOException;
import java.io.InputStream;
import java.util.ArrayList;
import java.util.List;

public class RssHandler {

// We don't use namespaces
private final String ns = null;


public List<RssItem> parse(InputStream inputStream, String autorlink, String link2) throws XmlPullParserException, IOException {
    try {
        XmlPullParser parser = Xml.newPullParser();
        parser.setFeature(XmlPullParser.FEATURE_PROCESS_NAMESPACES, false);
        parser.setInput(inputStream, null);
        parser.nextTag();
        return readFeed(parser, autorlink, link2);
    } finally {
        inputStream.close();
    }
}

private List<RssItem> readFeed(XmlPullParser parser, String rssLink, String autorpic) throws XmlPullParserException, IOException {
    parser.require(XmlPullParser.START_TAG, null, "rss");
    String title = null;
    String link = null;
    String pubDate = null;
    String autor = null;
    String autorimage = null;
    String imageUrlContent = null;
    String imageUrlDescription = null;
    String imageUrl = null;


    List<RssItem> items = new ArrayList<RssItem>();
    Boolean readItemContent = false;

    while (parser.next() != XmlPullParser.END_DOCUMENT) {
        if (parser.getEventType() != XmlPullParser.START_TAG) {
            continue;
        }
        String name = parser.getName();
        if (name.equals("item")) {
            readItemContent = true;
        }
        if (readItemContent) {
            if (name.equals("title")) {
                String dirtTitle = readValue(parser, "title");
                {
                    title = cleanTitle(dirtTitle);
                    Log.d("RssParserLink", "HtmlContent: " + title);
                }

            } else if (name.equals("link")) {
                link = readValue(parser, "link");
                //  Log.d("RssParserLink", "url: " + link);
            } else if (name.equals("pubDate")) {
                pubDate = readDate(parser);


            } else if (name.equals("description")) {

                // Parse the html description to get the image url
                String html = readValue(parser, "description");
                org.jsoup.nodes.Document docHtml = Jsoup.parse(html);
                Log.d("RssParserLink", "Descript: " + docHtml);
                Element imgElement = docHtml.select("img").first();
                if (imgElement != null) {

                    imageUrlDescription = imgElement.attr("src");
                    Log.d("RssParserLink", "Imagendes: " + imageUrlDescription);
                    imageUrl = imageUrlDescription;
                } else {

                    Log.d("RssParserLink", "Kein LinkAAA" + autor);
                }


            }/* else if (name.equals("content:encoded")) { 

                // Parse the html description to get the image url
                String html = readValue(parser,"content:encoded");

                org.jsoup.nodes.Document docHtml = Jsoup.parse(html);

                Element imgElement2 = docHtml.select("img").first();
                if (imgElement2 != null) {


                    imageUrlContent = imgElement2.attr("src");

                } else {
                    Log.d("RssParserLink", "Kein LinkBBB" + autor);
                }


            }*/


        }

if (imageUrlDescription != null) {
imageUrl = imageUrlDescription;
} else {
if (imageUrlContent != null) {
imageUrl = imageUrlContent;
}

        }

        autor = new String(rssLink);
        autorimage = new String(autorpic);

       // Log.d("RssParserLink", "RssItemsTest: " + autor + imageUrl);


         if (title != null && link != null && pubDate != null) {

        RssItem item = new RssItem(title, link, pubDate, autor, autorimage, imageUrl);
             Log.d("RssParserLink", "RssItemsTest: " + autor + imageUrl);
        items.add(item);
        title = null;
        link = null;
        pubDate = null;
        autor = null;
        autorimage = null;
        imageUrl = null;


         }


    }
    return items;


}

private String cleanTitle(String str) {

    str = str.replace("„", "");
    str = str.replace("“", "");
    str = str.replace("Ä", "Ä");
    str = str.replace("Ö", "Ö");
    str = str.replace("Ü", "Ü");
    str = str.replace("ß", "ß");
    str = str.replace("ä", "ä");
    str = str.replace("ö", "ö");
    str = str.replace("ü", "ü");
    str = str.replace("ẞ", "ß");
    str = str.replace(""", "");
    str = str.replace("á", "á");
    str = str.replace("ä", "ä");
    str = str.replace("ß", "ß");
    str = str.replace("ö", "ö");
    str = str.replace("ü", "ü");
    str = str.replace("“", "“");
    str = str.replace("„", "„");
    return str;
}

private String readValue(XmlPullParser parser, String key) throws XmlPullParserException, IOException {
    parser.require(XmlPullParser.START_TAG, ns, key);
    String link = readText(parser);
    parser.require(XmlPullParser.END_TAG, ns, key);
    return link;
}



private String readDate(XmlPullParser parser) throws XmlPullParserException, IOException {
    parser.require(XmlPullParser.START_TAG, ns, "pubDate");
    String date = readText(parser);
    DateFormat.parseDate(date.toString());
    parser.require(XmlPullParser.END_TAG, ns, "pubDate");
    return date;
}
// For the tags title and link, extract their text values.


private String readText(XmlPullParser parser) throws IOException, XmlPullParserException {
    String result = "";
    if (parser.next() == XmlPullParser.TEXT) {
        result = parser.getText();
        parser.nextTag();
    }
    return result;
}

private void skip(XmlPullParser parser) throws XmlPullParserException, IOException {
    if (parser.getEventType() != XmlPullParser.START_TAG) {
        throw new IllegalStateException();
    }
    int depth = 1;
    while (depth != 0) {
        switch (parser.next()) {
            case XmlPullParser.END_TAG:
                depth--;
                break;
            case XmlPullParser.START_TAG:
                depth++;
                break;
        }
    }
}

}

RssService.class

import android.app.IntentService;
import android.content.Context;
import android.content.Intent;
import android.content.SharedPreferences;
import android.os.Bundle;

import android.os.ResultReceiver;
import android.preference.PreferenceManager;
import android.support.v4.app.NotificationCompat;
import android.util.Log;

import org.xmlpull.v1.XmlPullParserException;

import java.io.IOException;
import java.io.InputStream;
import java.io.Serializable;
import java.net.URL;
import java.text.ParseException;
import java.text.SimpleDateFormat;
import java.util.Collections;
import java.util.Comparator;
import java.util.Date;
import java.util.List;
import java.util.Locale;
import java.util.TimeZone;

public class RssService extends IntentService implements Serializable {

private static final String RSS_LINK_1 = Irgendein link
private static final String RSS_LINK_2 = irgendein link


int image1 = R.drawable.rssapp;
int image2 = R.drawable.facebook;

public static final String ITEMS = "items";
public static final String RECEIVER = "receiver";

Tools tools;




public RssService() {
    super("RssService");
}


@Override
protected void onHandleIntent(Intent intent) {



    Log.d(Constants.TAG, "Service started");

    List<RssItem> rssItems = null;
    try {
        RssHandler parser = new RssHandler();
        rssItems = parser.parse(getInputStream(RSS_LINK_1), "Titel 1 ", String.valueOf(image1));
        rssItems.addAll(parser.parse(getInputStream(RSS_LINK_2), "Titel 2 ", String.valueOf(image2)));

        Collections.sort(rssItems, new CustomComparator());



    } catch (XmlPullParserException e) {
        Log.w(e.getMessage(), e);
    } catch (IOException e) {
        Log.w(e.getMessage(), e);
    }


    Bundle bundle = new Bundle();
    bundle.putSerializable(ITEMS, (Serializable) rssItems);
    ResultReceiver receiver = intent.getParcelableExtra(RECEIVER);
    receiver.send(0, bundle);


}




public class CustomComparator implements Comparator<RssItem> {


    public String cleanPub(String str) {
        str = str.replace(" +0000", "");
        str = str.replace("Dec", "12");
        str = str.replace("Nov", "11");
        str = str.replace("Oct", "10");
        str = str.replace("Sep", "09");
        str = str.replace("Aug", "08");
        str = str.replace("Jul", "07");
        str = str.replace("Jun", "06");
        str = str.replace("May", "05");
        str = str.replace("Apr", "04");
        str = str.replace("Mar", "03");
        str = str.replace("Feb", "02");
        str = str.replace("Jan", "01");
        str = str.replace(":", "");
        str = str.replace(" ", "");
        str = str.substring(4);
        return str;
    }

    @Override
    public int compare(RssItem o1, RssItem o2) {


        String[] formats = 
                {     //  "dMMyyyyHHmmss",
                        "ddMMyyyyHHmmss"// ,


                };
        for (String format : formats) {
            SimpleDateFormat sdf = new SimpleDateFormat(format);

            sdf.setTimeZone(TimeZone.getTimeZone("UTC"));


            try {

                Date date1 = sdf.parse(cleanPub(o1.getPublicationDate()));
                Date date2 = sdf.parse(cleanPub(o2.getPublicationDate()));
                return date2.compareTo(date1);
              /*  if (date2.before(date1))
                return -1;
                else if (date2.after(date1))
                  return +1; */

            } catch (ParseException ex) {
                Log.d("RssService", ex.getMessage());
            }


        }


        return 0;
    }
}

public InputStream getInputStream(String link) {
    try {
        URL url = new URL(link);
        return url.openConnection().getInputStream();
    } catch (IOException e) {
        Log.d("RssService", "Exception while retrieving the input stream", e);
        return null;
    }
}

RssItem.class

public class RssItem {

private final String title;
private final String link;
private final String pubDate;
private final String autor;
private final String autorimage;
private final String imageUrl;


public RssItem(String title, String link, String pubDate, String autor,String autorimage, String imageUrl) {
    this.title = title;
    this.link = link;
    this.pubDate = pubDate;
    this.autor = autor;
    this.autorimage = autorimage;
    this.imageUrl = imageUrl;

}


public String getTitle() {
    return title;
}

public String getLink() {
    return link;
}

public String getPublicationDate() {
    return pubDate;
}



public String getAutor() {
    return autor;
}

public String getAutorimage(){return autorimage;}

public String getImageUrl() {
    return imageUrl;
}

}

— geändert am 15.01.2015, 17:43:51

Das Problem ist, das die imageUrlContent dem Falschem Item (immer dem nächsten und teilweise dem übernächsten) zugeteilt wird......

hab in meiner Handler.class im Log.. folgendes geändert um mein Problem Verständlicher zu machen!

RssItem item = new RssItem(title, link, pubDate, autor, autorimage, imageUrl);
      ------>       Log.d("RssParserLink", "RssItemsTest: " + autor + title + imageUrl);  <------
        items.add(item);
        title = null;
        link = null;
        pubDate = null;
        autor = null;
        autorimage = null;
        imageUrl = null;

und dann kommt im Logcat unter dem Suchbegriff "RssItemsTest" das Raus:

     Sturm12 Epilog: Sturm12.at verabschiedet sichnull
     Sturm12 12 Momente:   Christopher, Didi und Jürgenhttp://www.sturm12.at/wp-content/uploads/10177915_10202716632569569_8573250731790600559_n-300x400.jpg
     Sturm12 12 Momente: Gordon, Peter und Ivan Sturm12 Sechs Jahre Sturm12.at in Zahlen 
     Sturm12 12 Momente: Julia, Peter und Andreashttp://www.sturm12.at/wp-content/uploads/diagramme-2-976x690.jpg
     Sturm12 12 Momente: Lukas, Werner und Daniel 
     Sturm12 Sturm12.at geht, die Statistik bleibt 
     Sturm12 12 legendäre Livetickerhttp://www.sturm12.at/wp-content/uploads/20140823-1035_1DCB5242-217x300.jpg
     Sturm12 12 legendäre  Fotoshttp://www.sturm12.at/wp-content/uploads/dsc_2368-500x332.jpg
     Sturm12 Torfabrik-Bonushttp://www.sturm12.at/wp-content/uploads/2009_Kharkiv.jpg

und hier zeigt sich schon der Fehler .... im Ersten Item ist der Link null ... was Falsch ist .... und der Link steht im nächsten Item!

siehe Feed: view-source:http://feeds.feedburner.com/Sturm12at?format=xml

egal was ich mache immer Falsch --- nur wenn ich im new RssItem if (imageUrl != null) bestimme ... kommts richtig .... dann fehlen aber die ganzen Items ohne Bilder ... was ja klar ist!

Also Ich hab keine Ahnung mehr!

— geändert am 19.01.2015, 16:37:57

1. Das mit den Variablen hatte ich schon, dann fehlen aber alle Items ohne Bildlink... Dann würde auch ein != imageUrl reichen

2. Ich hol mir die Bilder(falls vorhanden) um Sie als Titelbilder zu verwenden...

3. Die Items sind dann nur eine Übersicht, der Aktuellen Artikel und verlinken auf die jeweilige Seite

— geändert am 22.01.2015, 19:48:43