Построчное считывание файла и преобразование считанной строки в число (linux)

@Sid_Pic · Регистрация: 10.10.2019

Author24 — интернет-сервис помощи студентам

Здравствуйте, помогите пожалуйста, нужно последовательно преобразовывать строки файла в числа (желательно unsigned short) и записывать в массив. Ниже мой текущий нерабочий код:

C

 FILE * lMap;
 if ((lMap = fopen(fileName, "r")) == NULL) puts ("\e[1;1H\e[2K This file is not exist!"); else
 {
    lMap  = fopen(fileName, "r");
    fgets (nStr, sizeof (nStr), lMap); n = atoi(nStr); // преобразование в целое число здесь работает корректно
  
    for (unsigned short i = 0; i < n; i++)
    {
        if (i < n/2) { fgets (xStr, 3, lMap); itemX[i] = atoi(xStr); } else // здесь и ниже строки преобразуются в 0,
        if (i < n) { fgets (yStr, 3, lMap); itemY[i] = atoi(yStr); }          // хотя в файле указаны иные числа
 
    }
    fclose(lMap);
}

@L0M · 11.12.2019, 22:54

Sid_Pic, строка 4 лишняя.

Покажите содержимое файла из которого читаются данные.

@Sid_Pic · 12.12.2019, 19:21 **[ТС]**

Почему четвёртая - лишняя?
часть файла:
....
78
77
76
75
74
73
23
23
....

@palva · 12.12.2019, 20:26

Почему такой маленький буфер для чтения чисел? Если я правильно понимаю, после первого чтения буфер будет содержать символы '7','8','\0', типа всё правильно. Второе чтение даст '\n','\0','\0'. Следующее чтение даст '7','7','\0'. Наверно, вы не этого хотели?

Добавлено через 11 минут

Сообщение от Sid_Pic

Почему четвёртая - лишняя?

Вы уже открыли файл. Какой смысл открывать его повторно в том же режиме?

@L0M · 12.12.2019, 21:54

Sid_Pic, видимо вам нужно что-то типа такого:

C

#include <stdio.h>
#include <stdlib.h>
#include <math.h>
 
#define BUFLEN 80
 
int main(void)
{
    const char* fileName = "input.txt";
    char nStr[BUFLEN];
    int n;
    int itemX[100], itemY[100];
 
    FILE* lMap;
    if ((lMap = fopen(fileName, "r")) == NULL)
        puts("\e[1;1H\e[2K This file is not exist!");
    else
    {
        fgets(nStr, sizeof(nStr), lMap);
        n = atoi(nStr);
 
        for (unsigned short i = 0; i < n; ++i)
        {
            fgets(nStr, BUFLEN, lMap);
            if (i < n / 2)
                itemX[i] = atoi(nStr);
            else
                itemY[i - n / 2] = atoi(nStr);
 
        }
        fclose(lMap);
 
        for (unsigned short i = 0; i < n / 2; i++)
            printf("[%d, %d]\n", itemX[i], itemY[i]);
    }
 
    return 0;
}

Добавлено через 3 минуты
Я не очень понял, зачем вы читаете построчно, а потом переводите в целое.
Видимо, в это есть какой-то свой сакральный смысл?
На мой взгляд было бы проще пользоваться fscanf().

@Sid_Pic · 13.12.2019, 19:52 **[ТС]**

Сообщение от L0M

Я не очень понял, зачем вы читаете построчно, а потом переводите в целое.
Видимо, в это есть какой-то свой сакральный смысл?

Чтобы переменные не сливались, очевидно же, перевожу в целое чтобы пользоваться можно было.

Сообщение от L0M

На мой взгляд было бы проще пользоваться fscanf().

fscanf(lMap, "%s", value) почему-то вообще (null) записывает.

Сообщение от L0M

видимо вам нужно что-то типа такого

Ваш способ делает всё-таки делает то же самое, хоть и более правильно.

Добавлено через 34 минуты
Всем спасибо. Невероятно, но факт: при разделении циклов всё заработало

C

    FILE * lMap;
    if ((lMap = fopen(fileName, "r")) == NULL) puts ("\e[1;1H\e[2K This file is not exist!"); else
    {
      puts ("\e[2J");
       fgets (nStr, sizeof(nStr), lMap); n = atoi(nStr);
 
         for (unsigned short i = 0; i < n; i++)
        {
           fgets (nStr, 4, lMap);
           itemX[i] = atoi(nStr);
        }
  
        for (unsigned short i = 0; i < n; i++)
        {
           fgets (nStr, 4, lMap);
           itemY[i] = atoi(nStr);
        }
    }
         fclose (lMap);

@L0M · 13.12.2019, 20:02

Sid_Pic, вот же:

C

#include <stdio.h>
#include <stdlib.h>
#include <math.h>
 
int main(void)
{
    const char* fileName = "input.txt";
    int n;
    int itemX[100], itemY[100];
 
    FILE* lMap;
    if ((lMap = fopen(fileName, "r")) == NULL)
        puts("\e[1;1H\e[2K This file is not exist!");
    else
    {
        fscanf(lMap, "%d", &n);
 
        for (unsigned short i = 0; i < n; ++i)
        {
            if (i < n / 2)
                fscanf(lMap, "%d", &itemX[i]);
            else
                fscanf(lMap, "%d", &itemY[i - n / 2]);
        }
        fclose(lMap);
 
        for (unsigned short i = 0; i < n / 2; i++)
            printf("[%d, %d]\n", itemX[i], itemY[i]);
    }
 
    return 0;
}

Не?

Добавлено через 5 минут

Сообщение от Sid_Pic

Ваш способ делает всё-таки делает то же самое, хоть и более правильно.

А что делает "то же самое", но "более правильно"? Не работает, как и у вас?

Добавлено через 3 минуты
Интересно. То, что вы дописали в посте #6, несколько отличается по количеству считываемых данных от стартового топика. В #1 вы считывали n данных в два массива. В #6 вы считываете 2n данных в те же два массива.

@L0M Мозгоправ 1744 / 1038 / 468 Регистрация: 01.10.2018 Сообщений: 2,138 Записей в блоге: 2
	11.12.2019, 22:54	2
	Sid_Pic, строка 4 лишняя. Покажите содержимое файла из которого читаются данные. 1

@Sid_Pic 1 / 1 / 0 Регистрация: 10.10.2019 Сообщений: 11
	12.12.2019, 19:21 [ТС]	3
	Почему четвёртая - лишняя? часть файла: .... 78 77 76 75 74 73 23 23 .... 0

@palva 4240 / 2937 / 687 Регистрация: 08.06.2007 Сообщений: 9,817 Записей в блоге: 4
	12.12.2019, 20:26	4
	Почему такой маленький буфер для чтения чисел? Если я правильно понимаю, после первого чтения буфер будет содержать символы '7','8','\0', типа всё правильно. Второе чтение даст '\n','\0','\0'. Следующее чтение даст '7','7','\0'. Наверно, вы не этого хотели? Добавлено через 11 минут Сообщение от Sid_Pic Почему четвёртая - лишняя? Вы уже открыли файл. Какой смысл открывать его повторно в том же режиме? 1

	13.12.2019, 20:02

Построчное считывание файла и преобразование считанной строки в число (linux)

Решение