主页 / 问题 / 1439643
Accepted
Vladislav Shvetsov
Asked:2022-08-16 04:24:10 +0000 UTC2022-08-16 04:24:10 +0000 UTC

每次条件检查后递增是否正确？

  • 772

告诉我，在循环中的每个 if 之后都有 i += 1 是否正确？

from ping3 import ping

urls = ('ya.ru', '8.8.8.8', 'e1.ru', '192.168.0.99', 'google.com', '192.168.1.25', '192.168.1.26', 'ok.ru', '192.168.0.2', '192.168.0.1', '192.168.1.1', '192.168.1.2')

i=0
while i<len(urls):
    ping(urls[i])
    if ping(urls[i]) == None:
        print('None: ', urls[i])
        i += 1
    if ping(urls[i]) == False:
        print('False: ',urls[i])
        i += 1
    else:
        print('OK: ', urls[i])
        i += 1

代码可以工作，但让我感到困惑的是，他们在参考书中的任何地方都没有这样写，如果你不这样写，代码就不能正常工作。恐怕在大列表中会出现某种错误，而这在实际任务中根本看不到。

python if

3 个回答

  1. 大错特错。请参阅@StanislavVolodarskiy 的回答

    这更好：

    else:
    print('OK: ', urls[i])
i += 1

    删除其余部分i += 1。

    更好的是，因为它是遍历列表，所以使用列表遍历：

    for url in urls:
    ping(url)
    if ping(url) == None:
        print('None: ', url)
    if ping(url) == False:
        print('False: ',url)
    else:
        print('OK: ', url)

    另外，这里每个地址都被 ping 了 3 次，其中 1 次完全没用。这已经很糟糕了，因为结果可能不同。像这样正确：

    for url in urls:
    result = ping(url)
    if result == None:
        print('None: ', url)
    if result == False:
        print('False: ',url)
    else:
        print('OK: ', url)
    • 2
  2. Best Answer

    越界名单

    该函数ping总是返回None列表中的一个地址。如果地址列表中的最后一个地址没有响应，脚本会崩溃：

    def ping(url):
    return None

urls = ('a', )

i=0
while i<len(urls):
    ping(urls[i])
    if ping(urls[i]) == None:
        print('None: ', urls[i])
        i += 1
    if ping(urls[i]) == False:
        print('False: ',urls[i])
        i += 1
    else:
        print('OK: ', urls[i])
        i += 1 

    $ python temp.py
None:  a
Traceback (most recent call last):
  File "/home/sv/desk/stackoverflow/temp.py", line 12, in <module>
    if ping(urls[i]) == False:
IndexError: tuple index out of range

    错误的答案

    如果地址列表中连续的两个地址没有响应，则代码将为第二个地址打印“OK”。该函数ping总是返回None, 列表中的两个地址。我希望答案是

    None:  a
None:  b

    我们推出：

    def ping(url):
    return None


urls = ('a', 'b')

i=0
while i<len(urls):
    ping(urls[i])
    if ping(urls[i]) == None:
        print('None: ', urls[i])
        i += 1
    if ping(urls[i]) == False:
        print('False: ',urls[i])
        i += 1
    else:
        print('OK: ', urls[i])
        i += 1

    对于第二个地址，打印“OK”：

    None:  a
OK:  b

    工作版

    for url in urls:
    r = ping(url)
    v = 'OK'
    if r is None:
        v = 'None'
    if r is False:
        v = 'False'
    print(f'{v}: {url}')
    • 2

  3. 这是它应该如何工作的：

    for url in urls:
    print(f'OK: {url}' if (r:=ping(url)) else f'{r}: {url}')
    • 0

相关问题