Thread: Question

Originally Posted by Fox101

Code:

void hire(char firstName[][20], char lastName[][20],char class[][20], int id[], int *numEmploy)
{
	int ident=ZEROI;
	int index=ZEROI;
	int valid_id;
	if(*numEmploy >= MAX_EMPLOYEE)
		printf("You can not hire anymore employees");
	else
	{
		do
		{
			/*valid_id = TRUE;*/
			printf("Enter ID: ");
			scanf("%d", &ident);
			for(index = ZEROI; index < MAX_EMPLOYEE; index++)
			{
				if(id[index] == ident)
				{
					printf("Invalid ID\n");
					valid_id = FALSE;
					break;
				}
			}
			
		} while(valid_id == FALSE);
			
	id[index]=ident;
	printf("Enter first and last name: ");
	scanf("%s%s", firstName[*numEmploy], lastName[*numEmploy]);
	*numEmploy += 1;
	}
	
}

ZEROI=0 MAX_EMPLOYEE=5 *numEmploy=0, I am having issues storing the user input in id[]. I want it to store the value only if it is a unique id, and I want it to store in the next available space in the array. So the first time I run this function it should store it in id[0]. The next time in id[1], if it not the same as id[0]. My problem is I check to see if there is a copy but after the check, I cant get it to store the right value in the right space. I dunno if that makes sense...

Note the change in red, above.

The variable "index" is why that's happening. You need to use a different variable for the array index name, or you need to reset the index variable.

I'd just a different index variable name for id[index] = ...

Code:

for(i = 0; i < MAX_EMPLOYEE; index++)
{
    if(id[i] == ident)
    {
       printf("Invalid ID\n");
       valid_id = FALSE;
       break;
    }
}