Форум программистов, компьютерный форум CyberForum.ru

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
Drusha
6 / 6 / 0
Регистрация: 16.07.2013
Сообщений: 74
#1

Задача с ошибкой из книги (Шилтд) - C++

15.10.2013, 15:43. Просмотров 324. Ответов 6
Метки нет (Все метки)

Программа из книги Шилдт Г. С++ Базовый курс, 3-е издание,2010 (стр. 339) не работает.
Убрав ссылочный параметр из сток 15 и 35, вроде работает корректно. Вот и становиться интересно, это просто ошибка в книге или я что-то не правильно делаю?

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
#include<iostream>
#include<cstring>
#include<cstdlib>
using namespace std;
 
class sample
{
    char *s;
public: 
    sample();
    sample(const sample &ob);
    ~sample() { if(s) delete [] s; cout <<"S - free.\n";}
    void show() { cout << s << "\n";}
    void set(char *str);
    sample operator=(sample &ob);  //Если убрать сылочный параметр, программа заработает.
};
 
sample::sample()
{
    s = new char('\0');
}
 
sample::sample(const sample &ob)
{
    s = new char[strlen(ob.s)+1];
    strcpy(s, ob.s);
}
 
void sample::set(char *str)
{
    s = new char[strlen(str)+1];
    strcpy(s, str);
}
 
sample sample::operator=(sample &ob)  //Если убрать сылочный параметр, программа заработает.
{
    if(strlen(ob.s) > strlen(s)+1)
    {
        delete [] s;
        s = new char [strlen(ob.s)+1];
    }
    strcpy(s, ob.s);
    return *this;
}
 
sample input()
{
    char instr[80];
    sample str;
    
    cout << "Enter string: ";
    cin >> instr;
    str.set(instr);
    return str;
}
 
int main()
{
    sample ob;
    
    ob = input();
    ob.show();
    return 0;
}
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
15.10.2013, 15:43     Задача с ошибкой из книги (Шилтд)
Посмотрите здесь:

C++ Задача из книги Р.Лафоре содержит ошибки, исправьте пожалуйста
Задача из книги Страуструпа C++
Задача из книги Дейтел Как программировать на C++ C++
C++ Задача: написать программу электронной телефонной книги
Непонятная задача на массивы из книги Герберта Шилдта C++
C++ Задача из книги Липпмана - не могу написать красивый код
Задача 7 главы 7 из книги Р. Лафоре. Строки C++
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
MrGluck
Ворчун
Эксперт CЭксперт С++
6675 / 3856 / 510
Регистрация: 29.11.2010
Сообщений: 10,215
15.10.2013, 16:02     Задача с ошибкой из книги (Шилтд) #2
Похоже на то, что хотели реализовать оператор присвоений, который имеет следующий вид:
C++
1
Obj& operator= (const Obj &);
Но мне не понятно что вы здесь хотели сделать:
Цитата Сообщение от Drusha Посмотреть сообщение
sample input()
{
* * char instr[80];
* * sample str;
cout << "Enter string: ";
* * cin >> instr;
* * str.set(instr);
* * return str;
}
Вы создаете локальную переменную, изменяете её и как бы возвращаете, но на самом деле она стирается при выходе с тела функции и на её месте уже находится мусор.
Наверное вы это хотели получить:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
#include<iostream>
#include<cstring>
#include<cstdlib>
using namespace std;
 
class sample
{
    char *s;
public:
    sample();
    sample(const sample &ob);
    ~sample() { if(s) delete [] s; cout <<"S - free.\n";}
    void show() { cout << s << "\n";}
    void set(char *str);
    sample& operator= (const sample &ob); 
};
 
sample::sample()
{
    s = new char('\0');
    std::cout << "S()";
}
 
sample::sample(const sample &ob)
{
    s = new char[strlen(ob.s)+1];
    strcpy(s, ob.s);
    std::cout << "S(S &)";
}
 
void sample::set(char *str)
{
    s = new char[strlen(str)+1];
    strcpy(s, str);
}
 
sample& sample::operator= (const sample &ob)
{
    if(strlen(ob.s) > strlen(s)+1)
    {
        delete [] s;
        s = new char [strlen(ob.s)+1];
    }
    strcpy(s, ob.s);
    return *this;
}
 
sample* input()
{
    char instr[80];
    sample* str = new sample;
 
    cout << "Enter string: ";
    cin >> instr;
    str->set(instr);
    return str;
}
 
int main()
{
    sample* ob;
 
    ob = input();
    ob->show();
    delete ob;
    return 0;
}
alsav22
5415 / 4811 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
15.10.2013, 16:18     Задача с ошибкой из книги (Шилтд) #3
Цитата Сообщение от Drusha Посмотреть сообщение
Программа из книги Шилдт Г. С++ Базовый курс, 3-е издание,2010 (стр. 339) не работает.
И как это выглядит? У меня работает.
Миниатюры
Задача с ошибкой из книги (Шилтд)  
MrGluck
Ворчун
Эксперт CЭксперт С++
6675 / 3856 / 510
Регистрация: 29.11.2010
Сообщений: 10,215
15.10.2013, 16:25     Задача с ошибкой из книги (Шилтд) #4
alsav22, оператор= не распознает, хочет ссылку получать
alsav22
5415 / 4811 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
15.10.2013, 16:43     Задача с ошибкой из книги (Шилтд) #5
Написано: "не работает". Я понял так, что компилируется, но не работает. В студии компилируется и работает, mingv не компилирует (если константную ссылку сделать, то будет).
Drusha
6 / 6 / 0
Регистрация: 16.07.2013
Сообщений: 74
15.10.2013, 16:56  [ТС]     Задача с ошибкой из книги (Шилтд) #6
Я имел ввиду что не компилируется, выдает ошибку на 63 строку, но судя по всему не нравится ему перегрузка оператора "=" (в следующий раз буду яснее излагать). Dev-Cpp не компилирует, на сайте codepad.org тоже не получилось скомпилировать, для проверки на сайте я заменил 52 строчку на:
C++
1
 strcpy(instr, "tyttext");
После того как я убрал ссылочный параметр из строк 15 и 35 (operator=(sample ob)) программа вроде работает корректно.

Добавлено через 7 минут
Цитата Сообщение от alsav22 Посмотреть сообщение
(если константную ссылку сделать, то будет).
Проверил, работает. Странно что в книге обычно пишут когда из-за особенностей компилятора программа может работать не корректно или вообще не работать, а тут ничего не было написано.
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
15.10.2013, 16:58     Задача с ошибкой из книги (Шилтд)
Еще ссылки по теме:

C++ Задача из книги Страуструпа 4 глава векторы
Задача из книги Дейтелов Как программировать на C++ C++
C++ Задача из книги Дейтлов
Задача из книги Страуструпа C++
C++ Задача из книги Страуструпа (4.6.3)

Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
MrGluck
Ворчун
Эксперт CЭксперт С++
6675 / 3856 / 510
Регистрация: 29.11.2010
Сообщений: 10,215
15.10.2013, 16:58     Задача с ошибкой из книги (Шилтд) #7
По стандарту
12.8.23
The implicitly-declared copy/move assignment operator for class X has the return type X&; it returns the
object for which the assignment operator is invoked, that is, the object assigned to. An implicitly-declared
copy/move assignment operator is an inline public member of its class.
Так что все верно gcc/mingw делает. А VS попускает, разрешая создавать бессмысленную временную копию объекта при передаче по значению.
Кстати, насчет параметров, для вас, Drusha:
12.8.18
A user-declared copy assignment operator X::operator= is a non-static non-template member function of
class X with exactly one parameter of type X, X&, const X&, volatile X& or const volatile X&.
Добавлено через 44 секунды
Цитата Сообщение от Drusha Посмотреть сообщение
Странно что в книге обычно пишут когда из-за особенностей компилятора программа может работать не корректно или вообще не работать, а тут ничего не было написано.
Скорее всего, стандарт был принят уже после написания книги.
Yandex
Объявления
15.10.2013, 16:58     Задача с ошибкой из книги (Шилтд)
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru