Thread: Text files and validation

How about :

Code:

int getint(FILE *fp, char item[])
{
    int n;
    int min;
    int max;

    fscanf(fp, "%d%d", &min, &max);
  
    printf("Enter the %s: %d to %d: ", item, min, max);
    scanf("%d%*c", &n);
    while (n < min || n > max)
        {
        message("\nChoice not in range.\a");
        printf("Enter the %s: %d to %d: ", item, min, max);
        scanf("%d%*c", &n);
        }
    return(n);
}

Make a similar change in getreal() but use floats/doubles instead of ints.

This will avoid a lot of redundant code - since each call to get* has an fscanf into variables which are used just in that function, move the call there. It'll clean up the calling code significantly.

Won't that cause a problem with this function when it's called here (red)?:

Code:

/*========================menu()=================================================*/
int menu()
{
	int choice;

	title();

	/* display menu items */
	printf("\t1 = Add a record\n");
	printf("\t2 = Report\n");
	printf("\t3 = Delete a record\n");
	printf("\t4 = Change a record\n");
	printf("\t5 = Quit\n\n");

	/* display selection question and validate */
	choice = getint(1,5,"choice");

	return (choice);
}

anybody help me to solve my problem in my todays post?.................