Чи краще викликати функцію, яка не має ефекту в цей момент, якщо це покращує ясність коду?

У мене в програмі три програми (програма iOS). Лише одна з них є активною одночасно, тому я вимикаю видимість для двох з них і перемикаю видимість, коли користувач натискає кнопки. Погляди ініціалізуються як видимі, тому я вимикаю видимість у коді до того, як буде показано основний вигляд.

я можу зробити

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

для двох переглядів, але тепер третій (той, який повинен бути видимий на початку програми) не адресується. Я поставив

[view3 setAlpha:1.0f];

після перших двох, тому що я вважаю, що це ясно, що насправді три погляди, а не два, як можна було б подумати, побачивши код. Як це роблять інші програмісти? Це суто перевагу чи є якісь умовності?

Якщо дзвінок дуже важкий, очевидно, краще не телефонувати, коли це не потрібно, але мені було цікаво про такі дрібні речі, як мій приклад.

У вас інваріант:

Лише один перегляд (із 3) завжди активний (і видимий).

Тоді я пропоную вам надати функцію одночасно перемикати активність та видимість ВСІХ переглядів:

[setActiveView viewID:2]

Ця функція:

• перевірте, чи перегляд вже активний, уникаючи зайвої роботи
• встановити подання як активне та видиме
• встановіть інші 2 представлення як неактивні та невидимі

Він має багато переваг щодо необмеженого дзвінка setVisibility:

• доброзичливий: зателефонувавши до нього, це не створює проблем із продуктивністю
• оборонний: його єдиний параметр набагато складніше отримати, тоді як для setVisibilityцього важче запам’ятати, що діапазон значень є 0.0f - 1.0fі що лише один повинен бути встановлений1.0f
• стійкий: наступний хлопець не може випадково забути один із поглядів
• адаптується: додавання / видалення подання не вимагає ретельного вивчення всього коду програми, щоб знайти місце перемикачів, одну функцію (цю) потрібно оновити

В ідеалі, щоб допомогти застосувати інваріант, жодна інша функція не повинна мати змогу зіпсувати цей параметр ...

Альтернативна ідея: якщо ваша мета полягає в тому, щоб запобігти помилкам , що походять з - за люди забувають , що є три види , і зробити що - то з тільки два з них , що вони дійсно повинні робити з усіма з них, а потім зробити функцію , яка робить неможливим забути:

setViewVisibilities(0.0f, 0.0f, 1.0f)

Тепер у вас є щось набагато потужніше - час складання гарантує, що ви не забули . Якщо ви забудете параметр, компілятор буде кричати на вас. Це набагато корисніше, ніж коментарі чи непотрібний код, оскільки він створює суворий протокол з назвою, який застосовує властивість, про яку ви дбаєте.

Для випадку , коли view3не потрібно це видимість змінилася, ви можете додати деякий поведінку , де проходять спеціальне значення , як -1.0і nilчи що - то вздовж цих ліній означає «не змінити видимість перегляду на всіх». Це обходить проблему встановлення видимості без необхідності.

Я вважаю, що найкраще додавати коментар, що пояснює, що дзвінок є непотрібним (і чому).

^{(можливо, той факт, що дзвінок непотрібний, або що вам потрібен коментар з цього приводу, може бути запахом коду)}

У цьому конкретному випадку @Mattieu M. має правильне рішення.

У більш загальному випадку, коли не існує ніякого подібного перетворення, ви повинні запитати себе: Чи є якийсь - небудь шанс , майбутній програміст може зіпсувати це?

Відповідь, як правило, так. Що означає, так, вам слід додати дзвінок. Можливо, якась майбутня версія фреймворку починається з усіх переглядів OFF, а не ON.