Является ли перегрузка логических операций хорошей практикой или плохой практикой?

Это плохая практика в общем случае.

Причины:

• Неожиданное поведение: Перегруженные && и || теряют свойство ленивых вычислений (short-circuit evaluation). Оба операнда будут вычислены всегда, что может привести кSide effects и падению производительности.
• Потеря интуитивности: Пользователи ожидают стандартного поведения булевых операторов. Перегрузка делает код менее читаемым и предсказуемым.
• Сложности с отладкой: Нестандартное поведение операторов затрудняет поиск ошибок.

Исключения (редкие случаи):

• Типы, представляющие логические выражения: Например, в библиотеках для работы с символьными вычислениями или логическими схемами. В этом случае перегрузка может быть оправдана для построения выражений.
• Доменно-специфичные языки (DSL): Иногда перегрузка используется для создания более выразительного синтаксиса в узкоспециализированных областях.

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

Гораздо предпочтительнее использовать именованные функции для выполнения операций, которые не соответствуют стандартной логике булевых операторов.

// Пример плохой практики
class MyBool {
public:
    bool value;
    MyBool(bool v) : value(v) {}

    bool operator&&(const MyBool& other) const {
        // Оба операнда будут вычислены, даже если this->value == false
        return value && other.value;
    }

    bool operator||(const MyBool& other) const {
        // Оба операнда будут вычислены, даже если this->value == true
        return value || other.value;
    }
};

// Пример хорошей практики (использование именованных функций)
class SafeBool {
public:
    bool value;
    SafeBool(bool v) : value(v) {}

    bool and_op(const SafeBool& other) const {
        return value && other.value; // Здесь short-circuit работает на встроенном типе bool
    }

    bool or_op(const SafeBool& other) const {
        return value || other.value; // Здесь short-circuit работает на встроенном тиме bool
    }
};