Компилятор ругается на цикл for - C++

[Dimasik_777]

Есть код:

#include <iostream>
#include <fstream>
#include <sstream>
#include <string>
#include <vector>
#include <iterator>
#include <algorithm>
 
bool length_compare (const std::string&, const std::string&);
bool words_count_compare(const std::vector<std::string>&, const std::vector<std::string>&);
 
int main()
{
    setlocale (LC_ALL, "RUS");
    std::ifstream in("D://lab.txt");
    
    std::vector<std::vector<std::string> > lines;
    std::string line;
    int lineNumber = 0;
    while(std::getline(in, line))
    {
        std::cout << line << std::endl;
        std::istringstream iss(line);
        std::istream_iterator<std::string> iter(iss), eof;
        std::vector<std::string> line_vector(iter, eof);
        std::sort(line_vector.begin(), line_vector.end(), length_compare);
        std::ostringstream ost;
        ost << ++lineNumber;
        line_vector.insert(line_vector.begin(), ost.str());
        lines.push_back(line_vector);
    }
    std::cout << std::endl;
    std::cout << std::endl;
    std::sort(lines.begin(), lines.end(), words_count_compare);
    for (std::vector<std::string> line : lines)                                      // на вот этот
    {
        for(std::string word : line)                                                       // и на этот
            std::cout << word << " ";
        std::cout << std::endl;
    }
    return 0;
}
 
bool length_compare (const std::string &s1, const std::string &s2)
{
    return s1.size() < s2.size();
}
 
bool words_count_compare(const std::vector<std::string> &v1, const std::vector<std::string> &v2)
{
    return v1.size() < v2.size();
}

И ошибки:

[p51x]

Вам же явно написали: в С++98 не было улучшенного фора. Либо говорите компилятору, что используете новый стандарт, либо не используйте фишки нового.

[Cuprum5]

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

std::ifstream in("D://lab.txt");

- по-моему тут нужен обратный слэш. Напиши вначале программы после всех #include: using namespace std; чтобы не писать этот бред, т.е. чтобы не писать каждый раз std.

[p51x]

по-моему тут нужен обратный слэш

нет. можно и такой, только экранировать его не надо

Напиши вначале программы после всех #include: using namespace std; чтобы не писать этот бред, т.е. чтобы не писать каждый раз std.

По губам и рукам за такие советы.

[alexzk]

Во первых, такие циклы начинаются со стандарта 11, т.е. нужно указать в настройках, что это С++11.
Во-вторых

 for (const std::vector<std::string>& line : lines)                                      // на вот этот
    {
        for(const std::string word& : line)                                                       // и на этот
            std::cout << word << " ";
        std::cout << std::endl;
    }

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

[p51x]

оформляйте параметр цикла как ссылку

а еси ему это не надо? а если ему нужно копирование?

в таком виде

лучше через auto (прочитав как оно работает!), а то могут быть вариант

[alexzk]

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

а еси ему это не надо? а если ему нужно копирование?


лучше через auto (прочитав как оно работает!), а то могут быть вариант

К auto тоже ссылку нада типа const auto&, иначе, вероятнее всего, будет копирование. Уже сталкивался - дебажил...