Artikelbestand

[Devanther]

Welchen Kommentar genau meinst du?

 

[JStein52]

Da stehen zwei kommentare in der Methode und ich meine beide

 

[Devanther]

// Artikelnummer holen
// Methode verlassen und nichts weiter tun

Ja?

 

[JStein52]

jep

 

[Joose]

// Artikelnummer holen
// Methode verlassen und nichts weiter tun

Diese Kommentare sind Platzhalter von JStein52 um dir zu sagen hier musst DU noch Code ergänzen, damit die Methode das macht was du willst.

 

[Devanther]

Achso!

public void neuerArtikel(Artikel artikel)
    {
       artikel.gibNummer();


        for(Artikel aktuellerArtikel: lager){
              if (aktuellerArtikel.gibNummer() == nummer) {
                 System.out.println("Es existiert bereits ein Artikel mit dieser Nummer!");
                 return;// Methode verlassen und nichts weiter tun
              }
         }
       lager.add(artikel);
    }

Es kommt keine Fehlermeldung aber es funktioniert immer noch nicht.
Es können immer noch mehrere Artikel mit der selben Nummer eingefügt werden.

 

[JStein52]

if (aktuellerArtikel.gibNummer() == nummer)

so lange du Äpfel mit Birnen vergleichst wird es nicht funktionieren. Aber der return sitzt schon mal an der richtigen Stelle

Edit: ein Tip noch: gibNummer hat auch einen Rückgabewert ...

 

[Devanther]

public void neuerArtikel(Artikel artikel)
    {
       artikel.gibNummer();
       return;

        for(Artikel aktuellerArtikel: lager){
              if (aktuellerArtikel.gibNummer() == artikel) {
                 System.out.println("Es existiert bereits ein Artikel mit dieser Nummer!");
                 return;// Methode verlassen und nichts weiter tun
              }
         }
       lager.add(artikel);
    }

Ich bekomm die Fehlermeldung bei "==artikel" einfach nicht weg.
incomparable types: int and Artikel

 

[thecain]

Was gibts denn für Methoden auf Artikel? eventuell ein gibNummer (ja, das ist rhetorisch)?

Grundlagen, Grundlagen, Grundlagen.. seit 11 Seiten das Selbe...

Ach ja... jetzt ist das eine Return übrigens falsch... weisst du was return macht?

 

[Xyz1]

Lesen, lesen, lesen... verstehen... anwenden... UND DANN ERST Frage(n) stellen.
Sonst hat es ein Potential, ein sehr langes Thema mit viiielen Aufrufen zu werden.
Was ich eigentlich sagen wollte oder, ich halte mich raus aus dem Thema.

 

[JStein52]

ich halte mich raus aus dem Thema.

Zuerst mit BlueJ bequatschen und dich dann raushalten ...

Edit: Dabei war er mal soooo dicht davor.

nummer = artikel.gibNummer();

und alles war gut. Aber dann gings wieder in die falsche Richtung

 

[Devanther]

return, gibt etwas zurück.
Ich weiss es nicht, ich komme nicht weiter.
Immer wenn ich was verändere kommen Fehlermeldungen.

edit:

public void neuerArtikel(Artikel artikel)
    {
       artikel.gibNummer();
       return;

        for(Artikel aktuellerArtikel: lager){
              if (aktuellerArtikel.gibNummer() == artikel.gibNummer(){
                 System.out.println("Es existiert bereits ein Artikel mit dieser Nummer!");
                 return;// Methode verlassen und nichts weiter tun
              }
         }
       lager.add(artikel);
    }

So? Dann kommt bei der { eine Fehlermeldung -.-

 

[Joose]

Ich weiss es nicht, ich komme nicht weiter.
Immer wenn ich was verändere kommen Fehlermeldungen.

Dann lerne die Grundlagen und versuche dich an einfacheren Beispielen.
Du hast einfach noch große Lücken im Grundwissen und weißt nicht mal um die korrekte Syntax eines Java Programmes bescheid.
Außerdem musst du lernen die Fehlermeldungen die dir eine IDE (oder der Compiler) zeigt zu lesen und zu verstehen. Auch selbst im Internet nach Lösungen zu suchen gehört dazu. (nein gleich im Forum einen Thread aufmachen ist nicht nach einer Lösung suchen)
Und bevor du Änderungen am Code vornimmst solltest du dir im klaren sein welche Auswirkungen diese Änderungen am verhalten haben.

So? Dann kommt bei der { eine Fehlermeldung -.-

Wenn du schon sagst das eine Fehlermeldung kommt dann sag immer dazu welche Fehlermeldung! Wir wollen nicht raten!

if (aktuellerArtikel.gibNummer() == artikel.gibNummer(){

Schau dir doch nochmal das Kapitel zu if-Blöcken an und wie die Syntax dazu ausschaut und dann überlege dir warum du hier einen Fehler bekommst
Ansonsten einfach mal nach deiner Fehlermeldung googlen ... du bist nicht der erste der dieses Problem hat und es gibt unzähliche Foren und Seiten wo die Lösung schon geschrieben steht.

 

[JStein52]

Schau dir doch nochmal das Kapitel zu if-Blöcken an und wie die Syntax dazu ausschaut

Ja, aber er wird bei der Zeile davor schon eine Fehlermeldung kriegen wegen "unreachable Code"

 

[Devanther]

Als Fehlermeldung kommt )expected.
Ich hab keine Ahnung, wie ich den Fehler beheben kann -.-

 

[thecain]

Ich sehs in einem Blick... der Fehler ist:

)expected

Oder auch ") erwartet" oder auch "du hast eine schliessende Klammer vergessen!!!"

 

[mrBrown]

Wenn die Fehlermeldung sagt, dass er eine Klammer erwartet, wie könnte man das wohl beheben?
Hast du mal die Klammern in der Zeile gezählt?

 

[Devanther]

peinlich, aber ich kann den Fehler nicht beheben.
Wenn ich ne Klammer hinzufüge kommen überall im Quelltext Fehler angezeigt...

 

[JStein52]

Und was besagen die dann ? Ich denke mal dass er zu allem hinter deinem ersten return sagt: "unreachable code"

 

[Devanther]

Und was besagen die dann ? Ich denke mal dass er zu allem hinter deinem ersten return sagt: "unreachable code"

Nein, hinter return wird keine Fehlermeldung ausgegeben.
http://imgur.com/a/T6zBq

 

[JStein52]

Ok, d.h. du hast den zuletzt geposteten Code schon korrigiert und nur noch ein return drin ? Wie sieht er denn jetzt aus ?

 

[Devanther]

http://imgur.com/a/T6zBq
? Ich glaube wir reden einander vorbei...

 

[Harry Kane]

Muss mich noch ein letztes Mal zu Wort melden.
Wenn dem BlueJ an der markierten Stellen ausser "Error(s) found in class" keine weitere Erklärung oder Fehlermeldung mehr einfällt, ist es nicht für Einsteiger geeignet.
@TE Wenn du allerdings mit den bisherigen Hinweisen immer noch nicht imstande bist, den Fehler zu beheben (an der von BlueJ markierten Stelle fehlt nämlich wie schon mehrfach gesagt ein ")"), solltest du einfach mal Java lernen und nicht nur Aufgaben und Quellcode hier im Forum posten.
[edit] Ich habe gerade in NetBeans eine Testklasse mit unreachable statements erstellt. NB kan sie sogar ausführen, und bei Änderungen an der Klasse (die den unreachable Code aber nicht beheben) sehe ich auch die gemachten Änderungen auf der Konsole. Der javac steigt allerdings mit einer Fehlermeldung aus. KA was NB da im Hintergrund deichselt. Wenn BlueJ das aber ähnlich handhabt (also auch fehlerhaften Code irgendwie ausführen kann), kann es gut sein, das der TE vom unreachable code nix mitbekommt, wei BlueJ den Fehler nicht anzeigt und die Klasse bzw. Methode trotzdem bis zum return ausführt. Das Verhalten wäre allerdings eine guter Grund, auf was anderes umzusteigen.

 

[thecain]

") expected" hat er ja vorher erwähnt, das muss er ja von irgendwo haben...

@TE, mach jetzt einfach mal diese Klammer die er dir anmerkt...

Und ich bin wieder maximal passiver Leser hier.. das führt zu nix

 

[Devanther]

")") wenn ich das an die markierte Stelle hinzufüge kommt wieder ne Fehlermeldung. ohman^^

 

[Harry Kane]

WAS fügst du genau WO ein? Quelltext! Das wurde dir auch schon mehrfach gesagt.

 

[Devanther]

Ich weiss es nicht, kann jemand den vollständigen Quelltext der Methode
public void neuerArtikel(Artikel artikel) posten?
Egal was ich mache, es kommt immer irgendwo ne Fehlermeldung.

 

[Harry Kane]

Ich weiss es nicht

Was weisst du nicht?

 

[Devanther]

Wie ich die Klammer/Klammern was auch immer setzen muss.
Das die Methode funktioniert ohne dass immer ne Fehlermeldung kommt.

 

[Harry Kane]

Dann zeige deinen Quellcode mit der Fehlermeldung.

 

[Devanther]

Hab ich schon oben gepostet:
http://imgur.com/a/T6zBq

) expected

 

[Harry Kane]

Das war der Originalcode. Dann kam das hier von dir:

")") wenn ich das an die markierte Stelle hinzufüge kommt wieder ne Fehlermeldung. ohman^^

Schlussfolgerung: Du hast den Code geändert. Und den geänderten Code würde ich gerne sehen.

 

[JStein52]

Kann nicht mal endlich jemand die versteckte Kamera wegtun ...

 

[Harry Kane]

@JStein52 Heeee. Nicht stören. Ich bin gerade am trainieren.

 

[mrBrown]

Ich habe gerade in NetBeans eine Testklasse mit unreachable statements erstellt. NB kan sie sogar ausführen, und bei Änderungen an der Klasse (die den unreachable Code aber nicht beheben) sehe ich auch die gemachten Änderungen auf der Konsole. Der javac steigt allerdings mit einer Fehlermeldung aus. KA was NB da im Hintergrund deichselt. Wenn BlueJ das aber ähnlich handhabt (also auch fehlerhaften Code irgendwie ausführen kann), kann es gut sein, das der TE vom unreachable code nix mitbekommt, wei BlueJ den Fehler nicht anzeigt und die Klasse bzw. Methode trotzdem bis zum return ausführt. Das Verhalten wäre allerdings eine guter Grund, auf was anderes umzusteigen.

Müsste der Eclipse-Compiler sein, der kompiliert auch bei einfachen Fehlern - an den entsprechenden Stellen würde dann ne Exception geworfen werden. Zum entwickeln manchmal ganz praktisch

 

[JStein52]

der kompiliert auch bei einfachen Fehlern

Wobei ja gerade der Fall "unreachable Statement" kein Fehler ist der Exceptions werfen sollte. Ich frage mich sowieso wieso der Compiler mich da bevormundet und behauptet das wäre ein Fehler ?

 

[mrBrown]

Wobei ja gerade der Fall "unreachable Statement" kein Fehler ist der Exceptions werfen sollte. Ich frage mich sowieso wieso der Compiler mich da bevormundet und behauptet das wäre ein Fehler ?

Weil das in so ziemlich jedem Fall ein Fehler ist? Nicht erreichbarer Code ist doch kein Code, der „richtig“ ist.

Von mir aus könnte der Compiler da gern noch deutlich mehr anmeckern

 

[Devanther]

Also, wie lautet nun der Endcode?

 

[Harry Kane]

Was ist ein "Endcode"?

 

[Devanther]

Ja, die vollständige Methode von public void neuerArtikel(Artikel artikel).
Sodass kein Artikel mit der selben Nummer eingefügt werden kann und ne Fehlermeldung endlich mal kommt.

 

[Harry Kane]

und ne Fehlermeldung endlich mal kommt.

Du hast doch schon eine Fehlermeldung. Die mit dem fehlenden ")".

 

[Devanther]

Ich meine diese Fehlermeldung
System.out.println("Es existiert bereits ein Artikel mit dieser Nummer!");

 

[Harry Kane]

Ach so.

 

[JStein52]

Nicht erreichbarer Code ist doch kein Code, der „richtig“ ist.

Das nicht. Aber manchmal eine bequeme Möglichkeit den restlichen Code einer Methode vorrübergehend "lahmzulegen"

 

[Harry Kane]

    public void neuerArtikel(Artikel artikel){
        for(Artikel vorhandenerArtikel: lager){
            if(vorhandenerArtikel.gibNummer() == artikel.gibNummer(){
                System.out.println("Es existiert bereits ein Artikel mit dieser Nummer");
                return;
            }
        }
        lager.add(artikel);
    }

 

[JStein52]

Ich sag es nur ungern, aber das ist falsch. Guck mal nach den Klammern

 

[Meniskusschaden]

Ich vermute mal, @Harry Kane wollte es bewußt so weit vorbereiten, daß als Eigenleistung nur noch der Klammerfehler beseitigt werden muß. Es ist ja noch immer unklar, ob @Devanther die schon mal korrekt gesetzt hatte und es nur aufgrund anderer Fehlermeldungen wieder beseitigt hat.

 

[Devanther]

@Harry Kane
Ja, danke. Es geht alles. Das war jetzt einfach.
Ich habe mich nur auf die Klammer konzentriert, und den Fehler mit return nicht berücksichtigt bzw. artikel.gibNummer war falsch.

 

[Meniskusschaden]

Nur noch mal zur Erinnerung: Mit dem Vorschlag von @JStein52 aus Post 84 wäre das auch gegangen, ohne eine neue Schleife zu programmieren.

 

[Harry Kane]

Na dann so:

public void neuerArtikel(Artikel artikel){
   Artikel suchErgebnis = findeArtikel(artikel.gibNummer());
       if(suchErgebnis != null){
           System.out.println("Es existiert bereits ein Artikel mit dieser Nummer");
       }
       else{
            lager.add(artikel);
       }
}