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

@Drusha · Регистрация: 16.07.2013

Студворк — интернет-сервис помощи студентам

Программа из книги Шилдт Г. С++ Базовый курс, 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;
}

MrGluck · 15.10.2013, 16:02

Похоже на то, что хотели реализовать оператор присвоений, который имеет следующий вид:

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 · 15.10.2013, 16:18

Сообщение от Drusha

Программа из книги Шилдт Г. С++ Базовый курс, 3-е издание,2010 (стр. 339) не работает.

И как это выглядит? У меня работает.

MrGluck · 15.10.2013, 16:25

alsav22, оператор= не распознает, хочет ссылку получать

alsav22 · 15.10.2013, 16:43

Написано: "не работает". Я понял так, что компилируется, но не работает. В студии компилируется и работает, mingv не компилирует (если константную ссылку сделать, то будет).

@Drusha · 15.10.2013, 16:56 **[ТС]**

Я имел ввиду что не компилируется, выдает ошибку на 63 строку, но судя по всему не нравится ему перегрузка оператора "=" (в следующий раз буду яснее излагать). Dev-Cpp не компилирует, на сайте codepad.org тоже не получилось скомпилировать, для проверки на сайте я заменил 52 строчку на:

C++
1
 strcpy(instr, "tyttext");

После того как я убрал ссылочный параметр из строк 15 и 35 (operator=(sample ob)) программа вроде работает корректно.

Добавлено через 7 минут

Сообщение от alsav22

(если константную ссылку сделать, то будет).

Проверил, работает. Странно что в книге обычно пишут когда из-за особенностей компилятора программа может работать не корректно или вообще не работать, а тут ничего не было написано.

MrGluck · 15.10.2013, 16:58

По стандарту

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

Странно что в книге обычно пишут когда из-за особенностей компилятора программа может работать не корректно или вообще не работать, а тут ничего не было написано.

Скорее всего, стандарт был принят уже после написания книги.

Новые блоги и статьи Все статьи Все блоги /
Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование 8Observer8 05.03.2026 Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. На мобильном - сканируйте QR-код. Вращайте камеру одним пальцем,. . .	SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .	SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip Сканируйте QR-код на мобильном и вы увидите, что появится джойстик для управления главным героем. . . .	Реалии Hrethgir 01.03.2026 Нет, я не закончил до сих пор симулятор. Эта задача сложнее. Не получилось уйти в плавсостав, но оно и к лучшему, возможно. Точнее получалось - но сварщиком в палубную команду, а это значит, в моём. . .
Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .	SDL3 для Web (WebAssembly): Сборка библиотек: SDL3, Box2D, FreeType, SDL3_ttf, SDL3_mixer и SDL3_image из исходников с помощью CMake и Emscripten 8Observer8 27.02.2026 Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .	SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .

MrGluck Форумчанин 8216 / 5047 / 1437 Регистрация: 29.11.2010 Сообщений: 13,453
	15.10.2013, 16:25
	alsav22, оператор= не распознает, хочет ссылку получать 0

alsav22 5500 / 4895 / 831 Регистрация: 04.06.2011 Сообщений: 13,587
	15.10.2013, 16:43
	Написано: "не работает". Я понял так, что компилируется, но не работает. В студии компилируется и работает, mingv не компилирует (если константную ссылку сделать, то будет). 0