爲什麼這個代碼訪問向量超出了它的範圍？

Question

所以，我試圖將一些代碼帶到我正在開發的Qt項目中。 Motion類使用fstream將.txt文件中的一些控制點導入公共成員變量ctrlPos。例如，當我使用readCtrlPositions，然後嘗試使用writePositions訪問ctrlPos時，出現「矢量下標超出範圍」錯誤。

還有很多代碼，但希望這應該足以回答我的問題。我也是一個新手，所以如果運氣好的話，這不是太愚蠢。

運動類的頭：

#ifndef MOTION_H 
#define MOTION_H 

#include <vector> 

#include "DualQuaternion.h" 

class Motion 
{ 
public: 
    virtual ~Motion(); 
    virtual void readCtrlPositions(char*, char*); 
    virtual void writePositions(char*); 
    virtual void drawCtrlPositions(); 
    virtual void set(int, vector<DualQuaternion>); 
    virtual pair<int, vector<DualQuaternion>> get(); 

public: 
    vector<DualQuaternion> ctrlPos, c; 
    int numberOfPositions; 

}; 

#endif 

Motion類：

其中Qt中程序出現錯誤

#include <stdlib.h> 
#include <GL\glut.h> 

#include "motion.h" 
#include "Quaternion.h" 
#include "hMatrix.h" 
#include "hPoint.h" 

using namespace std; 

void Motion::readCtrlPositions(char *fileNameArg, char *t) 
{ 
    ifstream inFile(fileNameArg, ios::in); 

    if (!inFile) 
    { 
     cerr<<"File" << fileNameArg << "could not be opened" << endl; 
     exit(1); 
    } 

    int i; 

    inFile >> numberOfPositions; 

    Quaternion *RotationQuaternion = new Quaternion[numberOfPositions]; 

    for (i = 0; i<numberOfPositions; i++) 
     inFile >> RotationQuaternion[i]; 

    if (t == "v") 
    { 
     Vector *TranslationVector = new Vector[numberOfPositions]; 
     for (i = 0; i<numberOfPositions; i++) 
      inFile >> TranslationVector[i]; 
     ctrlPos.clear(); 
     for (i = 0; i<numberOfPositions; i++) 
     { 
      DualQuaternion dQ(RotationQuaternion[i], TranslationVector[i]); 
      ctrlPos.push_back(dQ); 
      cout << "first position from input: " << ctrlPos[i] << endl; 
     } 
     delete[] TranslationVector; 
    } 
    else if (t == "q") 
    { 
     Quaternion *TranslationQuaternion = new Quaternion[numberOfPositions]; 
     for (i = 0; i<numberOfPositions; i++) 
      inFile >> TranslationQuaternion[i]; 
     ctrlPos.clear(); 
     for (i = 0; i<numberOfPositions; i++) 
     { 
      DualQuaternion dQ(RotationQuaternion[i], TranslationQuaternion[i]); 
      ctrlPos.push_back(dQ); 
      cout << "first position from input: " << ctrlPos[i] << endl; 
     } 
     delete[] TranslationQuaternion; 
    } 

    delete[] RotationQuaternion; 

} 

void Motion::writePositions(char *fileNameArg) 
{ 
    ofstream outFile(fileNameArg, ios::out); 

    if (!outFile) 
    { 
     cerr<<"File" << fileNameArg << "could not be opened for writing" << endl; 
     exit(1); 
    } 

    int i; 

    outFile << numberOfPositions << endl << endl; 

    for (i = 0; i<numberOfPositions; i++) 
     outFile << ctrlPos[i].GetReal(); 
    outFile << endl; 
    for (i = 0; i<numberOfPositions; i++) 
     outFile << ctrlPos[i].GetDual(); 
} 

void Motion::set(int n, vector<DualQuaternion> p) 
{ 
    int i; 
    numberOfPositions = n; 
    ctrlPos.clear(); 
    for (i = 0; i<numberOfPositions; i++) 
     ctrlPos.push_back(p[i]); 
} 

pair<int, vector<DualQuaternion>> Motion::get() 
{ 
    return make_pair(numberOfPositions, ctrlPos); 
} 

void Motion::drawCtrlPositions() 
{ 

    vector <hMatrix> homogeneousMatricesForCtrlPositions; 
    for (int i=0; i<numberOfPositions; i++) 
    { 
     homogeneousMatricesForCtrlPositions.push_back(ctrlPos[i].dualQuaternionToHomogeneousMatrix().transpose()); 
     double MatrixforOpenGLStack[16]; 

     for (int i1=0; i1<4; i1++) 
      for (int i2=0; i2<4; i2++) 
       MatrixforOpenGLStack[4*i1+i2] = homogeneousMatricesForCtrlPositions.at(i).m[i1][i2]; 

     ::glPushMatrix(); 
     ::glMultMatrixd(MatrixforOpenGLStack); 
     glutSolidTeapot(0.15); 
     ::glPopMatrix(); 
    } 

} 

Motion::~Motion() 
{ 

} 

樣品的編號：

static Curve m; 
m.readCtrlPositions("input.txt", "v"); 
m.writePositions("output.txt"); //<--vector subscript out of range 
m.readCtrlPositions("output.txt", "q"); 
ctrlPos = m.get().second; 
numberOfPositions = m.get().first; 

Answer 1

在readCtrlPositions，t是char*，所以也不是t=="v"，也不是t=="q"將被評估爲true（如果兩個指針具有相同的地址，它將返回true）。所以你的函數將把numberOfPositions設置爲非零值，但永遠不會用任何值填充ctrlPos向量。

稍後，您將嘗試訪問0到numberOfPositions（非零）的ctrlPos元素，而ctrlPos向量爲空。這就是爲什麼你被報告訪問矢量超出其範圍！

用std::string替換char*是解決問題的簡單方法。如果需要將參數保留爲char*，則使用strcmp來比較字符串值而不是指針。

我還強烈建議您刪除您的numberOfPositions屬性，並簡單地使用ctrlPos.size()來代替。在這種情況下，它可以防止崩潰，保證您的類屬性完整。