Klasse Signal

[Heinrich500]

Hallo liebes Forum,
mir bereitet folgende Aufgabe den ganzen Tag schon Kopfzerbrechen:

Programmieren Sie eine Klasse Notsignalgeber. Bleibt eine gewisse Zeit (z. B. 30 s) ohne Bewegung, wird ein Voralarm ausgelöst. Ist der Träger bei Bewusstsein, wird er sich dann leicht bewegen, um den Alarm auszuschalten. Ansonsten wird z. B. nach weiteren 10 s der Hauptalarm ausgelöst. Die Klasse Notsignalgeber soll die wesentliche Steuerungslogik enthalten. Informationen über die Bewegung und die Zeit kommen von außen (s. u., das brauchen Sie nicht zu programmieren). Die Klasse Notsignalgeber soll einen Konstruktur und zwei Methoden bekommen:
• Der Konstruktor hat zwei Parameter für die Zeit bis zum Voralarm und die zusätzliche Zeit bis zum Hauptalarm. Ein Aufruf könnte so aussehen:
Notsignalgeber notsig = new Notsignalgeber(30,10);

• Die Methode tick() (ohne Parameter) wird von einem externen Zeitgeber jede Sekunde einmal aufgerufen. Sie soll die Lautstärke des Alarms zurückliefern (als Rückgabewert der Methode), also 0 für kein Alarm, 50 für Voralarm und 100 für Hauptalarm. Ein möglicher Aufruf könnte so aussehen:
int lautstaerke = notsig.tick();
Beim oben erzeugten Objekt soll bei den ersten 30 Aufrufen der Wert 0 geliefert werden, beim 31. bis 40. Aufruf der Wert 50, und ab dem 41. Aufruf der Wert 100 (sofern nicht zwischendurch eine Bewegung festgestellt wurde).

• Eine Methode bewegt(), die aufgerufen wird, wenn der Bewegungsmelder eine Bewegung erkannt hat. Diese Methode hat keine Parameter und keinen Rückgabewert. Ein möglicher Aufruf könnte so aussehen:
notsig.bewegt();
Nachdem diese Methode aufgerufen wurde, sollen die nächsten 30 Aufrufe von tick() wieder den Wert 0 liefern. Die beiden Methoden (und natürlich der Konstruktor) sollen von außen aufrufbar sein. Entsprechend muß die Klasse auch von außerhalb benutzbar sein. Nichts sonst darf von außen zugreifbar sein (auch nicht von anderen Klassen des gleichen Paketes aus). Sie dürfen beliebige Attribute verwenden, die aber nur von innerhalb dieser Klasse aus zugreifbar sein dürfen. Hinweis: Offenbar muss man sich die Parameter des Konstruktors abspeichern und über die Zeit (Anzahl Aufrufe von tick()) seit dem letzten Aufruf von bewegt() ” Buch fuhren“ (wie Sie das genau realisieren, ist Ihnen überlassen).



Mein Programm bis jetzt:

public class Notsignalgeber {

    private int vor;
    private int haupt;
    private int speichervor;
    private int speichernach;
    private int anzahlderAufrufevonTick=0;
   
   
    public Notsignalgeber(int vor, int haupt) {
        this.vor=vor;
        this.haupt=haupt;
        speichervor=vor;
        speichernach=haupt;
       
       
    }
   
    public int tick() {
        anzahlderAufrufevonTick++;
        if(anzahlderAufrufevonTick<=vor) return 0;
        if(anzahlderAufrufevonTick>vor && anzahlderAufrufevonTick<=haupt) return 50;
        return 100;
       
       
    }
   
   
    public void bewegt() {
       
    }
}

Ich weis nicht wie ich die Methoden bewegt und tick verketten soll, dass das klappt.
Kann mir da jmd bitte weiterhelfen ;(

 

[Java20134]

Erstell doch einfach eine Variable, die in der Methode tick() überprüft wird, ob sie 0 ist, dann kann der Rest folgen oder du dekrementierst nur und beendest dann die Methode. bewege() macht dann nichts anderes als diese Variable neu zusetzen.
Warum brauchst du die Variablen speichervor und speichernach?

 

[Meniskusschaden]

Du könntest bei Objekterzeugung und beim Aufruf von bewegt() einen Zähler initialisieren, und bei tick() dekrementieren und auswerten.

 

[Heinrich500]

Erstell doch einfach eine Variable, die in der Methode tick() überprüft wird, ob sie 0 ist, dann kann der Rest folgen oder du dekrementierst nur und beendest dann die Methode. bewege() macht dann nichts anderes als diese Variable neu zusetzen.
Warum brauchst du die Variablen speichervor und speichernach?

Weis ich nicht genau. Es stand im Hinweis, dass man die Werte aus dem Konstruktor speichern soll.
Wie soll das mit der 0 gehen.
Ich verstehe das nicht.

 

[Java20134]

Du musst die Argumente ja aber nicht zweimal zuweisen!

//als Beispielname counter
private int counter;

//Initialisierung im Konstruktor
counter = 30;

public int tick() {
        anzahlderAufrufevonTick++;
        if(counter != 0){
                 counter--;
                 return;
        }

        if(anzahlderAufrufevonTick<=vor) return 0;
        if(anzahlderAufrufevonTick>vor && anzahlderAufrufevonTick<=haupt) return 50;
        return 100;
  
  
    }
 
 
    public void bewegt() {
          counter = 30;
          anzahlderAufrufevonTick = 0;
    }

 

[Heinrich500]

Danke für deine Antwort.
Du setzt in bewegt den counter auf 30.
Wenn dann tick aufgerufen wird, wird der counter verringert, jedoch wird doch trotzdem anzahlderAufrufevonTick erhöht und bleibt dann nicht 30 mal 0 ?

 

[Java20134]

Habs geändert. War ein kleiner Denkfehler.
Es ist sogar noch viel einfacher, wenn du das oben genannte Bsp. noch einmal durchgehst, kannst du auch folgende Lösung nutzen:

public int tick() {
        anzahlderAufrufevonTick++;

        if(anzahlderAufrufevonTick<=vor) return 0;
        if(anzahlderAufrufevonTick>vor && anzahlderAufrufevonTick<=haupt) return 50;
        return 100;
 

    }
    public void bewegt() {
          anzahlderAufrufevonTick = 0;
    }

 

[Meniskusschaden]

Die Variable anzahlDerAufrufeVonTick wird eigentlich nicht benötigt. Du kannst direkt counter geeignet abfragen.

 

[Heinrich500]

Ach so.
Das ist ja ganz einfach.
anzahlderAufrufevonTick wird einfach zurückgesetzt.
Danke euch))))
Endlich verstanden.
Nur verwunderlich, dass die Aufgabe 13 Punkte bringt für so "wenig" .
Und auch der Hinweis: Offenbar muss man sich die Parameter des Konstruktors abspeichern?
Naja es geht ja so

 

[Meniskusschaden]

Nur verwunderlich, dass die Aufgabe 13 Punkte bringt für so "wenig" .
Und auch der Hinweis: Offenbar muss man sich die Parameter des Konstruktors abspeichern?

Na ja, die musst du schon abspeichern. Sonst wird's nicht funktionieren.

 

[Meniskusschaden]

Nur verwunderlich, dass die Aufgabe 13 Punkte bringt für so "wenig" .

Wenn man's selber löst, ist es etwas mehr Arbeit.

 

[Java20134]

Es müsste dann bei dir folgendes stehen:

public class Notsignalgeber {

    private int vor;
    private int haupt;
    private int anzahlderAufrufevonTick;
  
    public Notsignalgeber(int vor, int haupt) {
        this.vor = vor;
        this.haupt = haupt;
        this.anzahlderAufrufevonTick = 0;
    }
  
    public int tick() {
        anzahlderAufrufevonTick++;

        if(anzahlderAufrufevonTick <= vor) return 0;
        if(anzahlderAufrufevonTick > vor && anzahlderAufrufevonTick <= haupt) return 50;

        return 100;      
    }
  
  
    public void bewegt() {
       anzahlderAufrufevonTick = 0;
    }
}

 

[Heinrich500]

Ja genau. Das habe ich so.
Danke Java20134
Das ist doch das ganze Programm.

 

[Flown]

Naja, abgesehen davon, dass es nicht das macht was es soll.
Schreib doch Tests dazu, dann siehst du auch was fehlt:

import org.junit.jupiter.api.Assertions;
import org.junit.jupiter.api.BeforeEach;
import org.junit.jupiter.api.Test;

class NotsignalgeberTest {
 
  private Notsignalgeber signal;
  private int preAlarm = 10;
  private int mainAlarm = 10;
 
  @BeforeEach
  private void init() {
    signal = new Notsignalgeber(preAlarm, mainAlarm);
  }
 
  @Test
  void tickTest() {
    for (int i = 0; i < preAlarm; i++) {
      Assertions.assertEquals(Notsignalgeber.NO_ALARM, signal.tick());
    }
    for (int i = 0; i < mainAlarm; i++) {
      Assertions.assertEquals(Notsignalgeber.PRE_ALARM, signal.tick());
    }
    Assertions.assertEquals(Notsignalgeber.MAIN_ALARM, signal.tick());
  }
 
  @Test
  void bewegt() {
    for (int i = 0; i < preAlarm + mainAlarm; i++) {
      signal.tick();
    }
    Assertions.assertEquals(Notsignalgeber.MAIN_ALARM, signal.tick());
    signal.bewegt();
    Assertions.assertEquals(Notsignalgeber.NO_ALARM, signal.tick());
  }
}

 

[MoxxiManagarm]

Ich stelle hier mal was in den Raum, was ich bisher noch in keinem Code sehe:
"Ansonsten wird z.B. nach weiteren 10 s der Hauptalarm ausgelöst"

 

[Heinrich500]

Ich stelle hier mal was in den Raum, was ich bisher noch in keinem Code sehe:
"Ansonsten wird z.B. nach weiteren 10 s der Hauptalarm ausgelöst"

Aber das ist doch realisiert hier:

if(anzahlderAufrufevonTick > vor && anzahlderAufrufevonTick <= haupt) return 50;

 

[Meniskusschaden]

Hast du ein Beispiel für eine Zahl, die größer als 30 und kleiner als 10 ist?

 

[Heinrich500]

Ich glaube nicht

if(anzahlderAufrufevonTick > vor && (anzahlderAufrufevonTick+vor) <= (vor+haupt) return 50;

Würde das so gehen?

 

[MoxxiManagarm]

Die sicherste Variante wäre wenn du im Konstruktor gleich beide Werte addierst und haupt dann die Summe beider Werte enthält

 

[Heinrich500]

Ok danke für deinen Tipp