Có phải tốt hơn để gọi một chức năng không có hiệu lực tại thời điểm đó, NẾU nó cải thiện độ rõ của mã?

Tôi có ba lượt xem trong chương trình của mình (ứng dụng iOS). Chỉ một trong số chúng hoạt động cùng một lúc nên tôi tắt chế độ hiển thị cho hai trong số chúng và chuyển chế độ hiển thị khi người dùng nhấn nút. Các khung nhìn được khởi tạo là có thể nhìn thấy được vì vậy tôi đặt chế độ hiển thị trong mã trước khi chế độ xem chính hiển thị.

tôi có thể làm

[view1 setAlpha:0.0f];
[view2 setAlpha:0.0f];

đối với hai trong số các chế độ xem, nhưng bây giờ quan điểm thứ ba (quan điểm sẽ hiển thị khi bắt đầu ứng dụng) không được đề cập. tôi đặt một

[view3 setAlpha:1.0f];

sau hai lần đầu tiên, vì tôi nghĩ rõ ràng rằng trên thực tế có ba quan điểm chứ không phải hai như người ta có thể nghĩ khi xem mã. Làm thế nào để các lập trình viên khác làm điều này? Đó có phải là ưu tiên hoàn toàn hay có một số quy ước?

Nếu cuộc gọi rất nặng, rõ ràng tốt hơn là không gọi nó khi không cần thiết, nhưng tôi đã tự hỏi về những điều nhỏ nhặt như ví dụ của tôi.

Bạn có một bất biến:

Chỉ một chế độ xem duy nhất (trong số 3) là hoạt động (và hiển thị).

Sau đó, tôi khuyên bạn nên cung cấp một chức năng để chuyển đổi hoạt động và khả năng hiển thị của TẤT CẢ các chế độ xem cùng một lúc:

[setActiveView viewID:2]

Chức năng này sẽ:

• kiểm tra xem chế độ xem đã hoạt động chưa, tránh các công việc không cần thiết
• đặt chế độ xem là hoạt động và hiển thị
• đặt 2 chế độ xem khác là không hoạt động và vô hình

Nó có nhiều lợi thế về một cuộc gọi thô đến setVisibility:

• thân thiện: gọi nó không cần thiết không tạo ra vấn đề hiệu suất
• phòng thủ: tham số duy nhất của nó khó khăn hơn nhiều để botch, trong khi setVisibilityđó để nhớ rằng phạm vi của các giá trị là khó hơn 0.0f - 1.0fvà chỉ có một giá trị phải được đặt thành1.0f
• kiên cường: chàng trai tiếp theo không thể vô tình quên một trong những quan điểm
• có thể thích ứng: thêm / xóa chế độ xem không yêu cầu xem xét kỹ tất cả mã ứng dụng để tìm vị trí của các công tắc, một chức năng (cái này) cần được cập nhật

Lý tưởng nhất là để giúp thực thi bất biến, không có chức năng nào khác có thể gây rối với cài đặt này ...

Ý tưởng thay thế: nếu mục tiêu của bạn là ngăn chặn các lỗi xảy ra do mọi người quên có ba chế độ xem và làm điều gì đó chỉ với hai trong số họ thực sự nên làm với tất cả chúng, thì hãy tạo một chức năng khiến bạn không thể quên:

setViewVisibilities(0.0f, 0.0f, 1.0f)

Bây giờ bạn có một thứ gì đó mạnh mẽ hơn nhiều - thời gian biên dịch đảm bảo bạn không bị lãng quên . Nếu bạn quên một tham số, trình biên dịch sẽ mắng bạn. Điều này hữu ích hơn nhiều so với các bình luận hoặc mã không cần thiết, vì nó tạo ra một giao thức có tên nghiêm ngặt để thực thi tài sản mà bạn quan tâm.

Đối với trường hợp view3không cần nó tầm nhìn thay đổi, bạn có thể thêm một số hành vi nơi đi qua một giá trị đặc biệt như -1.0hoặc nilhoặc một cái gì đó dọc theo những đường có nghĩa là "không thay đổi khả năng hiển thị xem ở tất cả". Điều này xoay quanh vấn đề thiết lập khả năng hiển thị không cần thiết.

Tôi sẽ tin rằng việc thêm một bình luận giải thích rằng cuộc gọi là không cần thiết (và tại sao) là tốt nhất.

^{(có lẽ, thực tế là một cuộc gọi là không cần thiết hoặc bạn cần nhận xét về điều đó, có thể là mùi mã)}

Trong trường hợp cụ thể này, @Mattieu M. có giải pháp phù hợp.

Trong trường hợp tổng quát hơn, khi không có biến đổi tương tự, bạn phải tự hỏi: Có khả năng nào một lập trình viên tương lai có thể làm hỏng điều này không?

Câu trả lời thường là có. Có nghĩa là, vâng, bạn nên thêm cuộc gọi. Có thể một số phiên bản tương lai của khung bắt đầu với tất cả các lượt xem TẮT thay vì BẬT.