Замечания по MyString

[alexey-malov]

    char& operator[](size_t index) const;

• [x] Оператор позволяет изменить содержимое константной строки. Нужны 2 версии этого оператора

[alexey-malov]

1>c:\teaching\oop-2019\yambakov\oop\lab5\mystring\cmystring.cpp(10): warning C4267: 'return': conversion from 'size_t' to 'int', possible loss of data

• [x] Программа должна компилироваться без предупреждений компилятора во всех конфигурациях и платформах.

[alexey-malov]

int CompareStrings(const CMyString& a, const CMyString& b)
{
    size_t aLen = a.GetLength();
    size_t bLen = b.GetLength();

    int result = memcmp(a.GetStringData(), b.GetStringData(), std::min(aLen, bLen));
    return (result == 0) ? (aLen - bLen) : result;
}

• [x] Если длина 1 строки будет равна 2^32+1, а второй строки 1, И первые символы будут одинаковыми, то memcmp сравнит только первый символ и вернёт 0, а затем результат (aLen-bLen) будет усечён с 2^32 до 0 и функция ошибочно вернёт нам информацию о том, что строки равны.

[alexey-malov]

CMyString::CMyString(CMyString&& other)
{
    m_string = std::move(other.m_string);
    m_size = std::move(other.m_size);
    other.m_string = nullptr;
}

• [x] Строка-донор будет иметь прежнюю длину

[alexey-malov]

void CMyString::Clear()
{
    delete[] m_string;
    m_string = new char[1]{ "" };
    m_size = 0;
}

• [x] Выбрасывание исключения в new оставит строку в невалидном состоянии - при разрушении произойдёт неопределённое поведение

[alexey-malov]

CMyString operator+(const CMyString& a, const CMyString& b)
{
    CMyString result(a);
    result.Append(b);

    return result;
}

• [x] Почему бы не выполнить за одну операцию выделения памяти память под хранение суммы двух строк?

[alexey-malov]

• [x] Для метода Append написать тест, проверяющий добавление строки к самой себе

[alexey-malov]

bool operator==(const CMyString& a, const CMyString& b)
{
    return CompareStrings(a, b) == 0;
}

• [ ] CompareStrings сравнивает содержимое строк. Если строки имеют разную длину, то нет смысла сравнивать их содержимое в этом операторе и операторе !=

[alexey-malov]

bool operator==(const CMyString& a, const CMyString& b)
{
    return true;
    //CompareStrings(a, b) == 0;
}

bool operator!=(const CMyString& a, const CMyString& b)
{
    return true; //!(a == b);
}

bool operator<(const CMyString& a, const CMyString& b)
{
    return true;
    //CompareStrings(a, b) < 0;
}

bool operator>(const CMyString& a, const CMyString& b)
{
    return true;
    //CompareStrings(a, b) > 0;
}

bool operator<=(const CMyString& a, const CMyString& b)
{
    return true;
    //!(CompareStrings(a, b) > 0);
}

bool operator>=(const CMyString& a, const CMyString& b)
{
    return true;
    //!(CompareStrings(a, b) < 0);
}

• [x] Вот в таком виде все тесты успешно проходят. Их либо нет для операторов сравнения, либо они не проверяют работу операторов

[alexey-malov]

• [x] Операция вычитания итератора из difference_type не имеет смысла. Удалить

[alexey-malov]

• [x] Добавить возможность создать const_iterator из iterator

[alexey-malov]

template <typename Type>
inline bool CMyStringIterator<Type>::operator>(const CMyStringIterator& it) const
{
    return true; // m_ch > it.m_ch;
}

template <typename Type>
inline bool CMyStringIterator<Type>::operator<(const CMyStringIterator& it) const
{
    return true; //m_ch < it.m_ch;
}

template <typename Type>
inline bool CMyStringIterator<Type>::operator>=(const CMyStringIterator& it) const
{
    return true; //m_ch >= it.m_ch;
}

template <typename Type>
inline bool CMyStringIterator<Type>::operator<=(const CMyStringIterator& it) const
{
    return true; //m_ch <= it.m_ch;
}

• [x] Тесты операций сравнения итераторов проходят даже с такими реализациями