[已解决]迭代器

filk-shun

class leapyear:

    def __init__(self):
        self.year = 2019
        
    def __iter__(self):
        return self

    def __next__(self):
        while self.year:
            if (self.year % 4 == 0 and self.year % 100 != 0) or (self.year % 400 == 0):
                return self.year
                self.year = self.year - 1
            else:
                self.year = self.year - 1

>>> c= leapyear()
>>> for i in c:
        if i >= 2000:
                print(i)
        else:
                break

想问一下为什么这个代码输出的全是2016

最佳答案

月排行榜 / 总排行榜

jackz007

2019-5-3 22:23:50

      由于__next__() 每次开始时的 self . year 就是闰年，但却不是我们所需要的，所以，应该把 self . year 的减 1 操作安排在循环的入口处，这样，就会避免 self . year 无法步进的问题了。此外，楼主判断闰年的条件也是错误的，判断闰年不变的前提必须是 4 的倍数，在这个范围内，剔除100的倍数，保留400的倍数。

      楼主试试我修改的代码，看看是不是你想要的效果。

1. class LeapYear:
   
2.     def __init__(self , year = 2019):
   
3.         self . year = year
   
4.         
   
5.     def __iter__(self):
   
6.         return self
   
7. 
   
8.     def __next__(self):
   
9.         while True:
   
10.             self . year -= 1
    
11.             if self . year < 2000:
    
12.                 raise StopIteration
    
13.             else:
    
14.                 if (self . year % 4 == 0) and (self . year % 100 != 0 or self . year % 400 == 0):
    
15.                      break
    
16.         return self . year
    
17. 
    
18. c = LeapYear()
    
19. for i in c:
    
20.     print(i)

复制代码

跳转到最佳答案楼层

jackz007

      由于__next__() 每次开始时的 self . year 就是闰年，但却不是我们所需要的，所以，应该把 self . year 的减 1 操作安排在循环的入口处，这样，就会避免 self . year 无法步进的问题了。此外，楼主判断闰年的条件也是错误的，判断闰年不变的前提必须是 4 的倍数，在这个范围内，剔除100的倍数，保留400的倍数。

      楼主试试我修改的代码，看看是不是你想要的效果。

1. class LeapYear:
   
2.     def __init__(self , year = 2019):
   
3.         self . year = year
   
4.         
   
5.     def __iter__(self):
   
6.         return self
   
7. 
   
8.     def __next__(self):
   
9.         while True:
   
10.             self . year -= 1
    
11.             if self . year < 2000:
    
12.                 raise StopIteration
    
13.             else:
    
14.                 if (self . year % 4 == 0) and (self . year % 100 != 0 or self . year % 400 == 0):
    
15.                      break
    
16.         return self . year
    
17. 
    
18. c = LeapYear()
    
19. for i in c:
    
20.     print(i)

复制代码

_谪仙

你应该将return放在self.year = self.year - 1的后面，程序是不会执行return之后的语句