JPESEL - Pesel

Witam, czy jest ktoś w stanie dostrzec błąd? https://ideone.com/NcQnXS21

weź swój pesel i sprawdź, dlaczego jest zły

Mój wychodzi jako poprawny Czyli wróciłem do punktu wyjścia.

to podaję dobry pesel: 99040112345

Hej. Ja zrobiłem to tak. Na moim kopi działa dobrze wyniki są poprawne.
Na SPOJ nie działa. Dlaczego?

#include <iostream>
#include <string>
#include <cstdlib>
#include <math.h>
using namespace std;

int main()
{
	string pesel;
	string pomocnicza="";
	double wynik=0;
	int tab[11];
	short ile;
	cin >> ile;
	for (int k = 0; k < ile; k++)
	{
		cin >> pesel;

		for (int i = 0; i < 11; i++)
		{
			pomocnicza.push_back(pesel[i]);
			tab[i] = atoi(pomocnicza.c_str());
			pomocnicza = "";
		}
		tab[0] = tab[0] * 1;
		tab[1] = tab[1] * 3;
		tab[2] = tab[2] * 7;
		tab[3] = tab[3] * 9;
		tab[4] = tab[4] * 1;
		tab[5] = tab[5] * 3;
		tab[6] = tab[6] * 7;
		tab[7] = tab[7] * 9;
		tab[8] = tab[8] * 1;
		tab[9] = tab[9] * 3;
		tab[10] = tab[10] * 1;
		for (int j = 0; j < 11; j++)
		{
			wynik += tab[j];
		}
		if ((wynik - ((floor(wynik / 10)) * 10)) == 0)
			cout << "D" << endl;
		else
			cout << "N" << endl;
	}

    return 0;
}

z tego wniosek, że

1. przeczytaliśmy treść różnych zadań (co mało prawdopodobne)
2. przeczytałeś nieuważnie treść zadania - zdecydowanie bardziej prawdopodobne

Co za niedopatrzenie z mojej strony… Przyjęło.

Witam, ciągle wyrzuca mi błąd. Pdejrzewam, że ma to coś wspólnego z wczyywaniem danych za pomocą scanf (białe znaki) ponieważ gdy wpisze ciąg 11 cyfr program uważa, że to jedna wprowadzona (muszą być odzielone enterem).
Bardzo prosze o rade i z góry dziekuje
Link do kodu: https://ideone.com/b7VPaa8

scanf("%d", &pesel[i]);

wczytuje Ci całą liczbę, jeśli nie jest rozdzielona białym znakiem, a nie pojedynczą cyfrę. Możesz użyć czegoś w stylu

scanf(" %c ", &pesel[i]);
pesel[i] -= '0';

Wtedy wczyta jeden znak. Spacja przed i po %c jest po to, żeby zjadało line feed (enter) po ostatniej cyfrze i nie wczytywało go w następnym obiegu jako znak. ‘0’ odejmujemy, gdyż komputer wczytuje jedną cyfrę jako jej kod ASCII i trzeba odjąć kod zera, żeby otrzymać tę cyfrę.

Wtedy też będziesz musiał wyzerować tablicę pesel przed pierwszym użyciem:

memset(pesel, 0, sizeof(pesel));

Lub zadeklarować ją jako tablicę charów:

char pesel[12];

Dlaczego zwiększyłem rozmiar tablicy o 1? Gdyż wtedy będzie miejsce na ostatni znak ‘\0’, co pozwoli ci wczytywać pesel w następujący sposób:

scanf("%s", pesel);

Potem jednak w pętli będziesz musiał poodejmować kody zera.

I mała uwaga na koniec: w konsoli być może nie będzie chciało ci “ogarnąć”, że wpisałeś ostatni pesel, w takim wypadku trzeba wcisnąć CTRL-Z i potwierdzić enterem (wysyła to znak końca pliku do programu).

Powodzenia

BTW: Linijka

if(sprawdzenie(pesel)<=0) return 0;

pewnie nie spowoduje błędnej odpowiedzi w tym zadaniu, ale w takim wypadku powinieneś zgodnie ze specyfikacją zadania wypisać, że pesel jest błędny, a nie wychodzić z programu.

Witam, sędzia odrzuca a porgram działa jak powinien (prawie). Problem polega na tym, że potrzebuje wczytać liczby pesel jako char a nie int (wtedy lapie mi ciag 11 znakow jako jedna liczbe).
Prosze o pomoc,z gory dziekuje!

Dobrym pomysłem byłoby czytanie odpowiedzi w tematach, w których prosiłeś o pomoc… Post wyżej masz wszystko dokładnie opisane…

Dziekuje, przypadkiem powstal ten drugi post…

Czesc! Ja tez wstawiam swoj kod, bo na codeBlock’sie mi dziala, a sedzia na przyjmuje i nie wiem sama, co jest nie tak. WYDAJE sie logiczny Prosze o jakas wskazowke! Dzieki!! https://ideone.com/3UxOap14

program nie musi być logiczny, sędziego to nie interesuje, sędzia natomiast oczekuje poprawnych odpowiedzi

a twój program daje przecież niewłaściwą odpowiedź dla danych testowych

Dzień dobry

Nie chcę zakładać nowego tematu, aby nie zaśmiecać forum. Próbowałem się zmierzyć z tym zadaniem i coś mi się udało urodzić, ale poległem :(. Oto mój kod:

#include
using namespace std;
int main()
{
int pesel[10];
int wynik;
int i,k = 0;
int ile = 0;
cout << "Ile numerow pesel chcesz sprawdzic? ";
cin >> ile;

for (k; k <ile; k++)
{

for (i = 0; i <= pesel[10]; i++)
{
    cout << i + 1 ;
    cin >> pesel[i];

}
wynik = pesel[0]*1 + (pesel[1]*3) + (pesel[2]*7) + (pesel[3]*9) + (pesel[4]*1) + (pesel[5]*3) + (pesel[6]*7) + (pesel[7]*9) + (pesel[8]*1) + (pesel[9]*3) + (pesel[10]*1);


   cout << wynik << endl;
    if (wynik = 0 || wynik % 10 == 0)
         {
             cout << "D" << endl;
         }
         else
         {
             cout << "N" << endl;
         }

}

return 0;

}

Pierwszy mój problem - jak sprawić, aby podawać pesel nie jedna literka po jednej tylko cały naraz?
Drugi mój problem - o ile pierwszy pesel program sprawdzi poprawnie, o tyle każdy kolejny już się wysypuje (po podaniu pierwszej liczby od razu przechodzi do jakiegoś dziwnego wyniku). Nie rozumiem tego :(. Ktoś potrafiłby wytłumaczyć, dlaczego?

1. Naucz się korzystać z Ideone. Korzyści są dwie:
   - możesz podawać swój kod przez link do Ideone
   - możesz wygodnie testować swój kod.
2. Twój program może wypisywać TYLKO takie komunikaty jakich oczkuje autor. W przypadku tego zadania jest to “D” lub “N” i nic innego. Wypisanie "Ile numerów pesel chcesz sprawdzić? " od razu uwala zadanie.
3. Pisząc cin >> pesel[i] wczytujesz od razu cały pesel a nie jak się zapewne spodziewałeś cyfra po cyfrze. Zresztą w pętli for jest błąd nawet gdyby Twój sposób był poprawny bo wczytujesz do i <= pesel[10], który nie jest znany. Zapewne chciałeś wczytywać do i<=10. Ponadto wczytana liczba ma 11 cyfr i nie mieści się w typie int
4. Wykonanie instrukcji if powinno być dokładnie odwrotnie (zmienić “N” z “D”).
5. Jeśli chcesz cały pesel wczytać za jednym razem to możesz po prostu zrobić to np. tak:
   long long pesel;
   cin >> pesel;
   a później wyłuskiwać poszczególne cyfry
   Łatwiej będzie wczytać to jako string a później z niego wyłuskać poszczególne cyfry