ich habe ein prog geschrieben,das ein passwort nach eingabe speichern soll und beim ändern das alte abfragen soll,aber leider funktioniert das nicht.
das programm erkennt i.wie nicht, dass schon ein passwort eingegeben wurde.
kann mir einer helfen??

Quelltext:

#include <iostream>

using namespace std;
class Password{

private:
string currentPw;
static int counter;

public:
void setPw(string);
string getPw();
};

int Password::counter = 0;

int main()
{
void run();
run();

std::cin.get();
}

void run(){

Password password;
string newPw;

cout<<"altes Passwort: "<<password.getPw()<<endl;
cout<<"Bitte das neue Passwort eingeben\n";
cin>>newPw;

password.setPw(newPw);

std::cin.get();

}

void Password::setPw(string newPw){

if(currentPw!=""){

string eingabe;
cout<<"Bitte das alte Passwort eingeben"<<endl;
cin>>eingabe;

std::cin.get();

if(eingabe ==currentPw){

currentPw = newPw;
cout<<"Passwort geändert";

std::cin.get();

}else if(counter<3){

counter++;
cout<<"Falsches Passwort \nSie haben noch "<<3-counter<<" Versuche"<<endl;

cout<<"Bitte das neue Passwort eingeben\n";
cin>>newPw;

run();

std::cin.get();

}else{
cout<<"Programm gesperrt\n";

std::cin.get();
}

}else{

currentPw = newPw;

run();

std::cin.get();

}
}

string Password::getPw(){
return currentPw;
}

...
int main()
{
void run();
run();
...

ähhh - das hat Dir bestimmt der Compiler angemahnt. Also mal in die Warnings sehen ....

Tipp: "void run();" ist da falsch und braucht's nicht...

Gruß,

Simon2.

P.S.: Mit Codeeinrückungen sehen nicht nur wir, sondern auch Du besser, was los ist. (sind aber vermutlich TABs, die beim Copy/Paste hierhin verloren gegangen sind)

schaich

ja ich hab nur keinen anderen namen gefunden...
Oder was meinst du??

Wie der Namenskollege schon andeutete: Am "Namen" liegt's nicht.
Du definierst Dir da eine Variable namens "run" und die rufst Du in main() auf ... nicht "die run()", die Du außen definiert hast.

Gruß,

Simon2.

Dann sollte es wenigstens kompilieren.

Gruss Simon

schaich

EDIT:

void Password::setPw(string newPw){

if(currentPw!="")
{
    // hier kommt er nie an

vc++6.0

... generiert noch deutlich mehr Probleme. Sofern man nicht gezwungen ist vc++ 6.0 zu verwenden sollte man davon (u.a. wegen fehlernder Standardkonformität) eh gewaltigen Abstand nehmen.

simon.gysi schrieb:

vc++6.0

... generiert noch deutlich mehr Probleme. Sofern man nicht gezwungen ist vc++ 6.0 zu verwenden sollte man davon (u.a. wegen fehlernder Standardkonformität) eh gewaltigen Abstand nehmen.

Danke für den Hinweis. Leider bin ich beim momentanen Projekt gezwungen. Ansonsten sind mit konformere und komfortablere IDE / Compiler auch lieber.

#if !defined(PASSWORD_HPP__INCLUDED)
#define PASSWORD_HPP__INCLUDED

#if (_MSC_VER >= 1300)
#pragma once
#endif // (_MSC_VER >= 1300)

#include <string> // std::string

class Password
{
	std::string m_current;

public:
	Password(std::string const& password)
		: m_current(password)
	{}

public:
	bool set(std::string const& password, std::string const& new_password) { if (m_current != password) return false; m_current = new_password;  return true; }
	std::string const& get() const { return m_current; }
};

#endif // PASSWORD_HPP__INCLUDED

#include "password.hpp"
#include <iostream>
#include <string>

int main()
{
	Password password("test");

	for (unsigned char i(1); i <= 3; ++i)
	{
		std::cout << "Passwort: ";
		std::string password;
		std::getline(std::cin, password);
		std::cout << "Neues Passwort: ";
		std::string password_new;
		std::getline(std::cin, password

		if (password.set(password, new_password) == false)
			std::cerr << "FEHLER: Falsches Passwort eingegeben! (Noch " << 3 - i << " Versuche)" << std::endl;
		else break;
	}
	std::cout << "Passwort: " << password.get() << std::endl;
	std::cin.get();
}

So hast du es mal anständig unterteilt. Die Klasse Password lässt nur zu, das du ein neues Passwort setzt (set), wenn das Passwort stimmt. Insgesamt hat man 3 Versuche das Password richtig einzugeben (for 1 ... 3).

So in etwa das was du haben willst?

Wäre vllt jetzt ganz gut das m_current noch zu verschlüsseln usw