__iter __() thực hiện như một máy phát điện

Question

Tôi có một lớp đối tượng mà thực hiện một cử động __ iter __ sử dụng một máy phát bộ nhớ đệm (Tôi cũng có một phương pháp vô hiệu các bộ nhớ cache ITER) như sau:

def __iter__(self): 
    print("iter called") 
    if self.__iter_cache is None: 
     iter_seen = {} 
     iter_cache = [] 
     for name in self.__slots: 
      value = self.__slots[name] 
      iter_seen[name] = True 
      item = (name, value) 
      iter_cache.append(item) 
      yield item   
     for d in self.__dc_list: 
      for name, value in iter(d): 
       if name not in iter_seen: 
        iter_seen[name] = True 
        item = (name, value) 
        iter_cache.append(item) 
        yield item 
     self.__iter_cache = iter_cache 
    else: 
     print("iter cache hit") 
     for item in self.__iter_cache: 
      yield item 

Dường như với được làm việc ... Có bất kỳ gotchas tôi có thể không nhận thức được? Tôi có làm điều gì đó vô lý không?

Answer 1

Có vẻ như đó là một cách tiếp cận rất mong manh. Nó đủ để thay đổi bất kỳ __slots, __dc_list, __iter_cache nào trong quá trình lặp hoạt động để đưa đối tượng vào trạng thái không nhất quán.

Bạn cần cấm thay đổi đối tượng trong khi lặp hoặc tạo tất cả các mục trong bộ nhớ cache cùng một lúc và trả về một bản sao của danh sách.

Answer 2

container.__iter__() trả về đối tượng trình lặp. Các đối tượng iterator mình được yêu cầu để hỗ trợ hai phương pháp sau đây, mà cùng nhau tạo thành giao thức iterator:

iterator.__iter__() 

Trả về đối tượng iterator riêng của mình.

iterator.next() 

Trả lại mục tiếp theo từ vùng chứa.

Đó chính là điều mà mỗi máy phát có. Vì vậy, đừng sợ bất kỳ tác dụng phụ nào.

Answer 3

Nó có thể là tốt hơn để tách các iteration của đối tượng từ bộ nhớ đệm của các giá trị nó trả về. Điều đó sẽ đơn giản hóa quá trình lặp lại và cho phép bạn dễ dàng kiểm soát cách bộ nhớ đệm được thực hiện cũng như cho dù nó được kích hoạt hay không, ví dụ.

Một khác có thể là xem xét quan trọng là mã của bạn sẽ không tiên đoán xử lý tình huống mà đối tượng được lặp lại bị thay đổi giữa các lần gọi liên tiếp đến phương thức. Một cách đơn giản để giải quyết vấn đề đó là điền toàn bộ nội dung của bộ nhớ cache vào cuộc gọi đầu tiên và sau đó chỉ yield nội dung chứa trong mỗi cuộc gọi - và ghi lại hành vi.

Answer 4

Những gì bạn đang làm là hợp lệ mặc dù lạ. __slots hoặc __dc_list là gì ?? Nói chung tốt hơn là mô tả nội dung của đối tượng của bạn trong một tên thuộc tính, thay vì loại của nó (ví dụ: self.users chứ không phải self.u_list).

Bạn có thể sử dụng trang trí LazyProperty để đơn giản hóa điều này một cách đáng kể.

Chỉ cần trang trí phương pháp của bạn với @LazyProperty. Nó sẽ được gọi là lần đầu tiên, và người trang trí sau đó sẽ thay thế thuộc tính bằng các kết quả. Yêu cầu duy nhất là giá trị có thể lặp lại; nó không phụ thuộc vào trạng thái có thể thay đổi được. Bạn cũng có yêu cầu đó trong mã hiện tại của bạn, với tự của bạn .__ iter_cache.

def __iter__(self) 
    return self.__iter 

@LazyProperty 
def __iter(self) 
    def my_generator(): 
     yield whatever 
    return tuple(my_generator())