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

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

Восстановить пароль Регистрация
 
_Edith
 Аватар для _Edith
0 / 0 / 0
Регистрация: 02.04.2011
Сообщений: 99
05.01.2015, 11:41     Не могу найти ошибку. Хеш-таблицы #1
Программа работает, в принципе, правильно, но есть маленькие погрешности при поиске элементов. То есть мы точно знаем, что элемент такой есть, но его алгоритм его не может найти и выводит, что его нет. Помогите, пожалуйста разобраться.
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;
}
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Гром
 Аватар для Гром
199 / 118 / 10
Регистрация: 20.03.2009
Сообщений: 1,075
Записей в блоге: 15
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
 Аватар для _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] = занято;
                    }
                }
            }
Гром
 Аватар для Гром
199 / 118 / 10
Регистрация: 20.03.2009
Сообщений: 1,075
Записей в блоге: 15
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
 Аватар для _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);
}
Что думаете по этому поводу?
Yandex
Объявления
06.01.2015, 14:43     Не могу найти ошибку. Хеш-таблицы
Ответ Создать тему
Опции темы

Текущее время: 07:42. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru