请教python代码只执行一半

lxa000

试着改进一段python代码
作用是4个led顺序点亮，然后再倒序熄灭。

改完后运行4个led顺序点亮正常，后就不再变化了。
请教坛友：代码哪里不对了？应该怎样改进？
先谢谢！


#LED流水灯：先熄灭4个LED,然后逐个点亮4个LED,反复进行
from machine import Pin   
import time                 

def ledz():   
    ledpinz=[8,7,6,5]   
    leds=[]                        
    for i in range(4):            
        leds.append(Pin(ledpinz[i],Pin.OUT))        
    for n in range(4):              
         while True:                                                   
             for n in range(4):        
                 leds[n].value(1)     
                 time.sleep(0.3)      
             time.sleep(0.3)                       
def ledf():
    ledpinf=[5,6,7,8]   
    leds=[]                 
    for i in range(4):            
        leds.append(Pin(ledpinf[i],Pin.OUT))  
    for n in range(4):      
         while True:
            for n in range(4):      
                leds[n].value(0)
                time.sleep(0.3)     
            time.sleep(0.3)
try:
    while True:
         ledz()               
         ledf()                    
except KeyboardInterrupt:
    pass

only5784

ai分析的

junyee

lxa000 发表于 2025-1-6 23:36
按你说的改进后 如下 ：

1. import time
   
2. from machine import Pin
   
3. 
   
4. ledpins = [ Pin(v, Pin.OUT) for v in [8, 7, 6, 5] ]
   
5. ledstats = [0, 0, 0, 0]
   
6. 
   
7. 
   
8. def led_ctrl():
   
9.         for i,v in enumerate(ledstats):
   
10.                 ledpins[i].value(v)
    
11. ##
    
12. 
    
13. while True:
    
14.         for i in range(4):
    
15.                 ledstats[i] = 1
    
16.                 led_ctrl()
    
17.                 time.sleep(1)
    
18.                
    
19.         for i in range(4):
    
20.                 ledstats[3-i] = 0
    
21.                 led_ctrl()
    
22.                 time.sleep(1)

复制代码

佚天

while True:                                                   
             for n in range(4):        
                 leds[n].value(1)     
                 time.sleep(0.3)      
             time.sleep(0.3)   


执行到这段时，这死循环在这里出不去了。

junyee

看来是个新手。
给你写一个：

1. import time
   
2. from machine import Pin
   
3. 
   
4. ledpins = [ Pin(v, Pin.OUT) for v in [8, 7, 6, 5] ]
   
5. ledstats = [0, 0, 0, 0]
   
6. 
   
7. 
   
8. def led_ctrl():
   
9.         for i,v in enumerate(ledstats):
   
10.                 ledpins[i].value(v)
    
11. ##
    
12. 
    
13. while True:
    
14.         for i in range(4):
    
15.                 ledstats[i] = 1 if ledstats[i] == 0 else 0
    
16.                 led_ctrl()
    
17.                 time.sleep(1)

复制代码

aidn

ai修改from machine import Pin    import time                   def ledz():        """     控制 4 个 LED 逐个点亮的函数     """     ledpinz = [8, 7, 6, 5]        leds = []                             for i in range(4):                     leds.append(Pin(ledpinz[i], Pin.OUT))             for _ in range(4):  # 这里使用 _ 替代 n，避免与内层循环的变量名冲突         for led in leds:  # 直接遍历 leds 列表             led.value(1)                  time.sleep(0.3)               time.sleep(0.3)                         def ledf():     """     控制 4 个 LED 逐个熄灭的函数     """     ledpinf = [5, 6, 7, 8]        leds = []                      for i in range(4):                     leds.append(Pin(ledpinf[i], Pin.OUT))       for _ in range(4):  # 这里使用 _ 替代 n，避免与内层循环的变量名冲突         for led in leds:  # 直接遍历 leds 列表             led.value(0)             time.sleep(0.3)              time.sleep(0.3)  try:     """     主程序，不断交替执行逐个点亮和逐个熄灭的操作     """     while True:          ledz()                         ledf()                     except KeyboardInterrupt:     """     捕获键盘中断异常，通常用于在程序运行时通过键盘操作中断程序     """     pass

dianziwg

好奇，这是用来控制哪个上面的led啊？

lxa000

佚天 发表于 2025-1-6 09:21
while True:                                                   
             for n in range(4):      ...

非常感谢上面坛友的解答。

我会根据你们的指导继续试验。

lxa000

dianziwg 发表于 2025-1-6 08:55
好奇，这是用来控制哪个上面的led啊？

我外接了 4个led和 4个按键。

lxa000

刚才试了junyee，aidn两位坛友的代码，
都是顺序点亮顺序熄灭的 。
我想要顺序点亮，倒序熄灭的 。

junyee

13行 后改成：

1. 
   
2. while True:
   
3.         for i in range(4):
   
4.                 ledstats[i] = 1
   
5.                 led_ctrl()
   
6.                 time.sleep(0.5)
   
7.                
   
8.         for i in range(4):
   
9.                 ledstats[3-i] = 0
   
10.                 led_ctrl()
    
11.                 time.sleep(1)

复制代码

lxa000

junyee 发表于 2025-1-6 19:50
13行 后改成：

按你说的改进后 如下 ：

qyg123

查了一下发现RP2040单片机挺强大的，应该很好玩。

lxa000

junyee 发表于 2025-1-7 09:29

ok！
这回成了。
非常感谢！

lxa000

qyg123 发表于 2025-1-7 00:50
查了一下发现RP2040单片机挺强大的，应该很好玩。

是的 。
用这种单片机做啥子数控收音机/数控稳压电源/电子手表。。。
都不在话下。

所以现在暂时放下51
转到学习ESP32。

lxa000

junyee 发表于 2025-1-6 19:50
13行 后改成：

你昨天发的代码咋没了 ？

今天拍了一下段视频：
顺倒流水灯.rar (469.4 KB, 下载次数: 4)

2025-1-8 18:22 上传

点击文件名下载附件