Поиск объектов на C++

@generalghest · Регистрация: 16.01.2018

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

Доброго времени суток. Я пишу программу для коллекционера, использую контейнер <vector>. При этом я пытаюсь использовать поиск по объектам find() из библиотеки #algorythm , но сталкиваюсь с ошибкой: find: не является членом "std::vector<CollectionObject,std::alloc ator<_Ty> >" .
Подскажите пожалуйста, что я делаю не так. Спасибо

Код
main.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
#include <iostream>
#include <locale.h>
#include "collection.h"
#include "collectionobject.h"
#include <conio.h>
#include <windows.h>
#include <algorithm>
 
using namespace std;
 
int main(){
    SetConsoleCP(1251);
    SetConsoleOutputCP(1251);
    setlocale(LC_ALL, "RUS");
    Collection collection;
    string name, country;
    int index;
    short set = 1;
    while (set!=0){
        cout<<"\nНажмите 1 для добавления предмета\n";
        cout<<"Нажмите 2 для удаления предмета по индексу в коллекции\n";
        cout<<"Нажмите 3 для просмотра всей коллекции\n";
        cout<<"Нажмите 4 для удаления всей коллекции\n";
        cout<<"Нажмите 5 для поиска по Вашей коллекции\n";
        cout<<"Нажмите 6 для сортировки Вашей коллекции\n";
        cout<<"Нажмите 0 для выхода\n";
        cin>>set; getline(cin, name);
        switch(set){
        case 0: {
                cout<<"\nВсего доброго!";
                set = 0;
                break;
        }
        case 1: {
                //добавление объекта
                cout<<"Добавим предмет\n";
                cout<<"Введите название и страну откуда Вы его привезли\n";
                getline(cin, name);
                getline(cin, country);
                CollectionObject tmp(name, country);
                collection.Add(tmp);
                cout<<"Предмет добавлен!\n";
                break;
        }
        case 2: {
                //удаление объекта
                cin>>index;
                collection.Del(index-1);
                break;
        }
        case 3: {
                //вывод объектов
                collection.Print();
                break;
        }
        case 4: {
                //удаление всех объектов
                collection.ClearAll();
                break;
        }
        case 5: {
                //поиск объекта по индексу
                cin>>index;
                collection.Find(index);
                break;
        }
 
        case 6: {
                //сортировка объектов по алфавиту
                cout<<"Отсортируем Вашу коллекцию\n";
                sort(collection.begin(), collection.end());
                break;
 
                default: cout<<"\nВы не выбрали ни одного из вариантов\n"; set=100;;
         }
        }
    }
}

collection.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
#ifndef COLLECTION_H
#define COLLECTION_H
 
#include <vector>
#include <locale.h>
#include <algorithm>
#include "collectionobject.h"
 
class Collection{
private:
    vector<CollectionObject> collection;
 
public:
    Collection():collection{}
    {}
    void Add(CollectionObject someObject);
    void Del(int index);
    void Print();
    void ClearAll();
    void Find(int index);
    void Begin();
    void End();
    using iter = decltype(collection)::iterator;
    iter begin() { return collection.begin(); }
    iter end() { return collection.end(); }
};
 
#endif // COLLECTION_H

colleciton.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
#include <locale.h>
#include "collection.h"
#include <algorithm>
 
 
 
void Collection::Add(CollectionObject someObject){
    collection.push_back(someObject);
}
void Collection::Del(int index){
    collection.erase(collection.begin() + index);
}
 
void Collection::Print(){
    for(unsigned int i = 0 ; i < collection.size(); i++){
        cout<<(i+1)<<". ";
        collection.at(i).ShowName();
        collection.at(i).ShowCountry();
    }
}
 
void Collection::ClearAll(){
    collection.clear();
}
 
void Collection::Find(int index){
    collection.find(index);
}
 
void Collection::Begin() {
    collection.front();
}
 
void Collection::End() {
    collection.back();
}

collectionobject.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
#ifndef COLLECTIONOBJECT_H
#define COLLECTIONOBJECT_H
 
#include <string>
#include <iostream>
#include <locale.h>
#include <algorithm>  
 
using namespace std;
 
class CollectionObject{
private:
    string name;
    string country;
public:
    CollectionObject(string someName, string someCountry):name(someName), country(someCountry)
    {}
    void ShowName();
    void ShowCountry();
    bool operator<(const CollectionObject& rh) const;
 
};
 
#endif // COLLECTIONOBJECT_H

collectionobject.cpp

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
#include <locale.h>
#include <iomanip>
#include "collectionobject.h"
 
void CollectionObject::ShowName(){
    cout << setiosflags(ios::left) << setw(10) << name << "  ";
}
 
void CollectionObject::ShowCountry(){
    cout << setw(20) <<country<<"\n";
}
 
 
bool CollectionObject::operator<(const CollectionObject& rh) const
{
    return name < rh.name;
}

@oleg-m1973 · 06.06.2019, 11:00

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

дописал в скобках string name чтобы передать эту строковую переменную
Ошибка всё-равно не ушла:

Сделай для проверки

C++
1
2
3
4
class CollectionObject{
public:
    string name;
    string country;

@generalghest · 06.06.2019, 11:58 **[ТС]**

Ошибка ушла, но ничего не выводится после того как прошел поиск.

@oleg-m1973 · 06.06.2019, 12:01

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

Ошибка ушла, но ничего не выводится после того как прошел поиск.

А нашло что-нибудь?

@generalghest · 06.06.2019, 12:05 **[ТС]**

Не знаю. Вывода ведь нет

@oleg-m1973 · 06.06.2019, 12:07

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

Не знаю. Вывода ведь нет

Воспользуйся отладчиком

IGPIGP · 06.06.2019, 14:19

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

Вывода ведь нет

Поиск возвращает итератор. По итератору и выводите. Но если, итератор смотрит в end() (не нашли ни чего) то у if ветвь else пусть скажет "Ne"

C++
1
2
3
4
5
6
if (it != collection.end())
{
    size_t ind = it - collection.begin();
    std::cout << "Index: "<< ind << endl;
    std::cout << "Name: "<< it->name << endl;
}else std::cout<<"Ne-a!";

@generalghest · 06.06.2019, 20:32 **[ТС]**

Добавил if и теперь понял, что проблема в работе find. На все что я пишу он отвечает как раз Ne-a.
Может что-то с кодировкой?

Весь код:
main.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
#include <iostream>
#include <locale.h>
#include "collection.h"
#include "collectionobject.h"
#include <conio.h>
#include <windows.h>
#include <algorithm>
 
using namespace std;
 
int main() {
    SetConsoleCP(1251);
    SetConsoleOutputCP(1251);
    setlocale(LC_ALL, "RUS");
    Collection collection;
    string name, country, find_word;
    int index;
    short set = 1;
    while (set != 0) {
        cout << "\nНажмите 1 для добавления предмета\n";
        cout << "Нажмите 2 для удаления предмета по индексу в коллекции\n";
        cout << "Нажмите 3 для просмотра всей коллекции\n";
        cout << "Нажмите 4 для удаления всей коллекции\n";
        cout << "Нажмите 5 для поиска по Вашей коллекции\n";
        cout << "Нажмите 6 для сортировки Вашей коллекции\n";
        cout << "Нажмите 0 для выхода\n";
        cin >> set; getline(cin, name);
        switch (set) {
        case 0: {
            cout << "\nВсего доброго!";
            set = 0;
            break;
        }
        case 1: {
            //добавление объекта
            cout << "Добавим предмет\n";
            cout << "Введите название и страну откуда Вы его привезли\n";
            getline(cin, name);
            getline(cin, country);
            CollectionObject tmp(name, country);
            collection.Add(tmp);
            cout << "Предмет добавлен!\n";
            break;
        }
        case 2: {
            //удаление объекта
            cout << "Введите номер объекта которы Вы хотите удалить";
            cin >> index;
            collection.Del(index - 1);
            break;
        }
        case 3: {
            //вывод объектов
            collection.Print();
            break;
        }
        case 4: {
            //удаление всех объектов
            collection.ClearAll();
            cout << "Ваша коллекция очищена!";
            break;
        }
        case 5: {
            //поиск объекта
            cin >> find_word;
            collection.Find(find_word, name);
            break;
        }
 
        case 6: {
            //сортировка объектов по алфавиту
            cout << "Отсортируем Вашу коллекцию\n";
            sort(collection.begin(), collection.end());
            break;
 
        default: cout << "\nВы не выбрали ни одного из вариантов\n"; set = 100;;
        }
        }
    }
}

collection.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
#ifndef COLLECTION_H
#define COLLECTION_H
 
#include <vector>
#include <locale.h>
#include <algorithm>
#include "collectionobject.h"
 
class Collection{
private:
    vector<CollectionObject> collection;
 
public:
    Collection():collection{}
    {}
    void Add(CollectionObject someObject);
    void Del(int index);
    void Print();
    void ClearAll();
    void Find(string find_word, string name);
    void Begin();
    void End();
    using iter = decltype(collection)::iterator;
    iter begin() { return collection.begin(); }
    iter end() { return collection.end(); }
};
 
#endif // COLLECTION_H

collection.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
#include <locale.h>
#include "collection.h"
#include <algorithm>
 
 
 
void Collection::Add(CollectionObject someObject){
    collection.push_back(someObject);
}
void Collection::Del(int index){
    collection.erase(collection.begin() + index);
}
 
void Collection::Print(){
    for(unsigned int i = 0 ; i < collection.size(); i++){
        cout<<(i+1)<<". ";
        collection.at(i).ShowName();
        collection.at(i).ShowCountry();
    }
}
 
void Collection::ClearAll(){
    collection.clear();
}
 
void Collection::Find(string find_word, string name){
    std::find(collection.begin(), collection.end(), find_word);
    auto it = std::find(collection.begin(), collection.end(), find_word);//итератор на искомый объект
    if (it != collection.end())
    {
        size_t ind = it - collection.begin();
        std::cout << "Index: " << ind << endl;
        std::cout << "Name: " << it->name << endl;
    }
    else std::cout << "Ne-a!";
}
 
void Collection::Begin() {
    collection.front();
}
 
void Collection::End() {
    collection.back();
}

collectionobject.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
#ifndef COLLECTIONOBJECT_H
#define COLLECTIONOBJECT_H
 
#include <string>
#include <iostream>
#include <locale.h>
#include <algorithm>  
 
using namespace std;
 
class CollectionObject{
private:
    //string name;
    //string country;
public:
    string name;
    string country;
    CollectionObject(string someName, string someCountry):name(someName), country(someCountry)
    {}
    void ShowName();
    void ShowCountry();
    bool operator<(const CollectionObject& rh) const;
    //bool operator ==(const CollectionObject& rh) const;
    bool operator ==(const std::string& rh) const;
};
 
#endif // COLLECTIONOBJECT_H

collectionobject.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
#include <locale.h>
#include <iomanip>
#include "collectionobject.h"
 
void CollectionObject::ShowName(){
    cout << setiosflags(ios::left) << setw(20) << name << "  ";
}
 
void CollectionObject::ShowCountry(){
    cout << setw(10) <<country<<"\n";
}
 
 
bool CollectionObject::operator<(const CollectionObject& rh) const
{
    return name < rh.name;
}
 
/*bool CollectionObject::operator==(const CollectionObject& rh) const
{
    return name == rh.name;
}*/
 
bool CollectionObject::operator==(const std::string& rh) const
{
    return name == rh;
}

@oleg-m1973 · 06.06.2019, 20:35

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

Добавил if и теперь понял, что проблема в работе find. На все что я пишу он отвечает как раз Ne-a.
Может что-то с кодировкой?

Вполне возможно. Проверь на английском

@generalghest · 06.06.2019, 21:18 **[ТС]**

Нет. Что-то другое. Кодировку я уже лечил.

IGPIGP · 06.06.2019, 22:05

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

Нет. Что-то другое. Кодировку я уже лечил.

это не кодировка, а скорее локализация. Распечатайте все имена в коллекции и сравните с той строкой, которую задали как ключ поиска (и его распечатайте). Чудес не бывает.

@generalghest · 06.06.2019, 22:15 **[ТС]**

Я не совсем понял что нужно распечатывать. Все переменные из класса Collection? Ключ поиска это переменная find_word? Объясните пожалуйста что сравнивать.

IGPIGP · 06.06.2019, 23:18

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

Я не совсем понял что нужно распечатывать.

Почти поняли. Но слово "нужно" не совсем подходит. Попытка соответствует определению "можно", потому как, если зациклиться на том, что "нужно", то можно просидеть в ступоре неограниченное время. Отладка - набор попыток.

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

Все переменные из класса Collection?

имена (поля name), вы же среди них ищете. Вдруг у вас там все строки пустые или ещё чего-то странное с инициализацией случилось. Если коллекция слишком велика - сократите для теста до пары десятков объектов.

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

Ключ поиска это переменная find_word?

Да вроде больше и нет варинатов.

@zayats80888 · 06.06.2019, 23:27