-4

私のコードは次のとおりです。path1.updatePathInfo() は空の関数を呼び出す関数です。これを削除すると正常に動作します。理解できませんでした。

////////////////////////////////////////////////////main.cpp

#include <iostream>
#include<vector>
#include "CPath.h"
using namespace std;
int main()
{
    for(int i = 0;i<10;++i)
    {
        cout<<i;
        CPath path1;
        CPath path2;
        cross(path1,path2);
        path1.updatePathInfo();  //this is an function which call an empty function
                                 //if I delete this it works fine.I could't understand.
        path2.updatePathInfo();

    }
}

/////////////////////////////////////////////// /////////////CPath.h

#ifndef CPATH_H_
#define CPATH_H_
#include<iostream>
#include<vector>
#include<fstream>
#include<map>
#include<cmath>
#include<cstdlib>
class CPath
{
public:
    CPath();
    friend void cross(CPath & path1,CPath & path2);
    void updatePathInfo();
    std::vector< int > getPathByCode();
    void createRadomCode();
    std::vector< int > code;
    double pathLineLength;
};
#endif //CPATH_H_

/////////////////////////////////////////////// ////////////////////CPath.cpp

#include"CPath.h"
CPath::CPath()
{
    createRadomCode();
}

void cross(CPath & path1,CPath & path2)
{
    int max,min;
    max = rand()%8;
    min = rand()%8;
    if(min > max)
    {
        max ^= min;min ^= max;max ^= min;
    }
    if(0.9*100>rand()%100)
    {
        for(int i = min;i<=max;++i)
        {
            path1.code[i]^=path2.code[i];
            path2.code[i]^=path1.code[i];
            path1.code[i]^=path2.code[i];
        }
    }
}

void CPath::updatePathInfo()
{
    getPathByCode();
}

std::vector< int > CPath::getPathByCode()
{
    //this function has nothing 
}

void CPath::createRadomCode()
{
    for(int i = 0 ; i<8 ; ++i)
    {
        code.push_back(rand()%(i+1));
    }
}
4

1 に答える 1

1

CPath::getPathByCode()は値を返さず、結果は未定義の動作です。コンパイラはこれについて通知する必要はありませんが、すべての警告を有効にしてコンパイルすると、ほぼ確実に通知されます。

余談ですが、コードを正しくフォーマットした場合、この種のエラーははるかに見やすくなります。

于 2012-12-07T16:40:43.463 に答える