Programming Forums > Application Development > C++

RemoteC2 · Aug 9th, 2006, 10:43 PM

i created a program that prompts the user to give a psswrd, then the program stores the password. the program then checks the password. but when it gets to the part where it checks the password, no matter what i do, it never sees that the two strings are the same...i was wondering if i someone could point me in the right direction?

anyways, here is the code:

(Toggle Plain Text)

//store/check password
#include <iostream>
#include <cstdlib>
using namespace std;

class pswrd {
public:
      char ps[80], usr[80];
      int len;
      void set_ps (char *p) {strcpy(ps, p);}
      void check_ps () {
            printf("\nPassword: ");
            cin>>usr;
            if(usr!=ps) cout<<endl<<"Access Denied";
            else if(usr==ps) cout<<endl<<"Access Granted";
       }
};

int main ()
{
    pswrd ob1;
    char p[80];
    printf("Set Password: ");
    cin>>p;
    ob1.set_ps(p);
    ob1.check_ps();
    system("PAUSE");
}

Prm753 · Aug 9th, 2006, 11:07 PM

Try using the strcmp() function.

andro · Aug 9th, 2006, 11:21 PM

Or stop using C headers and functions, seeing as this is posted in the C++ forum, and use std::string.

Then you can use != and == on them.

ionexchange · Aug 9th, 2006, 11:24 PM

You might also want to initialize the strings ps and usr.

andro · Aug 10th, 2006, 3:43 AM

Quote:

Originally Posted by ionexchange

You might also want to initialize the strings ps and usr.

BS...

DaWei · Aug 10th, 2006, 7:02 AM

@ionexchange: I checked the rules and the one forbidding posting voodoo solutions seems to have disappeared. Please don't do it, anyway.

RemoteC2 · Aug 10th, 2006, 9:36 AM

the reason that i didnt initialize ps and usr, is b/c the program makes it so that the user inputs the password. i made this program just to see if i could..anways, thx everyone for the help. the program works now, and the final source is this:

(Toggle Plain Text)

//store/check password
#include <iostream>
#include <cstdlib>
using namespace std;

class pswrd {
public:
      char ps[80], usr[80];
      int len;
      void set_ps (char *p) {strcpy(ps, p);}
      void check_ps () {
            printf("\nPassword: ");
            cin>>usr;
            if(strcmp(usr,ps)) cout<<endl<<"Access Denied";
            else if cout<<endl<<"Access Granted";
       }
};

int main ()
{
    pswrd ob1;
    char p[80];
    printf("Set Password: ");
    cin>>p;
    ob1.set_ps(p);
    ob1.check_ps();
    system("PAUSE");
}

DaWei · Aug 10th, 2006, 10:14 AM

Quote:

else if cout<<endl<<"Access Granted";

You should remove that token. It's not functional, therefore it's confusing, therefore it's worthless.

Prm753 · Aug 10th, 2006, 10:20 AM

Quote:

Originally Posted by RemoteC2

the program works now, and the final source is this:

Quote:

Originally Posted by Dev-C++ compiler error

expected `(' before "cout"

This was the error:

 cpp Syntax (Toggle Plain Text) 
else if cout<<endl<<"Access Granted";
else if cout<<endl<<"Access Granted";

Easy to fix, though.

EDIT: Too slow, DaWei beat me to it...

andro · Aug 10th, 2006, 3:09 PM

You're still using C functions and C strings in a C++ program.

And what happens when I come along and enter my 81 character password? Looks like bad news.

Aug 9th, 2006, 10:43 PM	#1
RemoteC2 Programmer Join Date: Jan 2006 Posts: 55 Rep Power: 3	simple password checker i created a program that prompts the user to give a psswrd, then the program stores the password. the program then checks the password. but when it gets to the part where it checks the password, no matter what i do, it never sees that the two strings are the same...i was wondering if i someone could point me in the right direction? anyways, here is the code: (Toggle Plain Text) //store/check password #include <iostream> #include <cstdlib> using namespace std; class pswrd { public: char ps[80], usr[80]; int len; void set_ps (char p) {strcpy(ps, p);} void check_ps () { printf("\nPassword: "); cin>>usr; if(usr!=ps) cout<<endl<<"Access Denied"; else if(usr==ps) cout<<endl<<"Access Granted"; } }; int main () { pswrd ob1; char p[80]; printf("Set Password: "); cin>>p; ob1.set_ps(p); ob1.check_ps(); system("PAUSE"); } //store/check password #include <iostream> #include <cstdlib> using namespace std; class pswrd { public: char ps[80], usr[80]; int len; void set_ps (char p) {strcpy(ps, p);} void check_ps () { printf("\nPassword: "); cin>>usr; if(usr!=ps) cout<<endl<<"Access Denied"; else if(usr==ps) cout<<endl<<"Access Granted"; } }; int main () { pswrd ob1; char p[80]; printf("Set Password: "); cin>>p; ob1.set_ps(p); ob1.check_ps(); system("PAUSE"); }

Aug 9th, 2006, 11:07 PM	#2
Prm753 Professional Programmer Join Date: Oct 2005 Location: United States Posts: 447 Rep Power: 3	Try using the strcmp() function. __________________ The world's first athletic computer geek! The home of PrProgramsStudios How not to post a question: <-- Please don't reply

Aug 10th, 2006, 7:02 AM	#6
DaWei Resident Grouch Join Date: Jun 2005 Posts: 6,453 Rep Power: 10	@ionexchange: I checked the rules and the one forbidding posting voodoo solutions seems to have disappeared. Please don't do it, anyway. __________________ Abstraction doesn't make it impossible to write bad code; it makes it possible to write superior code. Contributor's Corner: Grumpy on C++ Exceptions DaWei on Pointers

Aug 10th, 2006, 9:36 AM	#7
RemoteC2 Programmer Join Date: Jan 2006 Posts: 55 Rep Power: 3	the reason that i didnt initialize ps and usr, is b/c the program makes it so that the user inputs the password. i made this program just to see if i could..anways, thx everyone for the help. the program works now, and the final source is this: (Toggle Plain Text) //store/check password #include <iostream> #include <cstdlib> using namespace std; class pswrd { public: char ps[80], usr[80]; int len; void set_ps (char p) {strcpy(ps, p);} void check_ps () { printf("\nPassword: "); cin>>usr; if(strcmp(usr,ps)) cout<<endl<<"Access Denied"; else if cout<<endl<<"Access Granted"; } }; int main () { pswrd ob1; char p[80]; printf("Set Password: "); cin>>p; ob1.set_ps(p); ob1.check_ps(); system("PAUSE"); } //store/check password #include <iostream> #include <cstdlib> using namespace std; class pswrd { public: char ps[80], usr[80]; int len; void set_ps (char p) {strcpy(ps, p);} void check_ps () { printf("\nPassword: "); cin>>usr; if(strcmp(usr,ps)) cout<<endl<<"Access Denied"; else if cout<<endl<<"Access Granted"; } }; int main () { pswrd ob1; char p[80]; printf("Set Password: "); cin>>p; ob1.set_ps(p); ob1.check_ps(); system("PAUSE"); }

Thread Tools
Show Printable Version Email this Page
Display Modes
Linear Mode Switch to Hybrid Mode Switch to Threaded Mode

Aug 9th, 2006, 11:21 PM	#3
andro Professional Programmer Join Date: Oct 2005 Location: California Posts: 290 Rep Power: 3	Or stop using C headers and functions, seeing as this is posted in the C++ forum, and use std::string. Then you can use != and == on them.

Aug 9th, 2006, 11:24 PM	#4
ionexchange Newbie Join Date: Oct 2005 Posts: 12 Rep Power: 0	You might also want to initialize the strings ps and usr.

Aug 10th, 2006, 3:09 PM	#10
andro Professional Programmer Join Date: Oct 2005 Location: California Posts: 290 Rep Power: 3	You're still using C functions and C strings in a C++ program. And what happens when I come along and enter my 81 character password? Looks like bad news.

Currently Active Users Viewing This Thread: 1 (0 members and 1 guests)

Similar Threads
Thread	Thread Starter	Forum	Replies	Last Post
OnlineTextEditor.Com!	Sane	Show Off Your Open Source Projects	43	Jun 16th, 2006 8:55 AM
[Python] Password Generator	bulio	Show Off Your Open Source Projects	2	Feb 28th, 2006 3:01 AM
password box	ragenuub	Visual Basic	5	Nov 15th, 2005 3:46 PM
Just a small password generator	Jessehk	Show Off Your Open Source Projects	3	Sep 16th, 2005 8:41 AM
Simple Function Questions	meverha1	C++	16	Sep 12th, 2005 1:25 AM

Programming Forums > Application Development > C++

simple password checker