Использование «is» для именования булевых переменных - плохая практика?

Question

Является ли присвоение логических значений, начинающихся с «это», плохой практикой сейчас? Мой менеджер считает, что «isAnything» является устаревшей и плохой практикой. Это правда?

myManager.isLame ? correct() : incorrect();

Answer 1

Он используется довольно часто на многих языках, но я не знаю, можно ли с уверенностью сказать, что это предпочтительный метод.

Я думаю, что важно иметь в виду последовательность и то, что все в данной команде используют одни и те же стандарты / стили.

Answer 2

Я бы не стал здесь использовать жесткие и быстрые правила.Хотя я нахожу такой префикс, как «Is», полезным для определения логического свойства, во многих случаях «Is» не будет лучшим выбором.

• Car.HasFlatTyre против Car.IsFlatTyre
• Cars.AreAllRed против Cars.IsAllRed
• и т. Д. *

Рекомендации по присвоению имен MSDN включают следующие соответствующие рекомендации.

Назовите булевы свойства утвердительной фразой (CanSeek вместо CantSeek).При желании вы также можете добавить префикс логических свойств к Is, Can или Has, но только там, где это добавляет значение.

Answer 3

isLame() очень распространено, и я считаю, что это не хромая.В Java это часть спецификации JavaBeans и, следовательно, довольно сложная практика.

Answer 4

Это вопрос стиля, и я видел это по-твоему много раз (и делаю это сам на многих языках).

Answer 5

Стилистически мой голос будет за hasValue или isNullOrEmpty.Использование умных ярлыков или однострочных, если такие заявления, однако, повсеместно плохо.Это значительно снижает читабельность кода и в большинстве языков не приводит к увеличению производительности.

Answer 6

Следуйте языковым соглашениям.Если нет соглашения:

Информация о типе пропуска в именовании переменных.

Включает is для логических значений.

boss.lame ? limp() : sprint()

Языки со строгой типизацией информации являются избыточными.

Для языков без строгой типизации информация по-прежнему избыточна, поскольку IDE и все доступные инструменты теперь помогают при наборе текста без необходимости запутывать имена.*is это глагол.is_lame() должен быть методом доступа, который возвращает логическое значение.

Answer 7

было бы лучше, если бы вы создали логическую переменную с чистым именем

boolean lame;

и сделайте метод для проверки его значения

isLame(){
  return lame;
}

Как правило, лучше вызывать метод через прямой доступ к переменной.