[C++] Errore acquisendo stringa di caratteri con lunghezza massima

[squalo92]

Salve ragazzi, sono nuovo, ho un errore stranissimo.
Questo è il codice:

#include <iostream>
#include <fstream>
#include <direct.h>
#include <string>

using namespace std;


class persona{
private:
	char nome[20];		
	char cognome[20];	
	char indirizzo[20];
	char nascita[20];	
	char email[20];		
	char telefono[20];
	char password[20];
	char username[20];
public:
	void acquisisci();
	void controlla_caratteri(char*,int);
};	


void persona::controlla_caratteri(char *vettore,int n)
{
	do
	{
		cin>>vettore;
		if(strlen(vettore)>n)
			cout<<"Errore,max "<<n<<" caratteri: ";
	}while(strlen(vettore)>n);
}

void persona::acquisisci()
{
	cout<<"\nMax 20 caratteri!";
	cout<<"\nInserisci nome: ";
	controlla_caratteri(nome,sizeof(nome));
	cout<<"Nome:"<<nome;
	cout<<"\nInserisci cognome: ";
	controlla_caratteri(cognome,sizeof(cognome));
	cout<<"\nInserisci indirizzo: ";
	controlla_caratteri(indirizzo,sizeof(indirizzo));
	cout<<"\nInserisci nascita: ";
	controlla_caratteri(nascita,sizeof(nascita));
	cout<<"\nInserisci email: ";
	controlla_caratteri(email,sizeof(email));
	cout<<"\nInserisci telefono: ";
	controlla_caratteri(telefono,sizeof(telefono));
	cout<<"\nInserisci password: ";
	controlla_caratteri(password,sizeof(password));
}


////////////////////////////////////////////////
int main (){
	persona P;
	P.acquisisci();
	return 0;
}

Ovviamente questo è un pezzettino del codice.
L'errore consiste che quando mando l'eseguibile, se scrivo più di 20 caratteri alla password, dopo giustamete(grazie al controllo) mi chiede di reinserirla, ma anche se la metto inferiore ai 20 caratteri, dopo si blocca il programma!
Qualcuno mi può aiutare???
grazie

[oregon]

Una volta che sfori con i caratteri in memoria, non serve a nulla controllare che la lunghezza massima sia stata superata.

In pratica, prima della if, il danno è già stato fatto.

Utilizza la

cin.getline

per l'input.

[squalo92]

Non so se è proprio così, perchè se faccio stampare con cout il nome, che ho inserito prima in modo errato e poi in modo corretto,quello che si prende il vettore è il secondo cin, non il primo.
Poi cosa ho notato è che se tolgo le ultime due righe dell'acquisisci, l'errore non fa più (cioè se tolgo le righe inerenti la password).

[Nikiki]

Non lo so se è per questo che ti si blocca il programma, ma un errore di sicuro c'è.
La funzione strlen() restituisce la lunghezza della stringa ma il valore restituito non conta il terminatore, quindi se inserisci 20 caratteri nell'array ne verranno inseriti 21 perchè viene aggiunto il terminatore e quindi vai a scrivere in una locazione di memoria fuori dall'array.
La tua if controlla se il valore è maggiore, ma anche se è uguale sfora.
Dovresti modificarlo in:
if(strlen(vettore)>=n)

[oregon]

Originariamente inviato da squalo92
Non so se è proprio così, perchè se faccio stampare con cout il nome, che ho inserito prima in modo errato e poi in modo corretto,quello che si prende il vettore è il secondo cin, non il primo.

Fidati ...

[Scara95]

Stesso argomento, stessa persona