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

Оценить правильность использования конструкторов, деструкторов в коде - C++

Восстановить пароль Регистрация
 
DJames-
0 / 0 / 0
Регистрация: 02.09.2012
Сообщений: 12
30.09.2012, 17:52     Оценить правильность использования конструкторов, деструкторов в коде #1
Всем доброго времени суток. Не могли бы вы оценить правильность использования конструкторов,деструкторов и динамического выделения памяти данной программы.

Вот код:
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
#include <iostream.h>
#include <string.h>
 
class String
{
      public:
             String(const String &);
             String(char* = "");
             ~String();
             
             friend ostream &operator<<(ostream &,String );
             
      private:
              char* ln;
              int sizeln;
      };
String::String(const String &s)
{
                     sizeln = s.sizeln;
                     ln = new char [sizeln];
                     
                     ln = s.ln;
                     
                     cout << "Constructor copy called!" << endl;
                     }      
String::String(char *sPtr)
{
                    sizeln = strlen(sPtr);
                    ln = new char [sizeln];
                    ln = sPtr;
                    
                    cout << "Constructor with parameters called!" << endl;
                    }
String::~String()
{
                 delete [] ln;
                 
                 cout << "Destructor called!" << endl;
                 }
 
ostream &operator<<(ostream &out,String s)
{
        out << s.ln << endl;
        
        return out;
        }                 
                 
                 
main()
{
      String s1 = "Classes on ";
      String s2 = "cpp";
      
      cout << s1 << s2;
      
      system("pause");
      
      return 0;
      }
Заранее спасибо!
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
30.09.2012, 17:52     Оценить правильность использования конструкторов, деструкторов в коде
Посмотрите здесь:

C++ Правильное использование конструкторов и деструкторов
Разработка классов, создание конструкторов и деструкторов C++
C++ Создание и разрушение объектов класса, исследование вызовов конструкторов и деструкторов
C++ Порядок вызова конструкторов/деструкторов
C++ Как реализовать набор конструкторов и деструкторов
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
DU
1477 / 1053 / 45
Регистрация: 05.12.2011
Сообщений: 2,279
30.09.2012, 18:22     Оценить правильность использования конструкторов, деструкторов в коде #2
1. нет оператора =
2. конструктор должен принимать const char* а не char*
3.
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
String::String(char *sPtr)
{
  sizeln = strlen(sPtr);
  ln = new char [sizeln]; //  вы выделении не учитывается один байт по завершающий нуль-символ.
  ln = sPtr; // простое присваивание указателей некорректно. нужно скопировать симовлы в
                // только что выделенный массив.
  cout << "Constructor with parameters called!" << endl;
}
 
 
String::String(const String &s)
{
  // тут такие же проблемы, как и у первого конструктора
  sizeln = s.sizeln;
  ln = new char [sizeln];
  ln = s.ln;
  cout << "Constructor copy called!" << endl;
}
Andsteadur
152 / 136 / 3
Регистрация: 23.05.2009
Сообщений: 275
30.09.2012, 18:23     Оценить правильность использования конструкторов, деструкторов в коде #3
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
String::String(const String &s)
{
                     sizeln = s.sizeln;
                     ln = new char [sizeln];
                     
                     ln = s.ln;
                     
                     cout << "Constructor copy called!" << endl;
                     }   
String::String(char *sPtr)
{
                    sizeln = strlen(sPtr);
                    ln = new char [sizeln];
                    ln = sPtr;
                    
                    cout << "Constructor with parameters called!" << endl;
                    }
ерунда. Выделяется память под строку, но переданная строка не копируется в выделенную память, а просто производится присваивание указателю ln переданного указателя.

C++
1
2
3
4
5
6
7
8
String::String(char *sPtr)
{
                    sizeln = strlen(sPtr);
                    ln = new char [sizeln];
                    std::copy(sPtr, sPtr + sizeln, ln);
                    
                    cout << "Constructor with parameters called!" << endl;
                    }
DJames-
0 / 0 / 0
Регистрация: 02.09.2012
Сообщений: 12
01.10.2012, 20:47  [ТС]     Оценить правильность использования конструкторов, деструкторов в коде #4
Есть один вопрос, а если копировать циклом for, сойдёт?
Yandex
Объявления
01.10.2012, 20:47     Оценить правильность использования конструкторов, деструкторов в коде
Ответ Создать тему
Опции темы

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