Что выбрать: наглядность или краткость логических условий?

Question

[AVAtarMod]

Есть два варианта написания логических условий с указателями в C++
1)

while (a_ptr || b_ptr) {
        uint currentDataA = (a_ptr) ? a_ptr->data : 0;
        uint currentDataB = (b_ptr) ? b_ptr->data : 0;
        ....
}

2)

while (a_ptr != nullptr || b_ptr != nullptr) {
        uint currentDataA = (a_ptr != nullptr) ? a_ptr->data : 0;
        uint currentDataB = (b_ptr != nullptr) ? b_ptr->data : 0;
        ....
}

Раньше я обычно выбирал 2 случай, но теперь я вижу,что наверное лучше использовать неявное преобразование типов? Какие у Вас мысли на этот счет?

Answer 1

[Василий Банников]

Явное лучше, чем неявное.
Код чаще читается, чем пишется.
Так что лучше второй вариант.

Comments

[Adamos]

Не думаю, чтобы у кого-то часто читающего код на С++ первый вариант вызывал какие-то сомнения. Скорее второй избыточен для этого конкретного случая.
А если оно в коде достаточно активно используется - я бы вообще вынес это дело в отдельную функцию, где этот случай проще будет отлавливать и дебажить, если что.

[Wataru]

Я тоже считаю, что тут первый случай чуть-чуть предпочтительнее. Ибо из названия видно, что это указатель и его появление в логических усовиях - очевидно проверка на не nullptr.

[Василий Банников]

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