Логическое сравнение == Перегрузка оператора

Question

Мне нужно сделать некоторое логическое сравнение и вернуть логический ответ.

Вот код из файла .cpp:

bool MyString::operator==(const MyString& other)const 
{ 
    if(other.Size == this.Size) 
    { 
      for(int i = 0; i < this.Size+1; i++) 
      { 
        if(this[i] == other[i]) 

          return true; 
      } 
    } 
    else 
      return false; 
} 

Вот что называется из main.cpp файла:

if (String1 == String4) 
{ 
    String3.Print(); 
} 
else 
{ 
    String4.Print(); 
} 

Здесь есть ошибки компиляции я получаю:

error: request for member `Size` in `this`, which is of non-class type `const MyString* const` 
error: no match for `operator[]` in `other[i]` 

Answer 1

Проблемы с кодом:

• this[i]: Вы, видимо, хотите получить доступ к г-й символ строки здесь. Это не так. Предполагая, что ваш класс перегружает operator[], вы хотите (*this)[i]. Кроме того, вы можете напрямую получить доступ к внутреннему представлению строки.

• if(this[i] == other[i]) return true;: Подумайте, что это значит относительно сравнения строк «A1» и «AB».

• for() {...}: Что происходит, когда вы выходите из цикла? Вам нужно что-то вернуть, если сравнения удастся выполнить через цикл, не возвращаясь.

Answer 2

this - указатель, поэтому вы должны его разыменовать:

this->Size; 

Кроме того, я думаю, что логика вашего operator== несовершенен - ​​вот, он возвращает true, если какой-либо из символов равно символа на той же позиции во второй строке. Изменение цикла для

 for(int i = 0; i < this->Size+1; i++) 
     { 
       if(this[i] != other[i]) 

         return false; 
     } 

и поставить return true; вместо последней части вашего кода (else п) для сравнения целых строк.

Как уже упоминалось Сет, вы не можете использовать operator[] на this как описано выше, - таким образом, он рассматривается как массив (т.е. this[i] действительно *(this + i) - так не то, что вы думаете, что это). Вместо этого выберите свой внутренний элемент хранения.

Answer 3

Вы не указали, можете ли вы использовать стандартные алгоритмы C++ или нет. Здесь иллюстрировали обе версии, используя рукописную петлю и std::equal алгоритм:

//#define USE_STD_ALGORITHM 1 // uncomment to enable std::equal version 
#include <cassert> 
#include <algorithm> 
#include <stdexcept> 

// NOTE: partial simplest definition for the test and presentation purposes only. 
struct MyString 
{ 
    MyString(char const* s, std::size_t size) : data(s), Size(size) {} 
    char const& operator[](std::size_t index) const; 
    bool operator==(const MyString& other) const; 
private: 
    char const* data; 
    std::size_t Size; 
}; 

char const& MyString::operator[](std::size_t index) const 
{ 
    if (index < Size) 
     return data[index]; 
    throw std::out_of_range("index invalid"); 
} 

bool MyString::operator==(const MyString& other) const 
{ 
    if (this->Size == other.Size) 
    { 
#ifdef USE_STD_ALGORITHM 
     return std::equal(data, data+Size, other.data); 
#else 
    bool equal = true; 
    for(std::size_t i = 0; i < this->Size; ++i) 
    { 
     if((*this)[i] != other[i]) 
     { 
      equal = false; 
      break; 
     } 
    } 
    return equal; 
#endif 
    } 

    return false; 
} 

int main() 
{ 
    char const* a = "abc"; 
    char const* b = "abc"; 
    MyString sa(a, 3); 
    MyString sb(b, 3); 
    assert(sa == sb); 

    char const* c = "adc"; 
    MyString sc(c, 3); 
    assert(!(sa == sc)); 

    char const* d = "ab"; 
    MyString sd(d, 2); 
    assert(!(sa == sd)); 
} 

Удачи!