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

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

Восстановить пароль Регистрация
 
Drusha
6 / 6 / 0
Регистрация: 16.07.2013
Сообщений: 74
15.10.2013, 15:43     Задача с ошибкой из книги (Шилтд) #1
Программа из книги Шилдт Г. С++ Базовый курс, 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
Ворчун
Эксперт С++
 Аватар для MrGluck
4927 / 2670 / 243
Регистрация: 29.11.2010
Сообщений: 7,427
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
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
15.10.2013, 16:18     Задача с ошибкой из книги (Шилтд) #3
Цитата Сообщение от Drusha Посмотреть сообщение
Программа из книги Шилдт Г. С++ Базовый курс, 3-е издание,2010 (стр. 339) не работает.
И как это выглядит? У меня работает.
Миниатюры
Задача с ошибкой из книги (Шилтд)  
MrGluck
Ворчун
Эксперт С++
 Аватар для MrGluck
4927 / 2670 / 243
Регистрация: 29.11.2010
Сообщений: 7,427
15.10.2013, 16:25     Задача с ошибкой из книги (Шилтд) #4
alsav22, оператор= не распознает, хочет ссылку получать
alsav22
5282 / 4801 / 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     Задача с ошибкой из книги (Шилтд)
Еще ссылки по теме:

Задача 7 главы 7 из книги Р. Лафоре. Строки C++
Задача из книги Дейтлов: напечатать таблицу двоичных, восьмеричных и шестнадцатиричных чисел C++
C++ Задача из книги Страуструпа 4 глава векторы

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

Или воспользуйтесь поиском по форуму:
MrGluck
Ворчун
Эксперт С++
 Аватар для MrGluck
4927 / 2670 / 243
Регистрация: 29.11.2010
Сообщений: 7,427
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     Задача с ошибкой из книги (Шилтд)
Ответ Создать тему
Опции темы

Текущее время: 16:27. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru