Książka adresowa jako struktura

Question

Piszę program - książka adresowa, w którym chcę mieć nazwisko, nr tel oraz nazwę ulicy mieszkańców. Przy czym mój program w nazwie ulicy ma zmieniać  literę B i Z na Y. Nie wiem jak mam wywołać taką funkcję, bo kompilator wyrzuca mi błędy. Jeszcze jedno pytanie: Czy jest możliwym określenie długości obiektu nazwaUlicy, który należy do struktury (wers 16.)  

#include <iostream>
#include <cstdlib>
#include <string>

using namespace std;

struct osoba
{
    string nazwisko;
    long long int nr_tel;
    char nazwaUlicy[20];
};

void usunZKsiazkiAdresowej (osoba &adres)
{
    int dlugosc = osoba.nazwaUlicy.size();

    for (int i = 0; i < dlugosc; i++)
    {
        if(adres.nazwaUlicy[i] == 'B' || adres.nazwaUlicy[i] == 'Z')
        {
           adres.nazwaUlicy[i] = 'Y';
        }

    }

}

int main()
{
    cout << "Witaj w ksiazce telefonicznej. Zaczynamy zabawe!" << endl << endl;

    int rozmiar;
    cin >> rozmiar;

    int *wielkosc = new int [rozmiar];
    osoba *mieszkancy = new osoba;
    for (int i = 0; i < rozmiar; i++)
    {
        cin >> mieszkancy->nazwisko >> mieszkancy->nr_tel >> mieszkancy->nazwaUlicy;
    }

    usunZKsiazkiAdresowej()


    delete [] wielkosc;
    delete mieszkancy;


    return 0;
}

 

Comments

Czy to ma byc C++, czy C?

---

@magda_19, osoba::nazwisko jest typu std::string, a osoba::nazwaUlicy to char[20]. WTF? Skąd ta niekonsekwencja?
 

---

@DragonCoder, C++

---

Nie wiem,  jakoś tak mi się char napisało

---

No to zmień tę tablicę na std::stringa i problem z linii 16 zniknie.

Answer 1

Co do linii 16. jeśli chcesz określić rozmiar tablicy typu char polecam skorzystać z funkcji sizeof(nazwaUlicy);

W funkcji main wydaję mi się, że powinnaś zrobić tablice dynamiczną typu osoba, z tego co widzę rezerwujesz jedną przestrzeń pamięci na "jedną osobę". 

osoba *mieszkancy = new osoba; - > osoba *mieszkancy = new osoba[rozmiar];

I w funkcji  usunZKsiazkiAdresowej musisz podesłać w argumencie strukturę jak to zadeklarowałaś, dlatego wyskakują błędy. 

A co do pytania długość nazwaUlicy można określić przez zmienną typu const nad strukturą, żeby była widoczna globalnie,albo dodanie po prostu do struktury zmiennej typu int przechowującą tą wartość. Ale nie wiem czy do końca oto Ci chodziło.

 

Comments

To już lepiej, jak już musi być tablica char, zamiast tego sizeof niech użyje strlen.

---

No nie, bo jak chce określić faktyczny rozmiar to używa sizeof, a jak długość do strlen. strlen () wyszukuje znak NULL i zlicza liczbę przekazanych adresów pamięci, więc faktycznie liczy liczbę elementów obecnych w ciągu przed znakiem NULL, a sizeof () zwraca rzeczywistą ilość pamięci przydzielonej dla operand.

---

A no tak macie racje, mój błąd.

Answer 2

Utworzyłam dodatkową zmienną w strukturze, dlugosc, ale nie wiem jak się do niej odwołać w pętli for, bo nadal nie chcę się kompilować

#include <iostream>
#include <cstdlib>
#include <string>

using namespace std;

struct osoba
{
    string nazwisko;
    long long int nr_tel;
    char nazwaUlicy[20];
    const int dlugosc = sizeof(nazwaUlicy);


};

void usunZKsiazkiAdresowej (osoba &adres)
{

    for (int i = 0; i < osoba.dlugosc; i++)
    {
        if(adres.nazwaUlicy[i] == 'B' || adres.nazwaUlicy[i] == 'Z')
        {
            adres.nazwaUlicy[i] = 'Y';
        }

    }

}

int main()
{
    cout << "Witaj w ksiazce telefonicznej. Zaczynamy zabawe!" << endl << endl;

    int rozmiar;
    cin >> rozmiar;

    int *wielkosc = new int [rozmiar];
    osoba *mieszkancy = new osoba[rozmiar];
    for (int i = 0; i < rozmiar; i++)
    {
        cin >> mieszkancy->nazwisko >> mieszkancy->nr_tel >> mieszkancy->nazwaUlicy;
    }

    //usunZKsiazkiAdresowej()


    delete [] wielkosc;
    delete mieszkancy;


    return 0;
}

 

Comments

void usunZKsiazkiAdresowej (osoba &adres)
{
 
    for (int i = 0; i < osoba.dlugosc; i++)
    {
        if(adres.nazwaUlicy[i] == 'B' || adres.nazwaUlicy[i] == 'Z')
        {
            adres.nazwaUlicy[i] = 'Y';
        }
 
    }
 
}


Na to:


void usunZKsiazkiAdresowej (osoba &adres)
{

    for (int i = 0; i < adres.dlugosc; i++)
    {
        if(adres.nazwaUlicy[i] == 'B' || adres.nazwaUlicy[i] == 'Z')
        {
            adres.nazwaUlicy[i] = 'Y';
        }

    }

}

Masz błąd w argumencie funkcji. 

---

Czyli teraz funkcja powinna powinna przechodzić po każdej literce w nazwie ulicy, tak aby zamieniała spotkane literki B lub Z na Y?

---

#include <iostream>
#include <cstdlib>
#include <string>

using namespace std;

struct osoba
{
    string nazwisko;
    long long int nr_tel;
    char nazwaUlicy[20];
    const int dlugosc = sizeof(nazwaUlicy);
};

void usunZKsiazkiAdresowej (osoba &adres)
{

    for (int i = 0; i < adres.dlugosc; i++)
    {
        if(adres.nazwaUlicy[i] == 'B' || adres.nazwaUlicy[i] == 'Z')
        {
            adres.nazwaUlicy[i] = 'Y';
        }
    }
}

int main()
{
    cout << "Witaj w ksiazce telefonicznej. Zaczynamy zabawe!" << endl << endl;

    int rozmiar;
    cin >> rozmiar;

    int *wielkosc = new int [rozmiar];
    osoba *mieszkancy = new osoba[rozmiar];
    for (int i = 0; i < rozmiar; i++)
    {
        cin >> mieszkancy->nazwisko >> mieszkancy->nr_tel >> mieszkancy->nazwaUlicy;
    }
    for (int i = 0; i < rozmiar; i++)
    {
        usunZKsiazkiAdresowej(mieszkancy[i]);
    }
    std::cout<<mieszkancy[0].nazwaUlicy;


    delete [] wielkosc;
    delete mieszkancy;


    return 0;
}

Tak

---

#include <iostream>
#include <cstdlib>
#include <string>
#include <vector>

struct osoba
{
    std::string nazwisko;
    long long int nr_tel;
    std::string nazwaUlicy;
};

void usunZKsiazkiAdresowej (osoba &adres)
{

    for (int i = 0; i < adres.nazwaUlicy.length(); i++)
    {
        if((toupper(adres.nazwaUlicy[i]) == 'B') || (toupper(adres.nazwaUlicy[i]) == 'Z'))
        {
            adres.nazwaUlicy[i] = 'Y';
        }
    }
}

int main()
{
    std::cout << "Witaj w ksiazce telefonicznej. Zaczynamy zabawe!" << std::endl << std::endl;

    int rozmiar;
    std::cin >> rozmiar;
    std::vector<osoba> mieszkancy;

    for (int i = 0; i < rozmiar; i++) {
        mieszkancy.push_back(osoba());
        std::cin >> mieszkancy[i].nazwisko;
        std::cin >> mieszkancy[i].nr_tel;
        std::cin >> mieszkancy[i].nazwaUlicy;
    }
    for (int i = 0; i < rozmiar; i++) {
        usunZKsiazkiAdresowej(mieszkancy[i]);
    }

    mieszkancy.clear();

    return 0;
}

Czy to nie wyglada lepiej?

---

Jak się już czepiamy co do "czystości", to powinieneś użyć jeszcze reserve i emplace_back. Nie potrzebne clear, to samo zrobi destruktor, a nawet zwolni pamięć, czego clear nie robi. Metoda at jest bezpieczniejsza od [].

---

emplace_back niewiele tu zmieni, bo osoba nie ma konstruktora parametrycznego.

---

Faktycznie. Celem użycia emplace_back jest uniknięcie tworzenia obiektu tymczasowego, który jest następnie kopiowany (lub przenoszony) do miejsca docelowego. A bez konstruktora nie ma to sensu. Żeby zadziałało emplace_back musiał by wywoływać konstruktor. Dzięki.