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

Ошибка в алгоритме перегрузки оператора присваивания - C++

Войти
Регистрация
Восстановить пароль
 
agiliter
1 / 1 / 0
Регистрация: 13.12.2012
Сообщений: 102
13.12.2012, 17:48     Ошибка в алгоритме перегрузки оператора присваивания #1
Добрый вечер. Пишу методы для класса по своему заданию. Столкнулся с проблемой перегрузки оператора присваивания для метода поиска. Привожу только часть кода с методом поиска Find и перегрузкой. Хочу разобраться непосредственно с алгоритмом. Может косяк в реализации? Буду благодарен за помощь
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
struct DataType
{
    string term;
    string text;
};
 
typedef struct DataType;
 
class SeqList
{
    private:
        DataType *ListItem;
        int size;
    public:             
        SeqList::SeqList(void) : size(0)
        {
            ListItem = new DataType[MaxSize];
        }
                bool Find(DataType &item) const         //Сканирование списка для нахождения соответствующего элемента.
        {
            int i = 0;
            if(!ListEmpty())
            {
                return false;
            }
            while(i < size && !(item.term == ListItem[i].term))  //здесь используется присваивание
                i++;
            if(i < size)
            {
                item.term = ListItem[i].term;
                return true;
            }
            else
                return false;
        }
        bool SeqList::operator ==(DataType &item)
        { 
            int i;
            for (i = 0; ListItem[i].term[i] == item.term[i]; i++) //не пойму как тут организовать, запутался 
            if(ListItem[i].term[i] == item.term[i])                   //индекс ListItem не i, но я не пойму что и как
                return true;                        
            return false;                           
        }
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
13.12.2012, 17:48     Ошибка в алгоритме перегрузки оператора присваивания
Посмотрите здесь:

Вопрос по поводу перегрузки оператора + C++
C++ Friend-функции перегрузки оператора
Насчет оператора присваивания C++
C++ Проверка оператора присваивания
C++ Ошибка из книги оператора перегрузки
Аналог перегрузки оператора индексирования (operator []) C++
Необходимость перегрузки оператора присваивания "=" C++
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
MrGluck
Ворчун
Эксперт С++
 Аватар для MrGluck
5791 / 3166 / 359
Регистрация: 29.11.2010
Сообщений: 8,469
13.12.2012, 17:55     Ошибка в алгоритме перегрузки оператора присваивания #2
agiliter, начнем с того, что это оператор сравнения, а не присваивания (=) , коий вы тоже должны, по-хорошему, реализовать, наряду с деструктором и конструктором копий.
C++
1
2
3
4
5
6
bool SeqList::operator ==(DataType &item)
{ 
    for (int i = 0; i < MaxSize; i++) // если заполнены не все - создайте член класса index, хранящий нужный размер
        if (ListItem[i].term != item.term) return false;
    return true;
}
agiliter
1 / 1 / 0
Регистрация: 13.12.2012
Сообщений: 102
13.12.2012, 18:20  [ТС]     Ошибка в алгоритме перегрузки оператора присваивания #3
Цитата Сообщение от MrGluck Посмотреть сообщение
agiliter, начнем с того, что это оператор сравнения, а не присваивания (=) , коий вы тоже должны, по-хорошему, реализовать, наряду с деструктором и конструктором копий.
разумеется, что это оператор сравнения. Но не суть. Деструктор, как и конструктор копий и другие методы - всё есть, я просто не стал загромождать пример. Привел нужные части
Цитата Сообщение от MrGluck Посмотреть сообщение
C++
1
2
3
4
5
6
bool SeqList::operator ==(DataType &item)
{ 
    for (int i = 0; i < MaxSize; i++) // если заполнены не все - создайте член класса index, хранящий нужный размер
        if (ListItem[i].term != item.term) return false;
    return true;
}
Нужный размер (количество элементов в списке) хранится в переменной size.
Вы здесь "пробегаете" по всем элементам ListItem[i] списка SeqList. А нужно же сравнивать только какой-то конкретный ListItem[???] с введенным item.term. Как именно указать нужный? Чтобы компилятор понял что с чем сравнивать (см. Find) - он жутко ругается
MrGluck
Ворчун
Эксперт С++
 Аватар для MrGluck
5791 / 3166 / 359
Регистрация: 29.11.2010
Сообщений: 8,469
13.12.2012, 18:25     Ошибка в алгоритме перегрузки оператора присваивания #4
agiliter, все еще проще
C++
1
2
3
4
bool SeqList::operator ==(DataType &item)
{ 
    return ListItem == item;
}
agiliter
1 / 1 / 0
Регистрация: 13.12.2012
Сообщений: 102
13.12.2012, 18:30  [ТС]     Ошибка в алгоритме перегрузки оператора присваивания #5
Цитата Сообщение от MrGluck Посмотреть сообщение
agiliter, все еще проще
C++
1
2
3
4
bool SeqList::operator ==(DataType &item)
{ 
    return ListItem == item;
}
error C2679: binary '==' : no operator found which takes a right-hand operand of type 'DataType' (or there is no acceptable conversion)
1> could be 'built-in C++ operator==(DataType *, DataType *)'
MrGluck
Ворчун
Эксперт С++
 Аватар для MrGluck
5791 / 3166 / 359
Регистрация: 29.11.2010
Сообщений: 8,469
13.12.2012, 18:44     Ошибка в алгоритме перегрузки оператора присваивания #6
Вы хотите сравнить массив и структуру, являющуюся типом элементов этого массива? Что это значит? Объект должен быть в массиве/ весь массив должен состоять из равных объекту элементов?

Добавлено через 1 минуту
И объясните, зачем вы это делаете:
C++
1
typedef struct DataType;
agiliter
1 / 1 / 0
Регистрация: 13.12.2012
Сообщений: 102
13.12.2012, 18:52  [ТС]     Ошибка в алгоритме перегрузки оператора присваивания #7
Цитата Сообщение от MrGluck Посмотреть сообщение
Вы хотите сравнить массив и структуру, являющуюся типом элементов этого массива? Что это значит? Объект должен быть в массиве/ весь массив должен состоять из равных объекту элементов?
Весь массив ListItem состоит из объектов/структур DataType. Нужно организовать поиск (находится ли данный термин - переменная term структуры DataType - в списке) в массиве объектов по введенному term


Короче, вот исходное задание, если так понятнее будет:
Текст помощи для некоторой программы организован в виде линейного списка. Каждая компонента текста помощи содержит термин (слово) и текст, содержащий пояснения к этому термину. Организовать вставку и удаление новых терминов в список. Также поиск: по введенному термину должен выводить термин и определение к нему

Добавлено через 5 минут
Цитата Сообщение от MrGluck Посмотреть сообщение
Добавлено через 1 минуту
И объясните, зачем вы это делаете:
C++
1
typedef struct DataType;
typedef struct нужно использовать по заданию
MrGluck
Ворчун
Эксперт С++
 Аватар для MrGluck
5791 / 3166 / 359
Регистрация: 29.11.2010
Сообщений: 8,469
13.12.2012, 18:59     Ошибка в алгоритме перегрузки оператора присваивания #8
C++
1
2
3
4
5
6
bool Find(DataType &item) const         //Сканирование списка для нахождения соответствующего элемента.
{
    for (int i = 0; i < size; i++)
        if (ListItem[i].term != item.term) return false;
    return true;
}
Хотя по мне, так надо либо указатель на элемент массива, либо индекс массива возвращать, ну или -1, если не нашлось. Функция не оправдывает своего названия, больше подходило бы isIn.

Добавлено через 5 минут
agiliter, ваш тайпдеф невалиден.
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
13.12.2012, 19:05     Ошибка в алгоритме перегрузки оператора присваивания
Еще ссылки по теме:

Особенонсти перегрузки оператора запятая C++
Ошибка перегрузки оператора C++
C++ Неправильная работа оператора присваивания после работы оператора суммирования
C++ Избыточное копирование объекта при реализации оператора умножения и оператора присваивания
C++ Целесообразность перегрузки оператора

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

Или воспользуйтесь поиском по форуму:
go
Эксперт C++
3584 / 1364 / 128
Регистрация: 16.04.2009
Сообщений: 4,528
13.12.2012, 19:05     Ошибка в алгоритме перегрузки оператора присваивания #9
Цитата Сообщение от agiliter Посмотреть сообщение
Весь массив ListItem состоит из объектов/структур DataType. Нужно организовать поиск (находится ли данный термин - переменная term структуры DataType - в списке) в массиве объектов по введенному term
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
#include <vector>
#include <iostream>
#include <algorithm>
 
struct Data
{
   int val;
   Data(int val_) : val(val_)
   {
      
   }
   bool operator==(const int& a)
   {
      return this->val == a;
   }
};
 
 
 
int main()
{
   std::vector<Data> vec = { 1, 2, 3, 4, 5 };
   std::cout << std::boolalpha << (std::find(std::begin(vec), std::end(vec), 4) != std::end(vec)) << std::endl;
   std::cout << std::boolalpha << (std::find(std::begin(vec), std::end(vec), 8) != std::end(vec)) << std::endl;
}
http://liveworkspace.org/code/3ZG8Xo
Если правильно понял.
Yandex
Объявления
13.12.2012, 19:05     Ошибка в алгоритме перегрузки оператора присваивания
Ответ Создать тему
Опции темы

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