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

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
_Edith
0 / 0 / 0
Регистрация: 02.04.2011
Сообщений: 99
#1

Не могу найти ошибку. Хеш-таблицы - C++

05.01.2015, 11:41. Просмотров 254. Ответов 4
Метки нет (Все метки)

Программа работает, в принципе, правильно, но есть маленькие погрешности при поиске элементов. То есть мы точно знаем, что элемент такой есть, но его алгоритм его не может найти и выводит, что его нет. Помогите, пожалуйста разобраться.
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
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
#include <iostream>
#include <clocale>
#include <string>
#include <windows.h>
 
using namespace std;
 
#define свободно true
#define занято false
 
const int n = 9;
bool fl[n];
 
int h(int key)
{
    return (key%5);
}
 
int menu()
{
    system("cls");
 
    cout << "ВЫБЕРИТЕ ИНТЕРЕСУЮЩИЕ ВАС ДЕЙСТВИЕ:" << endl;
    cout << "1 - поиск определенного элемента" << endl;
    cout << "2 - просмотр хеш таблицы" << endl;
    cout << "3 - удаление элемента из таблицы" << endl;
    cout << "4 - выход из программы" << endl;
 
    int a = -1;
 
    while ((a > 4) || (a < 1))
    {
        cout << "введите число, из списка действий" << endl;
        if (!(cin >> a))
            cin.clear();
    }
 
    return a;
}
 
int found(int data, int *a, int n)
{
    int key = h(data);
 
    if (fl[key] == занято)
    {
        if (a[key] == data)
            return key;
    }
 
    for (int i = key + 1; i < n; ++i)
    {
        if (fl[i] == занято)
        {
            if (a[key] == data)
                return i;
        }
    }
 
    for (int i = key - 1; i > -1; --i)
    {
        if (fl[i] == занято)
        {
            if (a[key] == data)
                return i;
        }
    }
 
    return -1;
}
int main()
{
    setlocale(LC_ALL, "");
 
    int a[n], data;
 
    int key, boolean;
    bool flag, work = true;
 
    for (int i = 0; i < n; ++i)
    {
        fl[i] = свободно;
        a[i] = 0;
    }
 
    cout << "Введите data's" << endl;
 
    while (cin >> data)
    {
        key = h(data);
 
        if (fl[key] == свободно)
        {
            a[key] = data;
            fl[key] = занято;
        }
 
        else
        {
            flag = false;
            for (int i = key + 1; i < n; ++i)
            {
                if (fl[i] == свободно && flag == false)
                {
                    a[i] = data;
                    flag = true;
                    fl[i] = занято;
                }
            }
                
            if (!flag && flag == false)
            {
                for (int i = key - 1; i > -1; i--)
                {
                    if (fl[i] == свободно)
                    {
                        a[i] = data;
                        flag = true;
                        fl[i] = занято;
                    }
                }
            }
                
            if (!flag)
            {
                cout << "превышен лимит, введенно больше чем " << n << " элементов";
 
                system("pause");
 
                break;
            }
        }
    }
    cin.clear();
 
    while (work)
        switch (menu())
        {
            case 1:
                system("cls");
 
                cout << "введите элемент который нужно найти" << endl;
                cin >> data;
 
                boolean = found(data, a, n);
 
                if (boolean == -1)
                {
                    cout << "такого элемента нет!" << endl;
                }
 
                else
                {
                    cout << "элемен " << data << " найден и его кэш - " << boolean << endl;
                }
 
                system("pause");
 
                break;
 
            case 2:
                system("cls");
 
                for (int i = 0; i < n; ++i)
                {
                    if (fl[i] == занято)
                    {
                        cout.width(3);
                        cout << right << i << ": ";
                        cout << a[i];
                        cout << endl;
                    }
                }
 
                system("pause");
 
                break;
 
            case 3:
                system("cls");
 
                cout << "введите элемент который нужно удалить" << endl;
                cin >> data;
 
                boolean = found(data, a, n);
 
                if (boolean == -1)
                {
                    cout << "такого элемента нет!" << endl;
                }
 
                else
                {
                    fl[boolean] = свободно;
                }
 
                system("pause");
 
                break;
 
            case 4:
                work = false;
        }
 
    return 0;
}
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
05.01.2015, 11:41
Здравствуйте! Я подобрал для вас темы с ответами на вопрос Не могу найти ошибку. Хеш-таблицы (C++):

хеш-таблицы - C++
Реализовать ассоциативный массив в виде хеш-таблицы с операциями добавления, поиска . Ключом массива должна быть строка, значением – целое...

Хеш таблицы - C++
Начал изучать хеш таблицы. Подскажите насчёт хеш таблиц с открытимы адрессами: - Должны ли мы инициализировать значение ключа...

Найти слова, повторяющиеся более одного раза, не могу найти ошибку - C++
#include &lt;iostream&gt; using namespace std; void obr1(char **s, char **mas, int n, int m) { int i; int k; char *tm; for(i...

Массивы. Посчитать количество положительных, найти минимальное, удалить строку с минимальным (Не могу найти ошибку) - C++
// Заданы матрицы X(8;4),Y(5;5),Z(6;9). // Для каждой из матриц определить строку, в которой находится наименьшее // количество...

Реализация хеш-таблицы - C++
Всем привет. Нужна помощь с заданием:

Не могу найти ошибку - C++
Здравствуйте,не могу понять в чем ошибка подскажите пожалуйста #include &quot;stdafx.h&quot; #include &lt;time.h&gt; #include &lt;cstdlib&gt; ...

Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
Гром
210 / 129 / 11
Регистрация: 20.03.2009
Сообщений: 1,103
Записей в блоге: 16
Завершенные тесты: 1
05.01.2015, 12:01 #2
Вероятно, здесь у вас ошибка:
C++
1
2
3
4
5
6
7
8
9
                for (int i = key - 1; i > -1; i--)
                {
                    if (fl[i] == свободно)
                    {
                        a[i] = data;
                        flag = true;
                        fl[i] = занято;
                    }
                }
Даже если вы в одну ячейку поместили элемент, то несмотря на установку флага, вы все еще продолжаете забивать ячейки этим же элементом, поскольку флаг не проверяете.

Если выделить вставку в отдельную функцию (возвращаемое значение - успешна ли вставка), то можно обойтись без флага. Кроме того, проверку позиции key можно безболезненно совместить с проверкой последующих. Плюс я несколько изменил порядок обхода - закольцевал его (дойдя до конца, начинаем с нулевого элемента)
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
bool insert(int data, int* htable, int* isEmpty, int n)
{
int key = h(data);
for (unsigned i = key; i < n; ++i)
 if (isEmpty[i])
  {
  htable[i] = data;
  isEmpty[i] = false;
  return true;
  }
for (unsigned i = 0; i < key; ++i)
 if (isEmpty[i])
  {
  htable[i] = data;
  isEmpty[i] = false;
  return true;
  }
return false;
}
Еще в этой строке: if (!flag && flag == false) оба условия эквивалентны. Ну и несколько сомнительно определять макроопределения типа свободно/занято, когда можно переименовать массив и писать просто:
C++
1
2
if (isEmpty[i])
 isEmpty[i] = false;
_Edith
0 / 0 / 0
Регистрация: 02.04.2011
Сообщений: 99
06.01.2015, 13:23  [ТС] #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
key = h(data);
 
        if (fl[key] == свободно)
        {
            a[key] = data;
            fl[key] = занято;
        }
 
        else
        {
            flag = false;
            for (int i = key + 1; i < n; ++i)
            {
                if (fl[i] == свободно && flag == false)
                {
                    a[i] = data;
                    flag = true;
                    fl[i] = занято;
                }
            }
                
            if (!flag && flag == false)
            {
                for (int i = key - 1; i > -1; i--)
                {
                    if (fl[i] == свободно)
                    {
                        a[i] = data;
                        flag = true;
                        fl[i] = занято;
                    }
                }
            }
Гром
210 / 129 / 11
Регистрация: 20.03.2009
Сообщений: 1,103
Записей в блоге: 16
Завершенные тесты: 1
06.01.2015, 14:39 #4
Да, будет:
C++
1
2
3
4
5
6
7
8
9
10
    while (cin >> data)
    {
        bool success = insert(data, a, fl, n);
        if (!success)
        {
            cout << "превышен лимит, введенно больше чем " << n << " элементов"; 
            system("pause"); 
            break;
        }
    }
_Edith
0 / 0 / 0
Регистрация: 02.04.2011
Сообщений: 99
06.01.2015, 14:43  [ТС] #5
Гром, Спасибо, кстати, я тут еще пофорсил интернет и есть теория, что хеш-функция изначально у меня была записана неправильно.
C++
1
2
3
4
int h(int key)
{
    return (key%5);
}
Как я прочитал, метод деления реализуется так:
C++
1
2
3
4
int h(int key, int n)
{
    return (key % n);
}
Что думаете по этому поводу?
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
06.01.2015, 14:43
Привет! Вот еще темы с ответами:

С++ не могу найти ошибку - C++
#include &quot;stdafx.h&quot; #include &lt;iostream&gt; #include &lt;math.h&gt; typedef double TMatrix; typedef double TVector; const int N=10; ...

Не могу найти ошибку - C++
Кусок программы. Нужно найти в файле необходимую услугу и записать её номер. У строки searching уже есть значение. Пример строки: Cвязь...

Не могу найти ошибку - C++
В общем суть программы состоит в том чтобы решить нелинейное уравнение x3+x2-4x-4=0 двумя методами хорд и итераций, есть исходник , вот...

Не могу найти ошибку - C++
В общем написал программу , только не понимаю ошибок


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

Или воспользуйтесь поиском по форуму:
Yandex
Объявления
06.01.2015, 14:43
Ответ Создать тему
Опции темы

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