Java Klassen und Instanzmethoden

[Gansa]

Hallo, ich habe hier eine einfache Klasse. Ich möchte in meiner Klasse Auto eine Methode schreiben (isFasterThan), das ein Auto übernimmt und die Geschwindigkeit des übergebenen Autos mit dem aktuellen vergleichen. Wenn das aufgerufene Auto Objekt einen größeren Wert hat, soll meine Methode true zurückgeben. Jedoch funktionier meine Methode nicht, es gibt mir ständig false zurück. Kann mir da jemand weiterhelfen? Ich habe die Klasse A und die Klasse Auto teilweise reimkopiert (das nötigste). Danke falls mir jemand weiterhelfen kann !

public class A {


    public static void main(String[] args) {

        Auto a = new Auto();
        Auto b = new Auto();

        
        b.repaint("Schwarz");
        b.schneller(350);

    

        System.out.println(a.toString());
        System.out.println(b.toString());
      
        System.out.println( a.isFasterThan(b));





    }
}

//Klasse Auto
        public boolean isFasteThan(Auto auto) {
        if (auto.maxSpeedKmH > this.maxSpeedKmH) {
            return true;
        } else {
            return false;
        }
    }
}

 

[LimDul]

Ich würde mal vermuten, dass die Methode "schneller" nicht das macht, was du erwartest.

Bzw. wo und wie wird das Feld maxSpeedKmH definiert und gesetzt?

 

[Gansa]

Ich würde mal vermuten, dass die Methode "schneller" nicht das macht, was du erwartest.

Bzw. wo und wie wird das Feld maxSpeedKmH definiert und gesetzt?In

Doch die habe ich schon ausgetestet es funktioniert. Ich glaube das Problem liegt daran das ich ein Objekt mit einer Instanzvariable vergleiche, aber ich wüsste nicht wie ich es anders machen soll. Ich erhöhe mit der Methode schneller die Geschwindigkeit von b und möchte es dann mit a vergleichen,

 

[LimDul]

Nein, zeig mal den Code den ich meinte.

 

[Gansa]

Nein, zeig mal den Code den ich meinte.

 

[LimDul]

Und wie ist der startwert von maxSpeed? (Kopier ggf. mal die gesamte Klasse Auto rein)

 

[kneitzel]

Also der Vergleich sieht richtig aus - nur eben kannst Du es deutlich kürzer schreiben: if (bedingung) return true; else return false; ist das Gleiche wie einfach ein return bedingung:

Wir sehen zu wenig Code. Wo setzt du die Höchstgeschwindigkeit? Wenn du weder in a noch in b die Höchstgeschwindigkeit setzt, sind diese Gleich (und damit ist die von a nicht größer als die von b)

Und dieses schneller rufst Du nur bei b auf ...

 

[Gansa]

Und wie ist der startwert von maxSpeed? (Kopier ggf. mal die gesamte Klasse Auto rein)

Im Konstrukt setze ich ihn, habe es nochmal mit rein kopiert.

 

[LimDul]

Wie wäre es mal mit dem Code per Copy & Paste zu kopieren.

Jetzt ist da auf einmal ein "Car" als Klasse und kein Auto mehr.

 

[Gansa]

Also der Vergleich sieht richtig aus - nur eben kannst Du es deutlich kürzer schreiben: if (bedingung) return true; else return false; ist das Gleiche wie einfach ein return bedingung:

Wir sehen zu wenig Code. Wo setzt du die Höchstgeschwindigkeit? Wenn du weder in a noch in b die Höchstgeschwindigkeit setzt, sind diese Gleich (und damit ist die von a nicht größer als die von b)

Und dieses schneller rufst Du nur bei b auf ...

 

[LimDul]

Und wie sieht die Klasse Car aus?

 

[LimDul]

package de.limdul.javaforum;

public class Auto {

    private String color;
    private int maxSpeedKmH;
    private String hersteller;

    public Auto() {

        this.color = "Silber";
        this.maxSpeedKmH = 150;
        this.hersteller = "Volkswagen";
    }

    @Override
    public String toString() {
        return "Car {color = " + this.color + ", maxSpeedKmh= " + this.maxSpeedKmH + ", manufacturer= "
                + this.hersteller + " }";
    }

    public void repaint(String color) {
        this.color = color;
    }

    public void schneller(int maxSpeedKmH) {
        this.maxSpeedKmH = maxSpeedKmH;

    }

    public boolean isFasterThan(Auto car) {

        if (car.maxSpeedKmH > this.maxSpeedKmH) {
            return true;
        } else {
            return false;
        }
    }

    public static void main(String[] args) {
        Auto a = new Auto();
        Auto b = new Auto();

        b.repaint("Schwarz");
        b.schneller(350);

        System.out.println(a.toString());
        System.out.println(b.toString());

        System.out.println(a.isFasterThan(b));
    }
}

Ausgabe:

Car {color = Silber, maxSpeedKmh= 150, manufacturer= Volkswagen }
Car {color = Schwarz, maxSpeedKmh= 350, manufacturer= Volkswagen }
true

 

[Gansa]

Und wie sieht die Klasse Car aus?

package de.limdul.javaforum;

public class Auto {

    private String color;
    private int maxSpeedKmH;
    private String hersteller;

    public Auto() {

        this.color = "Silber";
        this.maxSpeedKmH = 150;
        this.hersteller = "Volkswagen";
    }

    @Override
    public String toString() {
        return "Car {color = " + this.color + ", maxSpeedKmh= " + this.maxSpeedKmH + ", manufacturer= "
                + this.hersteller + " }";
    }

    public void repaint(String color) {
        this.color = color;
    }

    public void schneller(int maxSpeedKmH) {
        this.maxSpeedKmH = maxSpeedKmH;

    }

    public boolean isFasterThan(Auto car) {

        if (car.maxSpeedKmH > this.maxSpeedKmH) {
            return true;
        } else {
            return false;
        }
    }

    public static void main(String[] args) {
        Auto a = new Auto();
        Auto b = new Auto();

        b.repaint("Schwarz");
        b.schneller(350);

        System.out.println(a.toString());
        System.out.println(b.toString());

        System.out.println(a.isFasterThan(b));
    }
}

Ausgabe:

Das hatte ich doch Genauso oder? Bei mir kam jedesmal false ..

 

[LimDul]

Das hatte ich doch Genauso oder? Bei mir kam jedesmal false ..

Du hattest mal Car & mal Auto da stehen. Das spricht dafür das dein Code hier im Forum und der in deiner IDE nicht identisch waren.
Hast du vielleicht noch mehr Klassen die zufällig auch Car oder Auto heißen da rumliegen?

An sich funktioniert jetzt - das ist jetzt wirklich nur Copy & Paste deines Codes gewesen. Was kam den bei dir als Ausgabe?

 

[Gansa]

Du hattest mal Car & mal Auto da stehen. Das spricht dafür das dein Code hier im Forum und der in deiner IDE nicht identisch waren.
Hast du vielleicht noch mehr Klassen die zufällig auch Car oder Auto heißen da rumliegen?

An sich funktioniert jetzt - das ist jetzt wirklich nur Copy & Paste deines Codes gewesen. Was kam den bei dir als Ausgabe?

Ja das kann sein, da ich mehrere Applikationen habe und rumteste, ist ein ziemlicher Spaghetti Code geworden.. Es wundert mich aber trotzdem da ich auf die Namen eigentlich geachtet hatte. Naja jetzt funktioniert es aufjedenfall. Danke nochmals!

 

[fhoffmann]

Dass der Code

public boolean isFasteThan(Auto auto) {
    if (auto.maxSpeedKmH > this.maxSpeedKmH) {
        return true;
    } else {
        return false;
    }
}

kürzer geschrieben werden kann als

public boolean isFasteThan(Auto auto) {
    return auto.maxSpeedKmH > this.maxSpeedKmH;
}

wurde ja bereits (von Kneitzel) erwähnt.
Aber ist der logische Vergleich nicht umgekehrt:

public boolean isFasteThan(Auto auto) {
    return this.maxSpeedKmH > auto.maxSpeedKmH;
}

Und man könnte die Methode isFasteThan möglicherweise in isFasterThan umbenennen.