[sumit.xxx365]

I am trying to write a simple code to compare password, but am unable to do so.....
The program contains a class password which contains a constructor to initialize the actual password.......
there are also to other function...
the first one is to input the password
and the second one is to compare the input password to the actual one.....
If the two passwords are same then the program outputs "Correct Password"
else it says "incorrect Password"

The Problem: Whenever I run the program,TC++ crashes though there are no errors and warnings.....
the window just closes......
I think there must be a logical error....
But i cannot find where.....
Please guys helm me find out.....

The code is:

#include <iostream.h>
#include <conio.h>
#include <string.h>
class password
	{
	char pass[20];             		//Stores Actual Password
	char inpass[20];                        //Stores Input Password
   public:
	password(char []);                      //Initializes Actual Password
	void input();                           //Inputs the Password
	int comp();                             //Compares the Password
	};
void password::password(char p[])
	{
	int len=strlen(p);
	for(int i=0;i<len;i++)
		{
		pass[i]=p[i];
		}
	}
void password::input()
	{
	char temp;
	cout<<"Enter Password(!to end): ";
	for(int i=0;temp!='!';i++)
		{
                temp=getch();                    //After first Edit
		inpass[i]=temp;                  /
		cout<<"*";
		}
	}
int  password::comp()
	{
	int temp;
	temp=strcmp(pass,inpass);
	return temp;
	}
void main()
	{
        password pass("password");
	clrscr();	
	pass.input();
	if(pass.comp())
		cout<<"\nCorrect Password";
	else
		cout<"\nIncorrect Password";
	getch();
	}

This post has been edited by sumit.xxx365: 01 October 2010 - 02:05 AM

[Salem_c]

So where is the cin >> temp;
in your input function?

[sumit.xxx365]

oh thanks.....
I think i missed that.....
temp=getch();
but now the new edit give "correct password to all password"....

This post has been edited by sumit.xxx365: 01 October 2010 - 01:44 AM

[Salem_c]

At what character does strcpy() stop at?

Are any of your string manipulations dealing with this character?

[simeesta]

Don't use void main()!!!!

[sumit.xxx365]

Salem_c, on 01 October 2010 - 12:54 AM, said:

At what character does strcpy() stop at?

Are any of your string manipulations dealing with this character?

have not used strcpy() anywhere.....
do u mean strcmp()?
It just compares the string and returns 0 for false and 1 for true....(isn't that so?)

and I don't know what u mean by string manipulation....

simeesta, on 01 October 2010 - 01:04 AM, said:

Don't use void main()!!!!

umm why?
what benifits does using int main() and returning 0 have over void main?

[simeesta]

This is why.

Null terminate your string
inpass[i]='\0'; after you have got it from user input.

[sumit.xxx365]

simeesta, on 01 October 2010 - 01:13 AM, said:

This is why.

Null terminate your string
inpass[i]='\0'; after you have got it from user input.

umm sorry for my noobiness but i still don't understand what u mean.....

[simeesta]

void password::input()
	{
	char temp;
	cout<<"Enter Password(!to end): ";
	for(int i=0;temp!='!';i++)
		{
                temp=getch();                    //After first Edit
		inpass[i]=temp;                  /
		cout<<"*";
		}
        inpass[i-1]='\0'; //add this line to your code!
	}

and

void password::password(char p[])
	{
	int len=strlen(p);
	for(int i=0;i<len;i++)
		{
		pass[i]=p[i];
		}
        pass[i]='\0'; //add this too
	}

it will add a '\0' to the end of the array telling strcmp this is the end of it. '\0' is NULL and adding to the end of char array makes it a string.

This post has been edited by simeesta: 01 October 2010 - 03:29 AM

[Alex6788]

sumit.xxx365, on 01 October 2010 - 03:43 AM, said:

simeesta, on 01 October 2010 - 01:13 AM, said:

This is why.

Null terminate your string
inpass[i]='\0'; after you have got it from user input.

umm sorry for my noobiness but i still don't understand what u mean.....

Simeesta is right, if you want to be a good programmer than use int main and have it return 0; it's the standard and is much better.

[sumit.xxx365]

but what is i?
we have exited the for loop......
do u mean at the end of the string i should insert the null character.....
but how does it help in the code?

[simeesta]

sumit.xxx365, on 01 October 2010 - 10:54 AM, said:

but what is i?
we have exited the for loop......
do u mean at the end of the string i should insert the null character.....
but how does it help in the code?

The null character makes the difference between a char array and a string. When a function reads the null character this tells it it has reached the end. Say as as if

password[]="cheese";

Then

password[0] is 'c'
password[1] is'h'
...
password[5] is 'e'
password[6] is '\0' reached end of the string, stop reading it;
as password[7] is random data and you don't want strcmp to compare it to the input etc.

Hope this helps!

edit:

i is just the next value. the loop increments and then tests. so password[i] is the next character.

and change line 43 of main to
if(!pass.comp())
notice the not.
if they are equal strcmp will return 0 which is false.

i made a mistake:
void password::input()
should be i-1 NOT i see previous post!

This post has been edited by simeesta: 01 October 2010 - 03:34 AM

[Salem_c]

It helps the code because at present you're comparing "helloeshgshgs" with "helloh[sttsth[sh-ttijhnnn]inn" (or some other random garbage past the end of what you consider to be the real strings).

Until you make sure you add the \0 in the correct place for BOTH strings, you're not going to succeed with the strcmp().

BTW, what does strcmp() return when two strings are equal (not what you think at present).

Oh and I might suggest you ditch the 20+ year old TurboC++ compiler in favour for something belonging to this millenium. But I'm guessing your school forces obsolete tech on you from the start. Think about that and how well that is likely to prepare you for your career.

[simeesta]

If you will stop using turbo C++ i can recommend Codeblocks - with MinGW, an IDE for C and C++. You can download it here.

[sumit.xxx365]

Well I tried to implement the stuff u suggested....
but it's still not working......

#include <iostream.h>
#include <conio.h>
#include <string.h>
class password
    {
    char pass[20];                         //Stores Actual Password
    char inpass[20];                        //Stores Input Password
   public:
    password(char []);                      //Initializes Actual Password
    void input();                           //Inputs the Password
    int comp();                             //Compares the Password
    };
void password::password(char p[])
    {
    int len=strlen(p);
    for(int i=0;i<len;i++)
        {
        pass[i]=p[i];
        }
    pass[len]='\0';
    }
void password::input()
    {
    int t;
    char temp;
    cout<<"Enter Password(!to end): ";
    for(int i=0;temp!='!';i++)
        {
        t++;
        temp=getch();
        inpass[i]=temp;
        cout<<"*";
        }
    pass[t]='\0';
    }
int  password::comp()
    {
    int temp;
    temp=strcmp(pass,inpass);
    return temp;
    }
void main()
    {
password pass("password");
    clrscr();    
    pass.input();
    if(pass.comp())
        cout<<"\nCorrect Password";
    else
        cout<"\nIncorrect Password";               //This Line has no effect ://>
    getch();
    }

The warning that come are:
Line 50 has no effect...

This post has been edited by sumit.xxx365: 01 October 2010 - 08:53 PM