C++ nietypowy błąd - Code Blocks

Question

Hejka. Chciałabym się Was zapytać, co jest zrobione nie tak w kodzie, że wpisując pesel do konsoli, który powinien mieć 11 znaków, wyświetla komunikat "Błąd" i dodatkowo program wraca do początku z błędem "Niepoprawna plec!". A jeśli wpiszę długość peselu różną od 11 to wyskakuje komunikat "Błędu" i wraca do początku do "switcha" poprawnie.
Funkcja błędu:
 

void push_front()
{
    string imie, nazwisko, plec;
    int dzien, miesiac, rok_urodzenia;
    long pesel;
    cout << "WSTAWIANIE DANYCH NA POCZATEK LISTY" << endl;
    cout << "Podaj plec(Kobieta/Mezczyzna): ";
    cin >> plec;
    if(plec!="Kobieta" && plec!="Mezczyzna")
    {
        printf("Niepoprawna plec!");
        Sleep(3000);
        return;
    }
    cout << "Podaj imie: ";
    cin >> imie;
    cout << "Podaj nazwisko: ";
    cin >> nazwisko;
    cout << "Podaj dzien urodzenia: ";
    cin >> dzien;
    if(dzien>31 || dzien<1)
    {
        printf("Nieprawidlowy dzien! Sprobuj od nowa.");
        Sleep(3000);
        return;
    }
    cout << "Podaj miesiac urodzenia: ";
    cin >> miesiac;
    if(miesiac>12 || miesiac<1)
    {
        printf("Nieprawidlowy miesiac! Sprobuj ponownie.");
        Sleep(3000);
        return;
    }
    cout << "Podaj rok urodzenia: ";
    cin >> rok_urodzenia;
    if(rok_urodzenia>2019 || rok_urodzenia<1900)
    {
        printf("Nieprawidlowy rok! Sprobuj wypelnic dane jeszcze raz.");
        Sleep(3000);
        return;
    }
    std::string pesel_length_as_text = std::to_string (pesel);

    int pesel_length = pesel_length_as_text.length();
    cout << "Podaj swoj PESEL(powinien miec 11 cyfr): ";
    cin >> pesel;

    if(pesel_length != 11)
    {
        std::cout<<"Blad!";
        Sleep(3000);
        return;
    }
    else {
        std::cout<<"11";

    lista_systemu.push_front(plec[10]);
    lista_systemu.push_front(imie[30]);
    lista_systemu.push_front(nazwisko[30]);
    lista_systemu.push_front(dzien);
    lista_systemu.push_front(miesiac);
    lista_systemu.push_front(rok_urodzenia);
    lista_systemu.push_front(pesel);

}
}

 

Answer 1

long pesel; // deklarujesz tutaj zmienną pesel
std::string pesel_length_as_text = std::to_string (pesel); // zamieniasz int na string, pomimo,
                                                         // że int nie ma wczytanej wartości
 
int pesel_length = pesel_length_as_text.length();           // określasz długość zmiennej zawierającej losową wartość
cout << "Podaj swoj PESEL(powinien miec 11 cyfr): ";
cin >> pesel;                 // tu dopiero wczytujesz
 
if(pesel_length != 11)
{
    std::cout<<"Blad!";
    Sleep(3000);
    return;
}
else {
    std::cout<<"11";

Prześledź komentarze, które dopisałem do twojego kodu.

Comments

Hmm... rozumiem :) Spróbowałam zrobić w ten sposób jeszcze, jednak rezultat otrzymałam taki sam:
 

cout << "Podaj swoj PESEL(powinien miec 11 cyfr): ";
    cin >> pesel;
    std::string pesel_length_as_text = std::to_string (pesel);
    int pesel_length = pesel_length_as_text.length();

    if(pesel_length != 11)
    {
        std::cout<<"Blad!";
        Sleep(3000);
        return;
    }
    else {
        std::cout<<"11";

 

---

Zmień "long pesel" na "string pesel".

Skoro i tak od razu to chcesz zamienić na string.

A tak poza tym, maksymalna wartość long wynosi tyle samo co int, czyli 2,147,483,647.

Jeżeli chcesz używać większych wartości to używaj "long long".

Jednak żaden z powyższych typów zmiennych nie przechowa 11 cyfrowej liczby.

---

Wtedy program wyrzuca błąd w tej linijce:
 

std::string pesel_length_as_text = std::to_string (pesel);

O treści:
no matching function for call 'to_string(std::__cxxll::string&)'

---

Nie deklaruj zmiennej pesel jako zmienną liczbową tylko od razu jako string, wtedy nie będziesz musiała w ogóle konwertować tej zmiennej.

---

Nie wiem, czy dobrze zrozumiałam. Masz na myśli to?
 

string pesel_length = pesel.length();
    if(pesel_length != 11)
    {
        std::cout<<"Blad!";
        Sleep(3000);
        return;
    }

 

---

Nie. Chodzi mi o piątą linijkę twojego kodu.

---

length nie zwraca wartości typu std::string. Tak zrób:

if(pesel.length() != 11) { ... }

 

---

Zrobiłam w ten sposób:
 

cout << "Podaj swoj PESEL(powinien miec 11 cyfr): ";
    cin >> pesel;
    if(pesel.length() != 11)
        {
            std::cout<<"Blad!";
            Sleep(3000);
            return;
        }       
    else {
    std::cout<<"11";

Teraz jest dobrze. :) A tak zapytam, jak sprawić, żeby te dane, które wpisałam, zapisały się tak, jak je wpisałam, a nie w postaci liczb? Chodzi o to, żeby pojawiło się na ekranie:
Kobieta - Anna Kowalska - 01-01-1950 - PESEL: 50010102345
A wyskakuje coś takiego:
76
52
3578532
2382
I tak dalej :P

---

cout << plec << " - ";
cout << imie << " " << nazwisko << " - ";
cout << dzien << " - " << [reszta daty...];
cout << "PESEL: " <<  pesel;

 

---

Hmm... oki :) A czy to nie będzie kolidowało z tym?
 

    lista_systemu.push_front(plec[10]);
    lista_systemu.push_front(imie[30]);
    lista_systemu.push_front(nazwisko[30]);
    lista_systemu.push_front(dzien);
    lista_systemu.push_front(miesiac);
    lista_systemu.push_front(rok_urodzenia);
    lista_systemu.push_front(pesel.length());

 

---

lista_systemu to lista z bliblioteki <list> ?

Zadeklarowana globalnie?

---

Dokładnie tak. :)
 

#include <list>
#include <iostream>
#include <windows.h>
#include <string>

using namespace std;

list<int>lista_systemu; //utworzenie listy, która przechowuje liczby całkowite
int funkcja;

Tylko właśnie tutaj jest <int>, a dane są mieszane, jak można zabrać się za taki problem?

---

Dlaczego lista jest typu int, skoro chcesz tam przechowywać ciągi znaków?

Poza tym, jaki cel jest w wpisaniu w listę tych danych? Nie lepiej to zrobić jako struct?

lista_systemu.push_front(imie[30]); // dlaczego imie[30] ?

---

Tak tylko dodam, ze wzialem i przepisalem troche kodu.

https://pastebin.com/4Bww9YQH

Jak chcesz mozesz go edytowac, bo ten wedlug mnie ma wiekszy porzadek itd. Ale jak uwazasz. Musialem wyciac wszytsko co jest windowsowe, bo by mi sie nie kompilowalo. A co do reszty, to jak juz to musisz zrobic liste typu string a nie int, a najlepiej to strukture, w ktorej to przechowywac bedziesz. Pamietaj, ze uzywajac bibliothek windowsowych, odpalisz kod tylko na windowsie.

---

@gorgonek, użyj struktury:

struct wpis
{
	std::string imie;
	std::string nazwisko;
	std::string plec;
	uint8_t dzien;
	uint8_t miesiac;
	uint16_t rok_urodzenia;
	std::string pesel;
};



std::list<wpis>lista_systemu; 
...


wpis w;

std::cout << "Podaj imie: ";
std::cin >> w.imie;
std::cout << "Podaj nazwisko: ";
std::cin >> w.nazwisko;
std::cout << "Podaj dzien urodzenia: ";
std::cin >> w.dzien;
...


lista_systemu.push_back(w);

 

---

Hmm początkowo pomysł był taki, żeby przechowywała zmienne, które będą się dodawać. Mam na myśli numerowanie. Pewnie byłoby lepiej w strukturze, jednak... nigdy nie robiłam jej w C++, a w C to jest zmora wskaźników... Hmm chyba muszę Ciebie spytać wprost, czy istnieje jakiś sposób, aby zrobić ten projekt w 19 dni, mając wiedzę na poziomie bardzo podstawowym?
PS. imie[30] miało służyć temu, żeby ktoś nie zrobił spamu liter dowolnych i nie zajmował wiele pamięci, a czy to ma jakiś większy sens? Pewnie nie jest to potrzebne w tym wypadku, po prostu uznałam, że kod jest na tyle "łysy", że coś jednak w nim musi być :P

---

Struktury w C++ są dość łatwe (nie wiem jak to wygląda w przypadku C).

Przykład:

struct Nazwa {

int jakasLiczba;
string jakisString;
char znak;
}

int funkcja() {

Nazwa x;
x.jakasLiczba = 5;
x.jakisString = "abc";

cout << x.jakasLiczba;
}

 

---

początkowo pomysł był taki, żeby przechowywała zmienne

Problem w tym, że zmienne są różnego typu, a kontener jest homogeniczny i nie możesz dodawać do niego wartości różnych typów. Użycie struktury/klasy jest najbardziej racjonalne.

 

nie wiem jak to wygląda w przypadku C

Dokładnie tak samo :) C++ odziedziczyło struktury po C.

---

Zwykle uczyłam się na pamięć tych wszystkich struktur, bo nie rozumiałam tych gwiazdek i ampersandów, a też jak wiadomo, żeby zrozumieć nawet te najprostsze rzeczy, czasem wystarczy kilka skondensowanych słów, które trafią do użytkownika. Nie ukrywam, że mniej więcej połowa lub więcej rzeczy, o które pytacie, to sama nie wiem dokładnie o co pytacie, ale cóż... trzeba to przebrnąć :P
Spróbuję przekształcić cały kod na strukturę stosu i może coś z tego wyjdzie... i jak tylko skończę to wrzucę go tutaj niebawem :)

---

Spróbuj napisać co umiesz, a w razie problemów wrzuć kod, to pomożemy :)

Jak czegoś nie wiesz to pytaj.

---

Heh dziękuję :)) Hmm... tak teraz się zastanawiam... czy żeby uniknąć przekształcania całego kodu, nie można by było zawrzeć całej struktury w tą listę? Czyli tam zamiast inta, tak jak tutaj ktoś z Was napisał: <wpis>.

---

Oczywiście, że można. Wystarczy, że zamienisz typ listy z int na nazwę twojej struktury (w tym przypadku <wpis>)

---

@gorgonek,  przecież pokazałem Ci, jak to może wyglądać.

---

Hmm... to ciekawe. Gdy przekształciłam kod w taki sposób:
 

#include <list>
#include <iostream>
#include <windows.h>
#include <string>

using namespace std;

struct wpis
{
	std::string imie;
	std::string nazwisko;
	std::string plec;
	int dzien;
	int miesiac;
	int rok_urodzenia;
	std::string pesel;
};

wpis w;

list<wpis>lista_systemu; //utworzenie listy, która przechowuje liczby całkowite
int funkcja;

void Wyswietl_Dane()
{
    system("CLS");
    SetConsoleTextAttribute(GetStdHandle(STD_OUTPUT_HANDLE), 12);
    cout << "SYSTEM ZARZADZANIA OBIEGIEM INFORMACJI: " << endl;
    cout <<"-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-" << endl;
    for(list<wpis>::iterator w=lista_systemu.begin(); w!=lista_systemu.end(); ++w)
    {
        cout << *w << " ";

        cout << endl;
        cout << "-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-" << endl << endl;
        SetConsoleTextAttribute(GetStdHandle(STD_OUTPUT_HANDLE), 14);

    }
}

To jest jeszcze ta wersja windowsowa (ponieważ może być uruchamiana z tego komputera :P). Tutaj pokazuje błąd przy linijce 32:
cout << *w << " ";
A opis błędu mówi:
Error: no match for 'operator<<' (operand types are 'std::ostream {aka std::basic_ostream<char>}') and 'wpis'

---

cout << w->imie << ' ' << w->nazwisko << ' '  << /* itd. */;

 

---

Ale w tym przypadku "w" nie jest wskaźnikiem, więc odwołanie chyba jest za pomocą kropki?

---

W pętli w jest iteratorem.

---

A faktycznie, zasugerowałem się tą linijką:

wpis w;