Ненормальное поведение для обработки текстовых файлов в C

Question

Я пытаюсь моделировать поведение DNS-сервера, который у меня есть дБ по имени hosts.txt содержащий machine_names/IP_addresses, Например:

equipo_00/169.0.1.169
sala_oeste_01/200.1.2.200
sala_oeste_02/200.2.3.200
sala_oeste_03/200.3.4.200
MEMFIS_04/201.4.5.201
ICARO_05/200.5.6.200
equipo_06/169.6.7.169
sala_este_07/201.7.8.201
sala_este_08/201.8.9.201
CEC_09/189.9.10.189

Вот мой код:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <unistd.h>

int main(int argc, char** argv)
{
    char* machine, *ip_add;
    FILE* db;
    char* flag;
    char  tmp[256];
    char  par[256];
    printf("> ");
    scanf("%s", par);
    db = fopen("hosts.txt", "r");
    while(db != NULL && fgets(tmp, sizeof(tmp), db) != NULL)
    {
        if(strstr(tmp, par))
        {
            flag = "1"; // flag setting to 1 means find the line
            machine = strtok(tmp, "/");//here's supposed to get the value of the machine
            ip_add = strtok(NULL, "/");//here's supposed to get the value of the ip adress
        }
    }//while

    if(strcmp(flag, "1") == 0) //
    {
        printf("here\n");
    }
    else
    {
        flag = "0"; //flag setting to 0
        printf("there\n");
    }

    printf("flag= %s pc=%s server=%s\n", flag, machine, ip_add);
    return 0;
}

Мой код читается со стандартного ввода, значения машины и указывает на IP-адрес, который назначен на этот компьютер на сервере. Проблема в том, что программа не является нормальным поведением, я не знаю, как изменять мой код на ожидаемый вывод, например,

input: equipo_00
output: flag = 1 pc = equipo_00 server=169.0.1.169

Извините, если вопрос довольно глупо, я новичок на этом языке .. Спасибо всем за помощь и оправдывайте свой английский

Answer 1

Это то что ты делаешь:

Прочитайте строку из файла в tmp.

Проверьте, введен ли пользователь строка par присутствует в tmp.

Если да, вы идете вперед и токенизировать tmp на / и сделать указатель machine указать на первую часть и ip_add указать на следующую часть.

Обратите внимание, что machine а также ip_add являются только что указатели. Отказ И они указывают на различные показатели tmp. Отказ Поэтому, когда вы продолжите с цикла, вы читаете новую строку снова tmp, переписать его. Это вызывает проблему, и ваш указатель теперь указывает на измененную строку.

Чтобы избежать этого просто добавить перерыв после успешного матча:

if (strstr(tmp, par)) {
   flag = "1";
   machine = strtok(tmp, "/");
   ip_add = strtok(NULL, "/");
   break;       
}

Также ваш финал printf:

printf("flag= %s pc=%s server=%s\n", flag, machine, ip_add);

должно быть частью вашего if Тело, так что вы печатаете их, только если вы нашли матч. В настоящее время вы печатаете, даже если совпадение не найдено. В этом случае вы будете печатать мусор в качестве указателей server а также ip_add имеют не был инициализирован.

Answer 2

Похоже, ваша проблема может быть с flag Переменная. Использование строки для значения флага кажется очень необычным. Попробуй это:

int flag = 0; // always initialise your variables

if (strstr(tmp, par) == 0) {
    flag = 1; // just an integer value
}

if (flag) { // easier to test the value too!
    // ...
}

printf("flag=%d\n", flag); // ints use %d format value

В вашем коде выше, flag Переменная неиниляция в начале функции, которая не может дать неожиданные (и неопределенные) результаты.

Answer 3

Ниже приведен фиксированный код. Существует несколько незначительных ошибок, в основном пропавших. Подумайте, что произойдет, если машина не найдена.

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

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <unistd.h>

int main(int argc, char** argv)
{
    char* machine = "";
    char* ip_add = "";
    FILE* db;
    char* flag;
    char  tmp[256];
    char  par[256];
    printf("> ");
    scanf("%s", par);
    db = fopen("hosts.txt", "r");
    while(db != NULL && fgets(tmp, sizeof(tmp), db) != NULL)
    {
        char * found = strstr(tmp, par);
        if(found)
        {
            flag = "1"; // flag setting to 1 means find the line
            machine = strtok(found, "/");//here's supposed to get the value of the machine
            ip_add = strtok(NULL, "/");//here's supposed to get the value of the ip adress
            break;
        }
    }//while

    if(strcmp(flag, "1") == 0)
    {
        flag = "0"; //flag setting to 0
    }

    printf("flag= %s pc=%s server=%s\n", flag, machine, ip_add);
    return 0;
}

Answer 4

if(strcmp(flag, "1") == 0)

Вы не можете проверить это, потому что если флаг не был инициализирован, это неопределенное поведение

strcmp () поиск символа ' 0' в обоих аргументах