Refactor Wecker Script for Improved Readability and Functionality

- Combined IP retrieval and ping test into a single function 'check_ip'.
- Improved error handling for better script robustness.
- Removed unnecessary commented-out code and added comments for clarity.
- Ensured consistency in indentation and formatting.

Author: Michael, Haider
Date: 22.12.2023

A72_alarm_getter.sh

@@ -39,25 +39,23 @@ delete_expired_at_jobs() {
 # Verwende die Funktion, um überholte Jobs zu löschen
 delete_expired_at_jobs
 
-# Die IP-Adresse von Lisas-IPhone holen.
+# Die IP-Adresse von Lisas-IPhone holen und Ping-Test durchführen
 check_ip() {
   mac_address="e4:cd:d1"  # Ersetze dies durch die MAC-Adresse
   A72_ip=$(arp-scan --localnet | grep "$mac_address" | awk '{print $1}')
+  
+  if [ -z "$A72_ip" ]; then
+    echo "IP nicht gefunden. Ziel ist nicht erreichbar."
+    return 1
+  fi
+  
   echo "Gefundene IP: $A72_ip"
-}
 
-# check_ip -Funktion ausführen.
-check_ip
-
-# Datei zum Speichern der geplanten Weckzeiten
-weckzeit_datei="/root/iobroker_scripts/generel/.weckzeiten.txt"
-
-# Überprüfen, ob das Ziel über Ping erreichbar ist
-function is_target_reachable() {
+  # Ping-Test durchführen
   echo "teste ping $A72_ip"
   local ping_output=$(/bin/ping -q -w 3 -c 3 $A72_ip 2>&1)
   local ping_exit_code=$?
-
+  
   echo "$ping_output"  # Ausgabe des Ping-Befehls anzeigen
 
   if [ $ping_exit_code -eq 0 ] && [[ "$ping_output" == *"3 received"* ]]; then
@@ -66,45 +64,44 @@ function is_target_reachable() {
   else
     echo "Das Ziel ist nicht erreichbar. Exit Code: $ping_exit_code"
     return 1  # Fehler: Das Ziel ist nicht erreichbar
-    exit
-fi
+  fi
 }
 
-# Überprüfen, ob das Ziel erreichbar ist
-if is_target_reachable; then
-  # Ziel ist erreichbar, Alarmzeit über SSH abrufen
-  alarmTime=$(ssh -i /root/.ssh/A72 root@$A72_ip dumpsys alarm | grep -A 6 ":com.urbandroid.sleep.alarmclock.ALARM_ALERT" | grep "type=RTC_WAKEUP origWhen=" | cut -d= -f3 | cut -c1-16 | uniq)
+# check_ip -Funktion ausführen.
+check_ip || exit 1  # Beende das Skript, wenn das Ziel nicht erreichbar ist
 
-  if [ -n "$alarmTime" ]; then
-    # Zerlege das Datum und die Uhrzeit
-    datum=$(echo "$alarmTime" | cut -d ' ' -f 1)
-    uhrzeit=$(echo "$alarmTime" | cut -d ' ' -f 2)
+# Datei zum Speichern der geplanten Weckzeiten
+weckzeit_datei="/root/iobroker_scripts/generel/.weckzeiten.txt"
 
-    # Formatiere das Datum und die Uhrzeit neu
-    neues_format=$(date -d "$datum $uhrzeit" "+%H:%M %m/%d/%y")
+# Ziel ist erreichbar, Alarmzeit über SSH abrufen
+alarmTime=$(ssh -i /root/.ssh/A72 root@$A72_ip dumpsys alarm | grep -A 6 ":com.urbandroid.sleep.alarmclock.ALARM_ALERT" | grep "type=RTC_WAKEUP origWhen=" | cut -d= -f3 | cut -c1-16 | uniq)
 
-    # Subtrahiere 15 Minuten von der Zeit
-    lightsOn=$(date -d "$neues_format - 15 minutes" "+%H:%M %m/%d/%y")
+if [ -n "$alarmTime" ]; then
+# Zerlege das Datum und die Uhrzeit
+datum=$(echo "$alarmTime" | cut -d ' ' -f 1)
+uhrzeit=$(echo "$alarmTime" | cut -d ' ' -f 2)
 
-    echo "Nächster Wecker: $neues_format"
-    echo "Lichter einschalten um: $lightsOn"
+# Formatiere das Datum und die Uhrzeit neu
+neues_format=$(date -d "$datum $uhrzeit" "+%H:%M %m/%d/%y")
 
-    # Überprüfen, ob die Weckzeit bereits geplant ist
-    if grep -Fxq "$lightsOn" "$weckzeit_datei"; then
-      echo "Weckzeit ist bereits geplant für: $lightsOn"
-    else
-      echo "Weckzeit planen und in Datei speichern: $lightsOn"
-      echo "$lightsOn" > "$weckzeit_datei"
-      echo "/root/iobroker_scripts/schlafzimmer/wecker.sh > /root/wecker.log" | at "$lightsOn"
-      curl -u "Michaelis:u5ptufUFuDL6q4yEcSN3iqas5gtXNkN77Lx3cy3oX8UoSgFWdifYy9FVopv2Zwtu" -H "Icon:https://static.vecteezy.com/system/resources/previews/018/931/118/original/alarm-clock-icon-png.png" -H "Priority:High" -d "Alarm gestellt um: $neues_format" https://ntfy.michaelis.digital/$HOSTNAME
-    fi
-  else
-    echo "Kein Wecker gestellt. Lösche geplante Weckzeiten."
-    #curl -u "Michaelis:u5ptufUFuDL6q4yEcSN3iqas5gtXNkN77Lx3cy3oX8UoSgFWdifYy9FVopv2Zwtu" -H "Icon:https://static.vecteezy.com/system/resources/previews/018/931/118/original/alarm-clock-icon-png.png" -H "Priority:High" -d "Alarm gelöscht" https://ntfy.michaelis.digital/$HOSTNAME
+# Subtrahiere 15 Minuten von der Zeit
+lightsOn=$(date -d "$neues_format - 15 minutes" "+%H:%M %m/%d/%y")
 
-    rm "$weckzeit_datei"
-  fi
+echo "Nächster Wecker: $neues_format"
+echo "Lichter einschalten um: $lightsOn"
+
+# Überprüfen, ob die Weckzeit bereits geplant ist
+if grep -Fxq "$lightsOn" "$weckzeit_datei"; then
+  echo "Weckzeit ist bereits geplant für: $lightsOn"
 else
-  echo "Ziel ist nicht erreichbar."
+  echo "Weckzeit planen und in Datei speichern: $lightsOn"
+  echo "$lightsOn" > "$weckzeit_datei"
+  echo "/root/iobroker_scripts/schlafzimmer/wecker.sh > /root/wecker.log" | at "$lightsOn"
+  curl -u "Michaelis:u5ptufUFuDL6q4yEcSN3iqas5gtXNkN77Lx3cy3oX8UoSgFWdifYy9FVopv2Zwtu" -H "Icon:https://static.vecteezy.com/system/resources/previews/018/931/118/original/alarm-clock-icon-png.png" -H "Priority:High" -d "Alarm gestellt um: $neues_format" https://ntfy.michaelis.digital/$HOSTNAME
 fi
+else
+  echo "Kein Wecker gestellt. Lösche geplante Weckzeiten."
+  #curl -u "Michaelis:u5ptufUFuDL6q4yEcSN3iqas5gtXNkN77Lx3cy3oX8UoSgFWdifYy9FVopv2Zwtu" -H "Icon:https://static.vecteezy.com/system/resources/previews/018/931/118/original/alarm-clock-icon-png.png" -H "Priority:High" -d "Alarm gelöscht" https://ntfy.michaelis.digital/$HOSTNAME
 
+  rm "$weckzeit_datei"
+fi

wecker.sh

@@ -39,11 +39,32 @@ set_color() {
     iobroker state set zigbee.0.04cd15fffee03198.color "$color_value"
 }
 
-# Die IP-Adresse von Lisas-IPhone holen.
+# Die IP-Adresse von Lisas-IPhone holen und Ping-Test durchführen
 check_ip() {
   mac_address="e4:cd:d1"  # Ersetze dies durch die MAC-Adresse
   A72_ip=$(arp-scan --localnet | grep "$mac_address" | awk '{print $1}')
+  
+  if [ -z "$A72_ip" ]; then
+    echo "IP nicht gefunden. Ziel ist nicht erreichbar."
+    return 1
+  fi
+  
   echo "Gefundene IP: $A72_ip"
+
+  # Ping-Test durchführen
+  echo "teste ping $A72_ip"
+  local ping_output=$(/bin/ping -q -w 3 -c 3 $A72_ip 2>&1)
+  local ping_exit_code=$?
+  
+  echo "$ping_output"  # Ausgabe des Ping-Befehls anzeigen
+
+  if [ $ping_exit_code -eq 0 ] && [[ "$ping_output" == *"3 received"* ]]; then
+    echo "Das Ziel ist erreichbar."
+    return 0  # Erfolg: Das Ziel ist erreichbar
+  else
+    echo "Das Ziel ist nicht erreichbar. Exit Code: $ping_exit_code"
+    return 1  # Fehler: Das Ziel ist nicht erreichbar
+  fi
 }
 
 # check_ip -Funktion ausführen.
@@ -70,6 +91,7 @@ check_notification() {
 
   for ((i = 1; i <= $max_attempts; i++)); do
     # Führe den SSH-Befehl innerhalb eines Timeout aus
+    check_ip # IP holen
     output=$(ssh -i /root/.ssh/A72 root@$A72_ip 'dumpsys activity processes | grep com.urbandroid.sleep')
     echo $output
 
@@ -140,13 +162,13 @@ colortemp_step=$(bc <<< "$start_colortemp / $steps")
     set_brightness "$current_brightness"
     set_colortemp "$current_colortemp"
     
-    sleep 27  # Änderungen alle 30 (27) Sekunden
+    sleep 30  # Änderungen alle 30 Sekunden
 done
 
-
 # check_ip -Funktion ausführen.
 check_ip
 echo "$A72_ip"
+
 # Starten des Alarm-Scripts auf dem Handy.
 ssh -i /root/.ssh/A72 root@$A72_ip  '/data/scripts/morning-alarm.sh' > /root/morning_alarm.log