Цикл While странно себя ведет

@Encephalopathy · Регистрация: 04.06.2016

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

Пишу программу которая считывает из каждой строки массив неизвестной длины.Цикл while(который помечен комментом) странно себя ведет,например ставлю точку останова на 76 строке = программа ни разу не стопается,а если вызову while(false) то заходит в цикл один раз.Изначально планировал сделать while(strlen(s)>0).Visual studio 2015
Содержимое data.txt:
4 5 8 45 2
3 16 9 52 3
7 6 45 -8 -45 -2 45

C++

#include <stdio.h>
#include <conio.h>
#include <Windows.h>
#include <cmath>
 
using namespace std;
char* read_string(FILE *fp) {
 
    char c; int len = 0;
    long int pos = ftell(fp);
    while (1) {
 
        c = fgetc(fp);
 
        if (c == '\n') {
 
            break;
 
        }
 
        else len++;
 
        if (feof(fp)) break;
 
    }
    if (len == 0) return NULL;
    char *s = new char[len + 1];
    fseek(fp, pos, SEEK_SET);
    int i = 0;
    while (1) {
 
        c = fgetc(fp);
 
        if (c == '\n') {
 
            break;
 
        }
 
        else {
            
            s[i] = c;
            i++;
        }
 
        if (feof(fp)) break;
 
    }
    s[i] = '\0';
    return s;
 
}
 
int main()
{
 
    SetConsoleCP(1251);
    SetConsoleOutputCP(1251);
    FILE *fp = fopen("A:\\Users\\admin\\Desktop\\data.txt", "rt");
 
    if (!fp) {
 
        printf("Can't open file data.txt!"); getchar(); exit(1);
 
    }
    char* s = read_string(fp);
    int x = 1;
    while (true);//Этот цикл странно себя ведет
    {
        
        
        char **w = new char *[strlen(s) / 2];
        int num = 0;
        char *token = strtok(s, " ");
 
        while (token) {
 
            int len = strlen(token);
 
            w[num] = new char[len + 1];
 
            strcpy(w[num], token);
 
            num++;
 
            token = strtok(NULL, " ");
 
        }
        int *a = new int[num];
        for (size_t i = 0; i < num; i++)
        {
            a[i] = atoi(w[i]);
            printf("%d ", a[i]);
        }
        
            s = read_string(fp);
        
            
    }
    
    
 
 
 
    _getch();
    return 0;
}

@anapshy · 03.07.2017, 14:46

Encephalopathy, ; уберите

C++

1	while (true);//Этот цикл странно себя ведет

И память освобождать не забывайте.

@Antikl · 03.07.2017, 15:12

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

while (true);//Этот цикл странно себя ведет

во первых он бесконечен притом надо убрать ;, где условие выхода? или так и надо?

@Encephalopathy · 03.07.2017, 15:21 **[ТС]**

Спасибо теперь понял,условие изначально было strlen(s)>0 но поведение сбило столку и я поставил true.

Добавлено через 5 минут
anapshy, На счет памяти,не могли бы конкретнее подсказать?

@Antikl · 03.07.2017, 15:24

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

На счет памяти,не могли бы конкретнее подсказать?

C++

delete

используйте

Добавлено через 1 минуту
как пример

C++

#include <iostream>
using namespace std;
 
int main()
{
    int *a = new int; 
    int *b = new int; 
 
    delete b;
    delete a;
 
    return 0;
}

@Encephalopathy · 03.07.2017, 16:13 **[ТС]**

C++

#include <stdio.h>
#include <conio.h>
#include <Windows.h>
#include <cmath>
 
using namespace std;
char* read_string(FILE *fp) {
 
    char c; int len = 0;
    long int pos = ftell(fp);
    while (1) {
 
        c = fgetc(fp);
 
        if (c == '\n') {
 
            break;
 
        }
 
        else len++;
 
        if (feof(fp)) break;
 
    }
    if (len == 0) return NULL;
    char *s = new char[len + 1];
    fseek(fp, pos, SEEK_SET);
    int i = 0;
    while (1) {
 
        c = fgetc(fp);
 
        if (c == '\n') {
 
            break;
 
        }
 
        else {
            
            s[i] = c;
            i++;
        }
 
        if (feof(fp)) break;
 
    }
    s[i] = '\0';
    return s;
 
}
 
int main()
{
 
    SetConsoleCP(1251);
    SetConsoleOutputCP(1251);
    FILE *fp = fopen("A:\\Users\\admin\\Desktop\\data.txt", "rt");
 
    if (!fp) {
 
        printf("Can't open file data.txt!"); getchar(); exit(1);
 
    }
    char* s = read_string(fp);
    int x = 1;
    while (strlen(s)>0)
    {
        
        
        char **w = new char *[strlen(s) / 2];
        int num = 0;
        char *token = strtok(s, " ");
 
        while (token) {
 
            int len = strlen(token);
 
            w[num] = new char[len + 1];
 
            strcpy(w[num], token);
 
            num++;
 
            token = strtok(NULL, " ");
 
        }
        delete token;
        if (num)
        {
            int *a = new int[num];
            for (size_t i = 0; i < num; i++)
            {
                a[i] = atoi(w[i]);
                printf("%d ", a[i]);
            }
            printf("\n");
            delete []a;
            for (size_t i = 0; i < num; i++)
            {
                delete w[i];
            }
            delete []w;
        }
            s = read_string(fp);
        
            
    }
    
    
 
 
 
    _getch();
    return 0;
}

Так?

Добавлено через 45 секунд
Выдает ошибку HEAP CORRUPTION DETECTED

Добавлено через 1 минуту
Уже после того как выведет все строки из файла.

Добавлено через 8 минут
Все вроде разобрался.

Добавлено через 2 минуты
В конце файла программа почему-то считывала букву Я в качестве последней строки и пыталась создать массив с [0] элементов.Не понятно только откуда Я взялась,в файле её нет.

Добавлено через 32 минуты
Откуда Я взялось тоже разобрался.

@Antikl с++ 1282 / 523 / 225 Регистрация: 15.07.2015 Сообщений: 2,562
	03.07.2017, 15:12	3
	Сообщение от Encephalopathy while (true);//Этот цикл странно себя ведет во первых он бесконечен притом надо убрать ;, где условие выхода? или так и надо? 1

@Encephalopathy 71 / 71 / 55 Регистрация: 04.06.2016 Сообщений: 237
	03.07.2017, 15:21 [ТС]	4
	Спасибо теперь понял,условие изначально было strlen(s)>0 но поведение сбило столку и я поставил true. Добавлено через 5 минут anapshy, На счет памяти,не могли бы конкретнее подсказать? 0

	03.07.2017, 16:13