Thread: error about strcpy and strcmp

Well I gave you several things to change and you only changed two of them. Salary is still an array.

Does your compiler not warn you?

Originally Posted by kkkcj

Code:

if(id == P[i].id ==0){
            found =1;

this is mean if the id user enter same as id store in system. print all the details and another question is the compiler will display the details even though the user user enter the wrong ID

Then just delete the ==0 from the if expression.

but i'm declare the salary as the integer type ...i'm not sure which part you refer to and after i change the code become (id == P[i].id ) the compiler will straight print no record founded with this id.no matter id exit or not

>but i'm declare the salary as the integer type ...i'm not sure which part you refer to and after i change the code become (id == P[i].id )
salary should not be an array type for the same reason you didn't make id an array type in the structure. The names are char arrays because they have to be, so what you do with them (like call strcpy) is a little different from things that can be expressed as one number.

I mean, presumably P is an array of employees, so each P[i] has everything about one employee; their name arrays, their salary integer, etc. If salary stays an int[20] it's just a lot of extra space when you only need one number.

"P" is a rather terrible variable name by the way for something so central to the program.