Записать карту в файл с помощью for_each

Следующий код компилируется и выполняется, но в файл ничего не записывается, и я не могу понять, почему. Файл создается, но файл пуст. Я старался каждый раз удалять файл на всякий случай, но это не должно иметь значения

#include <algorithm>
#include <iostream>
#include <fstream>
#include <string>
#include <iterator>
#include <map>
#include <set>
using namespace std;

int main(){

    set<string> exclusionSet;
    ifstream stopWords("stopwords.txt");
    copy(istream_iterator<string>(stopWords),
            istream_iterator<string>(),
            inserter(exclusionSet, exclusionSet.end()));

    stopWords.close();

    //for_each(exclusionSet.begin(),
//          exclusionSet.end(),
//          [](const string& s){cout<<s<<endl;});

    map<string, int> wordMap;
    ifstream words("sample_doc.txt");
    copy_if (istream_iterator<string>(words),
            istream_iterator<string>(),
            inserter(exclusionSet, exclusionSet.end()),
            [=](const string& s){return exclusionSet.find(s)!=exclusionSet.end();});

    ofstream out("frequency.txt");

    for_each(begin(wordMap),
            end(wordMap),
            [&out](const pair<const string, int>& e){ out<<e.first<<" "<<e.second<<endl;});
    out.close();
    return 0;
}

Вы никогда ничего не вставляете в wordMap.

Miles Budnek 22.11.2018 06:48
Стоит ли изучать PHP в 2026-2027 годах?
Стоит ли изучать PHP в 2026-2027 годах?
Привет всем, сегодня я хочу высказать свои соображения по поводу вопроса, который я уже много раз получал в своем сообществе: "Стоит ли изучать PHP в...
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
В JavaScript одним из самых запутанных понятий является поведение ключевого слова "this" в стрелочной и обычной функциях.
Приемы CSS-макетирования - floats и Flexbox
Приемы CSS-макетирования - floats и Flexbox
Здравствуйте, друзья-студенты! Готовы совершенствовать свои навыки веб-дизайна? Сегодня в нашем путешествии мы рассмотрим приемы CSS-верстки - в...
Тестирование функциональных ngrx-эффектов в Angular 16 с помощью Jest
В системе управления состояниями ngrx, совместимой с Angular 16, появились функциональные эффекты. Это здорово и делает код определенно легче для...
Концепция локализации и ее применение в приложениях React ⚡️
Концепция локализации и ее применение в приложениях React ⚡️
Локализация - это процесс адаптации приложения к различным языкам и культурным требованиям. Это позволяет пользователям получить опыт, соответствующий...
Пользовательский скаляр GraphQL
Пользовательский скаляр GraphQL
Листовые узлы системы типов GraphQL называются скалярами. Достигнув скалярного типа, невозможно спуститься дальше по иерархии типов. Скалярный тип...
0
1
70
2

Ответы 2

Я бы предложил следующую версию исправлений

int main()
{
    map<string, int> wordMap;

    set<string> exclusionSet;
    ifstream stopWords("stopwords.txt");
    copy(istream_iterator<string>(stopWords), istream_iterator<string>(), inserter(exclusionSet, exclusionSet.end()));
    stopWords.close();

    vector<string> strings;    
    ifstream words("sample_doc.txt");
    copy_if (istream_iterator<string>(words), istream_iterator<string>(), inserter(strings, strings.end()),
        [=](const string& s)
        {
            return exclusionSet.find(s) == exclusionSet.end();
        });
    words.close();

    for(auto &a : strings)
        wordMap[a] = wordMap[a] + 1;

    ofstream out("frequency.txt");

    for_each(begin(wordMap), end(wordMap),
        [&out](const pair<const string, int>& e)
        {
            out << e.first << " " << e.second << endl;
        });

    out.close();
    return 0;
}

Основные отличия:

  1. чтение данных из файла sample_doc в векторе струны, проверка, находятся ли они в exclusionSet (исправьте ваш вызов find_if)
  2. формирование статистики частоты слов с помощью:

    for(auto &a : strings)
        wordMap[a] = wordMap[a] + 1;
    

И .... Я бы предпочел заменить для каждого на за на основе диапазона, например:

for(const auto &e: wordMap)
    out << e.first << " " << e.second << endl;

Очень мило спасибо! К сожалению, для этого назначения я не могу использовать диапазон на основе только для контейнерных функций stl.

Shane Murphy 23.11.2018 06:55

Мне нечего добавить к ответу @ snake_style, но я хотел бы отметить, что вы можете легко отказаться от сложности / затрат, связанных с использованием copy_if для фильтрации слов в sample_doc.txt, используя простой цикл while:

#include <algorithm>
#include <iostream>
#include <fstream>
#include <string>
#include <iterator>
#include <map>
#include <set>
using namespace std;

int main(){
    set<string> exclusionSet;
    ifstream stopWords("stopwords.txt");
    copy(istream_iterator<string>(stopWords),
            istream_iterator<string>(),
            inserter(exclusionSet, exclusionSet.end()));

    stopWords.close();

    map<string, int> wordMap;
    ifstream words("sample_doc.txt");
    string s;

    while(words >> s) if (exclusionSet.count(s)==0) ++wordMap[s];

    ofstream out("frequency.txt");

    for_each(begin(wordMap),
            end(wordMap),
            [&out](const pair<const string, int>& e){ out<<e.first<<" "<<e.second<<endl;});
    out.close();
    return 0;
}

Другие вопросы по теме