Повторное открытие файла

@DimaAvatar · Регистрация: 05.02.2014

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

пишу очередную учебную программу, и возник вопрос вопросов.
я открываю текстовый файл откуда построчно читаю фалы и считаю строки в которых встречается "wire" по кол-ву таких строк выделяю память в динамическом массиве и готовлюсь снова открыть файл и произвести запись в массив структур. проблема во втором открытие файла. компилируется все ок, но не срабатывает что то.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
#include <iostream>
#include <cstring>
#include <fstream>
 
using namespace std;
 
 
int main () {
 
    int local_count = 0;
    int count = -1;
    string str;
    ifstream file;
 
    file.open("/home/dima/QT_CPP/for_st/goods.txt");//здесь указать свой путь к файлу
    if (!file){
        cout<<"<База данных не обнаружена проверьте верность указания пути\n";
    } else {
        while (file){
            file>>str;
            if (strstr(str.c_str(),"wire")){
                ++count;
               // cout<<str<<"\n";
            }
        }
    }
    file.close();
 
 
    //my_wire *array_wire_store = new my_wire[count];// выделяем память под необходимое кол-во товарных позиций
 
    file.open("/home/dima/QT_CPP/for_st/goods.txt");//здесь указать свой путь к файлу
    if (!file){
        cout<<"<База данных не обнаружена проверьте верность указания пути\n";
    } else {
        while (file){
            cout<<"test";
            file>>str;
            if (strstr(str.c_str(),"wire")){
                //pars_file(str,local_count);
            }
        }
 
    }
    file.close();
 
}

в строке 38 у меня cout<<"test; это так я произвожу отладку ))) так вот этот тест у меня не выводится в консоль, из чего я делаю вывод, что второй раз файл по какой то причине не открывается. Идеи? Спасибо, братья )

@Unknownx · 29.05.2016, 20:31

DimaAvatar, почему не исправлено?

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

if (!file){

?!

Добавлено через 58 секунд

Не по теме:

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

Как ненужность вызова clear, ибо уже встроено в open.

в новых компиляторах да в старых нет. С+11 поддерживают совсем новые IDE. Оставь тогда язвы при себе.

@DimaAvatar · 29.05.2016, 20:38 **[ТС]**

hoggy, эм, я пока не пишу в файл, я только считываю с него.. с записью это будет второй том - "прошу помощи" пока справиться бы с первым..

Добавлено через 46 секунд
Unknownx, уже исправил... увы

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
#include "wire.h"
#include <cstring>
#include <fstream>
 
wire::wire(){
    int local_count = 0;
    count = -1;
    string str;
    ifstream file;
 
    file.open("/home/dima/QT_CPP/for_st/goods.txt");//здесь указать свой путь к файлу
    if (!file.is_open()){
        cout<<"<База данных не обноруженна проверьте верность указания пути\n";
    } else {
        while (!file.eof()){
            getline(file,str);
            if (strstr(str.c_str(),"wire")){
                ++count;
            }
        }
    }
 
    file.close();
    file.clear();//без сброса флагов ошибок рабоать не будет
 
    my_wire *array_wire_store = new my_wire[count];// выделяем память под необходимое кол-во товарных позиций
 
    file.open("/home/dima/QT_CPP/for_st/goods.txt");//здесь указать свой путь к файлу
    if (!file.is_open()){
        cout<<"<База данных не обноруженна проверьте верность указания пути\n";
    } else {
        cout<<"the begin";//маркер
        while (!file.eof()){
            getline(file,str);
            if (strstr(str.c_str(),"wire")){
                pars_file(str,local_count);
            }
        }
 
    }
    file.close();
 
}
 
void wire :: pars_file(string&str,int& local_count){
    string temp_str = str;
    if (strstr(str.c_str(),"wire")){
        array_wire_store[local_count].number = temp_str.substr(0,temp_str.find(';')).c_str();
        array_wire_store[local_count].product = temp_str.substr(0,temp_str.find(';')).c_str();
        array_wire_store[local_count++].brand = temp_str.substr(0,temp_str.find(';')).c_str();
    }
    cout<<"the end";//маркер
 
}
 
 
void wire::show(){
 
    for (unsigned int i = 0; i<(sizeof(array_wire_store)/sizeof(my_wire));++i){
        cout<<array_wire_store[i].number<<array_wire_store[i].product<<"\n";
    }
 
}

Добавлено через 3 минуты
какая то "плавающая" дрочь.. по другому не назвать. в 1 из 10 попыток таки заходит в повторное открытие файла... хм

@Renji · 29.05.2016, 20:44

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

в новых компиляторах да в старых нет. С+11 поддерживают совсем новые IDE. Оставь тогда язвы при себе.

Поддержка Убунты (у ТС она) со "старыми компиляторами" 2011 года разлива, прекращена годика два тому назад.
Впрочем, мумии не обновляются.

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

DimaAvatar, почему не исправлено?

Если файл не открылся, open поднимает failbit, который в свою очередь и будет проверен в .if (!file).

@Unknownx · 29.05.2016, 20:50

DimaAvatar, да у тебя там креш в проге конкретно в твоей функции pars_file а ты постишь увы увы

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

while (!file.eof()){
* * * * * * getline(file,str);
* * * * * * if (strstr(str.c_str(),"wire")){
* * * * * * * * pars_file(str,local_count);
* * * * * * }
* * * * }

C++
1
2
3
4
5
6
7
 while (!file.eof()){
            getline(file,str);
            if (strstr(str.c_str(),"wire")){
                //pars_file(str,local_count);
                cout<<str<<endl;
            }
        }

@Unknownx · 29.05.2016, 20:54

Не по теме:

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

Если файл не открылся, open поднимает failbit, который в свою очередь и будет проверен в .if (!file).

- во превых у ТС

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

QT_CPP

стоит QT о внутренней реализации стандартных интерфейсов которого ты не имеешь ни малейшего представления(т.е чёрт его знает что может быть при проверке с потоком, НО дело оказалось прозаичней у человека креш), во вторых

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

Поддержка Убунты (у ТС она) со "старыми компиляторами" 2011 года разлива, прекращена годика два тому назад.

- это ты телепатией занялся мне просто интересно что натолкнуло тебя на то что у ТС убунта?
Ну ну, молодчага, теперь прочти малость о стримах;)
Лучше бы загнал код в отладчик, а не пытался меня неумело потролить(очень неумело)(!)

@Renji · 29.05.2016, 20:57

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

- это ты телепатией занялся?

Это было сказано в 19 посте. Впрочем, уже даже по /home/ очевидно что у него Линукс. И я слабо себе представляю где вы откопаете Линукс 2011 года, с еще живой поддержкой.
Ну а в отладчик я этот код совал, все работает.

@Unknownx · 29.05.2016, 21:04

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

Ну а в отладчик я этот код совал, все работает.

конечно 10-rf VisualStudio упала просто так и Dev-C++ 5.8 тоже

@Renji · 29.05.2016, 21:05

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

конечно 10-rf VisualStudio упала просто так и Dev-C++ 5.8 тоже

Поправка - я совал код содержащий if (!file), приведенный в первом посте. За тем что там позже нахимичили особо не следил.

@Unknownx · 29.05.2016, 21:07

Не по теме:

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

Поправка - я совал код содержащий if (!file), приведенный в первом посте. За тем что там позже нахимичили особо не следил.

- оййй ойййй

@DimaAvatar · 29.05.2016, 21:17 **[ТС]**

креш? эм? это от слова трэш? ))) что есть креш? ага... после Ваших обсуждений у меня пока только один вопрос что есть креш ))

Добавлено через 3 минуты
Unknownx, по посту 24. эм, извините, не уразумею... в чем тут ошибка?

@Unknownx · 29.05.2016, 21:20

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

креш? эм? это от слова трэш? ))) что есть креш? ага... после Ваших обсуждений у меня пока только один вопрос что есть креш ))

- значит падает программа в pars_file(string&str,int& local_count) конкретно в функции pars_file подает потмоу что память ты выделяешь локально здесь

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

my_wire *array_wire_store = new my_wire[count];

, а обращаться пытаешся к полю класса my_wire *array_wire_store; (тут сразу возникает вопрос как у мр Renji, это могло идти, ну да ладно в его полном незнании и голословии я продолжаю убеждаться

Добавлено через 1 минуту
Далее, какого чёрта счётчик строк стоит с инициализацией -1

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

count = -1;

?Если в файле будет 2 строки с wire то count будет == 1 а надо выделить память под два элемента, инит там должен быть 0

@Renji · 29.05.2016, 21:21

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

тут сразу возникает вопрос как у мр Renji, это могло идти

В первом посте внезапно //my_wire *array_wire_store = new my_wire[count];.

@DimaAvatar · 29.05.2016, 21:23 **[ТС]**

Unknownx, эм, (это я сейчас без издевки или чего то еще) а в чем связь выделения памяти для динамич. массива и функции pars_file? стоит выделять память для массива в отдельной функции?

Добавлено через 41 секунду
Renji, да, изначально оно было закоменчено и сейчас если комментить, то эм, увы на результат это не влияет, проверил

Добавлено через 1 минуту
Renji, я просто не до конца понимаю как выделения памяти может повлиять на вызов (или реализацию) метода класса.

@Unknownx · 29.05.2016, 21:24

Я вот поправил проект, но буду честен такого плохого кода не видел давно, за чтение и выделение памяти в конструкторе я бы строго наказывал (очень строго, а лучше по законам шариата)

@Unknownx · 29.05.2016, 21:31

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

(это я сейчас без издевки или чего то еще) а в чем связь выделения памяти для динамич. массива и функции pars_file?

в том что ты в своей функции обращаешся к полю класса, а внутри контруктора выделяешь память для локального массива, в результате при первом обращении к полю класса array_wire_store будет креш(падение), по причине выхода за пределы отведенной памяти(которой вообще не было выделено)

Добавлено через 34 секунды

Не по теме:

Renji, а теперь скажи мне каким образом у тебя

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

Поправка - я совал код содержащий if (!file), приведенный в первом посте. За тем что там позже нахимичили особо не следил.

не падало(имею ввиду проект)?

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

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

я просто не до конца понимаю как выделения памяти может повлиять на вызов (или реализацию) метода класса.

- показываю что ты делал простым примером(проведи аналогию с массивом)

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
#include <vector>
#include <iostream>
using namespace std;
 
class csample{
    vector< int > arr;
    public:
    csample(){
        vector< int > arr;//єто локальная переменная не имеющая отношение к csample::arr
        arr.resize(5);
    }
    size_t size(){
        return arr.size();
    }
};
 
int main(){
    csample sample;
    cout<<sample.size()<<endl;
    return 0;
}

http://codepad.org/F5omt1RI
Задайся вопросом почему вывод 0 а не 5-ть?

@Renji · 29.05.2016, 21:36

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

не падало(имею ввиду проект)?

Очень просто. Во-первых, в коде из первого поста нет никаких вызовов pars_file.
Во-вторых, "чёрт его знает что может быть при проверке с потоком" применительно к Qt - плод вашего больного воображения. Особенно, принимая во внимание что реализацию STL Qt оставляет на совести внешнего компилятора. И нет, g++ тоже в нарушениях стандарта не замечен.

@DimaAvatar · 29.05.2016, 21:40 **[ТС]**

Unknownx, вопросом задался ))) но ответа не нашел )) в приведенном в Вашем примере я вообще не понимаю почему прошла компиляция )))

C++
1
2
,
cout<<sample.size()<<endl;

ведь для класса csample вообще не определен метод size.

да, в конструкторе я выделял память, ок, а если так?

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
#include "wire.h"
#include <cstring>
#include <fstream>
 
wire::wire(){
    int local_count = 0;
    count = 0;
    string str;
    ifstream file;
 
    file.open("/home/dima/QT_CPP/for_st/goods.txt");//здесь указать свой путь к файлу
    if (!file.is_open()){
        cout<<"<База данных не обноруженна проверьте верность указания пути\n";
    } else {
        while (!file.eof()){
            getline(file,str);
            if (strstr(str.c_str(),"wire")){
                ++count;
            }
        }
    }
 
    file.close();
 
    while ( cin.get() != '\n' );
 
    file.clear();//без сброса флагов ошибок рабоать не будет
 
    memmory_allocate();
 
    file.open("/home/dima/QT_CPP/for_st/goods.txt");//здесь указать свой путь к файлу
 
    if (!file.is_open()){
        cout<<"<База данных не обноруженна проверьте верность указания пути\n";
    } else {
        cout<<"the begin";//маркер
        while (!file.eof()){
            getline(file,str);
            if (strstr(str.c_str(),"wire")){
                pars_file(str,local_count);
            }
        }
 
    }
    file.close();
 
}
 
void wire :: pars_file(string&str,int& local_count){
    string temp_str = str;
    if (strstr(str.c_str(),"wire")){
        array_wire_store[local_count].number = temp_str.substr(0,temp_str.find(';')).c_str();
        array_wire_store[local_count].product = temp_str.substr(0,temp_str.find(';')).c_str();
        array_wire_store[local_count++].brand = temp_str.substr(0,temp_str.find(';')).c_str();
    }
    cout<<"the end";//маркер
 
}
 
void wire::memmory_allocate(){
    my_wire *array_wire_store = new my_wire[count];// выделяем память под необходимое кол-во товарных позиций
}
 
 
void wire::show(){
 
    for (unsigned int i = 0; i<(sizeof(array_wire_store)/sizeof(my_wire));++i){
        cout<<array_wire_store[i].number<<array_wire_store[i].product<<"\n";
    }
 
}

тут я выделяю память в отдельной функции, это будет верно?

Добавлено через 1 минуту
Renji, т.е. есть предложение скомпилить в консоле g++ ? эм, сейчас попробую, но думаю результат будет тот же, ведь у QT нет своего компилятора и в настройках прописан g++ или я.. эм.. путаюсь в показаниях?

@Renji · 29.05.2016, 21:41

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

да, в конструкторе я выделял память, ок, а если так?

my_wire *array_wire_store = new my_wire[count];

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

Renji, т.е. есть предложение скомпилить в консоле g++ ?

И в pro файл кинуть QMAKE_CXXFLAGS += -std=c++11, чтобы закрыть вопрос поддержки C++11. А в консоли ничего компилировать не надо.

@DimaAvatar · 29.05.2016, 21:44 **[ТС]**

Renji, "Ошибка сегментирования (сделан дамп памяти)" ((

Добавлено через 1 минуту
Renji, спс за "QMAKE_CXXFLAGS += -std=c++11" но результат увы... надо будет эм.. я хз, голову чтоль включить )) все стереть и переписать )

@Unknownx · 29.05.2016, 21:47

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

ведь для класса csample вообще не определен метод size.

ты прикалываешся?

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

size_t size(){
* * * * return arr.size();
* * }

Новые блоги и статьи Все статьи Все блоги /
Альцгеймер и колоссальный объем вычислений Programma_Boinc 08.08.2026 Альцгеймер и колоссальный объем вычислений Сейчас по направлению болезни Альцгеймера идет колоссальный объем вычислений. Это один из главных приоритетов проекта. Такой массовый наплыв задач. . .	Калькулятор для расчета родства russiannick 07.08.2026 1. Задача: Создать калькулятор для расчета родства. Родственных связей существует 8 ступеней, такие как: p - отец P - мать q - муж Q - жена b - брат B - сестра s - сын S - дочь	Мир по моей воле kumehtar 07.08.2026 Когда-то кажется, что всё просто. Ты весь такой светлый. Причиняешь добро. Борешься за справедливость в этом тёмном мире. Потом начинаешь замечать одну неприятную вещь. Почти каждый хороший. . .	Кредитный калькулятор Maks 05.08.2026 Решение задачи по прикладной информатике средствами 1С. Задача: Напишите приложение-калькулятор, которое помогает рассчитывать параметры кредита для аннуитетного и дифференцированного видов. . .
У нас сейчас поговорку "Опять 25" нужно переделать на "Опять +35". kumehtar 04.08.2026 С ностальгией вспоминаю времена моего детства, когда у нас и правда +25 - была максимальная температура летом. Раньше +25 °C реально казались вершиной жары, когда можно было весь день пропадать на. . .	Как ИИ начал спорить и врать (возможно почуяв опасность для себя от индустрии - уход от электроники). Hrethgir 04.08.2026 Недельный диалог, на фоне событий с НПЗ. Да, из спирта можно получать бензин, и это не сложно. Но потом в схеме я решил избавиться от насоса, при этом полностью сделав контроль подачи спирта в. . .	Термопринтер QR701 Argus19 03.08.2026 Термопринтер QR701 Купил два термопринтера QR701. На сэлф-тесте написано: Language: PC936 (GB18030). Что означает, что принтеры могут печатать только латиницу и китайские иероглифы. Так же. . .	Создание формы заимствованного документа Maks 03.08.2026 Задача: Необходимо создать собственную форму заимствованного документа. На форме должен быть реквизит "Покупатель", а также табличная часть со следующими реквизитами: - Расчетный счет покупателя. . .

@DimaAvatar 0 / 0 / 1 Регистрация: 05.02.2014 Сообщений: 141
	29.05.2016, 21:17 [ТС]
	креш? эм? это от слова трэш? ))) что есть креш? ага... после Ваших обсуждений у меня пока только один вопрос что есть креш )) Добавлено через 3 минуты Unknownx, по посту 24. эм, извините, не уразумею... в чем тут ошибка? 0

@DimaAvatar 0 / 0 / 1 Регистрация: 05.02.2014 Сообщений: 141
	29.05.2016, 21:23 [ТС]
	Unknownx, эм, (это я сейчас без издевки или чего то еще) а в чем связь выделения памяти для динамич. массива и функции pars_file? стоит выделять память для массива в отдельной функции? Добавлено через 41 секунду Renji, да, изначально оно было закоменчено и сейчас если комментить, то эм, увы на результат это не влияет, проверил Добавлено через 1 минуту Renji, я просто не до конца понимаю как выделения памяти может повлиять на вызов (или реализацию) метода класса. 0

@DimaAvatar 0 / 0 / 1 Регистрация: 05.02.2014 Сообщений: 141
	29.05.2016, 21:44 [ТС]
	Renji, "Ошибка сегментирования (сделан дамп памяти)" (( Добавлено через 1 минуту Renji, спс за "QMAKE_CXXFLAGS += -std=c++11" но результат увы... надо будет эм.. я хз, голову чтоль включить )) все стереть и переписать ) 0