Làm thế nào để tuyên bố này có thể được đơn giản hóa?

Question

Làm thế nào để tuyên bố này có thể được đơn giản hóa? Nó tạo dấu cộng: http://i.stack.imgur.com/PtHO1.png

Nếu câu lệnh được hoàn thành, thì khối được đặt theo tọa độ x và y.

for y in range(MAP_HEIGHT): 
    for x in range(MAP_WIDTH): 
     if (x%5 == 2 or x%5 == 3 or x%5 == 4) and \ 
      (y%5 == 2 or y%5 == 3 or y%5 == 4) and \ 
      not(x%5 == 2 and y%5 == 2) and \ 
      not(x%5 == 4 and y%5 == 2) and \ 
      not(x%5 == 2 and y%5 == 4) and \ 
      not(x%5 == 4 and y%5 == 4): 
      ... 

Answer 1

Điều này tương tự:

if (x % 5 == 3 and y % 5 > 1) or (y % 5 == 3 and x % 5 > 1): 

Answer 2

Có hai tầm thường sửa:

• cache kết quả của x % 5 và y % 5
• Sử dụng in hoặc xích < để kiểm tra các giá trị:

Bên cạnh đó, các thử nghiệm cho <= 4 (hoặc < 5) thực sự thừa vì mỗi giá trị của lx và ly sẽ < 5.

for y in range(MAP_HEIGHT): 
    for x in range(MAP_WIDTH): 
     lx = x % 5 # for local-x 
     ly = y % 5 # for local-y 
     if lx > 1 and y > 1 and \ 
      not (lx == 2 and ly == 2) and \ 
      not (lx == 4 and ly == 2) and \ 
      not (lx == 2 and ly == 4) and \ 
      not (lx == 4 and ly == 4): 

Hoặc bạn chỉ có thể giữ một danh sách của các bộ thực sự cho phép:

cross_fields = [(2, 3), (3, 2), (3, 3), (3, 4), (4, 3)] 

for y in range(MAP_HEIGHT): 
    for x in range(MAP_WIDTH): 
     if (x % 5, y % 5) in cross_fields: 

Answer 3

Xây dựng về câu trả lời Konrad, bạn có thể đơn giản hóa nó hơn nữa :

for y in range(MAP_HEIGHT): 
    for x in range(MAP_WIDTH): 
     lx = x % 5 # for local-x 
     ly = y % 5 # for local-y 
     if (1 < lx < 5 and 1 < y < 5 and 
      (lx, ly) not in ((2, 2), (4, 2), (2, 4), (4, 2))): 

Answer 4

trả lời thứ hai của Konrad: -

cross_fields = [(2, 3), (3, 2), (3, 3), (3, 4), (4, 3)] 

for y in range(MAP_HEIGHT): 
    for x in range(MAP_WIDTH): 
    if (x % 5, y % 5) in cross_fields: 

có lẽ là tốt nhất.

Tuy nhiên, tôi sẽ góp phần: -

for y in range(MAP_HEIGHT): 
    for x in range(MAP_WIDTH): 
    lx = x % 5 
    ly = y % 5 
    if (lx > 1 and ly == 3) or (ly > 1 and lx == 3): 

Answer 5

Về cơ bản bạn đang ốp lát một mô hình nhị phân 5x5. Đây là một biểu hiện rõ ràng về điều đó:

pattern = [[0, 0, 0, 0, 0], 
      [0, 0, 0, 0, 0], 
      [0, 0, 0, 1, 0], 
      [0, 0, 1, 1, 1], 
      [0, 0, 0, 1, 0]] 

for y in range(MAP_HEIGHT): 
    for x in range(MAP_WIDTH): 
     if pattern[x%5][y%5]: 
      ... 

Cách tiếp cận rất đơn giản và chung này sẽ cho phép dễ dàng sửa đổi mẫu.

Answer 6

Giải pháp chung để tối ưu hóa chức năng logic như thế này là Karnaugh map. Bảng chân lý của bạn sẽ là hình chữ nhật cộng với hình dạng bạn muốn với các hàng và cột là các bài kiểm tra mô đun của bạn.