C++ przeszukanie wektora

Question

Siemka, może mi tutaj ktoś na szybko doradzić, jak przeszukać wektor?

std::vector<Patient> patients;
 
//dalszy kod
 
patients.push_back(Patient{id, name, surname, age, phone_number});

Na przykład chciałbym przeszukać wszystkich pacjentów i znaleźć ID 2. Jak to zrobić? Wiem, ze na pewno trzeba będzie użyć pętli, ale... jakaś sugestia, co dalej?

Answer 1

unsigned int id = 10; // powiedzmy, że to jest id szukanego pacjenta
for( std::vector<Patient>::iterator i = patients.begin(); i < patients.end(); i++){
    if(i->id == id){
        // tutaj wypisujesz dane pacjenta
        break; // zakładam, ze jednemu pacjentowi przypisane jest unikalne id więc kończę pętlę
    }
}

W powyższym kodzie dla uproszczenia założyłem, że pole id klasy Patient jest publiczne.

Comments

Hmm... jednej rzeczy nie rozumiem. Wypisuję sobie dane, no dobra, ale jak wpiszę tam na przykład taką linijkę, to wywala mi błąd:

  std::cout << "Podaj id pacjenta: ";
            int id;
            std::cin >> id;

            for(std::vector<Patient>::iterator i = patients.begin(); i < patients.end(); i++)
                {
                    if(i -> id == id)
                    {
                        std::cout << "   " << id[i] << "      " << name[i] << "      " << surname[i] << "      " << age[i] << "       " << phone_number[i] << "\n";
                     // tutaj wypisujesz dane pacjenta
                    break; // zakładam, ze jednemu pacjentowi przypisane jest unikalne id więc kończę pętlę
                    }
                }

 

---

W powyższym kodzie i jest wskaźnikiem na element zawarty w kontenerze typu vector, więc jak chcesz wypisywać to robisz to tak:

unsigned int id = 10; // powiedzmy, że to jest id szukanego pacjenta
for( std::vector<Patient>::iterator i = patients.begin(); i < patients.end(); i++){
    if(i->id == id){
        // tutaj wypisujesz dane pacjenta
        std::cout<<"id pacjenta: "<<i->id<<" imię pacjenta: "<<i->name; 
        break; // zakładam, ze jednemu pacjentowi przypisane jest unikalne id więc kończę pętlę
    }
}

 

---

No i fajnie, dziękuję za wyjaśnienie tego. Teraz musiałbym zrobić funkcję zaprzyjaźnioną, bo moje atrybuty są prywatne (id, nazwisko, etc).

Zrobiłem dwie funkcje zaprzyjaźnione:

bool patient_id(const Patient& a)
{
    return a.id;
}
bool patient_name(const Patient& a)
{
    return a.name;
}

Tyle, że... w konsoli wywalił mi błąd:

C:\Users\User\Desktop\Baza danych C++\main.cpp||In function 'bool patient_name(const Patient&)':|
C:\Users\User\Desktop\Baza danych C++\main.cpp|88|error: cannot convert 'const string {aka const std::basic_string<char>}' to 'bool' in return|

 

---

Zauważ, że w nagłówku tych funkcji zadeklarowałeś, że zwracaną wartością będzie wartość logiczna bool (true lub false), natomiast w definicji funkcji oczekujesz zwrotu id, które jest liczbą np. int lub innym typem np. string.

Wniosek: Zmień bool w nagłówku na faktyczny typ zmiennej id.

PS. To niekoniecznie muszą być funkcje zaprzyjaźnione (nie widzę pliku nagłówkowego - bo przy deklaracji musiałoby być słówko friend). Tutaj mówisz o tzw. getterach, czyli funkcjach które zwracają np. prywatne pola klas. Istnieją również settery, czyli metody ustawiające prywatne pola klas ;)

Answer 2

Prosto i elegancko:

#include <algorithm>
vector<Patient>::iterator it = find_if(pacjenci.begin(), pacjenci.end(), [](Patient p){ return p.id == 2; });

 

Answer 3

Dlaczego w tym kodzie:

unsigned int id = 10; // powiedzmy, że to jest id szukanego pacjenta
for( std::vector<Patient>::iterator i = patients.begin(); i < patients.end(); i++){
    if(i->id == id){
        // tutaj wypisujesz dane pacjenta
        std::cout<<"id pacjenta: "<<i->id<<" imię pacjenta: "<<i->name; 
        break; // zakładam, ze jednemu pacjentowi przypisane jest unikalne id więc kończę pętlę
    }
}

Wyszukuje mi tylko i wyłącznie osobę /rekord na pierwszej pozycji w wektorze, a nie przeszukuje całego wektora? Na przykład jak próbuję znaleźć osobę w środku lub na końcu, to wywala mi błąd, zamiast wskazać poprawny wpis.

Comments

Ten kod jest w porządku. Jaki błąd Ci "wywala"?

Na marginesie, ten kod mógłby być odrobinę bardziej przejrzysty:

for(const auto &patient : patients){
    if(patient.id == id){
        // tutaj wypisujesz dane pacjenta
        std::cout<<"id pacjenta: "<<patient.id<<" imię pacjenta: "<<patient.name; 
        break; // zakładam, ze jednemu pacjentowi przypisane jest unikalne id więc kończę pętlę
    }
}

 

---

Znaczy, po prostu... Wyszukuje mi za każdym razem tylko i wyłącznie dane, które znajdują się przy pierwszym pacjencie. Jeżeli chcę wyszukać ID/Imię u osób, które znajdują się w wektorze na kolejnych pozycjach, to po prostu nic mi nie znajduje.

---

Ten kod jest poprawny. Sprawdź, czy Twój wektor rzeczywiście zawiera elementy, których się tam spodziewasz, np. wypisz je sobie przed wyszukaniem.

---

Hmm... na pewno zawiera.

http://imgur.com/a/vJfUf

---

To pokaż jeszcze kontekst, czyli kod za to odpowiedzialny.