Může být. Jen ještě tip k zamyšlení.
Funkce OdesliZapnuto() je velmi podobná s funkcí OdesliVypnuto(), resp. obsahuje hodně redundantního kódu, což nikdy není dobře. Představ si např., že se tě změní doména, nebo filename php skriptu, nebo GET parametry. Opravíš to ve funkci OdesliZapnuto(), ale na funkci OdesliVypnuto() zapomeneš, nebo ji přehlédneš.
Velmi snadno bys mohl mít jen funkci Odesli(bool Zapnuto):
Kód: Vybrat vše
void Odesli(bool Zapnuto) {
if (client.connect(serverName, 80)) {
client.print("GET http://xxx.eu/logger.php?text=Cidlo%20cislo%20");
client.print(ID_MODULU);
if (Zapnuto) {
client.println("%20zapnuto");
} else {
client.println("%20vypnuto");
}
client.println("Host: xxx.eu");
client.println("Accept: text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,*/*;q=0.8");
client.println("Connection: close");
client.println();
delay (500);
client.stop();
}
}
V loopu pak při detekci zapnutí budeš volat:
a při detekci vypnutí:
Dalším krokem by pak mohlo být, že funkce nebude znát jen tyto 2 fixní stavy, ale bude mít jako argument pointer na char a předáš jí vždy text, který chceš zalogovat...