Форум программистов, компьютерный форум, киберфорум
Наши страницы
С++ для начинающих
Войти
Регистрация
Восстановить пароль
 
Warezovvv
9 / 9 / 3
Регистрация: 09.12.2012
Сообщений: 219
#1

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

11.08.2015, 13:29. Просмотров 265. Ответов 3
Метки нет (Все метки)

Добрый день. Сейчас было небольшое собеседование и одна из задачек была такая
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;
}
В комментариях к коду я указывал ошибки. Все ли я указал, самому интересно.
0
Лучшие ответы (1)
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
11.08.2015, 13:29
Я подобрал для вас темы с готовыми решениями и ответами на вопрос Какие есть ошибки в коде? (C++):

Какие есть ошибки в коде
#include &lt;iostream&gt; using namespace std; class A { int num1, num2; public:...

Какие ошибки есть в коде и что можно улучшить?
Всем привет, есть вот такой код, подскажите, что у меня здесь не правильно и...

Можете сказать есть в коде ошибки или какие-то советы?
#include &lt;iostream&gt; #include &lt;conio.h&gt; #include &lt;string&gt; using namespace...

Какие ошибки в коде?
В конце получаю b=-1#IND00 #include&lt;stdio.h&gt; #include&lt;math.h&gt; void main() {...

Какие ошибки в коде?
#include &lt;iostream&gt; #include &lt;math.h&gt; using namespace std; int main() {...

Не понимаю, какие в моем коде ошибки
#include &lt;iostream&gt; char board = {'-','-','-','-','-','-','-','-','-',}; ...

3
ForEveR
В астрале
Эксперт С++
7994 / 4753 / 651
Регистрация: 24.06.2010
Сообщений: 10,547
Завершенные тесты: 3
11.08.2015, 13:34 #2
Warezovvv,
1) Деструктор A должен быть виртуальным, B не обязательно.
2) Наследование должно быть открытое.
3) Нет перегрузки конструктор копирования/оператора присваивания.
4) Некорректный порядок инициализации. size должен инициализироваться до buf, но этого не произойдет по причине порядка указания полей в классе или должен использоваться параметр sz.
Это так, самые грубые ошибки.
0
Убежденный
Ушел с форума
Эксперт С++
15941 / 7252 / 1176
Регистрация: 02.05.2013
Сообщений: 11,637
Записей в блоге: 1
Завершенные тесты: 1
11.08.2015, 13:50 #3
Навскидку, из того, что бросается в глаза:

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
ForEveR
В астрале
Эксперт С++
7994 / 4753 / 651
Регистрация: 24.06.2010
Сообщений: 10,547
Завершенные тесты: 3
11.08.2015, 13:59 #4
Лучший ответ Сообщение было отмечено Warezovvv как решение

Решение

По-хорошему, код должен выглядеть +- так (не переделывал на 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();
}
2
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
11.08.2015, 13:59
Привет! Вот еще темы с решениями:

Какие ошибки содержатся в представленном коде:
Какие ошибки содержатся в представленном коде: сlass Тest { stаtіc voіd...

Какие ошибки содержатся в приведенном коде?
void f1 ( int x, int y ) { float rez; rez = sin ( x )+float ( a%y ); void...

Подскажите пожалуйста какие ошибки здесь есть - или что не соблюдено.
Доброе утро! Я начинаю только осваивать программирование.Читаю - решаю задачи...

в коде ниже есть ошибки. в чем они и как их устранить?
structure brother { char name; into age; sister kin; }a; ...


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

Или воспользуйтесь поиском по форуму:
4
Ответ Создать тему
Опции темы

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