Sử dụng Python `isinstance` có đúng không?

Question

Tôi có một mảng 2D gồm các khối khác nhau, tất cả đều được kế thừa từ Chặn. Tôi muốn kiểm tra xem khối mà tôi nhấp vào là một khối loại Dirt, như thế này:

clickedblock = getClickedBlock() 

if isinstance(clickedblock, Dirt): 
    place a block 

else: 
    don't place a block 

Tôi đã nghe nói rằng isinstance là xấu, và nên tránh vì nó tạo ra dĩa trong mã. Những gì thời gian sẽ isinstance là tốt để sử dụng?

Một giải pháp rườm rà khác cho vấn đề của tôi là có một trường Block gọi là 'id' và sau đó kiểm tra xem nó có bằng một số hằng số có nghĩa là Dirt hay không. Nhưng điều đó nghe có vẻ khá tệ và dễ bị sai lầm hơn là đơn giản isinstance.

Answer 1

Ví dụ của bạn có vẻ giống như trường hợp sử dụng hợp pháp của isinstance().

Nó không phải là isinstance() là xấu, thường đa hình có thể được sử dụng cho cùng một mục đích (mà kết quả trong mã sạch hơn trong nơi lớp được sử dụng).

Nhưng đôi khi, isinstance() là những gì bạn cần. Ví dụ, cách pythonic phát hiện xem một biến là chuỗi hay không là isinstance(var, basestring).

Answer 2

tôi nghĩ rằng tôi muốn thay đổi nó trở nên giống như hơn:

PLACEABLE_TYPES = [ Dirt ] 
if isinstance(clickedblock, PLACEABLE_TYPES): 
    place the block 
else: 
    don't place the block 

nhưng ý tưởng trong các ý kiến ​​của:

if clickedblock.is_placeable(that_place): 
    place the block 
else: 
    don't place the block 

cũng có công đức.

Answer 3

Nếu bạn không muốn sử dụng, bạn có các tùy chọn khác. Truyền thống giải pháp vịt gõ:

try: 
    clickedblock_place = clickedblock.place 
except AttributeError: 
    # don't place block 
else: 
    clickedblock_place() 

Hoặc bạn có thể sử dụng hasattr:

if hasattr(clickedblock, 'place'): 
    clickedblock.place() 

Tôi hầu như không bao giờ sử dụng isinstance trừ kiểm tra lên (? Hoặc là nó xuống) hệ thống phân cấp thừa kế, chẳng hạn, ví dụ, nếu bạn cần phải biết nếu một tên trỏ đến một str HOẶC unicode:

if isinstance(str1, basestring): 
    blah, blah, blah 

may mắn, Mike