search

gregalli02 / Agrumino

This is the code for Agrumino Lemon with Telegram
MIT License
1 stars 0 forks source link

Switch case instead if and else :) #1

Open cicoph opened 3 years ago

cicoph commented 3 years ago

Ciao e complimenti per il tuo lavoro. Ho letto il tuo codice e avendo del tempo ho pensato di poter condividere con te questa soluzione di "stile" del codice. Fammi sapere cosa ne pensi. Alla prossima! ... if (myBot.getNewMessage(msg)) { String message = msg.text.toLowerCase() unsigned int senderId = msg.sender.id switch (message) { case 'temperatura': myBot.sendMessage(senderId, "misuro la temperatura..."); myBot.sendMessage(senderId, "ci sono " + t + "° C"); break; case 'umidità': myBot.sendMessage(senderId, "misuro l'umidità..."); myBot.sendMessage(senderId, "l'umidità è " + h + "%"); break; case 'luce': myBot.sendMessage(senderId, "misuro la luce..."); myBot.sendMessage(senderId, "la luce presente è " + l); break; case 'batteria: myBot.sendMessage(senderId, "misuro la batteria..."); myBot.sendMessage(senderId, "la batteria ha " + b + "V"); break; default: String reply; reply = (String)"Ciao " + msg.sender.username + (String)". Cosa vuoi sapere di Conny?"; myBot.sendMessage(senderId, reply); break; } } ...

gregalli02 commented 3 years ago

Ciao! Grazie mille! Non ci avevo pensato sinceramente, ero rimasta al codice "originale" con l'else if! Lo modifico e metto entrambe le versioni, grazie!

On Sun, 6 Jun 2021, 11:40 Francesco Loddo, @.***> wrote:

Ciao e complimenti per il tuo lavoro. Ho letto il tuo codice e avendo del tempo ho pensato di poter condividere con te questa soluzione di "stile" del codice. Fammi sapere cosa ne pensi. Alla prossima! ... if (myBot.getNewMessage(msg)) { String message = msg.text.toLowerCase() unsigned int senderId = msg.sender.id switch (message) { case 'temperatura': myBot.sendMessage(senderId, "misuro la temperatura..."); myBot.sendMessage(senderId, "ci sono " + t + "° C"); break; case 'umidità': myBot.sendMessage(senderId, "misuro l'umidità..."); myBot.sendMessage(senderId, "l'umidità è " + h + "%"); break; case 'luce': myBot.sendMessage(senderId, "misuro la luce..."); myBot.sendMessage(senderId, "la luce presente è " + l); break; case 'batteria: myBot.sendMessage(senderId, "misuro la batteria..."); myBot.sendMessage(senderId, "la batteria ha " + b + "V"); break; default: String reply; reply = (String)"Ciao " + msg.sender.username + (String)". Cosa vuoi sapere di Conny?"; myBot.sendMessage(senderId, reply); break; } } ...

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/gregalli02/Agrumino/issues/1, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUIXOJ2UBK5GLCQDQEYSL53TRM7BZANCNFSM46FXCISQ .

Cereal84 commented 2 years ago

Ciao ad entrambi. @cicoph: crea una PR/MR (Pull Request o Merge Request) con il tuo fix/enhancement. Questo è un buon modo (per te e per @gregalli02 ) per impratichirsi con git e imparare a collaborare su Git, volendo potete fare anche una code review. Bisoga sfruttare queste occasioni per imparare :)