Fehler Methode

[Nirvana]

Was ist in der methode falsch=?
In Netbeans steht: Fehler: invalid method declaration,

//Methode um eine Seitenvektoren zu erzeugen.
   public Vektor2D 1Seite(){
   Vektor2D v1 = new Vektor2D(); 
   v1 = p1.diff(p2);
   return v1;

diff ist eine Operation zwischen zwei Punkten, die die Differenz der beiden ausrechnet und einen Vektor zurückgibt

 

[Gast2]

da fehlen drei } Klammern. Eine beim Konstruktor, eine bei der Methode, die letzte bei der Klasse.

 

[Nirvana]

Hallo, das war aber nur ein fehler in der übertragung ins forum.

Ich hab den fehler, die variablen hab ich falsch bezeichnet!

<identifier expected>
invalid method declaration
return type required
steht auf der seite als fehler bei netbeans

 

[Gast2]

Dann poste doch mal den Quellcode der ganzen Klasse.

 

[Nirvana]

So das hat sich nun schon erledigt.
nur gibt er mir plötzlich bei der ausgabe der methode eine java.lang.NullPointerException aus.
Warum wenn ich die differenz zwischen zwei Punkten ausrechne..

 

[Gast2]

... weil irgendwo eine Referenz null ist. Poste deinen Code, den Stacktrace und wir können dir helfen.

 

[Nirvana]

public class Dreieck2D {
  
    private Punkt2D p1;
    private Punkt2D p2;
    private Punkt2D p3;

    public Dreieck2D(Punkt2D p1, Punkt2D p2, Punkt2D p3) {
        p1 = this.p1;
        p2 = this.p2;
        p3 = this.p3;

    }

    public Vektor2D p1Seite() {
        return this.p1.diff(this.p2); 

 // diff ist Methode von Punkt2D, gibts Vektor zurück, der die Differenz der beiden  Punkte ist
    }
}

In einer anderen klasse wollte ich das ausprobieren:

public static void main(String[] args) {
        Punkt2D p1 = new Punkt2D(0, 0);
        Punkt2D p2 = new Punkt2D(3, 0);
        Punkt2D p3 = new Punkt2D(1, 4);

        Dreieck2D A = new Dreieck2D(p1, p2, p3);
        System.out.println( p1.diff(p2));
        System.out.println( A.p1Seite());
}

Bei der methode für den Punkt gibt es mir die vektor normal aus.
Bei der Methode für das Dreieck bekomme ich eine NullPointerException

Genauso mit dem Code vom Anfang krieg ich die NullPointerException

 

[Kiri]

Das

    public Dreieck2D(Punkt2D p1, Punkt2D p2, Punkt2D p3) {
        p1 = this.p1;
        p2 = this.p2;
        p3 = this.p3;

    }

müsste so lauten

    public Dreieck2D(Punkt2D p1, Punkt2D p2, Punkt2D p3) {
        this.p1 = p1;
        this.p2 = p2;
        this.p3 = p3;

    }

 

[Gast2]

p1 = this.p1;

Diese Zuweisung ist falsch. Damit weißt du der lokalen Variable p1 des Konstruktor den Wert der Instanzvariable zu, und die ist null. Du musst die Zuweisung umdrehen.

 

[Nirvana]

Und daran sitze ich jetzt so lange und mir ist das nicht aufgefallen? :lol:


Danke,lg

 

[Nirvana]

Noch eine Frage:
Muss ich beim Dreieck vorher noch überprüfen ob es wohldefiniert ist, oder ist das egeal?
Weil beim Kreis muss man z.B überprüfen, dass der Radius nicht <0 ist

 

[Gast2]

Woher sollen wir das wissen? Das sollte doch in deiner Aufgabe stehen. Dein Programm wird auch ohne diese Prüfung funktionieren.

 

[Landei]

Woher sollen wir das wissen? Das sollte doch in deiner Aufgabe stehen. Dein Programm wird auch ohne diese Prüfung funktionieren.

Genau. Was ein "Dreieck" in einem Programm bedeuten soll, bestimmt der Programmierer. Allerdings wäre es für andere Programmierer leichter verständlich, wenn es mit der üblichen Definition von "Dreieck" arbeiten würde (und diese gegebenenfalls auch über eine Prüfung erzwingt).

 

[ARadauer]

Noch eine Frage:
Muss ich beim Dreieck vorher noch überprüfen ob es wohldefiniert ist, oder ist das egeal?
Weil beim Kreis muss man z.B überprüfen, dass der Radius nicht <0 ist

das ist eine fachliche Anforderung, java wirds egal sein was dein radius ist...

 

[AquaBall]

Noch eine Frage:
Muss ich beim Dreieck vorher noch überprüfen ob es wohldefiniert ist, oder ist das egeal?
Weil beim Kreis muss man z.B überprüfen, dass der Radius nicht <0 ist

Drei Punkte ergeben IMMER ein Dreieck.
Definitionsschwierigkeiten könnte es nur dann geben, wenn Punkte zusammenfallen.
Das ist aber letztlich ein Frage wer dir die Punkte Erzeugt.
(Anders wäre es, wenn du die SeitenLÄNGEN vorgegeben hättest. Da kann es Widersprüche geben.)

PS: Warum sollte ein Kreis keinen negativen Radius haben dürfen?

 

[Crian]

Warum sollte ein Kreis keinen negativen Radius haben dürfen?

Der Radius ist definiert als der Abstand zweier Dinge. Abstände sind nie negativ.

 

[AquaBall]

Das mag ja als Definition korrekt sein, aber welche Konstruktion ... würde das stören?

Und oben stand ja im Raum:

beim Kreis muss man Radius überprüfen

.
Und meine Frage war:
warum?
wo?
was, wenn nicht?

 

[Gast2]

Das mag ja als Definition korrekt sein, aber welche Konstruktion ... würde das stören?

Umfang, Durchmesser, ...

Der Abstand der Kreispunkte zum Mittelpunkt ist der Radius oder Halbmesser des Kreises, er ist eine positive reelle Zahl

 

[AquaBall]

Naja, wieder eine Definition,
... die aber meine Frage nicht beantwortet.

Im Gegenteil: Sehr oft wird gerade beim Kreis das Vorzeichen des Umfangs genutzt, um die "Laufrichtung" festzulegen.
* positiv = Im Uhrzeigersinn
* negativ = gegen den Uhrzeigersinn
oder es kann damit auch "oben" und "unten" unterschieden werden.
oder im 3DBereich kann auch "innen" und "außen" markiert werden, sogar bei Polygonen und Dreicken.

Wenn man diese Unterscheidung aber nicht braucht spricht nichts gegen einen negativen Raduis.

Andererseits kann aber aus den Längen 1/2/100 nie ein Dreieck gebildet werden, solange wir uns nicht über Hyperbolische Räume ... unterhalten.

Somit ist ein negativer Radius nicht wirklich ein Problem.
Wenn doch, dann muss der TO konsequenterweise sehr wohl überprüfen, ob die Punkte seines Dreiecks (vorallem im 2D-Bereich) im mathematisch/geometrischem Sinne positiv angeordnet sind.
Sonst hat er kein Dreieck konstruiert (positiv), sondern ein Dreieck ausgeschnitten (negativ).

Kurz gesagt: Wenn die Positivität des Radius relevant ist, dann muss auch die Laufrichtung des Dreiecks überprüft werden.
Oder eben beides nicht.

Meine Frage wann und wo und warum der Radius für eine darstellende Anwendung überprüft werden "MUSS" ist für mich nicht beantwortet.