Übungsaufgabe zu Klassen, Methoden ...

[bronko00]

Hallo,

ich habe zu folgende Übungsaufgabe:

Implementieren Sie eine Klasse Radio mit folgenden Attributen:
•eingeschaltet, wenn ein Radio an oder aus ist.
•lautstaerke, wie laut spielt das Radio Musik? (Die Lautstärke soll nur im Bereich von 0 bis 10 liegen.)
•frequenz, die die Frequenz des gewählten Senders angibt (Erlaubter Frequenzbereich ist zwischen 85.0 und 110.0).

Implementieren Sie zu der Klasse zwei Konstruktoren der Art:
•Radio()
•Radio(boolean einaus, int laut, double freq)

Zu der Klasse Radio sollen folgende Methoden implementiert werden:
•lauter(), leiser(): Diese Methoden sollen die Lautstärke ändern (nur möglich im Zustand an).
•an(), aus(): Diese Methoden sollen den Zustand des Attributs eingeschaltet ändern.
•public String toString()Diese Methode soll Informationen über den internen Zustand als String zurückgeben. Es soll eine Zeichenkette der Form "Radio an: Freq=98.4, Laut=2" zurückgeben.

•waehleSender(double frequenz) Diese Methode soll eine Frequenz speichern. Ist die gewählte Frequenz außerhalb der erlaubten Frequenzbereichs, so soll die Frequenz 99.9 gewählt werden.


Das ist meine Lösung: (ich glaube jedoch, dass da einiges falsch ist)

Public class Radio () {

private boolean eingeschaltet;
private int lautstaerke;
private double frequenz;


public boolean lautMin () {
if (lautstaerke < 0){
boolean lautstaerkeMin = true;
return lautstaerkeMin; }
else
lautstaerkeMin = false;
return lautstaerkeMin;}


public boolean lautMax () {
if (lautstaerke > 15) {
boolean lautstaerkeMax = true;
return lautstaerkeMax;}
else
lautstaerkeMax = false;
return lautstaerkeMax;}


public boolean freqMin () {
if (frequenz < 85.0){
frequenzMin = true;
return frequnzMin; }
else
freuenzMin = false;
return frequenzMin;}


public boolean freqMax () {
if (lautstaerke > 110.00){
frequenzMax = true;
return frequenzMax;}
else
freuqenzMax = false;
return frequenzMax; }



public Radio () {}

public Radio (boolean einaus, int laut, double freq) {
eingeschaltet = einaus;
lautstaerke = laut;
frequenz = freq;}



public int lauter () {
if (eingeschaltet == true && lautstaerkeMax == false) {
lautstaerke = lautstaerke +1;
return lautstaerke;}

else if ( eingeschaltet == true && lautstaerkeMax == true)
system.out.println("Maximale Lautstaerke");
}

public int leiser () {
if (eingeschaltet == true && lautstaerkeMin == false) {
lautstaerke = lautstaerke +1;
return lautstaerke;}

else if ( eingeschaltet == true && lautstaerkeMin == true)
system.out.println("Minimale Lautstaerke");
}




public boolean an() {
if(eingeschaltet = false) {
eingeschaltet = true;
return eingeschaltet;}
else
System.out,println("Radio ist bereits an");

public boolean aus() {
if(eingeschaltet = true) {
eingeschaltet = false;
return eingeschaltet;}



public void String toString ()
System.out,println("Radio" +eingeschaltet+": Frequenz=" +frequenz+", Lautstärke:" +lautstaerke+);

public double waehleSender(double frequenz){
if(frequenz < 85.0 || frequenz > 110.00) {
double speicher1 = 99.0
return speicher 1; }
else
double speicher1 = frequenz;
return speicher 1;

}

 

[bronko00]

Habs hinbekommen, nun sieht es so aus:
Gibt es dennoch Verbesserungsvorschläge von euch?


public class Aufg2 {

/**
* @param args
*/
public static void main(String[] args) {
// TODO Auto-generated method stub

}



private boolean eingeschaltet;
private int lautstaerke;
private double frequenz;
private int lautstaerkeMin=0;
private int lautstaerkeMax = 15;


public void Radio () {}

public void Radio (boolean einaus, int laut, double freq) {
eingeschaltet = einaus;
lautstaerke = laut;
frequenz = freq;}



public int lauter () {
if (eingeschaltet == true && lautstaerke < lautstaerkeMax) {
lautstaerke = lautstaerke +1;
return lautstaerke;}

else if ( eingeschaltet == true && lautstaerke >= lautstaerkeMax)
System.out.println("Maximale Lautstaerke");
return lautstaerke;
}

public int leiser () {
if (eingeschaltet == true && lautstaerke > lautstaerkeMin) {
lautstaerke = lautstaerke -1;
return lautstaerke;}

else if ( eingeschaltet == true && lautstaerke <= lautstaerkeMin)
System.out.println("Maximale Lautstaerke");
return lautstaerke;
}




public boolean an() {
if(eingeschaltet = false) {
eingeschaltet = true;
return eingeschaltet;}
else
System.out.println("Radio ist bereits an");
return eingeschaltet;}

public boolean aus() {
if(eingeschaltet = true) {
eingeschaltet = false;}
return eingeschaltet;}



public String toString (){
String temp = new String("Radio" +eingeschaltet+": Frequenz=" +frequenz+", Lautstärke:" +lautstaerke);
return temp;}

public double waehleSender(double frequenz){
if(frequenz < 85.0 || frequenz > 110.00) {
double speicher1 = 99.0;
return speicher1;}
else
return frequenz;}

}

 

[knilch]

Hi,
Zu deinem Code:

public int lauter () {
if (eingeschaltet == true && lautstaerke < lautstaerkeMax) {
lautstaerke = lautstaerke +1;
return lautstaerke;}

else if ( eingeschaltet == true && lautstaerke >= lautstaerkeMax)
System.out.println("Maximale Lautstaerke");
return lautstaerke;
}

public int leiser () {
if (eingeschaltet == true && lautstaerke > lautstaerkeMin) {
lautstaerke = lautstaerke -1;
return lautstaerke;}

else if ( eingeschaltet == true && lautstaerke <= lautstaerkeMin)
System.out.println("Maximale Lautstaerke");
return lautstaerke;
}

Die Lauter und leiser Methode kannst du auch so schreiben:

public int leiser () {
public int lauter () {
if (eingeschaltet == true && lautstaerke < lautstaerkeMax)
   return (++lautstaerke);

else if ( eingeschaltet == true && lautstaerke >= lautstaerkeMax)
   System.out.println("Maximale Lautstaerke");
   return lautstaerke;
}

else 
     return 0; // falls ausgeschaltet, 0 zurückgeben
}


public int leiser () {
if (eingeschaltet == true && lautstaerke > lautstaerkeMin)
   return (--lautstaerke);}

else if ( eingeschaltet == true && lautstaerke <= lautstaerkeMin) {
   System.out.println("Maximale Lautstaerke");
   return lautstaerke;
}

else 
     return 0; // falls ausgeschaltet, 0 zurückgeben
}

zu folgenden Zeilen:

public boolean an() {
if(eingeschaltet = false) {
eingeschaltet = true;
return eingeschaltet;}
else
System.out,println("Radio ist bereits an");

public boolean aus() {
if(eingeschaltet = true) {
eingeschaltet = false;
return eingeschaltet;}

if(eingeschaltet = false) {.. }

und

if(eingeschaltet = true) {..}

bei beiden if- Abfragen machst du eine Zuweisung, statt einer Abfrage. Die Abfrage geht in etwa in dieser Art:

if(eingeschaltet == false) {.. }

und

if(eingeschaltet == true) {..}

oder

if(!eingeschaltet) {.. }

und

if(eingeschaltet) {..}