Klassen wie würde das in einer neuen class aussehen

[Jimyx]

hallo zusammen,
ich bin neu in der Javawelt und versuche momentan langsam von der strukturierten Programmierung in die objektorientierte überzugehen. Hierzu lerne ich ua. an einem privaten Übungsprojekt.Das Umfangreiche Programm soll im ersten schritt nur einen Rss-Stream einlesen und auswerten. Dieser wird ann anschließend in weiteren schritten weiterverarbeitet. Ich habe heute hierzu mehr oder weniger strukturiert folgenden code erstellt, der auch funktioniert und das tut was er soll.

import java.io.IOException;
import java.net.MalformedURLException;
import java.net.URL;
import java.util.ArrayList;
import java.util.List;
import com.sun.syndication.feed.synd.SyndEntry;
import com.sun.syndication.feed.synd.SyndFeed;
import com.sun.syndication.io.FeedException;
import com.sun.syndication.io.SyndFeedInput;
import com.sun.syndication.io.XmlReader;


public class RssReader {

    public static void main(String[] args) throws IllegalArgumentException, FeedException, IOException {
    	
    	URL feedUrl = new URL("");
        SyndFeedInput input = new SyndFeedInput();
        SyndFeed feed = input.build(new XmlReader(feedUrl));
        
        List <SyndEntry> entries = new ArrayList<SyndEntry>();
        entries = feed.getEntries();
        
			for (SyndEntry entry:entries){
				
	               System.out.println(entry.getTitle());
	               String Split = entry.getDescription().getValue();  
	               String[] Array = Split.split("<br />");
	               
	               int i = 0; 
	               int l = Array.length;
	    
	               while (i < l){
	            	   
	            	   System.out.println(Array[i]);
	            	   i++;
	            	       	   
	               }
	                                
	               System.out.println(entry.getLink() + "\n");
         
        }
    	
    }
    
}

Die Xml enthält steuerungszeichen, also <br /> , nach denen ich splitte. Soweit so gut. Jetzt habe ich mir gedacht um den objektorientierten Ansatz zu wählen und es etwas üben zu können möchte ich das ganze aus der main rausnehmen und in eine eigene Klasse bsp. ReadXml{} packen. Ich bin mit dem Aufbau einer Klasse also konstrukoren und methoden, datenkaspelung vertraut jedoch noch in der Anwendung unerfahren und es ist mir leider nicht gelungen das umzusetzen. Kann mir vielleicht jemand der erfahrenen Entwickler etwas unter die Arme greifen und mal zeigen wie meine Codeschnipsel in einer eigenen Klasse mit einem Konstruktor und einer Methode aussehen würde ?


Beste Grüße

Jimyx

 

[hdi]

Das ist doch alles schon ganz gut ausgelagert?! Du hast doch eigene Klassen: SyndFeedInput, SyndFeed, SyndEntry und sogar eine statische Hilfsklasse Split.* (Ob die jetzt von dir kommen oder einer anderen API spielt ja keine Rolle)

*Ne moment, das ist ja nur ein String. Randbemerkung: Variablennamen schreibt man mit kleinem Anfangsbuchstaben. Sonst denkt man, das ist ne Klasse, wie ich gerade...

Dieses Programm tut halt nicht wirklich was sinnvolles.. Klar, du kannst dir ne Klasse schreiben die eine List/Array an SyndFeeds nimmt und dann deine Schleife in eine Methode dieser Klasse auslagern. Aber das wäre hier schon Overkill imho. Also bei dem Schnipsel, den du hier gezeigt hast, würd ich sagen: Kein Bedarf an weiteren Klassen.

Wenn du diese Sache im Programm jetzt an verschiedenen Stellen machen willst dann ist es was anderes. Aber auch da würd erstmal eine Klasse mit einer statischen Methode ausreichen imho.

 

[timbeau]

Hi, ist ganz einfach. statt:

class Test {
... main () {
LOGIK 
}
}

einfach:

class Test {

public void methodenname() {
LOGIK
}


public static void main (){
Test test = new Test();
test.methodenname();
}
}

Solltest du noch etwas aus der Main übergeben wollen, beispielsweise eine URL, kannst du dies sowohl einem Konstruktor oder aber der Methode mit übergeben.

 

[hdi]

Man kann das mit der Objektorientiertheit auch übertreiben. Man kann hier nicht von einem Objektzustand o.ä. reden. Was höchstens noch Sinn macht ist die Logik in eine eigene Methode auszulagern (raus aus der main). Meinetwegen diese dann noch in eine eigene Klasse, aber bitte statisch. Was timbeau gepostet hat schießt aber über das Ziel hinaus. Das in ein Objekt zu kapseln bringt dir gar nichts, außer den Umstand, erstmal ein Objekt erzeugen zu müssen um deine Logik ausführen zu können.

edit: jm2c. Ich hab's anfangs auch voll übertrieben mit Klassen und Objekten. Erster hinweis darauf dass das nix mit einem Objekt zu tun hat ist wenn es keinerlei "Zustand" gibt der sich zur Laufzeit ändert. Wie gesagt, ich finde deinen Code absolut in Ordnung so wie er ist. Zumindest in diesem kleinen Kontext.

 

[timbeau]

Gut, aber was wäre, wenn das Programm jetzt innerhalb eines anderen Programms genutzt werden soll um einen RSS-Feed zu bearbeiten? Ich finde man kann ruhig anfangen sowas als Objekt zu konstruieren.

 

[hdi]

Wie gesagt in eine eigene Klasse auslagern kann er machen. Aber das kann auch ne Hilfsklasse mit einer statischen Methode readRssFeed(URL url) sein. Wozu der Umweg über ein "Objekt" (was aus Design-Sicht nicht wirklich eines wäre, weil es eh nix hat oder kann, es bietet nur eine gewisse Logik, die von seinem Zustand (den es gar nicht gibt) total unabhängig ist).

Kurz gesagt: Wenn es keine (sinnvollen) Instanz-Variablen gibt, dann ist das gesamte Objekt überflüssig. Und eine sinnvolle Instanz-Variable ist keine, die man erzeugt nur um einen Parameter im Objektkontext abzuspeichern, der sowieso nur von einer einzigen Methode verwendet wird.

 

[Jimyx]

Hallo,
danke für die Antworten beide Beiträge haben mir irgendwie helfen können. Der Rss-Reader soll an anderer Stelle nochmal abgerufen und aktualisiert werden können, daher wollte ich diese Funktion auslagern. Ich werde mich in die Thematik der statischen Methoden mal einlesen.

Beste Grüße

Jimyx