Tại sao việc sử dụng len (SEQUENCE) trong các giá trị điều kiện được Pylint xem là không chính xác?

Xem xét đoạn mã này:

from os import walk

files = []
for (dirpath, _, filenames) in walk(mydir):
    # more code that modifies files
if len(files) == 0: # <-- C1801
    return None

Tôi đã bị báo động bởi Pylint với thông báo này liên quan đến dòng với câu lệnh if:

[pylint] C1801: Không sử dụng len(SEQUENCE)làm giá trị điều kiện

Quy tắc C1801, thoạt nhìn, nghe có vẻ không hợp lý lắm và định nghĩa trên hướng dẫn tham khảo không giải thích được tại sao đây là vấn đề. Trong thực tế, nó hoàn toàn gọi nó là sử dụng không chính xác .

len-as-condition (C1801) : Không sử dụng len(SEQUENCE)làm giá trị điều kiện Được sử dụng khi Pylint phát hiện việc sử dụng len (chuỗi) không chính xác trong các điều kiện.

Những nỗ lực tìm kiếm của tôi cũng đã thất bại trong việc cung cấp cho tôi một lời giải thích sâu sắc hơn. Tôi hiểu rằng thuộc tính độ dài của chuỗi có thể được đánh giá một cách lười biếng và __len__có thể được lập trình để có tác dụng phụ, nhưng có một câu hỏi đặt ra là liệu một mình có đủ vấn đề để Pylint gọi việc sử dụng đó không chính xác hay không. Do đó, trước khi tôi đơn giản cấu hình dự án của mình để bỏ qua quy tắc, tôi muốn biết liệu tôi có thiếu điều gì trong lý luận của mình không.

Khi nào việc sử dụng len(SEQ)như là một giá trị điều kiện có vấn đề? Những tình huống chính nào mà Pylint đang cố gắng tránh với C1801?

Khi nào việc sử dụng len(SEQ)như là một giá trị điều kiện có vấn đề? Những tình huống chính nào mà Pylint đang cố gắng tránh với C1801?

Nó không thực sự có vấn đề khi sử dụng len(SEQUENCE)- mặc dù nó có thể không hiệu quả (xem bình luận của chepner ). Bất kể, Pylint kiểm tra mã để tuân thủ hướng dẫn kiểu PEP 8 , trong đó nêu rõ rằng

Đối với các chuỗi, (chuỗi, danh sách, bộ dữ liệu), sử dụng thực tế là các chuỗi trống là sai.

Yes: if not seq:
     if seq:

No:  if len(seq):
     if not len(seq):

Là một lập trình viên Python thỉnh thoảng, người bỏ qua giữa các ngôn ngữ, tôi coi len(SEQUENCE)cấu trúc này dễ đọc và rõ ràng hơn (Tiết lộ rõ ​​ràng là tốt hơn sau đó ẩn ý). Tuy nhiên, bằng cách sử dụng thực tế là một chuỗi trống đánh giá Falsetrong bối cảnh Boolean được coi là nhiều hơn nữa.

Lưu ý rằng việc sử dụng len (seq) trên thực tế là bắt buộc (thay vì chỉ kiểm tra giá trị bool của seq) khi sử dụng mảng NumPy.

a = numpy.array(range(10))
if a:
    print "a is not empty"

dẫn đến một ngoại lệ: ValueError: Giá trị thật của một mảng có nhiều hơn một phần tử là không rõ ràng. Sử dụng a.any () hoặc a.all ()

Và do đó đối với mã sử dụng cả danh sách Python và mảng NumPy, thông báo C1801 ít hữu ích hơn.

Đây là một vấn đề trong môn vị, và nó không còn được xem xét len(x) == 0 là không chính xác.

Bạn không nên sử dụng trần len(x) như một điều kiện. So sánh len(x)với một giá trị rõ ràng, chẳng hạn như if len(x) == 0củaif len(x) > 0 là hoàn toàn tốt đẹp và không bị cấm bởi PEP 8.

Từ PEP 8 :

# Correct:
if not seq:
if seq:

# Wrong:
if len(seq):
if not len(seq):

Lưu ý rằng việc kiểm tra rõ ràng về độ dài không bị cấm. Các Zen của Python khẳng định:

Rõ ràng là tốt hơn so với ngầm.

Trong sự lựa chọn giữa if not seqvà if not len(seq), cả hai đều ngầm nhưng hành vi là khác nhau. Nhưng if len(seq) == 0hayif len(seq) > 0 là so sánh rõ ràng và trong nhiều bối cảnh hành vi chính xác.

Trong pylint, PR 2815 đã sửa lỗi này, lần đầu tiên được báo cáo là vấn đề 2684 . Nó sẽ tiếp tục phàn nàn về if len(seq), nhưng nó sẽ không còn phàn nàn về if len(seq) > 0. Bản PR đã được hợp nhất 2019-03-19 vì vậy nếu bạn đang sử dụng pylint 2.4 (phát hành 2019-09-14), bạn sẽ không thấy vấn đề này.

Pylint đã thất bại cho mã của tôi và nghiên cứu đã dẫn tôi đến bài viết này:

../filename.py:49:11: C1801: Do not use `len(SEQUENCE)` to determine if a sequence is empty (len-as-condition)
../filename.py:49:34: C1801: Do not use `len(SEQUENCE)` to determine if a sequence is empty (len-as-condition)

Đây là mã của tôi trước đây:

def list_empty_folders(directory):
"""The Module Has Been Build to list empty Mac Folders."""
for (fullpath, dirnames, filenames) in os.walk(directory):
    if len(dirnames) == 0 and len(filenames) == 0:
        print("Exists: {} : Absolute Path: {}".format(
            os.path.exists(fullpath), os.path.abspath(fullpath)))

Đây là sau khi sửa mã của tôi. Bằng cách sử dụng int() attribute, tôi dường như đã thỏa mãn Pep8 / Pylint và dường như không có tác động tiêu cực đến mã của tôi:

def list_empty_folders(directory):
"""The Module Has Been Build to list empty Mac Folders."""
for (fullpath, dirnames, filenames) in os.walk(directory):
    if len(dirnames).__trunc__() == 0 and len(filenames).__trunc__() == 0:
        print("Exists: {} : Absolute Path: {}".format(
            os.path.exists(fullpath), os.path.abspath(fullpath)))

Sửa chữa của tôi

Bằng cách thêm .__trunc__()vào chuỗi, nó dường như đã giải quyết nhu cầu.

Tôi không thấy sự khác biệt trong hành vi, nhưng nếu có ai biết chi tiết cụ thể mà tôi đang thiếu, xin vui lòng cho tôi biết.