Ошибка метода поиска

Я пытаюсь выполнить метод поиска, в котором проверяются 2 идентификатора, в которых хранится только целое число.

Во-первых, у меня есть база клиентов. Я ввожу имя, фамилию, идентификатор и адрес друг после друга в этом порядке, а затем они сразу же сохраняются в файл.

Когда пользователь вводит идентификационную карту, вызывается этот метод поиска и проверяется весь файл, чтобы увидеть, является ли идентификатор уникальным или нет. Если это не так, то возвращается 0, иначе возвращается 1.

Теперь проблема вот в чем. Когда я ввожу идентификатор, уникальный он или нет, он продолжает работать, но затем, когда он выводит то, что я написал, для ИМЯ и ФАМИЛИИ он показывает только первую запись, которую я там сохранил (как застрял в какой-то буфер), идентификатор и адрес выводятся нормально.

Файл также НЕ обновлен, что означает, что файл сохранения не произошел. Теперь, когда я удаляю этот метод, добавление работает нормально, но у меня не будет доступа к сравнению идентификаторов.

Любые предложения, почему это происходит? и если возможно любая идея, как я могу это исправить? это похоже на то, что всякий раз, когда я использую этот метод поиска, весь файл запускается с самого начала и застревает там. Я пробовал использовать метод с логическим значением, но все равно безрезультатно. Когда я попытался использовать его с логическим значением вместо строки «if (customerID (scanf («% d», & cCheck)) == 1)», я сделал это == TRUE, это дало мне ошибку, что вывод всегда будет == FALSE, поскольку данные не равны NULL.

и о TRUE и FALSE действительны в моем случае, так как у меня есть логическое перечисление typedef в common.h

Код выглядит следующим образом [публикация всего файла]: речь идет о методах [void addCustomer()] и [int customerID (int cCheck), но я опубликовал все, поскольку некоторые из них взаимосвязаны.

РЕДАКТИРОВАТЬ!!! - Идентификаторы, даже если они НЕ уникальны, все равно принимаются...

/*
 * CustomerMainMenu.c
 * Author: DodoSerebro
 *
 * This class will output the Customer's Main Menu and re-directs to the
 * corresponding section
 *
 */
#include<io.h>
#include<fcntl.h>
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include <ctype.h>
#include "..\Headers\common.h"
#include "..\Headers\customerManagement.h"

static FILE *cfp;
static customer c;
#define STRUCTSIZE sizeof (customer)


/** This is the Customers's Main Menu in which the various sections can be
 *  accessed from here
 */
boolean  customerMainMenu()
{



    int optionC;
    clrscr();

    copyright();


    printf ("\n\n\n\n\t\t    ************* Customer's Main Menu *************\n \n \n");

    printf ("Press [1] to add a new Customer\n");
    printf ("Press [2] to edit a Customer\n");
    printf ("Press [3] to list all Customers\n");
    printf ("Press [4] to Show a Customer's last Order\n");
    printf ("Press [5] to go back to Main Menu\n\n\n");


    if (scanf ("%d",&optionC) == 1)
    {
        switch (optionC)
        {

        case 1:
        {
            clrscr();
            getchar();
            addCustomer();
            break;
        }
        case 2:
        {
            printf ("Edit a Customer\n");
            break;
        }

        case 3:
        {
            clrscr();
            listCustomers();
            system ("PAUSE");
            break;
        }
        case 4:
        {
            printf ("Customer's Last Order\n");
            break;
        }
        case 5:
        {
            system ("PAUSE");
            break;
        }
        default:
        {
            if (optionC != 1 || optionC != 2 || optionC != 3 || optionC != 4 || optionC !=5)
            {
                clrscr();
                printf ("Invalid option!\n");
                system ("PAUSE");
                customerMainMenu();
            }
            break;
        }
        }
    }
    return TRUE;

}

/**
 *  This following method will append a customer to the
 *  database at the end of the file
 *
 *  */

void addCustomer ()
{
    char ch;
    copyright();

    printf ("\n\n\n\n\t\t    ************* Add Client **********\n \n \n");

    if ((cfp = fopen ("customers.dat","a+b")) == NULL)
    {
        fputs("Can't open customers.dat file\n",stderr);
    }



    printf ("\tThis will add another customer to the the database\n");
    printf ("\tPress 'Y' to confirm or 'N' to return to the Client Main Menu\n\tWITHOUT adding a customer\n");
    ch = getchar();

    if (ch == 'n' || ch == 'N')
    {
        customerMainMenu();
    }
    else if (ch == 'y' || ch == 'Y')
    {
        fflush(stdin);
        clrscr();
        printf ("\n\n\n\n\t\t    ************* Add Client **********\n \n \n");
        printf ("Please enter Name:\n");
        while (scanf ("%s", c.name) == 0 || cCheck(c.name,100) == FALSE);
        {

        }


        printf ("Please Enter Surname: \n");
        while (scanf ("%s",c.surname) == 0 && cCheck (c.surname,100) == FALSE);
        {

        }
        printf ("Please Enter ID Card, [NOTE! Only numbers are allowed!]\n");
        int cCheck;
        if (customerID(scanf ("%d",&cCheck)) == 1)
        {
            printf ("ID already Taken, Client exists!\n");
            printf ("Do you want to enter another ID? 'Y' for Yes and 'N' to return to Main Menu\n");
            ch = getchar();
            if (ch == 'Y' || ch == 'y')
            {
                scanf ("%d",&cCheck);
                customerID(cCheck);
                c.ID = cCheck;
            }
            else
            {
                customerMainMenu();
            }
        }
        else
        {
            c.ID = cCheck;
        }


        getchar();

        printf ("Please Enter Address:\n");
        gets(c.address);


        fwrite (&c,STRUCTSIZE, 1, cfp);

        printf ("For Testing purposes:\n");
        printf (" %s\n %s\n %s\n %d\n", c.name, c.surname, c.address, c.ID);
        askAnother();


    }
    else
    {
        printf ("\nInvalid choice! Either Y or N is accepted\n");
        system ("PAUSE");
        getchar();
        clrscr();
        addCustomer();
    }
}

void listCustomers()
{

    if ((cfp = fopen ("customers.dat","rb")) == NULL)
    {
        fputs("Can't open customers.dat file\n",stderr);
        printf ("Returning to Customer Main Menu");
        system ("PAUSE");
        customerMainMenu();
    }

    rewind (cfp);
    while (fread (&c,STRUCTSIZE,1,cfp)==1)
    {
        printf ("Customer: %s %s ID: %d\n", c.surname, c.name, c.ID);
    }
    fclose (cfp);


}


void askAnother()
{
    printf ("Do you want to add another Customer?\n");
    printf ("Enter 'Y' for yes and 'N' to return to the Main Menu\n");

    char input;
    input = getchar();

    if (input == 'Y' || input == 'y')
    {
        getchar();
        addCustomer();
    }
    else if (input == 'N'|| input == 'n')
    {

        fclose (cfp);
        customerMainMenu();

    }
    else
    {

        printf ("Invalid Option! Only Y or N are allowed\n");
        system ("PAUSE");
        clrscr();
        askAnother();

    }

}


boolean cCheck(char *test, int max)
{
    int x;
    for (x =0; x<max; x++)
    {
        if (isdigit(test[x]))
        {
            return FALSE;
        }
        if (x==max)
        {
            return TRUE;
        }
        x++;

    }
    return TRUE;
}

/**
 *  This method will compare the ID passed from the ID of the customer to check
 *  whether it is exists or not. If it exists it will output 1 otherwise it
 *  will output -1. This will make sure that the Person's ID is unique
 *
 */


int customerID (int cCheck)
{

    rewind (cfp);
    while (fread (&c,STRUCTSIZE,1,cfp)==1)
    {
        if (c.ID == cCheck)
        {
            return 1;
        }
        else
        {
            return 0;
        }
    }

    return 1;
}

РЕДАКТИРОВАТЬ!!

Загрузка изображения, чтобы показать, что я имею в виду, если я не понимаю

(Обратите внимание, как имя и фамилия отличаются от введенных) !http://s017.radikal.ru/i443/1212/c8/1ea9bc56d980.jpg

Далее показано что у меня в файле (только один файл)!http://s017.radikal.ru/i431/1212/49/2a0df6acf9ec.jpg


person DodoSombrero    schedule 20.12.2012    source источник
comment
Теперь дайте мне номер строки, где это не удается, не ждите, что я прочитаю весь этот беспорядок.   -  person    schedule 21.12.2012
comment
145 - это место, где он НАЧИНАЕТ терпеть неудачу. Затем SearchMethod начинается со строки 273.   -  person DodoSombrero    schedule 21.12.2012


Ответы (1)


if (customerID(scanf ("%d",&cCheck)) == 1)

Здесь вы являетесь возвращаемым значением функции scanf() функции customerID(), в то время как на самом деле вы хотели передать отсканированное значение cCheck.

Вы можете разделить вызов функции и чтение ввода, как показано ниже:

   if(scanf ("%d",&cCheck) != 1) {
   // Input error handling
   }
   else if (customerID(cCheck) == 1) {
    ...

Другая проблема заключается в том, что у вас есть функция с именем cCheck, а также локальная переменная с именем cCheck. Переименуйте один из них соответствующим образом.

person P.P    schedule 20.12.2012
comment
Спасибо большое, теперь методы работают, но есть небольшая проблема. После того, как я добавляю запись, она НЕ добавляется к файлу, что означает, что файл сохранения игнорируется. - person DodoSombrero; 21.12.2012