OOP Exception Handling - NullPointerException

[Don_Alfredo]

guten morgen,

public Fahrzeuge(FahrzeugeNode owner) throws NullPointerException
Der Konstruktor der Klasse soll soabgeändert werden, dass es zukunftig
nur noch möglich ist, einen Fahrzeugknoten (Vornamen und Nachnamen des Besitzers) für einen bestimmten, vorgegebenen Besitzer zu erzeugen und komplett leere Fahrzeugknoten sollen vermieden werden.
Sollte der übergebene owner null sein, soll eine NullPointerException
geworfen werden. Andernfalls ist das Fahrzeugknoten mit owner zu initialisieren – hierfür
verwende ich die setFahrzeugNodeRoot() Methode.

mein bisheriger Vorschlag, der allerdings nicht funktioniert. ich hab irgendwie den überblick verloren und steh wie der sprichwörtliche ochse vorm berg :rtfm:

public Fahrzeuge(FahrzeugeNode owner) throws NullPointerException {
        if (this.root == null) {
            throw new NullPointerException(
                    "blablablabla");
        }
    }

und die set-Methode für das einfügen

der eintrag soll den Index 0 zugewiesen bekommen und meiner FahrzeugKnoten Liste hinzugefügt werden

wie mache ich das mit dem index und wie füge ich den knoten in meine liste hinzu?

 public void setFahrzeugeRoot(FahrzeugeNode o) {

        setFahrzeugeRoot(root);
        entries.add(root);

Kann mir da jmd bitte einen freundlichen Hinweis geben wie ich das umsetzen kann/soll und ob das bisherige richtig ist?

mit verzweifelten grüßen

don

 

[Der Müde Joe]

Man darf nie, ich wiederhole NIE ein RuntimeException werfen (throws). Dazu gehört auch eine NPE. Die lässt man fliegen. Das ist ein Fehler der zu vermeiden ist.

 

[javimka]

Von welchem Typ ist entries? ArrayList? Dann entries.add(0,root). Ansonsten sehe ich nicht so richtig, was falsch sein könnte.

 

[Don_Alfredo]

ist leider in der Aufgabenstellung so vorgeschrieben, daran kann ich nix ändern :bahnhof:

//edit: annähernd zeitgleich gepostet: ja ist eine arraylist

 

[Der Müde Joe]

>ist leider in der Aufgabenstellung so vorgeschrieben, daran kann ich nix ändern

sofort beim Prof, Assi oder was auch immer beschweren. Man deklariert nie NPEs ;-)

EDIT:
Das sind immer grobe programmier fehler.

 

[Der Müde Joe]

Ansonsten. Was mich grad irritiert, ist die zweite Methode.
setFahrzeugRoot(... o)
und dann darin:
setFahrzeugeRoot(root)

was ist root?
Sicher eine globale Variabel. Schlusendlich eine Endlosschleife (rekursion)?

 

[ThreadPool]

[...]
Das sind immer grobe programmier fehler.

Quellen?

Sun sieht das pragmatischer:
Unchecked Exceptions — The Controversy (The Java™ Tutorials > Essential Classes > Exceptions)

[...]

One case where it is common practice to throw a RuntimeException is when the user calls a method incorrectly. For example, a method can check if one of its arguments is incorrectly null. If an argument is null, the method might throw a NullPointerException, which is an unchecked exception.

[...]

Here's the bottom line guideline: If a client can reasonably be expected to recover from an exception, make it a checked exception. If a client cannot do anything to recover from the exception, make it an unchecked exception.

 

[Schumi]

Vielleicht ein kleines Missverständniss. Joe meint, dass Du das werfen der NPE nicht deklarieren sollst (also throws NPE in der Methodendeklaration) werfen an sich (also throw new NPE...) kannst Du natürlich schon machen. Sicher dass das deklarieren der Exception auch so gefordert ist?

Ansonsten würde ich:

 if (owner == null)

abfragen und dann die Exception werfen.

 

[tfa]

Ich glaube Joe meint, man sollte keine RTEx in der throws-Klausel bei Methodendefinitionen angeben. Das ist in der Tat überflüssig - obwohl besonders gefährlich ist es auch wieder nicht.
RuntimeExceptions kann man natürlich schon werfen, dafür sind die ja da.

 

[ThreadPool]

Ah...stimmt, da hab ich ihn missverstanden.

 

[Don_Alfredo]

erstmal vielen dank für eure tipps und hinweise. :toll: ich war mir nicht bewusst, welche eine kleine diskussion entsteht aufgrund einer exception :shock:

root ist mein oberster knoten, sozusagen meine wurzel, ein Automobil sozusagen.

Wie löse ich dann die daraus entstehende Endlosschleife?

gruß

don

 

[javimka]

Indem du setFahrzeugeRoot(root) nur dann aufrufst, wenn es nötig ist. Vielleicht wäre es so besser:

 public void setFahrzeugeRoot(FahrzeugeNode o) {
        
        setFahrzeugeRoot(o);
        entries.add(o)

 

[Der Müde Joe]

>Vielleicht wäre es so besser:

nö. Wenn du setFahrzeugeRoot(blub) in der Methode aufrufst, rufst du die Methode wieder auf, darin dan wieder, und wieder. Einfach in der Methode nicht die Methode wieder aufrufen.

 

[javimka]

Stimmt natürlich, das war Blödsinn
Bevor wir nicht wissen, das die Methode bezwecken soll und wie die interne Struktur der Klasse aussieht, können wir nur raten, wie sie aussehen müsste.

 

[Don_Alfredo]

also root ist mein Wurzelknoten, der Anfang, das oberste Element. Es stellt den Knoten Automobil/Fahrzeug dar. Von diesem Knoten sollen verschiedene Abzweigungen weggehen.

 public void setFahrzeugeRoot(FahrzeugeNode o) {
 
        setFahrzeugeRoot(root);
        entries.add(0,root);

Ich soll lt. Aufgabenstellung mit Hilfe von setFahrzeugeRoot() eine Wurzelelement "owner" initialisieren. Diese Methode soll dementsprechend in der Nachfolgenden Methode aufgerufen werden in der

public Fahrzeuge(FahrzeugeNode owner) throws NullPointerException {
        if (this.root == null) {
            throw new NullPointerException(
                    "blablablabla");
        }
    }

damit genau das passieren sollte. Nur ich weiß nicht wie ich das machen soll? :rtfm:


ich hoffe ich konnte ein wenig Klarheit einbringen. ich bedanke mich schon im voraus für eure hilfestellungen

gruß

don

 

[Don_Alfredo]

die obigen punkte haben sich alle geklärt, vielen dank dafür. jetzt gibts aber ein neues problem. es soll eine main-methode das ganze testen. dazu hab ich mal folgendes verfasst, was allerdings nicht funktioniert. es soll entweder lt. aufgabenstellung die exception geworfen werden oder aber mit try&catch die exception abgefangen werden, sodass das ganze aber auch noch weiterläuft.
ich soll einen leeren Knoten einfügen, einen leeren Knoten löschen und einen beliebigen nicht vorhandenen.

   public static void main(String[] args) throws NullPointerException,
            NodeNotFoundException {
        Fahrzeuge myFahrzeuge = new Fahrzeuge();

        try {
            myFahrzeuge.insertNode(null);

        } catch (Exception e) {
            // TODO Auto-generated catch block
            e.printStackTrace();
        }

      }

ist das hier so semantisch korrekt und kann das überhaupt funktionieren? wie bekomme ich das programm dazu, noch weiter zu arbeiten?

gruß

don

 

[javimka]

Das Programm arbeitet schon weiter, gleich nach dem try-catch Block, also bei zeile 12. Nur ist dort gleich das Ende deines Programms. Es gibt ja gar nichts mehr, was es noch arbeiten könnte.

 

[Schumi]

Du nutzt allerdings schon wieder (oder immer noch) die

throws NullPointerException

Deklaration. Mach das mal weg.

throws NodeNotFoundException

ist im Grunde schon ok allerdings bezweifle ich, dass es sinnvoll ist, das die main Exceptions wirft. Aber wenn Du ein throws deklarierst, solltest Du dann auch irgendwo werfen. Oder macht dies der Konstruktor von Fahrzeuge?