Question

我正在学习Python作为我的第二种编程语言（如果不计算HTML / CSS / Javascript，我的第一个真正的编程语言）。我正在尝试构建一些有用的东西作为我的第一个真正的应用程序 - 一个IRC机器人，当通道中发生某些事情时通过短信提醒人们。根据某人的请求，我（尝试）建立调度首选项，人们可以选择不在当天的X和Y小时之间收到警报。

无论如何，这是我遇到问题的代码：

db = open("db.csv")
for line in db:
            row = line.split(",")  # storing stuff in a CSV, reading out of it 
            recipient = row[0]     # who the SMS is going to
            s = row[1]             # gets the first hour of the "no alert" time range
            f = row[2]             # gets last hour of above
            nrt = []               # empty array that will store hours
            curtime = time.strftime("%H")  # current hour
            if s == "no":          
                    print "They always want alerts, sending email"  # start time will = "no" if they always want alerts
                    # send mail code goes here
            else:
                    for hour in range(int(s), int(f)): #takes start, end hours, loops through to get hours in between, stores them in the above list 
                            nrt.append(hour)
                    if curtime in nrt: # best way I could find of doing this, probably a better way, like I said I'm new
                            print "They don't want an alert during the current hour, not sending"  # <== what it says
                    else:
                            # they do want an alert during the current hour, send an email
                            # send mail code here

我遇到的唯一问题是脚本只会以一条线（或类似的东西）循环，因为我每次只得到一个结果，即使我在CSV文件中有多个条目

Answer 1

如果这是常规CSV文件，则不应尝试自行解析。使用标准库csv module。

以下是文档的简短示例：

import csv
reader = csv.reader(open("some.csv", "rb"))
for row in reader:
    print row

Answer 2

您的计划中至少有两个错误：

curtime = time.strftime("%H")
...
for hour in range(int(s), int(f)):
    nrt.append(hour)
# this is an inefficient synonym for
# nrt = range(int(s), int(f))

if curtime in nrt:
    ...

首先，curtime是一个字符串，而nrt是一个整数列表。 Python是强类型的，因此两者不可互换，并且不会比较相等：

'4' == 4 # False
'4' in [3, 4, 5] # False

此修订后的代码解决了这个问题，并且比生成列表和搜索当前小时更有效：

cur_hour = time.localtime().tm_hour
if int(s) <= cur_hour < int(f):
    # You can "chain" comparison operators in Python
    # so that a op1 b op2 c is equivalent to a op1 b and b op2c
    ...

上述问题没有解决的第二个问题是，如果小时数在午夜左右（例如s = 22和f = 8），你的程序将无法正常运行。

这些问题都不一定与“脚本只能通过其中一条线路循环”有关，但你没有给我们足够的信息来弄清楚为什么会这样。提出问题的一种更有用的方法是发布简短但完整代码段，其中显示您正在观察的行为，以及示例输入和生成的错误消息（如果有）（以及追溯）。

Answer 3

你有没有尝试过更简单的东西？只是为了看看Python如何实际读取您的文件：

db = open("db.csv")  
for line in db:  
    print line

csv文件的格式可能有问题。例如，当您在Windows环境中打开Unix文件时，就会发生这种情况。在这种情况下，整个文件看起来像单个字符串，因为Windows和Unix有不同的行分隔符。所以，我不知道你的问题的某些原因，但提出要朝这个方向思考。

<强>更新你的循环体有多种方式：

当s为"no"时："They always want alerts, sending email"将被打印。
当s不是"no"且curtime in nrt时："They don't want an alert during the current hour, not sending"将被打印。
当s不是"no"且curtime in nrt为假（最后else）时：无将被打印，并且没有采取其他行动

您不应该在最后print分支中放置一些else语句吗？

此外，您的代码段的确切输出是什么？是"They always want alerts, sending email"吗？

Answer 4

我会检查条件中的逻辑。你循环构造应该工作。

Answer 5

明确连续的内容。使用0,1,2 ... n实际上是你的错误，并且它使代码在将来很难为自己或其他人阅读。因此，让我们使用方便的元组来显示我们从一行中期待的东西。这种工作就像代码作为文档

db = open("db.csv")
for line in db.readlines():
    recipient, start_hour, end_hour = line.split(",")
    nrt = []
    etc...

这向您的代码的读者显示了您希望包含一行的内容，并且在您第一次运行它时它会向您显示您的错误：）

Answer 6

您可以在Python Download

中查看现有的写得很好的IRC机器人

帮助Python循环怪异？

6 个答案: