Блог

Этот код дает мне исключительные данные, потому что я использовал !pinFound . Поэтому я хочу, чтобы детали были в состоянии. Это означает, pinFound что если я хочу получить результат, то он должен дать мне pinFound только результат или, если я хочу !pinFound результат, тогда он должен дать мне только !pinFound результат.

Я не хочу, чтобы оба результата печатались одновременно , плюс у меня есть несколько функций для чтения данных. Поэтому я не хочу повторять while(fgets(…)) в основной функции снова и снова.

 #include <stdio.h>
#include <stdlib.h>
#include <string.h>
#define STRING_LEN 200

int i;
char line[STRING_LEN], *lineOne = NULL, *numbers[5], pinFind[STRING_LEN], *pinFound = NULL;  
int find(FILE * fname, char *findPin){
    while(fgets(line, STRING_LEN, fname)){  
        lineOne = strtok(line, "n");
        numbers[0] = strtok(lineOne, ",");
        for(i = 1; i < 5; i  )
            numbers[i] = strtok(NULL, ",");
        pinFound = strstr(numbers[2], findPin);
        if(!pinFound)
            return line;    
    }
}

int main(){
    FILE * fp1 = fopen("file.csv", "r");
    printf("Enter the pin code: ");
    scanf("%s", pinFind);

    find(fp1, pinFind);
    for(i=0; i<5; i  )
        printf("%sn", numbers[i]);

    return 0;
}
  

Если я правильно понял ваш вопрос, вы хотели бы извлечь логику итерации файла из остальной части обработки.

Базовое изменение, которое позволило бы вам получить это, было бы просто:

 #include <stdbool.h>

// parse next line, return true if line was parsed
bool nextLine(FILE * fname, char *findPin)
{
    if (fgets(line, STRING_LEN, fname))
    {
        lineOne = strtok(line, "n");
        numbers[0] = strtok(lineOne, ",");
        for (int i = 1; i < 5; i  )
            numbers[i] = strtok(NULL, ",");
        pinFound = strstr(numbers[2], findPin);
        return true;
    }
    else
    {
        return false;
    }
}
  

А затем в main:

 FILE * fp1 = fopen("file.csv", "r");

printf("Enter the pin code: ");
scanf("%s", pinFind);

while (nextLine(fp1, pinFind))
{
    // if you are here, line was parsed, so
    // check the value of 'pinFound'

    if (pinFound)
        doSomething(numbers);
    else
        doSomethingElse(numbers);
}

fclose(fp1);
  

Для домашнего проекта это должно более или менее сработать, но я бы рекомендовал инкапсулировать состояние между nextLine вызовами в отдельную структуру, вместо того, чтобы сохранять его глобальным. Делать все эти переменные глобальными — плохая идея, но создание i глобальной переменной — особенно опасная плохая идея.