[423frewq4f23]

Chciałem sobie napisać program w C++ wyznaczający jakąś ilość czterocyfrowych kodów PIN, w ramach ćwiczeń na studia. Program się kompiluje ale wypisuje error jak na screenie. Ktoś mógłby podpowiedzieć co jest źle bo szukałem w necie ale nie rozumiem chyba i tak.

https://pastebin.com/81AXy0Um

#pytanie #programowanie #cpp #kiciochpyta #informatyka

[SmieszneZakola]

zmien <=komb na <komb. I staraj się więcej nie używać <= @423frewq4f23

[SmieszneZakola]

@423frewq4f23:
W tym pierwszym for'ze nie widzę zamykającego }, ale to moze mi się tak zle na telefonie wyświetla po prostu

[KonwersatorZabytkow]

#include <iostream>
#include <cstdlib>
#include <ctime>

std::string PIN;

std::string PINY[6561];
int komb=6561;
int yesornot;

int main()
{
for (int k = 0; k <= komb; k++)
{
//generowanie pinu
srand(time(NULL));
int a = rand()%9+1;
int b = rand()%9+1;
int c = rand()%9+1;
int d = rand()%9+1;

std::string AA = std::tostring(a);
std::string BB = std::tostring(b);
std::string CC = std::tostring(c);
std::string DD = std::tostring(d);

PIN = AA + BB +

[KonwersatorZabytkow]

@423frewq4f23:
Oto lista poprawek, które wprowadziłem:

Dodałem nagłówek <string>, ponieważ używasz typu std::string.
Zmieniłem zmienną komb na stałą (const int komb), ponieważ rozmiar tablicy PINY powinien być ustalony na etapie kompilacji.
W pętli for zamieniłem warunek k <= komb na k < komb, ponieważ indeksy tablicy zaczynają się od 0, a nie od 1.
Przeniosłem inicjalizację generatora liczb losowych srand(time(NULL)) na początek programu, aby uniknąć generowania tych samych liczb przy każdej

[423frewq4f23]

@SmieszneZakola: Miałeś rację poprawiłem to i poprawiłem jeszcze tylko if(PINY[i]==PIN)
{yesornot = 1;
break;}else {yesornot = 0; k=k-1;}
Na:
if(PINY[i]==PIN)
{yesornot = 1;
k=k-1;
break;}else {yesornot = 0;}
I ruszyło
A dlaczego jest taki problem z tymi znakami <= jeśli mogę spytać?

[SmieszneZakola]

@423frewq4f23:
Bo odwolujesz się do elementu tablicy poza zakresem.
Jak tablica ma 100 elementów( int i[100] ) to ostatni element ma index 99 a nie 100 a używając <= probujesz dostać się do tego 100 nieistniejącego elementu a nie do 99

[Krolik]

@SmieszneZakola: I właśnie dlatego uważam, że początkującym należy zakazać używania pętli for.

[select21]

@423frewq4f23: #!$%@? jak patrze na te kody pisane na wykopie to sie slabo robi. niech zrobia jak w markdownie codeblock