Проблема с массивом векторов, возможно, с индексированием

Question

Мне трудно понять, почему мой массив векторов не вводит строку. ...

#include<bits/stdc++.h>
using namespace std;
int main () {
    int r;
    cin>>r;
    vector <int> v[r];
    for (int i=0; i<r; i++) {
        for (int j=0; j<i; j++) {
        int x;
        cin>>x;
        v[i].push_back(x);
        }
    }
    for (int i=0; i<r; i++) {
        for (size_t j=0; j<v[i].size(); j++){
            cout<<v[i][j];
        }    
        cout<<endl;
    }

}

... с входом ...

5
7
3 8
8 1 0
2 7 4 4
4 5 2 6 5

... выводит ...

7
38
810
2744

... с пустая строка в начале вывода.

Answer 1

Вы видели пустую строку в начале вывода, потому что v[0] был пуст. Вы можете исправить это следующим образом:

#include<bits/stdc++.h>
using namespace std;
int main () {
    int r;
    cin>>r;
    // vector <int> v[r];
    //                ^
    // This is variable length array which is not really legal in C++. Use:
    vector<vector<int>> v;
    v.resize(r);
    for (int i=0; i<r; i++) {
        for (int j=0; j<=i; j++) {
        //             ^^
        // This loop had 0 iterations when i == 0, 1 when i == 1, ..., 4 when i == 4.
        // So you need to do one more iteration each time.
            int x;
            cin>>x;
            v[i].push_back(x);
        }
    }
    for (int i=0; i<r; i++) {
        for (size_t j=0; j<v[i].size(); j++){
            cout<<v[i][j];
        }    
        cout<<endl;
    }

}

Также:

• , включая bits/stdc++.h, плохо: Почему я не должен #include ? , вы должны включать только то, что вам нужно:

#include <iostream>
#include <vector>

• иногда using namespace std плохо: Почему "использование пространства имен std;" считается плохой практикой? .

Answer 2

Проблема только в одной строке (комментарий к тексту кода ниже). Также я изменил ваш код, чтобы он был более похож на c ++.

//#include<bits/stdc++.h> //Include all libraries is a bad practice. Include just what you need
#include <vector>
#include <iostream>

using namespace std;

int main () {
    int r;
    cin >> r;

    vector<vector<int>> v(r, vector<int>());    //more secure than c-style vector<int> v[r]
                                                //and create the 1st level of vector
    for (int i = 0; i < r; ++i)
         for (int j = 0; j < i; ++j) {          //Here is the issue
             v[i].emplace_back();
             cin >> v[i].back();
         }

    for (auto& i : v) {                         //Iterate over the 1st level of vector
        for (auto& j : i)                       //Iterate over the 2nd level of vector
            cout << j << ' ';
        cout << '\n';
    }

    return 0;
}