-1

Нужно открыть несколько файлов, в них случайным образом добавляются данные из массива объектов(одно из свойств - имя файла куда нужно сохранить текущий объект), затем все файлы закрываются.

Как создавать объект fstream сразу в конце вектора?

    fstream tmp_fp;
    vector <fstream*> vFP;
    for (int i = 1; i < 5; ++i) {
        string name = to_string(i) + ".txt";
        fstream tmp_fp(name,fstream::out); // так нельзя после текущей итерации указатель будет ссылаться на несуществующий объект
        vFP.push_back(&tmp_fp);
    }

Всем спасибо за обсуждение, тот ответ который был правильным удалили, вот его реализация:

    fstream tmp_fp;
    vector <fstream> vFP;
    for (int i = 0; i < 4; ++i) {
        string name = to_string(i) + ".txt";
        vFP.emplace_back(name,fstream::out);
    }

    for (int i = 0; i < 4; ++i) {
        for (int j = 0; j < 5; ++j) { // как бы заполнение файлов данными
            vFP[i] << j << endl;
        }
    }

    for (int i = 0; i < 4; ++i) {
        vFP[i].close(); 
    }
11
  • а чем вас не устраивает отдельная работа с файлами? т.е поработали с одним файлом и закрыли, затем тоже самое делаем с остальными файлами?
    – perfect
    13 сен 2015 в 15:39
  • Писать в 5 файлов параллельно? А оно Вам точно надо? Просто внутри фукнции создайте ifstream, обрабатывающий один объект и в цикле вызовите ее 5 раз.
    – igumnov
    13 сен 2015 в 15:39
  • 2
    @xttz, если Вам для практических нужд, то просто вместо fstream из крестов возьмите FILE * из чистого C.
    – avp
    13 сен 2015 в 15:54
  • 1
    @xttz 50 млн объектов в памяти удерживать? что то здесь не то...
    – perfect
    13 сен 2015 в 16:06
  • 1
    @VladD я слышал когда-то такую фразу про windows "не нужно бороться с системой - нужно пользоваться", она как раз и к С++ подходит, зачем явно использовать динамическое выделение памяти и прочее, если нас пытаются от этого оградить. вектор это и так динамический массив, так зачем в нем еще указатели на объекты хранить.
    – xttz
    13 сен 2015 в 16:57

3 ответа 3

3

Создавать fstream динамически (new/delete). А в вашем варианте он просто имеет слишком узкую область видимости.

2
  • вектор это и так динамический контейнер, вопрос лишь в том как в него засунуть не указатель на <fstream*> а сам объект <fstream>;
    – xttz
    13 сен 2015 в 15:42
  • Вектор - да. А вот в строке fstream tmp_fp(name,fstream::out); создается объект fstream, который существует одну итерацию цикла. Вам же нужно fstream создать динамически, который будет жить дольше. Вектор тут не при чем.
    – Nex
    13 сен 2015 в 15:45
3

Класс std::fstream имеет удаленный конструктор копирования, но имеет конструктор перемещения. Поэтому вы можете использовать вектор объектов std::fstream, например, следующим образом:

const size_t N = 5;

std::vector<std::fstream> vFP;
vFP.reserve( N );

for ( size_t i = 1; i < N; ++i ) 
{
    std::string name = std::to_string( i ) + ".txt";
    vFP.push_back( std::fstream( name, std::ios::out ) );
}

То есть создавая временный объект типа std::fstream в вызове функции push_back

Вместо функции push_back вы могли бы использовать emplace_back( name, std::ios::out ).

Но в любом случае вам лучше сначала зарезервировать память в векторе под требуемое число элементов:

vFP.reserve( N );

Более того, вы даже можете использовать стандартный алгоритм std::generate_n для создания вектора. Например,

#include <iostream>
#include <fstream>
#include <vector>
#include <string>
#include <algorithm>
#include <iterator>

int main()
{
    const size_t N = 5;

    std::vector<std::fstream> vFP;
    vFP.reserve( N );

    std::generate_n( std::back_inserter( vFP ), N, 
                     [i = 1] () mutable { return std::fstream( std::to_string( i++ ) + ".txt", std::ios::out ); } );  
}    
0

Можно хранить сами fstream: vector<fstream> vFP; и создавать fstream на месте с помощью emplace_back(): vFP.emplace_back(name, fstream::out);.

Либо, также храня сами fstream в векторе, создавать и после перемещать временный объект tmp_fp: vFP.push_back(move(tmp_fp));

16
  • 1
    получите ошибку шаблона при компиляции
    – perfect
    13 сен 2015 в 15:51
  • @perfect нет, ошибок компиляции нет, но при попытке например закрыть файлы vFP[i].close() программа вылетает.
    – xttz
    13 сен 2015 в 15:53
  • 1
    @xttz я про ошибку компиляции. У меня ругается на то, что у basic_fstream явно удалён конструктор копий. Что само по себе непонятно, т.к., насколько я знаю, emplace_back никак не трогает конструктор копий.
    – user190490
    13 сен 2015 в 16:17
  • 1
    @Wolf Ramovsky ошибка в том что для того чтобы что то положить в вектор оно должно иметь перегруженные операторы = и < . насколько я помню, а у fstream и ofstream таких операторов нет.
    – perfect
    13 сен 2015 в 16:18
  • 1
    @WolfRamovsky использую mingw64 с флагом -std=c++11 и все отлично работает.
    – xttz
    13 сен 2015 в 16:20

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.