请问这样定义Python函数是否正确？附时间转换代码

函数定义方式的问题 #

你的代码里函数定义顺序有误：convert函数定义在main函数之后，当main函数内部调用convert时，Python解释器还没加载这个函数，会直接抛出NameError。正确的做法是把convert函数放在main函数前面定义。

代码里的其他问题 #

除了函数顺序，还有这些错误需要修正：

• 缩进错误：main函数内的所有语句都没缩进，Python靠缩进识别代码块，必须给time = input(...)、if判断这些语句加上缩进（一般用4个空格）。
• 语法错误：if convert(time) >= 7.0 and <= 8.0写法不合法，得明确两边的比较对象，改成7.0 <= convert(time) <= 8.0或者convert(time) >=7.0 and convert(time) <=8.0都可以。
• 类型错误：elif time >=12.0里的time是用户输入的字符串，不能直接和数值比较，得先把转换后的时间存成变量，再用这个变量做判断。
• 时间转换逻辑错了：convert函数里nm = int(min)/6完全不对，分钟转小时应该除以60（1小时=60分钟），比如30分钟是30/60=0.5，而且没必要把结果转成int，否则会丢失小数部分，直接返回int(hours) + int(mins)/60就行。

修正后的代码 #

def convert(time):
    hours, mins = time.split(":")
    return int(hours) + int(mins) / 60

def main():
    time = input("What time is it? ")
    converted_time = convert(time)
    if 7.0 <= converted_time <= 8.0:
        print("breakfast time")
    elif 12.0 <= converted_time <= 13.0:
        print("lunch time")
    elif 18.0 <= converted_time <= 19.0:
        print("dinner time")

if __name__ == "__main__":
    main()

内容的提问来源于stack exchange，提问作者Shaila De Guzman