Реализовать класс vector

@Silverhorn · Регистрация: 04.07.2019

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

Недавно я получил тестовое задание на которое мне дали 24 чеса. Я всё сделал и отправил ответ, но спустя почти неделю мне прислали отказ мол я набрал мало балов
Задание:
Реалізувати клас vector в основі якого лежить динамічний масив типу int (розмір повинен динамічно змінюватися в залежності від кількості елементів). У даному класі повинні бути заімплементовані наступні функції:
- Додати в кінець масиву
- Вставити по індексу
- Видалити один елемент за індексом
- Видалити всі елементи
- Знайти індекс елементу за значенням
- Перегрузити оператор присвоєння
- Реалізувати конструктор копіювання

Комментарий модератора

Реализовать класс vector в основе которого лежит динамический массив типа int(размер должен меняться в зависимовсти от количества элементов). В данном классе должны быть реализованы следующие функции:
- Добавить в конец массива
- Вставить по индексу
_ Удалить элемент с заданным индексом
- Удалить все элементы
_ Найти индекс по значению
- Перегрузить operator=
- Реализовать копиконструктор

Решение:

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
Vector.h
#include <iostream>
#include <cstdlib>
#include <string>
 
namespace task {
 
class vector 
{
public:
    vector();
    vector(std::size_t size);
    vector(const vector &vec);
    ~vector();
 
    vector& operator=(vector vec);
    int& operator[](size_t &index) const;
 
    void push_back(int &&value);
    void insert(const std::size_t &index, int &&value);
    bool pop(const std::size_t &index);
    std::size_t getIndex(const int &value) const;
    void clear();
    
    std::size_t size() const;
    bool empty();
 
private:
    int *_array;
    std::size_t _size;
};
 
class VectorException : public std::exception 
{
    std::string _msg;
public:
    VectorException(const std::string &msg);
    const char *what() const noexcept override;
};
 
} //end namespace task
 
#endif //VECTOR_H

Vector.cpp

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
111
112
113
114
115
116
117
118
#include "Vector.h"
 
namespace task {
 
vector::vector()
    : _array(nullptr)
    , _size(0)
{
 
}
 
vector::vector(std::size_t size)
    : _array(static_cast<int*>(malloc(size*sizeof(int))))
    , _size(size)
{
 
}
 
vector::vector(const vector &vec)
    : _array(static_cast<int*>(malloc(vec.size()*sizeof(int))))
    , _size(vec.size())
{
    std::copy(vec._array, vec._array + _size, _array);
}
 
vector::~vector()
{
    clear();
}
 
vector& vector::operator=(vector vec)
{
    std::swap(*this, vec);
    return *this;
}
 
int& vector::operator[](std::size_t &index) const
{
    return _array[index];
}
 
void vector::push_back(int &&value)
{
    if (!(_array = static_cast<int*>(realloc(_array, (size()+1)*sizeof(int)))))
        throw VectorException("Memory allocation error");
    _array[_size++] = std::move(value);
}
 
void vector::insert(const std::size_t &index, int &&value)
{
    if (index >= size()) {
        if (!(_array = static_cast<int*>(realloc(_array, (index+1)*sizeof(int)))))
            throw VectorException("Memory allocation error");
        _array[index] = std::move(value);
        _size = index+1;
    } else {
        push_back(std::move(value));
        for(std::size_t i = size()-1; i >= 0; --i) {
            if (index == i) 
                break;
            std::swap(_array[i], _array[i-1]);
        }
    }
}
 
bool vector::pop(const std::size_t &index)
{
    if (index >= size() || empty())
        return false;
    for(std::size_t i = index; i < size()-1; ++i) {
        std::swap(_array[i], _array[i+1]);
    }
    if (!(_array = static_cast<int*>(realloc(_array, (size()-1)*sizeof(int)))))
        throw VectorException("Memory allocation error");
    _size--;
    return true;
}
 
std::size_t vector::getIndex(const int &value) const
{
    for(std::size_t i = 0; i < _size; ++i) {
        if (value == _array[i])
            return i;
    }
    throw VectorException("Item not found");
}
 
void vector::clear()
{
    if (_array) {
        free(_array);
        _array = nullptr;
    }
    _size = 0;
}
 
std::size_t vector::size() const
{
    return _size;
}
 
bool vector::empty()
{
    return _size == 0 ? true : false;
}
 
VectorException::VectorException(const std::string &msg)
    : _msg(msg)
{
 
}
 
const char *VectorException::what() const noexcept
{
    return _msg.c_str();
}
 
} //end namespace task

Пожалуйста дайте анализ. Что не так? Я очень буду благодарен!

@ТабуретY · 04.07.2019, 17:07

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

Пожалуйста дайте анализ

Этот раздел в самом низу, раздел с++ https://www.cyberforum.ru/cpp-beginners/ в самом верху

@nonedark2008 · 05.07.2019, 01:08

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

vector& operator=(vector vec);

C++
1
vector& operator=(const vector &vec);

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

C++
1
int& operator[](size_t &index) const;

Позволяет редактировать содержимое константного объекта - это ошибка. size_t &index не имеет смысла, должно было быть просто size_t

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

C++
1
void push_back(int &&value);

int && - не имеет смысла для базовых типов, достаточно просто int. Если уж так хотелось показать знание rvalue, то нужно было хотя бы реализовать конструктор перемещения и соответствующий оператор. А так, ты лишь показываешь именно незнание вопроса.

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

C++
1
bool empty();

C++
1
bool empty() const;

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

C++
1
2
3
vector::vector(std::size_t size)
* * : _array(static_cast<int*>(malloc(size*sizeof(int))))
* * , _size(size)

Остуствует проверка на случай, когда malloc вернет nullptr. Да и вообще, если пишешь на C++, то используй new и delete.

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

std::swap(*this, vec);

Без перемещающих конструктора и оператора будет крайне неэффективным.

Ну и еще всякая фигня на счет дублирования кода.

@rikimaru2013 · 05.07.2019, 01:19

Silverhorn, это хорошо, что вам отказали. Наоборот подтяните хвосты и на следующем собеседовании будете более уверенным и больше шансов заполучить лучше условия для себя.

@_stanislav · 05.07.2019, 06:45

Не по теме:

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

это хорошо, что вам отказали. Наоборот подтяните хвосты и на следующем собеседовании будете более уверенным и больше шансов заполучить лучше условия для себя.

позитивный настрой, в плохом видеть хорошее, одобряю.

@Silverhorn · 05.07.2019, 11:26 **[ТС]**

Спасибо вам большое, я всё понял и буду учиться больше)

Новые блоги и статьи Все статьи Все блоги /
Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .	SDL3 для Web (WebAssembly): Сборка SDL3 и Box2D из исходников с помощью 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. . .
Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»

@rikimaru2013 2549 / 1208 / 358 Регистрация: 30.11.2013 Сообщений: 3,826
	05.07.2019, 01:19
	Silverhorn, это хорошо, что вам отказали. Наоборот подтяните хвосты и на следующем собеседовании будете более уверенным и больше шансов заполучить лучше условия для себя. 2

@Silverhorn 1 / 1 / 0 Регистрация: 04.07.2019 Сообщений: 2
	05.07.2019, 11:26 [ТС]
	Спасибо вам большое, я всё понял и буду учиться больше) 0