Какие есть ошибки в коде?

@Warezovvv · Регистрация: 09.12.2012

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

Добрый день. Сейчас было небольшое собеседование и одна из задачек была такая

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
/*
    What problems do you see in this code
*/
 
class A
{
public:
    A(int s)
        : size(s)
        , buf(new char[size])
    {
        memset(buf, ‘A’, size);
    }
 
    ~A()
    {
        delete buf; //need to be delete[]
    }
 
    char operator [](int index)
    {
        return buf[index];
    }
         //No opearator = for "char = int"
private:
    char* buf;
    int   size;
};
 
class B : A
{
public:
    B(int s)
        : size(s)
, buf(new int[size])
    {
        memset(buf, ‘B’, size);  // can't memet int array with Chars
    }
 
    ~B() << need to be virtual
    {
        delete buf; // need to delete[] buf
    }
        //No operator []
 
private:
    int* buf;
    int  size;
};
 
void foo()
{
    A* a = new A(10);
    A* b = new B(20);
    a[5] = b[10];
    delete b; //double delete a because B haven't virtual destructor
    delete a;
}

В комментариях к коду я указывал ошибки. Все ли я указал, самому интересно.

@ForEveR · 11.08.2015, 13:34

Warezovvv,
1) Деструктор A должен быть виртуальным, B не обязательно.
2) Наследование должно быть открытое.
3) Нет перегрузки конструктор копирования/оператора присваивания.
4) Некорректный порядок инициализации. size должен инициализироваться до buf, но этого не произойдет по причине порядка указания полей в классе или должен использоваться параметр sz.
Это так, самые грубые ошибки.

Убежденный · 11.08.2015, 13:50

Навскидку, из того, что бросается в глаза:

1. Порядок инициализации членов в классах A и B.

2. Конструкторы A и B полезно сделать explicit.

3. Для размера лучше использовать тип size_t, не int.

4. Деструктор A должен быть виртуальным. B - нет.

5. Деструкторы A и B должны использовать форму "delete []".

6. Нет конструктора копирования и оператора присваивания.

7. operator[] следует делать в двух версиях: а) константная,
возвращающая элемент массива, б) неконстантная, возвращающая
ссылку на него (см. пункт 9).

8. memset в конструкторе B некорректен.

9. foo: "a[5] = b[10]" не сработает.

10. foo: здесь не double delete, а просто UB.

11. В классе B отсутствует "operator =".

12. Отсутствует функция main.

13. Из-за того, что используется закрытое наследование B от A,
код вообще не будет компилироваться.

14. Конструктор B не инициализирует базовый класс A.

@ForEveR · 11.08.2015, 13:59

По-хорошему, код должен выглядеть +- так (не переделывал на size_t, но по хорошему размер/индексация должны идти через size_t).
Ну и memset в B все равно кривой, но как минимум инициализирует весь массив.

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
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
class A
{
public:
    A(int s)
        : buf(new char[s]),
        size(s)
    {
        memset(buf, 'A', size);
    }
 
    A(const A& rhs) : buf(0), size(rhs.size)
    {
       buf = new char[size];
       memcpy(buf, rhs.buf, size);
    }
 
    A& operator = (const A& rhs)
    {
       A tmp(rhs);
       swap(tmp);
       return *this;
    }
 
    virtual ~A()
    {
        delete[] buf; //need to be delete[]
    }
 
    char& operator [](int index)
    {
        return buf[index];
    }
 
    char operator [] (int index) const
    {
       return buf[index];
    }
         //No opearator = for "char = int"
private:
    void swap(A& other)
    {
       std::swap(buf, other.buf);
       std::swap(size, other.size);
    }
 
    char* buf;
    int   size;
};
 
class B : public A
{
public:
    B(int s)
        : A(s), buf(new int[s]), size(s)
    {
        memset(buf, 'B', size * sizeof(int));  // can't memet int array with Chars
    }
 
    B(const B& rhs) : A(rhs), buf(0), size(rhs.size)
    {
       buf = new int[size];
       memcpy(buf, rhs.buf, rhs.size * sizeof(int));
    }
 
    B& operator = (const B& rhs)
    {
       B tmp(rhs);
       swap(tmp);
       return *this;
    }
 
    ~B()
    {
        delete[] buf; // need to delete[] buf
    }
 
    int& operator [](int idx)
    {
       return buf[idx];
    }
 
    int operator [](int idx) const
    {
       return buf[idx];
    }
 
private:
    void swap(B& other)
    {
       std::swap(buf, other.buf);
       std::swap(size, other.size);
    }
 
    int* buf;
    int  size;
};
 
void foo()
{
    A* a = new A(10);
    A* b = new B(20);
    (*a)[5] = (*b)[10];
    delete b; //double delete a because B haven't virtual destructor
    delete a;
}
 
int main()
{
   foo();
}

Новые блоги и статьи Все статьи Все блоги /
Переходник USB-CAN-GPIO Eddy_Em 20.03.2026 Достаточно давно на работе возникла необходимость в переходнике CAN-USB с гальваноразвязкой, оный и был разработан. Однако, все меня терзала совесть, что аж 48-ногий МК используется так тупо: просто. . .	Оттенки серого Argus19 18.03.2026 Оттенки серого Нашёл в интернете 3 прекрасных модуля: Модуль класса открытия диалога открытия/ сохранения файла на Win32 API; Модуль класса быстрого перекодирования цветного изображения в оттенки. . .	SDL3 для Desktop (MinGW): Рисуем цветные прямоугольники с помощью рисовальщика SDL3 на Си и C++ 8Observer8 17.03.2026 Содержание блога Финальные проекты на Си и на C++: finish-rectangles-sdl3-c. zip finish-rectangles-sdl3-cpp. zip	Символические и жёсткие ссылки в Linux. algri14 15.03.2026 Существует два типа ссылок — символические и жёсткие. Ссылка в Linux — это запись в каталоге, которая может указывать либо на inode «файла-ИСТОЧНИКА», тогда это будет «жёсткая ссылка» (hard link),. . .
[Owen Logic] Поддержание уровня воды в резервуаре количеством включённых насосов: моделирование и выбор регулятора ФедосеевПавел 14.03.2026 Поддержание уровня воды в резервуаре количеством включённых насосов: моделирование и выбор регулятора ВВЕДЕНИЕ Выполняя задание на управление насосной группой заполнения резервуара,. . .	делаю науч статью по влиянию грибов на сукцессию anaschu 13.03.2026 прикрепляю статью	SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:	Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .

@ForEveR В астрале 8049 / 4806 / 655 Регистрация: 24.06.2010 Сообщений: 10,562
	11.08.2015, 13:34
	Warezovvv, 1) Деструктор A должен быть виртуальным, B не обязательно. 2) Наследование должно быть открытое. 3) Нет перегрузки конструктор копирования/оператора присваивания. 4) Некорректный порядок инициализации. size должен инициализироваться до buf, но этого не произойдет по причине порядка указания полей в классе или должен использоваться параметр sz. Это так, самые грубые ошибки. 0

Убежденный Ушел с форума 16481 / 7444 / 1187 Регистрация: 02.05.2013 Сообщений: 11,616 Записей в блоге: 1
	11.08.2015, 13:50
	Навскидку, из того, что бросается в глаза: 1. Порядок инициализации членов в классах A и B. 2. Конструкторы A и B полезно сделать explicit. 3. Для размера лучше использовать тип size_t, не int. 4. Деструктор A должен быть виртуальным. B - нет. 5. Деструкторы A и B должны использовать форму "delete []". 6. Нет конструктора копирования и оператора присваивания. 7. operator[] следует делать в двух версиях: а) константная, возвращающая элемент массива, б) неконстантная, возвращающая ссылку на него (см. пункт 9). 8. memset в конструкторе B некорректен. 9. foo: "a[5] = b[10]" не сработает. 10. foo: здесь не double delete, а просто UB. 11. В классе B отсутствует "operator =". 12. Отсутствует функция main. 13. Из-за того, что используется закрытое наследование B от A, код вообще не будет компилироваться. 14. Конструктор B не инициализирует базовый класс A. 1

Какие есть ошибки в коде?

Решение