Ошибка при использовании vector (в конце, после вывода результатов выскакивает ошибка, но сам результат коректен)

@Duku · Регистрация: 05.05.2013

Суть в следующем, мы считываем введеные значения и выводим этот массив

Проблема такова что массив выводиться, корректно, но в конце вывода выскакивает ошибка суть которой я не оч. понял, но связана она с циклом, помогите разобраться.

P.S. я полный Нуб и ДЦПшник, так что если вам не сложно, выражайтесь во всех подробностях... Спасибо!!!))

C++

#include <iostream>
#include <vector>
 
 
 
using namespace std;
 
int main()
{
    
    vector<int> v;
    int el;
    while (cin >> el)
    {
        v.push_back(el);
    }
    for(int i = 0; i < sizeof(v); i++)
    {
        cout << v[i] << "\n";
    }
 
 
    cin.get();
    return 0;
}

@Senarist · 16.08.2017, 01:24

Duku,

C++

#include <iostream>
#include <vector>
 
using namespace std;
 
int main()
{
 
    vector<int> v;
    int el;
    cin >> el;
    while (el != 0)
    {
        v.push_back(el);
        cin >> el;
    }
    for (int i = 0; i < v.size(); i++)
    {
        cout << v[i] << "\n";
    }
 
        cin.get();
    cin.get();
    return 0;
}

nd2 · 16.08.2017, 01:28

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

C++

1	i < sizeof(v);

C++

1	i < v.size();

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

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

выражайтесь во всех подробностях...

sizeof(v) выдаст размер объекта вектор, а не количество элементов в нём. Метод size(), класса std::vector, выдаёт количество элементов в векторе.

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

выскакивает ошибка суть которой я не оч. понял

Выходишь за границу вектора (обращаешься по индексу к несуществующему элементу).

@easybudda · 16.08.2017, 03:04

Duku, при таком подходе к вводу чисел поток чистить надо

C++

#include <iostream>
#include <vector>
#include <limits>
 
int main() {
    int i;
    std::vector<int> vec;
 
    while (std::cin >> i)
        vec.push_back(i);
 
    std::cin.clear();
    std::cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n');
 
    for (int n : vec)
        std::cout << n << " ";
    std::cout << std::endl;
 
    std::cin.get();
    return 0;
}

Код

5
4
3
2
1
enough
5 4 3 2 1

@Duku · 16.08.2017, 12:08 **[ТС]**

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

C++

#include <iostream>
#include <vector>
 
 
using namespace std;
 
int main()
{
    int N = 0;
    
    vector<int> v(N);
    cin >> N;
    int el;
    el = 0;
    while (v.size() < N)
    {
        cin >> el;
        v.push_back(el);
    }
    int pr;
    for(int i = 0; i < N; i++)
    {
        pr = i % 2;
        switch (pr)
        {
        case 0: cout << v[i] << endl;
        break;
        }
    }
    cin.get();
    return 0;
}

Добавлено через 3 минуты
Всем спасибо за отзывы!!!)
Если можно краткую оценку моего "говнокода", пойдет или мне пойти убиться о стену?))

@Senarist · 16.08.2017, 12:17

Duku, замудрил с выводом конечно)

C++

#include <iostream>
#include <vector>
 
using namespace std;
 
int main()
{
 
    vector<int> v;
    int el;
    cin >> el;
    while (el != 0)
    {
        v.push_back(el);
        cin >> el;
    }
    for (int i = 0; i < v.size(); i++)
    {
        if (v[i] % 2 == 0) 
            cout << v[i] << "\n";
    }
 
    cin.get();
    cin.get();
    return 0;
}

@easybudda · 16.08.2017, 12:30

C++

//...
for ( int n : v )
    if ( ! (n & 1) )
        std::cout << n << " ";

другой вариант:

C++

#include <iostream>
#include <vector>
#include <algorithm>
#include <iterator>
 
int main() {
    std::vector<int> vec = { 1, 2, 3, 4, 5, 6 };
    std::copy_if(vec.begin(), vec.end(), std::ostream_iterator<int>(std::cout, " "), [](const int& n) { return !(n & 1); });
    return 0;
}