String wird richtig in konsole geschrieben aber falsch an Methode übergeben

[Framehacker]

Hallo

Ich weiß jetzt nicht ob ich hier richtig bin da es hier um das verhalten von Strings geht dachte ich es jedenfalls falls ich hier trozdem falsch bin bitte Verschieben.

Ich versuche gerade mittels eine for-schleife durch Verschiedene Url´s zu gehen die sich nur durch eine zahl unterscheiden (seite 1 - irgendwas). und mir den von der Entsprechenden url den Quellcode zurückliefern lassen.

    for(int i = 1; i < Math.ceil(kp/10);i++){

        quelle = qcread.GetQuellCode("https://www.blablub.com/blub/list?page="+i+"&s=rl");
        anfang = quelle.indexOf(startString);
        System.out.println("https://www.blablub.com/blub/list?page="+i+"&s=rl");
        
        if(i == 4) {
            System.out.println(quelle);
        }
}

die getQuellCode methode sieht so aus

public String GetQuellCode(String s) {
        try {
            ur = new URL(s);

            if(ur != null){
            Scanner sc;
            sc = new Scanner(ur.openStream());
            while(sc.hasNextLine()){
                stb.append(sc.nextLine() + "\n");
            }
            return stb.toString();
            }
        } catch (Exception ex) {
                Logger.getLogger(QuellCodeReader.class.getName()).log(Level.SEVERE, null, ex);
        }
        
        
        
        
        
        return "";

}

Das Seltsame ist bei jeden schleifen Durchlauf wird zwar die url richtig in die Konsole geschrieben auch die If anweisung wird richtig erst beim 4 ten Durchlauf durchlaufen allerdings wird mir immer nur der quellcode von der ersten url also "https://www.blablub.com/blub/list?page=1&s=rl" ausgegeben. wenn ich jetzt die url nicht zusammensetze also sie z.B. "https://www.blablub.com/blub/list?page=4&s=rl" schreibe wird der quellcode richtig von seite 4 ausgegeben.

Ich bin da gerade recht ratlos habe schon versucht den String mit einen Stringbuilder zusammenzusetzen und String.format habe ich auch probiert das Ergebnis war immer dasselbe:bahnhof:

 

[eRaaaa]

Das ist alles nicht ganz schlüssig und nachvollziehbar, einfacher wäre ein kleines ausführbares Beispielprogramm.

allerdings wird mir immer nur der quellcode von der ersten url also "https://www.blablub.com/blub/list?page=1&s=rl" ausgegeben

Wo denn? In dem Code gibt es ja nur bei i=4 eine Ausgabe vom Quellcode.
Des weiteren sieht man nicht wo du den Stringbuilder stb erstellst, bzw, demnach würde es bedeuten, dass du den Quellcode von jeder Seite dem Builder anhängst, siehst du deshalb vlt. immer nur den 1. Quellcode, bzw. denkst dass du ihn siehst, der Rest steht einfach immer nur etwas weiter hinten (weil er eben angehängt wird)?

Ansonsten zeige doch mal mehr Code

 

[Altairograph]

Eine andere Erklärung wäre dass das verschiedene Abschnitte ein und derselben website sind und du immer den kompletten Quelltext der Seite bekommst (allerdings ist mir das jetzt nur spontan eingefallen, ich weiß nicht ob das wirklich eine Möglichkeit ist). Versuch doch mal dir bei jedem Durchlauf den Quelltext ausgeben zu lassen und schau ob du immer die gleiche Ausgabe bekommst. Und wenn ja dann lass dir dazu noch die Länge des Quelltexts ausgeben. Wenn die sich bei jedem Durchlauf verändert ist es wahrscheinlich so wie eRaaaa vermutet.

 

[Framehacker]

Sry habe gedacht das wäre ausreichend habe aber wie gefordert das ganze mal ausgelagert und ein Beispielprogramm draus gemacht.

die main

package test;


public class Test {

    /**
     * @param args the command line arguments
     */
    public static void main(String[] args) {
        
        QCReader qcread = new QCReader();
        String quelle;
        
        for(int i = 1; i == 10;i++){

        quelle = qcread.GetQuellCode("https://www.blablub.com/blub/list?page="+i+"&s=rl");
        System.out.println("https://www.blablub.com/blub/list?page="+i+"&s=rl");
        
        if(i == 4) {
            System.out.println(quelle);
        }
        }

        
        
        
    }

    
}

die Klasse QCReader

package test;

import java.net.URL;
import java.util.Scanner;


public class QCReader {
    URL ur;
    StringBuilder stb = new StringBuilder();
    
        public String GetQuellCode(String s) {

        
        try {
            ur = new URL(s);

            if(ur != null){
            Scanner sc;
            sc = new Scanner(ur.openStream());
            while(sc.hasNextLine()){
                stb.append(sc.nextLine() + "\n");
            }
            return stb.toString();
            }
        } catch (Exception ex) {
               
        }
        
        return "";

}
    
    
}

Die Konsolen Ausgabe ist

https://www.blablub.com/blub/list?page=1&s=rl
https://www.blablub.com/blub/list?page=2&s=rl
https://www.blablub.com/blub/list?page=3&s=rl
https://www.blablub.com/blub/list?page=4&s=rl
[Quellcode von https://www.blablub.com/blub/list?page=1&s=rl] (was ich nicht verstehe eigendlich sollte hier der Quellcode von https://www.blablub.com/blub/list?page=4&s=rl kommen sollte.)
https://www.blablub.com/blub/list?page=5&s=rl....

Schreibe ich die for-schleife allerdings so also gebe Direkt die url von Seite 4 ein

        for(int i = 1; i < 10;i++){

        quelle = qcread.GetQuellCode("https://www.blablub.com/blub/list?page=4&s=rl");
        System.out.println("https://www.blablub.com/blub/list?page=4&s=rl");
        
        if(i == 4) {
            System.out.println(quelle);
        }
        }

bekomme ich als ausgabe

https://www.blablub.com/blub/list?page=4&s=rl
https://www.blablub.com/blub/list?page=4&s=rl
https://www.blablub.com/blub/list?page=4&s=rl
https://www.blablub.com/blub/list?page=4&s=rl
[Quellcode von https://www.blablub.com/blub/list?page=4&s=rl] (hier wird der richtige quellcode ausgegeben obwohl eigendlich nichts anderes an der Methode übergeben wird als oben.)
https://www.blablub.com/blub/list?page=4&s=rl....

 

[Bqg_stylo]

Hey bin gerade neue hier und dies ist mein erster Post!
Mir fallen an deinem Code 3 nicht so schöne Sachen auf:

1. Klasse Qcreader: try catch Block - du fängst jegliche Exception ohne diese durch zumindest eine Ausgabe zu quittieren... Das sollte man nie machen weil man so sporadische Fehler niemals angezeigt bekommt! Also bitte unbedingt zumindest ne Ausgabe machen!

2. Klasse Qcreader - Ich meine das es quasi zum gutem Ton gehört nur eine Return Anweisung pro Methode zu verwenden

3. Klasse Test:

        for(int i = 1; i == 10;i++)

Das soll sicher anders sein oder?

Bitte bearbeite Punkt 1 und fange mal gar keine Exception - ich bin gespannt!

Grüße Stylo

 

[Framehacker]

1. Klasse Qcreader: try catch Block - du fängst jegliche Exception ohne diese durch zumindest eine Ausgabe zu quittieren... Das sollte man nie machen weil man so sporadische Fehler niemals angezeigt bekommt! Also bitte unbedingt zumindest ne Ausgabe machen!

Danke für den hinweis da die Methode ansich funktioniert kommt es garnicht zur exception und da das ganze nur zum fehler reproduzieren da ist habe ich es einfach so gelassen. Im finalen Programm mach ich das den aber anders .

2. Klasse Qcreader - Ich meine das es quasi zum gutem Ton gehört nur eine Return Anweisung pro Methode zu verwenden

Return bricht ja meines Wissens nach die Methode ab und gibt halt das angegebene zurück das zweite Return steht da weil mein netbeans sonst meckert.

3. Klasse Test:

for(int i = 1; i == 10;i++)

Das soll sicher anders sein oder?

ups sollte eigentlich

for(int i = 1; i < 10;i++)

sein steht auch in meinen Code so hatte ich aber vergessen hier zu ändern^^ .

 

[eRaaaa]

Leider bist du auf meine Vermutung nicht eingegangen, verschiebe mal deine Zeile 9 in der QCReader Klasse aus deinem Beispielprogramm nach Zeile 12 , sprich also in die Methode!

Anders kann ich mir das nicht erklären :autsch:

 

[Bqg_stylo]

Hey ich habe mir die Mühe gemacht und dein Programm getestet und ich muss dir sagen es funktioniert!!!

Ich vermute du hast da irgendwo den Denkfehler mit der Domain. Vermutlich ist der Quelltext nicht unterschiedlich??? Mit meiner domain funktionierts.... Wenn du mir mal die richtige domain gibst, teste ich es auch gerne mal da aus... mit blablub kann keiner testen :shock:

Übrigens hängst du den quelltext immer an den schon vorhandenen an somit wird dir bei

if(i == 4) {
	            System.out.println(quelle);
	        }

der Quelltext von allen bisher durchgegangenen Seiten angezeigt...

 

[Framehacker]

Leider bist du auf meine Vermutung nicht eingegangen, verschiebe mal deine Zeile 9 in der QCReader Klasse aus deinem Beispielprogramm nach Zeile 12 , sprich also in die Methode!

Anders kann ich mir das nicht erklären :autsch:

Gott bin ich Blöd ja das ist es gewesen habe erst gedacht du gingst darauf ein

Ich bin da gerade recht ratlos habe schon versucht den String mit einen Stringbuilder zusammenzusetzen und String.format habe ich auch probiert das Ergebnis war immer dasselbe

Als ich das Probiert habe habe ich daran gedacht in meiner klasse allerdings nicht.

Jetzt habe ich das

StringBuilder stb = new StringBuilder();

in die GetQuellCode Methode gepackt und nun gehts. Hat also wie du gesagt hast alles immer schön hinten dran gehängt und vorne war halt immer Seite 1.

Naja aufjedenfall ein Großes Danke für dich und ein facepalm für mich.