Erste Schritte Weblog-Auswertung

[Devanther]

http://imgur.com/a/Ta8tN

Jetzt kommt in anzahlZugriffe() selbst ne Fehlermeldung.

 

[JStein52]

Dann fang mal mit der ersten an ...

 

[Devanther]

Leider keine Ahnung -.-

 

[mrBrown]

wie lautet denn die erste Fehlermeldung?

 

[JStein52]

Deshalb sagt dir ja dieses komische BlueJ was das Problem ist

 

[Joose]

Jetzt kommt in anzahlZugriffe() selbst ne Fehlermeldung.

Das ist etwas was dir kein Buch wirklich beibringen kann -> du musst die Fehlermeldungen lesen und verstehen können. Danach musst du auch lernen wie du das Problem lösen kannst!
Manchmal reicht es sich den Code nochmal genauer anzuschauen (Verschreiber, falsche Klammerung oder so) oder aber man muss in die Dokumentation schauen oder im Internet nach der Lösung suchen.

 

[Devanther]

bei Int kommt: cannot find symbol - class int

bei ++ kommt: bad operand type int[] for unary operator'++'

bei gesamt kommt: cannot find symbol - variable gesamt

 

[mrBrown]

bei Int kommt: cannot find symbol - class int

Nein, das steht da nicht - achte auf Groß- und Kleinschreibung.

bei ++ kommt: bad operand type int[] for unary operator'++'

bei gesamt kommt: cannot find symbol - variable gesamt

Erstmal den ersten, dann die anderen

 

[Devanther]

cannot find symbol - class Int

 

[mrBrown]

cannot find symbol - class Int

Und gibt es Int?

Tipp: Guck dir mal deine anderen Methoden an.

 

[JStein52]

cannot find symbol - class Int

Und ????

 

[Devanther]

uuups hab aus versehen Int statt int geschrieben.

Bleiben noch 2 kleine Fehler.

 

[JStein52]

weisst du wie man zu einer Variablen etwas addiert (subtrahiert, multipliziert, dividiert ...)

 

[Devanther]

zugriffeInStunde = zugriffeInStunde +1 ?

 

[JStein52]

Im Prinzip richtig. Nur musst u evtl. nicht eins addieren sondern vielleicht was anderes ..
Und das ganze auch nicht zu zugriffeInStunde ...

 

[Joose]

"zugriffeInStunde" ist ein Array oder? Wie addierst du zu einem Array den Wert 1 hinzu? Was passiert da deiner Meinung nach?

 

[Devanther]

Das Array hat eine add.Methode?

 

[Joose]

Nein hat es nicht, lies dir nochmal die Grundlagen zu Arrays durch
Du sollst die Grundlagen verstehen und NICHT IMMER RATEN!

 

[JStein52]

Nein hat es nicht,

Und selbst wenn sollst du nichts zu dem Array addieren.

 

[Devanther]

Und selbst wenn sollst du nichts zu dem Array addieren.

Was genau muss ich nun machen?

 

[mrBrown]

Lösch den gesamten Methodeninhalt, und beschreib einfach mal mit Worten ganz unabhängig vom Code, was die Methode machen muss.

 

[JStein52]

Lesen, lesen, lesen, dir andere Beispiele in dieser Klasse anschauen und nachdenken

 

[Devanther]

Ohman^^ ich habe keine Ahnung warum das falsch sein sollte.

public int anzahlZugriffe() {
    for (int aktuellerZugriff : zugriffeInStunde) {
        int gesamt = zugriffeInStunde[stunde]++;
    }     
    return gesamt;
    }

Ist das jetzt richtig? Es kommt keine Fehlermeldung mehr...nur noch bei gesamt

 

[mrBrown]

Das:

Lösch den gesamten Methodeninhalt, und beschreib einfach mal mit Worten ganz unabhängig vom Code, was die Methode machen muss.

Auf Code-Ebene kommst du offensichtlich noch nicht weiter, also beschreib erstmal was du da machen musst.

 

[Devanther]

/**
* Eine Klasse, die das Protokoll eines Webservers
* in Hinsicht auf Zugriffe pro Stunde auswertet.
*
* @author David J. Barnes und Michael Kölling.
* @version 31.07.2011
*/
public class ProtokollAuswerter
{
    // Hier werden die Zugriffe für die Stunden gehalten
    private int[] zugriffeInStunde;
    // Verwendung eines LogdateiLesers
    private LogdateiLeser leser;
   
    private int stunde;
   
    private int gesamt;
    /**
     * Erzeuge ein Exemplar, das die Zugriffe in den
     * Stunden eines Tages zählt.
     */
    public ProtokollAuswerter(String Sebastian)
    {
        // das Array-Objekt erzeugen, das die Zugriffe
        // in den einzelnen Stunden eines Tages zählt.
        zugriffeInStunde = new int[24];
        // Den Leser für die Logdatei erzeugen.
        leser = new LogdateiLeser(Sebastian);
    }

    /**
     * Analysiere die in der Logdatei erfassten Zugriffsdaten.
     */
    public void analysiereStundendaten()
    {
        while(leser.hasNext()) {
            Logeintrag eintrag = leser.next();
            int stunde = eintrag.gibStunde();
            zugriffeInStunde[stunde]++;
        }
    }
    public int anzahlZugriffe() {
    for (int aktuellerZugriff : zugriffeInStunde) {
        int gesamt = zugriffeInStunde[stunde]++;
    }     
    return gesamt;
    }
   
    /**
     * Gib die Anzahl der Zugriffe in den Stunden eines
     * Tages nach Stunden sortiert auf der Konsole aus.
     * Diese Werte sollten zuerst mit einem Aufruf von
     * 'analysiereStundendaten' berechnet werden.
     */
    public void stundendatenAusgeben()
    {
        System.out.println("Stunde: Zugriffe");
        for(int stunde = 0; stunde < zugriffeInStunde.length; stunde++) {
            System.out.println(stunde + ": " + zugriffeInStunde[stunde]);
        }
    }
   
    /**
     * Gib die Zeilen der Logdatei auf der Konsole aus.
     */
    public void logdateiAusgeben()
    {
        leser.datenAusgeben();
    }
}

Ist das richtig? Es kommt gar keine Fehlermeldung mehr.

 

[mrBrown]

Nein, das ist wieder so ein Versuch den richtigen Code zu raten, wobei leider völliger Unsinn raus kommt.

 

[Devanther]

Ich bin ratlos
Kann mich nicht jemand erlösen?

 

[mrBrown]

Kann mich nicht jemand erlösen?

Nein.

Aber Hilfe bekommst du trotzdem:

Lösch den gesamten Methodeninhalt, und beschreib einfach mal mit Worten ganz unabhängig vom Code, was die Methode machen muss.

 

[Devanther]

OK angenommen.

Also zugriffeInStunde ist ein Array mit einer Länge von 24. Und da stehen die Anzahl der Zugriffe drin.
Über dieses Array (zugriffeInStunde) muss man mit der Methode anzahlZugriffe iterieren.
Und den Wert von allen Elementen in ZugriffeInStunde muss man zu gesamt hinzuaddieren!

 

[Harry Kane]

Es kommt gar keine Fehlermeldung mehr.

... obwohl eigentlich eine kommen müsste. Dass sie nicht kommt, kommt daher, weil ProtokollAuswerter seit neuestem die Instanzvariablen stunde und gesamt hat, die ausserhalb der anzahlZugriffe-Methode gar nicht verwendet werden, wohl aber gleichnamige lokale Variablen.

 

[JStein52]

weil ProtokollAuswerter seit neuestem die Instanzvariablen stunde und gesamt hat

Die hatte er ja da hingeschrieben damit keine Fehlermeldung mehr kommt ..

 

[mrBrown]

Also zugriffeInStunde ist ein Array mit einer Länge von 24. Und da stehen die Anzahl der Zugriffe drin.
Über dieses Array (zugriffeInStunde) muss man mit der Methode anzahlZugriffe iterieren.
Und den Wert von allen Elementen in ZugriffeInStunde muss man zu gesamt hinzuaddieren!

Dann fang dabei mal von hinten an - du brauchst offensichtlich gesamt - was ist das denn?

 

[Devanther]

gesamt ist eine Variable die die ZugriffeInStunde speichert.

 

[mrBrown]

gesamt ist eine Variable die die ZugriffeInStunde speichert.

gesamt soll eine Variable sein, die das Array zugriffeInStunde speichert?

 

[Devanther]

gesamt ist eine Variable, die die Elemente des Arrays zugriffeInStunde speichert.

 

[mrBrown]

Sollten das vorhin nicht noch aufaddierte Werte sein, und nicht die einzelnen Elemente?

 

[Devanther]

Ich meine die einzelnen Elemente.
Ohman^^

 

[mrBrown]

Sicher, dass du irgendwo einzelne Elemente speichern musst? Soll die Methode nicht eher eine Summe zurückgeben?

 

[Devanther]

richtig die gesamtsumme soll herausgegeben werden.

 

[mrBrown]

Welcher Typ muss die dann sein?

 

[Devanther]

int

 

[mrBrown]

Und welchen Wert muss die Variable haben, bevor irgendwelche Werte aufaddiert werden?

 

[Devanther]

0

 

[mrBrown]

Dann schrieb doch genau das mal in die Methode (und die Schleife lässt du Weg!)

 

[Devanther]

int gesamt = 0;
return gesamt;

Ist falsch

 

[mrBrown]

Zeig mal die gesamte Methode und poste immer auch die Fehlermeldung, wenn irgendwas falsch ist (außer die ist nur "Ist falsch", dann lösch deine IDE)

 

[Devanther]

public int anzahlZugriffe() {
    for (int aktuellerZugriff : zugriffeInStunde) {
        int gesamt = 0;
    }     
    return gesamt;
    }

Keine Fehlermeldung.

 

[mrBrown]

Dann schrieb doch genau das mal in die Methode (und die Schleife lässt du Weg!)

Lies noch mal den letzten Teil.

 

[thecain]

Oder lass es einfach sein mit Java...

Sorry, musste mal sein.

 

[Devanther]

public int anzahlZugriffe() {
    {
        int gesamt = 0;
    }   
    return gesamt;
    }

Oder lass es einfach sein mit Java...

Sorry, musste mal sein.

Ne, will es lernen.