@homelessJames: Du müsstest Dir merken wann die letzte Ausgabe war und ausrechnen ob die schon länger als 5 Minuten zurück liegt. Dazu eignen sich die Datentypen aus dem `datetime`-Modul.
Um das sauber zu lösen brauchst Du eine Klasse. Brauchst Du im Grunde auch jetzt schon, denn das `handle()` einfach so auf `bot` zugreift, dürfte eigentlich nicht sein. Funktionen sollten unabhängig funktionieren und keine Abhängigkeiten zu globalen Variablen haben. So schafft man sich sonst ganz schnell unübersichtliche Programme die man nicht mehr, oder nur schwer versteht.
In diese Versuchung kommt man gar nicht erst wenn man auf Modulebene nur Code stehen hat, der Konstanten, Funktionen, und Klassen definiert. Das Hauptprogramm steht üblicherweise in einer Funktion die `main()` heisst.
Abkürzungen bei Namen sollte man vermeiden. `message` ist klarer als `msg`.
Dateien die man öffnen, sollte man auch wieder schliessen. Hier ist die ``with``-Anweisung recht praktisch. `file` ist der Name des eingebauten Dateityps, den sollte man nicht an andere Werte binden.
Wenn man Wahrheitswerte meint, sollte man `True` und `False` verwenden und nicht 1 und 0.
Wenn das Programm auf einem Unix/Linux läuft, würde ich die Schleife am Ende durch `signal.pause()` ersetzen.
Wenn die anderen Wochentage genauso aufgebaut sind, dann braucht man kein ``if`` pro Tag im Quelltext, denn das würde ja auf Programmieren durch kopieren und einfügen hinaus laufen, sondern eine Schleife über die Namen der Wochentage. Den Code für den einzelnen Tag kann man aufgrund dieser Zeichenkette formulieren. Auch sollte man Konstante Werte wie den Pfad zu den Texten nur einmal im Programm stehen haben. Sonst macht das nur unnötig Arbeit wenn man das mal anpassen möchte, und ist fehleranfällig, weil man aufpassen muss jede Kopie des Wertes exakt gleich zu ändern.
Ich lande dann als Zwischenergebnis bei so etwas (ungetestet):
Code: Alles auswählen
#!/usr/bin/env python
# -*- coding: utf-8 -*-
from __future__ import absolute_import, division, print_function
import os
import signal
from functools import partial
import telepot
SECRET_TOKEN = 'Geheimer Token.'
BASE_PATH = '/home/pi/bots/Schulbot'
WEEK_DAYS = [
'montag', 'dienstag', 'mittwoch', 'donnerstag',
'freitag', 'samstag', 'sonntag',
]
def handle(bot, message):
chat_id = message['chat']['id']
command = message['text'].lower()
for weekday in WEEK_DAYS:
if weekday in command or command == weekday[:2]:
path = os.path.join(BASE_PATH, weekday.capitalize() + '.txt')
with open(path, 'r') as day_file:
bot.sendMessage(chat_id, text=day_file.read())
break
def main():
bot = telepot.Bot(SECRET_TOKEN)
bot.message_loop(partial(handle, bot))
print('Schulbot gestartet.')
signal.pause()
if __name__ == '__main__':
main()
Hier kommt man noch ohne Klasse aus, aber wenn Du ein ”cooldown” einbauen möchtest, dann musst Du Dir selbst Zustand über `handle()`-Aufrufe hinweg merken der nicht in der `Bot`-Klasse gekapselt ist, also brauchst Du eine eigene Klasse.
Ich würde mir auch überlegen ob die Wochentagsdateinamen unbedingt mit einem Grossbuchstaben anfangen müssen. Dann könnte man sich nämlich das `capitalize()` sparen.
