Program sprawdzajacy haslo

Question

Witam, mam problem z moim programem sprawdzajacym hasla. A mianowicie nie wyswietla on nigdy "True".

Zeby haslo bylo prawidlowe musi:

-miec od 6 do 20 znakow,

-miec conajmniej po 1 duzej literze, malej literze oraz cyfrze

Oto link:
https://pastebin.com/L8M5kDsX

Answer 1

  if(ile_big==0 || ile_small == 0 || ile_liczb==0 )   //sprawdza czy byla chociaz jedna mala lub duza litera i cyfra
    cout<<"False"<<endl;                                // jesli nie to zwraca False
    else
    cout<<"True"<<endl;   

Dałeś ten warunek 'else' do innego if'a. Zauważ, że jest za klamrą zamykającą warunek (dlugosc>=6 && dlugosc <=20).

Answer 2

Proponuję zrobić to tak:

#include <iostream>
#include <string>

using namespace std;

int main()
{
	int ile;
	cin >> ile;    //ile testow

	for (int i = 1; i <= ile; i++)
	{
		string password;
		int ile_big = 0;  //deklaracja wszystkich zmiennych niezbednych do sprawdzania
		int ile_small = 0;
		int ile_liczb = 0;
		cin >> password;      //wczytuje haslo

		int dlugosc = password.length();        //jego dlugosc

		bool good = false;


		if (dlugosc < 6 || dlugosc > 20)   //jesli haslo ma mniej niz 6 znakow lub wiiecej niz 20 to zwroc False
			good = false;
		else // po co tamto wczesniejsze skoro taka role pelni else
		{

			for (int j = 0; j<dlugosc; j++) // do dlugosc
			{
				if (password[j] <= 90 && password[j] >= 65)
					ile_big++;

				else if (password[j] <= 122 && password[j] >= 97)
					ile_small++;

				else if (password[j] <= 57 && password[j] >= 48)
					ile_liczb++;
			}

			if (ile_big == 0 || ile_small == 0 || ile_liczb == 0)
				good = false;
			else
				good = true;

			// lub good = !(ile_big == 0 || ile_small == 0 || ile_liczb == 0);
		}

		if (good) cout << "True\n";
		else cout << "False\n";
			
		
	}
}

Pamiętaj, że przy sprawdzaniu czy znak jest duża/wielka litera czy liczba masz takie funkcje jak: isupper, islower, isdigit. Poza tym masz błąd w pętli j<dlugosc-1 powinno być j<dlugosc

Comments

Dzięki, skorzystam z tych funkcji ;)

---

Pętla jest dobra, metoda length zwraca ilość znaków ale indeksujemy je w tablicy od zera. Twoja pętla wychodziłaby po za zakres.

---

Dzięki za uzupełnienie mnie, ale mam uwagę - pętla indeksowana od 0 do password.length()-1 nie sprawdziłaby całego hasła (przykład "1 AA112a").

---

Ach tak, masz rację :)