Quel est le problème dans ce code python? ça a l'air bien

Question

condition = float(29.0) 

while True: 
     GPIO.output(17, GPIO.HIGH) 
     if temp_c < condition: 
       break 

La valeur temp_c est un flottant et change toutes les secondes. Lorsque la valeur temp_c est supérieure à 29,0, je souhaite que la sortie GPIO soit activée. Qu'est-ce qui ne va pas?

Answer 1

Vous définissez la broche 17 à HIGH, mais elle ne remonte pas à LOW. Vous devez définir explicitement revenir au niveau bas lorsque la condition est atteinte:

while True: 
    GPIO.output(17, GPIO.HIGH) 
    if temp_c < condition: 
     GPIO.output(17, GPIO.LOW) 
      break 

Mais le flux d'exécution n'est pas vraiment bon. Seraient mieux:

GPIO.output(17, GPIO.HIGH) 
while temp_c >= condition: 
    pass 
GPIO.output(17, GPIO.LOW) 

Ici, quelques choses doivent noter:

• Si une broche est réglée à un niveau donné, il reste à ce niveau. Il n'est pas nécessaire de le configurer à plusieurs reprises, comme vous l'avez fait en appelant GPIO.output(17, GPIO.HIGH) dans votre boucle while;
• Vous souhaitez attendre qu'une condition spécifique soit remplie. Par conséquent, il vous suffit de boucler tant qu'il n'est pas satisfait. Ceci est simplement fait par while not condition: pass, ou while not condition: continue (pass est une déclaration vide). Cela permet de se débarrasser du while True, qu'il est préférable d'éviter (bien que parfois nécessaire dans des contextes particuliers).

Answer 2

Vous appelez la fonction de sortie à chaque fois et de briser la boucle si la température est inférieure à 29. Si vous voulez que la valeur élevée si la température est supérieure à 29:

condition = float(29.0) 

while True: 
     if temp_c > condition: 
      GPIO.output(17, GPIO.HIGH) 
      break 

Answer 3

Vous devrez tourner hors de la broche GPIO afin de voir tout changement.

Voici un correctif immédiat à votre problème:

if temp_c < condition: 
    GPIO.output(17, GPIO.LOW) 
    break 

Mais vous ne devez pas définir l'état de la broche à chaque itération. Une mise en œuvre plus efficace de ce que vous voulez serait ceci:

condition = float(29.0) 
GPIO.output(17, GPIO.HIGH) 
while True : 
    if temp_c < condition : 
     GPIO.output(17, GPIO.LOW) 
     break 

Vous pouvez le rendre plus efficace en déplaçant la condition if à la condition while, mais vous devez le faire en gardant à l'esprit le reste de votre temps bloc de code.