String в шаблонном классе

@Ишук · Регистрация: 25.04.2012

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

Как сделать так, чтобы в Map<int, std::string> A; работало со string'ом
С интами чарами и т.д. все хорошо работает, а на string прога вылетает, как это можно исправить?

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
#ifndef MAP_H
#define MAP_H
#include<iostream>
 
template <typename T1, typename T2>
struct T
{
    T1 key;
    T2 dat;
};
 
template <typename T1, typename T2>
class Map
{
    T<T1,T2>* data;
 
public:
    size_t Size;
    Map();
    void insert(T1, T2);    //Добавить ключ-значение
    void remove(T1);        //Удалить ключ-значение
    void show();            //Вывод на экран всех слотов
    void show_slot(size_t); //Вывести на экран по индексу ключ-значение
    void clear();           //Уничтожить ассоциативный массив
 
    bool check_key(T1);     //Проверить ключ на существование
    T2 set_data(T1);        //Получить значение по ключу
    size_t set_index(T1);   //Получить значение по индексу
 
    Map<T1,T2>& operator=(Map<T1,T2> &map2); //перегрузка оператора присваивания
 
 
};
 
#endif // MAP_H

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
119
120
121
122
123
124
125
126
127
128
129
130
#include "map.h"
 
template <typename T1, typename T2>
Map<T1, T2>::Map()
{
    data = new T<T1, T2>[0];
    Size = 0;
}
 
template <typename T1, typename T2>
void Map<T1, T2>::insert(T1 k, T2 d)
{
    T<T1,T2>* buff = new T<T1,T2>[Size];
    for (size_t i = 0; i<Size; i++)
    {
        buff[i] = data[i];
    }
 
    delete[] data;
    Size++;
    data = new T<T1, T2>[Size];
 
    for (size_t i = 0; i<Size; i++)
    {
        data[i] = buff[i];
    }
    data[Size-1].key = k;
    data[Size-1].dat = d;
    delete[] buff;
}
 
template <typename T1, typename T2>
void Map<T1,T2>::remove(T1 k)
{
    size_t k1 = set_index(k);
    if (check_key(k) == true)
    {
        T<T1,T2>* buff = new T<T1,T2>[Size];
        for (size_t i = 0; i<Size; i++)
        {
            buff[i] = data[i];
        }
 
        delete[] data;
        Size--;
        data = new T<T1, T2>[Size];
 
        for (size_t i = 0; i<k1; i++)
        {
            data[i] = buff[i];
        }
        for (size_t i = k1+1; i<Size+1; i++)
        {
            data[i-1] = buff[i];
        }
    }
    else std::cout << "This key not exist" << std::endl;
 
}
 
 
template <typename T1, typename T2>
bool Map<T1, T2>::check_key(T1 k)
{
    size_t count = 0;
    for (size_t i = 0; i<Size; i++)
        if (data[i].key == k)
            count = 1;
    if (count == 1) return true;
    else return false;
}
 
template <typename T1, typename T2>
T2 Map<T1, T2>::set_data(T1 k)
{
    for (size_t i = 0; i<Size; i++)
        if (data[i].key == k)
        {
            return data[i].dat;
            break;
        }
    return 0;
}
 
template <typename T1, typename T2>
size_t Map<T1, T2>::set_index(T1 k)
{
    for (size_t i = 0; i<Size; i++)
        if (data[i].key == k)
        {
            return i;
            break;
        }
    return 0;
}
 
template <typename T1, typename T2>
void Map<T1, T2>::show()
{
    for (size_t i = 0; i<Size; i++)
    std::cout << data[i].key << " " << data[i].dat << std::endl;
 
}
 
template <typename T1, typename T2>
void Map<T1, T2>::show_slot(size_t n)
{
    if (n<Size)
        std::cout << data[n].key << ' ' << data[n].dat << std::endl;
    else std::cout << "This element not exist" << std::endl;
}
 
template <typename T1, typename T2>
void Map<T1, T2>::clear()
{
    delete[] data;
    Size = 0;
}
 
template <typename T1, typename T2>
Map<T1,T2>& Map<T1, T2>::operator=(Map<T1,T2> &map2)
{
    delete[]data;
    Size = map2.Size;
    data = new T<T1,T2>[Size];
    for (size_t i = 0; i<Size; i++)
    {
        data[i] = map2.data[i];
    }
}

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
#include <iostream>
#include "map.h"
#include "map.cpp"
#include <QString>
 
 
int main()
{
    Map<int, std::string> A;
    A.insert(12,"k");
    A.insert(5,"h");
    A.insert(7,"t");
    A.insert(3,"f");
 
    std::cout << A.check_key(5) << std::endl;
 
    std::cout << A.set_data(5) << std::endl;
 
    A.remove(7);
 
    A.show_slot(0);
    A.show();
 
    Map<int, char> B;
    B = A;
    B.show();
 
    return 0;
}

@TheCalligrapher · 17.02.2017, 03:15

Сообщение от Ишук

а на string прога вылетает

Вылетает? Вы что-то выдумываете. Ваш код в принципе некомпилируем. У вас нет подходящего оператора присваивания для B = A;.

Также в коде не выполняется Правило Трех (см. Распространенные ошибки)

---

В методе insert написана какая-то ерунда. Во-первых, не ясно, зачем нужен какой-то промежуточный buff. Во-вторых, вот в этом месте

C++
1
2
3
4
5
6
7
8
9
    ...
    Size++;
    data = new T<T1, T2>[Size];
 
    for (size_t i = 0; i<Size; i++)
    {
        data[i] = buff[i];
    }
    ...

происходит вылет за пределы массива buff. Неудивительно, что все падает.

@Ишук · 17.02.2017, 03:16 **[ТС]**

TheCalligrapher, Если изначально написать так:

C++
1
2
3
4
5
 Map<int, char> A;
    A.insert(12,'k');
    A.insert(5,'h');
    A.insert(7,'t');
    A.insert(3,'f');

то все спокойно компилируется. Когда со string'ом, он начинает уже не работать, если закомментить оператор присваивания, то прога просто вылетает, а если разкомментить, ругается на присваивание. Хотелось бы понять, как все это подстроить под string

@TheCalligrapher · 17.02.2017, 03:28

Сообщение от Ишук

то все спокойно компилируется.

Ничего не понял. Еще раз: в вашем коде не компилируется B = A;. Что вы там "изначально" написали не имеет значения.

А если у ваc не "падает" Map<int, char> A - так это ничего не значит вообще. Ваш код некорректен из-за выхода за пределы массива. А "вылетает" он у вас или не "вылетает" роли не играет. Это вопрос тупого везения. Сегодня ваш Map<int, char> A не вылетает, а завтра начнет вылетать.

Сообщение от Ишук

Хотелось бы понять, как все это подстроить под string

Исправить очевидные баги в insert (и в прочих местах).

Сообщение от Ишук

C++
1
2
return data[i].dat;
break;

Интересно также, в чем глубинный смысл таких комбинаций.

Сообщение от Ишук

C++
1
2
3
4
5
6
template <typename T1, typename T2>
void Map<T1, T2>::clear()
{
  delete[] data;
  Size = 0;
}

Функция clear приводит объект в некорректное состояние. Попытки сделать insert после clear будут "падать".

@Ишук · 17.02.2017, 03:37 **[ТС]**

С insert'ом понял

C++
1
2
3
4
 for (size_t i = 0; i<Size-1; i++)
    {
        data[i] = buff[i];
    }

А в clear стоило добавить

C++
1
data = new T<T1, T2>[0];

А вот как исправить косяк с оператором присваивания пока что не очень понятно

Добавлено через 3 минуты
Хотя не, он даже работает, если поставить те же типы
Вроде бы теперь все работает и не падает

@TheCalligrapher · 17.02.2017, 03:47

Сообщение от Ишук

С insert'ом понял

C++
1
2
3
4
for (size_t i = 0; i<Size-1; i++)
{
    data[i] = buff[i];
}

Верно. Но тем не менее цель этого buff мне в упор не ясна. Почему бы просто сразу не выделить новый массив (на 1 больше) и просто скопировать старые данные прямо в него? Без промежуточного buff.

Ну и идиому "copy-and-swap" тоже не мешало бы изучить (см. например реализацию оператора присваивания в Шаблон класса одномерного массива)

Сообщение от Ишук

А в clear стоило добавить

C++
1
data = new T<T1, T2>[0];

Тоже верно. Хотя кто-то может сказать, что ваши странные массивы нулевого размера new T<T1, T2>[0] лучше просто заменить на nullptr и все будет работать.

Сообщение от Ишук

А вот как исправить косяк с оператором присваивания пока что не очень понятно
Хотя не, он даже работает, если поставить те же типы
Вроде бы теперь все работает и не падает

Ну так вопрос в том, что именно вам нужно. Если вы хотите оператор, который работает с разными типами - это одно. А с одинаковыми - это другое. Однако оператор присваивания обычно принимает свой аргумент по константной ссылке.

И деструктора и конструктора копирования у вас в коде я по-прежнему не вижу.

Новые блоги и статьи Все статьи Все блоги /
Ритм жизни 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 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»