0

Помогите найти ошибку в коде.

Задание глобальное:"Создать класс String".

На одном из этапов надо добавить в класс метод int String::length(), который возвращает длину строки и перегрузить для класса операцию [] так, чтобы можно было обращаться и менять элементы строки: char& operator[] (int). С помощью указанных методов поменять местами первый и последний символы в строке.

Пример:

На входе: "Lorem Ipsum has been"

На выходе: "norem Ipsum has beeL"

#include <iostream>
#include <cstring>
using namespace std;

class String{
    int len;
    char *str;
public:
    String (): len(0), str(nullptr){}
    String(const String&src);
    String(const char *str);
    ~String();
    int length(String &);
    char& operator [](int index){
        return str[index];
    }
    friend istream &operator >>(istream &in, String &s);
    friend ostream &operator <<(ostream &out, String s);
};
String::String(const String &src){
    len = src.len;
    this->str = new char[len+1];
    strncpy(str, src.str, len);
    this->str[len] = '\0';
}
String::String(const char*str){
    len = strlen(str);
    this->str = new char*[len+1];
    strncpy(this->str, str, len);
    this->str[len] = '\0';
}

String::~String(){
    delete [] str;
}


int String::length(String &s){
    len = strlen(s.str);
    return len;
}

istream &operator >>(istream &in, String &s){
    char *buf = new char[100001];
    in.getlin(buf, 100000);
    s.len = strlen(buf);
    s.str = new char[s.len];
    for(int i = 0; i < s.len; ++i)
        s.str[i] = buf[i];
    delete [] buf;
    return in;
}
ostream &operator <<(ostream &out, String s){
    for(int i = 0; i < s.len; ++i)
        out << s.str[i];
    return out;
}

int main(){
    String s;
    cin >> s;
    char tmp = s[s.length];
    s[s.length] = s[0];
    s[0] = tmp;
    cout << s;
    return 0;
}
8
  • Исправил this->str = new char*[len]; на this->str = new char*[len+1];
    – Trulimon
    27 фев 2019 в 12:12
  • Для чего хранится одновременно len и нуль-терминатор в буфере? В разных случаях используется то одно (len), то другое (строковые функции, которые опираются на нуль-терминаторы).
    – free_ze
    27 фев 2019 в 12:24
  • s[s.length] = s[0]; не скомпилируется (в классе нет ни поля с таким именем, ни перегрузки квадратных скобок) и в рантайме будут проблемы, ибо в конструкторах явно в эту позицию (подразумеваемая len) вставлятеся нуль-терминатор.
    – free_ze
    27 фев 2019 в 12:29
  • Отсутствует деструктор - память не освобождается.
    – free_ze
    27 фев 2019 в 12:30
  • По поводу len и буфер не понял вопроса. Почему неправильно s[s.length] = s[0] ? Деструктор сейчас добавлю.
    – Trulimon
    27 фев 2019 в 12:32

1 ответ 1

2

Кое-что несколько подзапущено...

Вот, смотрите. Я бы еще добавил запрет выхода за границу массива в операторе [].

#include <iostream>
#include <cstring>
using namespace std;

class String
{
    size_t len;
    char *str;
public:
    String (): len(0), str(nullptr){}
    String(const String&src);
    String(const char *str);
    ~String() { delete[] str; }
    size_t length() const { return len; };
    char& operator [](size_t index)
    {
        return str[index];
    }
    char operator [](size_t index) const
    {
        return str[index];
    }
    friend istream &operator >>(istream &in, String &s);
    friend ostream &operator <<(ostream &out, const String& s);
};

String::String(const String &src)
{
    len = src.len;
    this->str = new char[len+1];
    strcpy(str, src.str);
}

String::String(const char*s)
{
    len = strlen(s);
    str = new char[len+1];
    strcpy(str, s);
}

istream &operator >>(istream &in, String &s)
{
    char *buf = new char[100001];
    in.getline(buf, 100000);
    delete[] s.str;
    s.len = strlen(buf);
    s.str = new char[s.len+1];
    strcpy(s.str,buf);
    delete [] buf;
    return in;
}
ostream &operator <<(ostream &out, const String& s)
{
    return out << s.str;
}

int main()
{
    String s;
    cin >> s;
    char tmp = s[s.length()-1];
    s[s.length()-1] = s[0];
    s[0] = tmp;
    cout << s;
    return 0;
}
6
  • @Harry - добавляя проверку выхода за границы в оператор [], вы нарушаете соглашение стандарта C++ о том, что [] не проверяют границы (их проверяет at()) - и таким образом, учите молодого человека писать плохо поддерживаемые программы.
    – gbg
    27 фев 2019 в 13:42
  • @gbg Учебный класс вы называете стандартом? :) И еще - покажите это место в стандарте, что оператор [] не должен проверять выход за границы. Не в vector или там string, а в самопальных вещах. А потом идите спорить со Страуструпом, который даже в вектор этот функционал засовывает в своих Программировании... или A Tour of C++ :)
    – Harry
    27 фев 2019 в 13:47
  • Ваш хамский тон и посылание во все стороны плохо сочетаются с ублюдошными смайлами. Стандартная библиотека ведет себя очень просто [] - не проверяют диапазон, at() - проверяет. Если джуны начнут совать в свои реализации [] проверку на диапазон (вместо того, чтобы назавать это at()) - путаницы будет только больше.
    – gbg
    27 фев 2019 в 14:18
  • @gbg, причем тут стандарт и какую путаницу может создасть проверка(проверка, это не обьязательно генерация иключения)? И кто мешает кому то написать свой собственный string для удобства выполнения проекта, совершенно не похожего на стандартный?... 27 фев 2019 в 14:29
  • @gbg Ну, если по существу вам нечего сказать, на этом и остановимся. Пар выпустили? :)
    – Harry
    27 фев 2019 в 15:03

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.