Compiler geht nicht (Dekorator Muster)

[pindakaas]

Hallöchen ich bins schonwieder, :toll:
diesmal bin ich dabei ein Dekorator Muster zu schreiben.
Der Dekorator der hier zu Implementieren bleibt soll Rabatt heißen und alle Objekte die
Betrag implementieren Dekorieren können und eine ganzzahligen Prozentsatz vom Preis abziehen.

Auf dem Papier dachte ich es sollte so funktionieren aber der Compiler gibt ne Fehlermeldung, die leider nichts aussagt da mein Compiler irreparabel kaputt zu sein scheint. Aber hier ist mein Versuch:

Ich freu mich auf eure Antworten ^^

/*für neuankömmlinge geändert*/
public class Rabatt extends Preis
{
    public Preis preis; int rabatt =0;
    
    public Rabatt(Preis preis,int a){
        this.preis=preis;
        rabatt = a;
    }
    
    public int berechneBetrag(){
        return preis.getBetrag()*(1-(rabatt/100));
    } 
    
    
}

Das Vorgegebene Interface und die vorgegebene Klasse.

public interface Betrag {
int berechneBetrag();
}

public class Preis implements Betrag {
int derPreis;
public Preis(int p) { derPreis = p>=0 ? p : 0; }
public int berechneBetrag() { return derPreis; }
}

für interessierte hier die Aufgabestellung:

Gegeben sind das Interface Betrag und die Klasse Preis. Programmieren Sie eine Klasse
Rabatt, die jedes Objekt dekorieren kann, das zu einer Klasse gehört, die Betrag implementiert.
Die Klasse Rabatt soll einen ganzzahligen Prozentsatz verwalten, der beim Aufruf der Methode
berechneBetrag von dem berechneten Betrag des dekorierten Objekts abgezogen wird.

 

[SlaterB]

willst du die Fehlermeldung einfach nicht posten oder hast du tatsächlich kein laufendes Java bei dir?
im letzteren Fall installiere das doch erstmal neu oder kaufe dir einen (anderen) Computer bevor hier jede Arbeit anfangen kann?

in Anbetracht dieses etwas dreisten Anteils vorerst nach Hausaufgaben verschoben,
wenn auch sonst auf etwas höheren Niveau als die meisten reinen Anfragen in diesem Bereich

 

[maki]

aber der Compiler gibt ne Fehlermeldung, die leider nichts aussagt

Was sagt er denn worüber?

 

[Gonzo17]

    public int berechneBetrag(){
        return preis.getBetrag()*(1-(a/100));
    }

Da ist wohl einiges schief gelaufen. Wo wird die Variable

preis

denn deklariert? Und wieso denn

a

?

a

ist eine lokale Variable im Konstruktor, wie willst du denn dadrauf zugreifen? Übrigens übergibst du ja genau deshalb den Wert dieser Variable an eine andere Variable, damit sie fürs Objekt nutzbar ist...

 

[pindakaas]

sorry hab wieder eine Variante die eigentlich anders aussah nach rumbasteln hier falsch gepostet.
Die Fehlermeldung enthält leider nur den Pfad zu der .java Datei das liegt an meiner Programmierumgebung das hat auch neuinstallieren nicht ändern können. Ich hab versucht einfach mal mit javac zu compilieren aber dann werden die Klassen nich automatisch importiert und solche Sachen und das zu lernen war bis jetzt mehr Aufwand als einfach den Fehler ohne Aussagekräftige Fehlermeldung zu finden.

so sollte der Adapter eigentlich aussehen:

public class Rabatt extends Preis
{
    public Preis preis; int rabatt =0;
    
    public Rabatt(Preis preis/*hier stand kurz Preis wird nachträglich geändert*/,int a){
        this.preis=preis;
        rabatt = a;
    }
    
    public int berechneBetrag(){
        return preis.getBetrag()*(1-(rabatt/100));
    } 
    
    
}

PS: Das hier ist keine Hausaufgabe sowas bekommen wir nicht ich versuche einfach nur zu verstehen warum mein Compiler das hier nicht Compilieren will, die Aufgabe ist aus ner Altklausur

 

[SlaterB]

dein Programm ist ungeeignet, den Compiler (oder genauer eine noch nicht einmal genannte Entwicklungsumgebung?) zu testen,
es sei denn es gibt speziell dort Fehler und anderswo nicht,

fange mit

public class Test
{
    public static void main(String[] args)
    {
        System.out.println("Hello World");
    }
}

an, geht das? baue verschiedene Fehler ein, falsche Klassennamen, Aufrufe, Klammern usw.,
teste mit mehreren Klassen wenn das alles geht usw., poste Screenshots

 

[pindakaas]

Wenn das Programm richtig ist geht es auch nur Fehlermeldungen werden halt nicht als Fehlermeldung ausgegeben, nur Laufzeitfehler bzw verursachte Exceptions werden richtig benannt.

Sorry falls ich verwirrung stifte ich werde mir mal angucken wie ich das mit javac hinbekomme, dann kann ich im Zweifel daher ne Fehlermeldung bekommen die Sinn macht.


PS: dein Programm spuckt "hallo world" aus

 

[SlaterB]

in Anbetracht deiner sonstigen Posting will ich mal extra Arbeit machen und selber einen Screenshot posten wie das ganze bei mir aussieht (beachte die exakten Fehlermeldungen!),
freilich nur mit kopierten Code in einer Klasse,
was du mit mehreren Dateien, packages usw. alles falsch machen kannst ist dabei nicht berücksichtigt

wenn du Hilfe zu deinem Compiler-Problem willst, musst du viele viele viele viele viele viele viele viele viele Informationen posten,
oder gleich Eclipse bzw. vergleichbares installieren falls noch nicht vorhanden

zurückverschoben, Titel geändert

 

[pindakaas]

okay ich werd mich jetzt erstmal darum kümmern das ich vernünftige Fehlermeldungen bekomme, das geht so nicht weiter, auch wenn ich eigentlich keine zeit für sowas hab.

PS: danke für den screenshot dachte mir schon das Preis zu subclassen hier falsch ist und ich eigenlich Betrag implementieren muss, das gibt zwar auch wieder irgend nen Fehler aber.. wie gesagt das regel ich jetzt erstmal

 

[VfL_Freak]

Moin,

okay ich werd mich jetzt erstmal darum kümmern das ich vernünftige Fehlermeldungen bekomme, das geht so nicht weiter, auch wenn ich eigentlich keine zeit für sowas hab.

aber alle anderen hier haben die Zeit ???:L 😱

Gruß
Klaus

 

[Noctarius]

Ich vermute Editor + Javac 😉

 

[pindakaas]

Moin,



aber alle anderen hier haben die Zeit ???:L 😱

Gruß
Klaus

Nein so war das natürlich nicht gemeint ich bin nur gerade am lernen und mich jetzt mit sowas zu beschäftigen und zwei Stunden mit einem Problem zu ringen weil ich keine Fehlermeldungen hab die Sinn machen ist da einfach doof.
Aber jetzt hab ich nen anderen Editor der mir direkt javac Fehlermeldungen gibt jetzt kanns endlich weitergehen^^.

nur um Missverständnissen vorzubeugen: Danke an alle für ihre Hilfe und Zeit !

 

[Dow Jones]

Hallo Pindakaas,
wenn du ohnehin gerade am Lernen bist solltest du dir am besten gleich angewöhnen mit Interfaces statt mit Klassen zu arbeiten.

public class Rabatt implements Betrag
{
    public Betrag betrag; int rabatt =0;
    ...    
}

so ist das nicht nur "schöner", es entspricht auch eher der Aufgabenstellung. 😉

 

[pindakaas]

Hey sauber zu dem Schluss bin ich auch gerade gekommen das wirft jedoch ein Problem auf
wenn ich das so mache sieht meine Testumgebung so aus

public class Test{
	
	public static void Test(){
		Preis  p = new Preis(5);
		Rabatt Rab = new Rabatt(p,5);
	}
}

nur sollte eigentlich dort stehen denn er Typ sollte erhalten bleiben

public class Test{
	
	public static void Test(){
		Preis  p = new Preis(5);
		p = new Rabatt(p,5);
	}
}

das gibt natürlich

 incompatible types
found   : Rabatt
required: Preis

das funktioniert aber halt nur mit subclasses oder überseh ich was?

 

[schnelle_hilfe]

public class Test{
    
    public static void Test(){
        Betrag  p = new Preis(5);
        p = new Rabatt(p,5);
    }
}

 

[SlaterB]

zu spät aber ausführlicher:

das funktioniert eben nur wenn man konsequent aufs Interface setzt:

Betrag b = new Preis(5);
b = new Rabatt(b, 5);

wobei du dann Rabatt evtl. noch umbauen musst, damit dann da ein beliebiges Betrag-Objekt übergeben kann,
nicht nur einen Preis (wie in Dow Jones Posting bereits angedeutet),
wenn Rabatt nur Preis akzeptiert, dann ist es relativ akzeptabel, dass Rabatt auch von Preis erbt

 

[pindakaas]

sauber jetzt gehts alles hier die Finalen klassen:

Danke für die Geduld ^^

public interface Betrag {
    int berechneBetrag();
}

public class Preis implements Betrag{
    
    int derPreis;
    
    public Preis(int p) { derPreis = p>=0 ? p : 0; }
    
    public int berechneBetrag() { return derPreis; }
}

public class Rabatt implements Betrag
{
    public Betrag preis; int rabatt =0;
    
    public Rabatt(Betrag preis,int a){
        
        this.preis=preis;
        rabatt=a;
    }
    
    public int berechneBetrag(){
        return preis.berechneBetrag()*(1-(rabatt/100));
    } 
    
    
}

public class Test{
	
	public static void main (String[] args){
		Betrag  p = new Preis(5);
		System.out.println("Preis : "+p);
			
		p = new Rabatt(p,5);
		System.out.println("Mit Rabatt"+p);
	}
}

 

[Kathi144]

Hallo,
warum übergibst du in der Testumgebung einen String[]?
Viele Grüße

 

[VfL_Freak]

Moin,

warum übergibst du in der Testumgebung einen String[]?

Meinst Du etwa diese Zeile ???:L 😱

public static void main (String[] args)

Dann solltest Du Dich mal mit den Basics des Aufbaus eines Java-Programms auseinander setzen !!
Jedes Programm benötigt eine main-Methode mit diesem Aufbau, bei dem im String-Array die Argumente ans Programm übergeben werden (können) .....
Siehe bspw. hier:
Applikationen

Gruß
Klaus