Ошибка с записью в vector из файла

@ЗеХель · Регистрация: 04.11.2014

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

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

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
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
#include <string>
#include <iostream>
#include <fstream>
#include <vector>
 
using namespace std;
 
#define SIZE 80
 
void read_file(string);
void sort_text(vector<string>);
void fill_output_file(vector<string>);
void check_file_name(string, ofstream &);
 
int main()
{
    setlocale(LC_ALL, "Russian");
    cout << "Добро пожаловать %username% ! \n";
    cout << "Введите имя начального файла для записи:\n";
    string name;
    getline(cin, name);
    ofstream fout;
    fout.open(name, ios:: _Noreplace);
 
    check_file_name (name, fout);
 
    cout << "Для окончания введите END\n";
    cout << "Введите строки для записи не более 80 символов:\n";
    string buffer;
    cin.get();
    getline(cin, buffer);
    while (buffer != "END")
    {
        
        if (size(buffer) > SIZE)
        {
            for (int i = size(buffer); i > SIZE; i--)
            {
                buffer[i] = NULL;
            }
            cout << "Строка слишком большая, обрезана до 80 символов.\n";
            cout << "Введите строку для записи не более 80 символов:\n";
        }
 
        fout << buffer << endl;
        getline(cin, buffer);
    }
    fout.close();
 
    read_file(name);
 
    system("pause");
}
 
//проверка имени на уникальность.
void check_file_name(string name, ofstream  &fout)
{
    while (!fout.is_open())
    {
        cout << "Файл с таким именем уже существует, хотите ввести другое имя? (Y/N) \n";
        string answer;
        cin >> answer;
        if (answer == "Y" || answer == "y")
        {
            cout << "Введите имя файла для записи:\n";
            cin.get();
            getline(cin, name);
            fout.open(name, ios::_Noreplace);
        }
        else fout.open(name, ios::out);
    }
} 
 
//запись слов из начального файла в вектор для сортировки.
void read_file(string name)
{
    ifstream fin;
    vector <string> word;
    string buffer2;
    fin.open(name);
        if (!fin.is_open())
        {
            cout << "Файл не существует.\n";
        }
        while (!fin.eof())
        {
            fin >> buffer2;
            word.push_back(buffer2);
        }
        fin.close();
        sort_text(word);
}
 
//сортировка текста.
void sort_text(vector<string> word)
{
    string example = "zyxwvutsrqponmlkjihgfedcba";
    vector <string> checked_words;
    int k = 0;
    bool flag = false;
    for (int i(0); i < word.size(); i++)
    {
        string buffer = word[i];
        while (buffer[k] != '\0')
        {
            if (buffer[k] == example[k])
            {
                k++;
                flag = true;
            }
            else
            {
                flag = false;
                break;
            }
        }
        if (flag == true)
        {
            checked_words.push_back(buffer);
        }
    }
    fill_output_file(checked_words);
}
 
//заполнение конечного файла после сортировки.
void fill_output_file(vector<string> checked_words)
{
    ofstream fout2;
    string name;
    cout << "Введите имя конечного файла для записи:\n";
    getline(cin, name);
    fout2.open(name, ios::_Noreplace);
 
    check_file_name(name, fout2);
 
    for (int i(0); i < checked_words.size(); i++)
    {
        fout2 << checked_words[i] << endl;
    }
    fout2.close();
}

@Nosey · 05.01.2016, 00:26

C++
1
2
3
4
5
        while (!fin.eof()) // пока не достигли конца файла читаем, конец файла это не последняя запись в файле, это "логический символ" расположенный после всего файла, тобишь это такая абстракция.
        {
            fin >> buffer2; // !!пытаемся!! прочитать из файла строку ( если не удаётся, то buffer2 сохраняет старое значение)
            word.push_back(buffer2); // и мы его добавляем.
        }

Решение :
сдлеать проверку удалось ли прочитать строку:

C++
1
2
//Считываем строку.
if (fin){//удалось прочитать строку}

Или воспользоваться способом прочитать файл предоставляемый стандартной библиотекой:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
#include <vector>
#include <string>
#include <fstream>
#include <iterator>
.......
 
    std::ifstream fin(name);
    if (!fin)
    {
        std::cout << "Файл не существует.\n";
        return;
    }
    std::vector<std::string> word = std::vector<std::string>(std::istream_iterator<std::string>(fin), std::istream_iterator<std::string>());
    fin.close();

@ЗеХель · 05.01.2016, 00:52 **[ТС]**

С первым вариантом не сработало, я вставил его перед 87 строкой, но ничего не изменилось, а что со вторым? Я не совсем понимаю, что нужно сделать, можно подробнее, пожалуйста)

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

C++
1
std::vector<std::string> word = std::vector<std::string>(std::istream_iterator<std::string>(fin), std::istream_iterator<std::string>());

@Nosey · 05.01.2016, 01:02

ЗеХель,
Первый вариант - вставить приведённое выше условие после

87 строки в вашем первом листинге, т.е. сразу после чтения строки (fin >> buffer2;). (мы же проверяем, прочиталась ли строка, или нет)

Второй вариант - заменить всю функцию read_file на приведённый мной выше код и добавить подключение приведённых заголовочных файлов(к вашему списку добавилось подключение

Code
1
#include <iterator>

).
Конечно оставив вызов sort_text(word);

@ЗеХель · 05.01.2016, 01:06 **[ТС]**

Я перепутал, я поставил после, но не сработало

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

после 87 строки

А во втором варианте уже под мой случай сделано что ли?

@Nosey · 05.01.2016, 01:12

C++
1
2
3
4
5
6
7
8
9
10
        while (!fin.eof())
        {
            fin >> buffer2;
            if (!fin)
            {
                  //не удалось прочитать строку - добавлять её не надо -> на выход.
                  break;
            }
            word.push_back(buffer2);
        }

И да второй вариант уже для вас, ибо ваш вариант отличается от других вариантов только именем файла )))

@ЗеХель · 05.01.2016, 01:12 **[ТС]**

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

заменить всю функцию read_file

И этот вариант не работает, последнее значение считывает дважды. Может дело в ключевом слове END для окончания ввода?

@Nosey · 05.01.2016, 01:24

Сообщение от ЗеХель

И этот вариант не работает, последнее значение считывает дважды. Может дело в ключевом слове END для окончания ввода?

От end'a не зависит, вроде

Если вы попробовали все варианты и они не работают ( компилируются и не работают) то кидайте последний листинг кода вашего и содержимое файла с которым программа работает.

@ЗеХель · 05.01.2016, 02:25 **[ТС]**

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

Если вы попробовали все варианты и они не работают ( компилируются и не работают) то кидайте последний листинг кода вашего и содержимое файла с которым программа работает.

Вот листинг. дублирует последнее слово, которое подходит по условию - состоит только из z, zy, zyx.... и так вплоть до cba.

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
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
#include <string>
#include <iostream>
#include <fstream>
#include <vector>
 
using namespace std;
 
#define SIZE 80
 
void read_file(string);
void sort_text(vector<string>);
void fill_output_file(vector<string>);
void check_file_name(string, ofstream &);
void output_text(string);
 
int main()
{
    setlocale(LC_ALL, "Russian");
    cout << "Добро пожаловать %username% ! \n";
    cout << "Введите имя начального файла для записи:\n";
    string name;
    getline(cin, name);
    ofstream fout;
    fout.open(name, ios:: _Noreplace);
 
    check_file_name (name, fout);
 
    cout << "Для окончания введите END\n";
    cout << "Введите строки для записи не более 80 символов:\n";
    string buffer;
    cin.get();
    getline(cin, buffer);
    while (buffer != "END")
    {
        
        if (size(buffer) > SIZE)
        {
            for (int i = size(buffer); i > SIZE; i--)
            {
                buffer[i] = NULL;
            }
            cout << "Строка слишком большая, обрезана до 80 символов.\n";
            cout << "Введите строку для записи не более 80 символов:\n";
        }
 
        fout << buffer << endl;
        getline(cin, buffer);
    }
    fout.close();
 
    read_file(name);
 
    system("pause");
}
 
//проверка имени на уникальность.
void check_file_name(string name, ofstream  &fout)
{
    while (!fout.is_open())
    {
        cout << "Файл с таким именем уже существует, хотите ввести другое имя? (Y/N) \n";
        string answer;
        cin >> answer;
        if (answer == "Y" || answer == "y")
        {
            cout << "Введите имя файла для записи:\n";
            cin.get();
            getline(cin, name);
            fout.open(name, ios::_Noreplace);
        }
        else fout.open(name, ios::out);
    }
} 
 
//запись слов из начального файла в вектор для сортировки.
void read_file(string name)
{
    ifstream fin;
    vector <string> word;
    string buffer2;
    fin.open(name);
        if (!fin.is_open())
        {
            cout << "Файл не существует.\n";
        }
        while (!fin.eof())
        {
            fin >> buffer2;
            word.push_back(buffer2);
        }
        fin.close();
        sort_text(word);
}
 
//сортировка текста.
void sort_text(vector<string> word)
{
    string example = "zyxwvutsrqponmlkjihgfedcba";
    vector <string> checked_words;
    int k = 0;
    bool flag = false;
 
    for (int i(0); i < word.size(); i++)
    {
        string buffer = word[i];
 
        while (buffer[k] != '\0')
        {
            if (buffer[k] == example[k])
            {
                k++;
                flag = true;
            }
            else
            {
                flag = false;
                break;
            }
        }
        if (flag == true)
        {
            checked_words.push_back(buffer);
            flag = false;
        }
    }
    fill_output_file(checked_words);
}
 
//заполнение конечного файла после сортировки.
void fill_output_file(vector<string> checked_words)
{
    ofstream fout2;
    string name;
    cout << "Введите имя конечного файла для записи:\n";
    getline(cin, name);
    fout2.open(name, ios::_Noreplace);
 
    check_file_name(name, fout2);
 
    for (int i(0); i < checked_words.size(); i++)
    {
        fout2 << checked_words[i] << endl;
    }
    fout2.close();
    output_text(name);
}
 
//вывод конечного файла в консоли.
void output_text(string name)
{
    string temp;
 
    ifstream fin2;
    fin2.open(name);
 
    cout << "Состав файла после обработки:\n";
    while (!fin2.eof())
    {
        fin2 >> temp;
        cout << temp << endl;
    }
    fin2.close();
}

@Nosey · 05.01.2016, 03:03

ЗеХель,

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
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
#include <iostream>
#include <vector>
#include <string>
#include <algorithm>
#include <iterator>
#include <fstream>
 
#define SIZE 80
 
void read_file(const std::string&);
void sort_text(const std::vector<std::string>&);
void fill_output_file(const std::vector<std::string>&);
void check_file_name(std::string&, std::ofstream &);
void output_text(const std::string&);
 
int main()
{
    setlocale(LC_ALL, "Russian");
    std::cout << "Добро пожаловать %username% ! \n";
    std::cout << "Введите имя начального файла для записи:\n";
    std::string name;
    std::getline(std::cin, name);
    std::ofstream fout(name);
//  fout.open(name, std::ios::_Noreplace);
 
//  check_file_name(name, fout);
 
    std::cout << "Для окончания введите END\n";
    std::cout << "Введите строки для записи не более 80 символов:\n";
    std::string buffer;
    std::cin.get();
    std::getline(std::cin, buffer);
    while (buffer != "END")
    {
        if (buffer.size() > SIZE)
        {
            std::cout << "Строка слишком большая, обрезана до 80 символов.\n";
            std::cout << "Введите строку для записи не более 80 символов:\n";
            fout.write(buffer.c_str(), SIZE);
            fout << std::endl;
        }
        else
        {
            fout << buffer << std::endl;
        }
        std::getline(std::cin, buffer);
    }
    fout.close();
 
    read_file(name);
 
    system("pause");
}
 
//проверка имени на уникальность.
void check_file_name(std::string& name, std::ofstream& fout)
{
    while (!fout.is_open())
    {
        std::cout << "Файл с таким именем уже существует, хотите ввести другое имя? (Y/N) \n";
        std::string answer;
        std::cin >> answer;
        if (answer == "Y" || answer == "y")
        {
            std::cout << "Введите имя файла для записи:\n";
            std::cin.get();
            std::getline(std::cin, name);
            fout.open(name);
//          fout.open(name, std::ios::_Noreplace);
        }
        else
            fout.open(name, std::ios::out);
    }
}
 
//запись слов из начального файла в вектор для сортировки.
void read_file(const std::string& name)
{
    std::ifstream fin(name);
    if (!fin)
    {
        std::cout << "Файл не существует.\n";
        return;
    }
    std::vector<std::string> word = std::vector<std::string>(std::istream_iterator<std::string>(fin), std::istream_iterator<std::string>());
    fin.close();
    sort_text(word);
}
 
//сортировка текста.
// Это самая страннаяя сортировка котору я видел.
void sort_text(const std::vector<std::string>& word)
{
    std::string example = "zyxwvutsrqponmlkjihgfedcba";
    std::vector<std::string> checked_words;
    int k = 0;
    bool flag = false;
 
    for (std::size_t i = 0; i < word.size(); i++)
    {
        std::string buffer = word[i];
 
        while (buffer[k] != '\0')
        {
            if (buffer[k] == example[k])
            {
                k++;
                flag = true;
            }
            else
            {
                flag = false;
                break;
            }
        }
        if (flag == true)
        {
            checked_words.push_back(buffer);
            flag = false;
        }
    }
    fill_output_file(checked_words);
}
 
//заполнение конечного файла после сортировки.
void fill_output_file(const std::vector<std::string>& checked_words)
{
    std::string name;
    std::cout << "Введите имя конечного файла для записи:\n";
    std::getline(std::cin, name);
    std::ofstream fout(name);
//  fout2.open(name, std::ios::_Noreplace);
 
    check_file_name(name, fout);
 
    std::copy(checked_words.begin(),checked_words.end(),std::ostream_iterator<std::string>(fout));
 
    fout.close();
    output_text(name);
}
 
//вывод конечного файла в консоли.
void output_text(const std::string& name)
{
    std::cout << "Состав файла после обработки:\n";
 
    std::ifstream fin(name);
    if (!fin)
    {
        std::cout << "Файл не существует.\n";
        return;
    }
    std::copy(std::istreambuf_iterator<char>(fin), std::istreambuf_iterator<char>(), std::ostreambuf_iterator<char>(std::cout));
    fin.close();
}

nord_v · 05.01.2016, 03:17

Сообщение от ЗеХель

C++
1
2
3
4
5
6
7
cout << "Состав файла после обработки:\n";
while (!fin2.eof())
{
    fin2 >> temp;
    cout << temp << endl;
}
fin2.close();

C++
1
2
3
4
5
6
7
 
    cout << "Состав файла после обработки:\n";
    while (fin2 >> temp)
    {
        cout << temp << endl;
    }
    fin2.close();

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

Сообщение от ЗеХель

C++
1
2
3
4
5
6
7
8
9
10
11
fin.open(name);
 if (!fin.is_open())
{
     cout << "Файл не существует.\n";
 }
while (!fin.eof())
{
    fin >> buffer2;
    word.push_back(buffer2);
}
fin.close();

C++
1
2
3
4
5
6
7
8
9
        if (!fin.is_open())
        {
            cout << "Файл не существует.\n";
        }
        while (fin >> buffer2)
        {
             word.push_back(buffer2);
        }
        fin.close();

@ЗеХель · 05.01.2016, 03:24 **[ТС]**

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

ЗеХель,

Спасибо большое! Осталось разобраться с некоторыми нюансами:
1) Что есть библотека <Algorithm>
2)

C++
1
std::copy(std::istreambuf_iterator<char>(fin), std::istreambuf_iterator<char>(), std::ostreambuf_iterator<char>(std::cout));

Объясни, пожалуйста, как работает это строка и где вставить тут '\n' или endl можно

Croessmah · 05.01.2016, 03:31

Сообщение от ЗеХель

Что есть библотека <Algorithm>

http://www.cplusplus.com/reference/algorithm/

Сообщение от ЗеХель

Объясни, пожалуйста, как работает это строка

C++
1
2
3
4
5
std::copy(//копируем последовательность,
   std::istreambuf_iterator<char>(fin), //начиная с данного итератора ,
   std::istreambuf_iterator<char>(), //заканчивая вот этим (не включая),
   std::ostreambuf_iterator<char>(std::cout) //и всё что копируем отправляем в последовательность, начало которой задано данным итератором
);

@ЗеХель · 05.01.2016, 03:32 **[ТС]**

nord_v, это реально удобней выглядит, спасибо)

nord_v · 05.01.2016, 03:36

Сообщение от ЗеХель

это реально удобней выглядит, спасибо)

Дело не в удобстве, а в том, что у тебя, последняя считанная строка, два раза выводится на консоль (или в вектор помещается). Подумай, что у тебя происходит, когда EOF считывается?

@Nosey · 05.01.2016, 03:47

ЗеХель,
1) Что есть библотека <Algorithm>
2) std::copy из <Algorithm>

C++
1
2
template< class InputIt, class OutputIt >
OutputIt copy( InputIt first, InputIt last, OutputIt d_first );

Copies the elements in the range, defined by [first, last), to another range beginning at d_first

копирует элементы из диапазона [first, last) в другой диапазон начинающийся с d_first.

std::istreambuf_iterator

std::istreambuf_iterator is a single-pass input iterator that reads successive characters from the std::basic_streambuf object for which it was constructed

Однопроходный input iterator читающий символы из std::basic_streambuf объекта переданного в конструктор.

std::ostreambuf_iterator

std::ostreambuf_iterator is a single-pass OutputIterator that writes successive characters into the std::basic_streambuf object for which it was constructed.

по образу и подобию верхнего.

Сообщение от ЗеХель

Объясни, пожалуйста, как работает это строка и где вставить тут '\n' или endl можно

std::ostreambuf_iterator и std::ostream_iterator принимают в конструкторе вторым параметром разделитель.
И я ошибся у себя в верхнем коде :
сохранение в файл вот такое должно быть:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
//заполнение конечного файла после сортировки.
void fill_output_file(const std::vector<std::string>& checked_words)
{
    std::string name;
    std::cout << "Введите имя конечного файла для записи:\n";
    std::getline(std::cin, name);
    std::ofstream fout(name);
//  fout2.open(name, std::ios::_Noreplace);
 
    check_file_name(name, fout);
 
    std::copy(checked_words.begin(),checked_words.end(),std::ostream_iterator<std::string>(fout,"\n"));
 
    fout.close();
    output_text(name);
}

И при этом output_text не трогать.

@ЗеХель · 05.01.2016, 03:52 **[ТС]**

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

что у тебя происходит, когда EOF считывается?

Я это понимал, поэтому изначально пробовал условие на корректность считывание ставить, но не помогло.

nord_v · 05.01.2016, 03:56

Сообщение от ЗеХель

Я это понимал, поэтому изначально пробовал условие на корректность считывание ставить, но не помогло.

Не там ставил значит. Проверку нужно сразу после чтения ставить, или как у меня, так тоже после чтения поток проверяется.

@ЗеХель · 05.01.2016, 17:42 **[ТС]**

Nosey, Спасибо за объяснение

Новые блоги и статьи Все статьи Все блоги /
сукцессия микоризы: основная теория в виде двух уравнений. anaschu 11.01.2026 https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/	WordPad для Windows 11 Jel 10.01.2026 WordPad для Windows 11 — это приложение, которое восстанавливает классический текстовый редактор WordPad в операционной системе Windows 11. После того как Microsoft исключила WordPad из. . .	Classic Notepad for Windows 11 Jel 10.01.2026 Old Classic Notepad for Windows 11 Приложение для Windows 11, позволяющее пользователям вернуть классическую версию текстового редактора «Блокнот» из Windows 10. Программа предоставляет более. . .	Почему дизайн решает? Neotwalker 09.01.2026 В современном мире, где конкуренция за внимание потребителя достигла пика, дизайн становится мощным инструментом для успеха бренда. Это не просто красивый внешний вид продукта или сайта — это. . .
Модель микоризы: классовый агентный подход 3 anaschu 06.01.2026 aa0a7f55b50dd51c5ec569d2d10c54f6/ O1rJuneU_ls https:/ / vkvideo. ru/ video-115721503_456239114	Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ФедосеевПавел 06.01.2026 Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ВВЕДЕНИЕ Введу сокращения: аналоговый ПИД — ПИД регулятор с управляющим выходом в виде числа в диапазоне от 0% до. . .	Модель микоризы: классовый агентный подход 2 anaschu 06.01.2026 репозиторий https:/ / github. com/ shumilovas/ fungi ветка по-частям. коммит Create переделка под биомассу. txt вход sc, но sm считается внутри мицелия. кстати, обьем тоже должен там считаться. . . .	Расчёт токов в цепи постоянного тока igorrr37 05.01.2026 / * Дана цепь постоянного тока с сопротивлениями и источниками (напряжения, ЭДС и тока). Найти токи и напряжения во всех элементах. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа и. . .

@ЗеХель 24 / 21 / 3 Регистрация: 04.11.2014 Сообщений: 283
	05.01.2016, 03:32 [ТС]
	nord_v, это реально удобней выглядит, спасибо) 0

@ЗеХель 24 / 21 / 3 Регистрация: 04.11.2014 Сообщений: 283
	05.01.2016, 17:42 [ТС]
	Nosey, Спасибо за объяснение 0