双重向量超出范围

huangapple
go评论99阅读模式
英文:

Double vector out of range

问题

  1. #include <iostream>
  2. #include <string>
  3. #include <vector>
  5. using std::cin;
  6. using std::cout;
  7. using std::endl;
  8. using std::string;
  9. using std::vector;
  11. int main() {
  12.     int n, x, y, z, xres, yres, zres;
  13.     cin >> n;
  14.     vector<vector<int>> vec(n);
  16.     while (n--) {
  17.         vector<int> aux(3);
  18.         cin >> aux.at(0) >> aux.at(1) >> aux.at(2);
  19.         vec.push_back(aux);
  20.     }
  21.     for (int i = 0; i < vec.size(); i++) {
  22.         for (int j = 0; j < vec[i].size(); j++) {
  23.             cout << vec.at(i).at(j) << " ";
  24.         }
  25.         cout << endl;
  26.     }
  27.     cout << vec.at(0).at(0);
  28.     return 0;
  29. }

为什么for循环有效，但尝试直接访问元素会产生超出范围的错误，错误消息表示向量的大小为0？我认为for循环也只是将一些数字放在i和j的位置上。输入如下所示：

  1. 3
  2. 1 2 3
  3. 1 2 3
  4. 1 2 3
  1. terminate called after throwing an instance of 'std::out_of_range'
  2.   what():  vector::_M_range_check: __n (which is 0) >= this->size() (which is 0)
  3. Aborted (core dumped)

请注意，你的代码存在一个问题，它在创建vector<vector<int>> vec(n)后，又使用vec.push_back(aux)添加了多余的空向量，这可能导致问题。你应该改为使用vec[i] = aux来填充vec。这样，你的代码应该像下面这样：

  1. #include <iostream>
  2. #include <string>
  3. #include <vector>
  5. using std::cin;
  6. using std::cout;
  7. using std::endl;
  8. using std::string;
  9. using std::vector;
  11. int main() {
  12.     int n, x, y, z, xres, yres, zres;
  13.     cin >> n;
  14.     vector<vector<int>> vec(n);
  16.     for (int i = 0; i < n; i++) {
  17.         vector<int> aux(3);
  18.         cin >> aux.at(0) >> aux.at(1) >> aux.at(2);
  19.         vec[i] = aux;
  20.     }
  22.     for (int i = 0; i < vec.size(); i++) {
  23.         for (int j = 0; j < vec[i].size(); j++) {
  24.             cout << vec[i][j] << " ";
  25.         }
  26.         cout << endl;
  27.     }
  28.     cout << vec[0][0];
  29.     return 0;
  30. }
英文: 
  1. #include &lt;iostream&gt;
  2. #include &lt;string&gt;
  3. #include &lt;vector&gt;
  6. using std::cin;
  7. using std::cout;
  8. using std::endl;
  9. using std::string;
  10. using std::vector;
  14. int main() {
  15.     int n, x, y, z, xres, yres, zres;
  16.     cin &gt;&gt; n;
  17.     vector&lt;vector&lt;int&gt;&gt; vec(n);
  19.     while(n--) {
  20.         vector&lt;int&gt; aux(3);
  21.         cin &gt;&gt; aux.at(0) &gt;&gt; aux.at(1) &gt;&gt; aux.at(2);
  22.         vec.push_back(aux);
  24.     }
  25.     for ( int i = 0; i &lt; vec.size(); i++ ) {
  26.         for (int j = 0; j &lt; vec[i].size(); j++) {
  27.             cout &lt;&lt; vec.at(i).at(j) &lt;&lt; &quot; &quot;;
  28.             }
  29.         cout &lt;&lt; endl;
  30.     }
  31.     cout &lt;&lt; vec.at(0).at(0);
  32.     return 0;
  33. }

Why do the for loops work but trying to access an element directly produces an out of range error which says that the vector is of size 0? I would think that the for loops also only put some numbers in place o i and j. The input is like this:

  1. 3
  2. 1 2 3
  3. 1 2 3
  4. 1 2 3
  1. terminate called after throwing an instance of &#39;std::out_of_range&#39;
  2.   what():  vector::_M_range_check: __n (which is 0) &gt;= this-&gt;size() (which is 0)
  3. Aborted (core dumped)

答案1

得分: 2

你的向量最初大小为三，然后你再添加了三个项目。

尝试这个：

  1. vector<vector<int>> vec; // 初始大小为零
  3. while (n--) {
  4.     vector<int> aux(3);
  5.     cin >> aux.at(0) >> aux.at(1) >> aux.at(2);
  6.     vec.push_back(aux); // 添加新项目
  7. }

或者这个：

  1. vector<vector<int>> vec(n); // 初始大小为3
  3. for (int i = 0; i < vec.size(); ++i) {
  4.     vector<int> aux(3);
  5.     cin >> aux.at(0) >> aux.at(1) >> aux.at(2);
  6.     vec[i] = aux; // 覆盖现有项目
  7. }
英文:

Your vector initially has a size of three, and then you add three more items to it.

Try this

  1. vector&lt;vector&lt;int&gt;&gt; vec; // initial size zero
  3. while(n--) {
  4.     vector&lt;int&gt; aux(3);
  5.     cin &gt;&gt; aux.at(0) &gt;&gt; aux.at(1) &gt;&gt; aux.at(2);
  6.     vec.push_back(aux); // add new item
  8. }

or this

  1. vector&lt;vector&lt;int&gt;&gt; vec(n); // initial size 3
  3. for (int i = 0; i &lt; vec.size(); ++i) {
  4.     vector&lt;int&gt; aux(3);
  5.     cin &gt;&gt; aux.at(0) &gt;&gt; aux.at(1) &gt;&gt; aux.at(2);
  6.     vec[i] = aux;           // overwrite existing item
  8. }

答案2

得分: 2

你创建的初始向量包含 n 个空向量：

  1. vector<vector<int>> vec(n);

稍后你推入了 n 个非空向量。此后，外部向量包含 2 * n 个向量，前 n 个向量是空的。

你应该使用：

  1. vector<vector<int>> vec;
  2. vec.reserve(n); // 分配必要的存储空间，但目前大小为 0
  3. while(n--)
  4. {
  5.     vector<int> aux(3);
  6.     cin >> aux.at(0) >> aux.at(1) >> aux.at(2);
  7.     vec.push_back(std::move(aux)); // std::move 避免了拷贝
  8. }

或者

  1. vector<vector<int>> vec(n, std::vector<int>(3));
  3. // vec 现在完全分配了; 只需填充现有元素
  4. for (auto& v : vec)
  5. {
  6.     std::cin >> v[0] >> v[1] >> v[2];
  7. }
英文:

The initial vector you create contains n empty vectors:

  1. vector&lt;vector&lt;int&gt;&gt; vec(n);

Later you push n nonempty vectors. After this the outer vector contains 2 * n vectors and the first n of these are empty.

You should use

  1. vector&lt;vector&lt;int&gt;&gt; vec;
  2. vec.reserve(n); // allocate the necessary storage, but the size remains 0 for now
  3. while(n--)
  4. {
  5.     vector&lt;int&gt; aux(3);
  6.     cin &gt;&gt; aux.at(0) &gt;&gt; aux.at(1) &gt;&gt; aux.at(2);
  7.     vec.push_back(std::move(aux)); // std::move avoids a copy here
  8. }

or

  1. vector&lt;vector&lt;int&gt;&gt; vec(n, std::vector&lt;int&gt;(3));
  3. // vec is completely allocated now; just fill existing elements
  4. for (auto&amp; v : vec)
  5. {
  6.     std::cin &gt;&gt; v[0] &gt;&gt; v[1] &gt;&gt; v[2];
  7. }

huangapple
  • 本文由 发表于 2023年5月28日 20:45:04
  • 转载请务必保留本文链接：https://go.coder-hub.com/76351565.html
  • c++
  • multidimensional-array
  • vector
匿名

发表评论

匿名网友

:?: :razz: :sad: :evil: :!: :smile: :oops: :grin: :eek: :shock: :???: :cool: :lol: :mad: :twisted: :roll: :wink: :idea: :arrow: :neutral: :cry: :mrgreen:

确定