Разве кто-то в здравом уме станет присваивать что-то самому себе? И, если уж так вышло, не является ли это ошибкой? И не лучше ли тогда вместо

MyClass& operator=(const MyClass& other) {
    if(this != &other) {
         * * *
    }
}

писать

MyClass& operator=(const MyClass& other) {
    assert(this != &other)
    * * *
}