0

我试图在 C++ 程序中采用一个简单的命令行参数来触发不同的程序行为——当你包含一个“y”(或任何以 y 开头的字符串——我不在乎)时,程序会显示一些中间结果.
当我和

ccal pix.txt

一切正常。

当我使用

ccal pix.txt 是的

它运行正常,显示我的像素并在最后崩溃。

还,

ccal pix.txt 没有

在不显示 pix 的情况下运行一切正常(就像它应该的那样),最后仍然崩溃。

这是相关代码-我做错了什么?

void dumpFloatMatrix(Mat m){
for(int i = 0; i < m.cols; i++){
    for(int j = 0; j < m.rows; j++){
        char *buff = new char[10];
        sprintf(buff, "%5.1f ", m.at<float>(i,j));
        cout << buff;
        delete buff;
    }
    cout << endl;
}
 }
int main(int argc, char *argv[]){
char* outFile;
bool showPix = false;

// Take in command line args
switch(argc){
case 3:
    if(strncmp(argv[2], "y", 1) == 0)
        showPix = true;
    outFile = argv[1];
    break;
case 2:
    outFile = argv[1];
    break;
default:
    cout << "Usage: ccal INPUT_LIST_FILE" << endl;
    return -1;
}
Mat cameraMatrix(3, 3, CV_32FC1);
dumpFloatMatrix(cameraMatrix);
return 0;
}

奇怪的是,即使我将案例 3 中的测试切换为以下内容:

        if(argv[2][0] == 'y')

我仍然得到相同的行为。我无法为我的生活找出原因。

4

2 回答 2

1

固定大小的缓冲区对我来说是一个警告信号。

作为故障排除步骤,更改

sprintf(buff, "%5.1f ", m.at<float>(i,j));

int const used = sprintf(buff, "%5.1f ", m.at<float>(i,j));
assert(used < 10);

除此之外,使用动态分配显然很荒谬。如果固定大小的缓冲区就足够了,只需使用本地自动数组变量即可。当您使用它时,堆栈空间很便宜,因此通过使缓冲区足够大来防止溢出。

void dumpFloatMatrix( Mat m )
{
    char buff[400];
    for(int i = 0; i < m.cols; i++){
        for(int j = 0; j < m.rows; j++){
            int const used = sprintf(buff, "%5.1f ", m.at<float>(i,j));
            assert(used * sizeof *buff < sizeof buff);
            cout << buff;
        }
        cout << endl;
    }
}
于 2013-01-12T21:00:16.080 回答
1

对不起,但这纯粹是疯狂:

for(int j = 0; j < m.rows; j++){
    char *buff = new char[10];
    sprintf(buff, "%5.1f ", m.at<float>(i,j));
    cout << buff;
    delete buff;
}

为 10 字节数组调用 new/delete 将花费 16-32 字节的内存加上您想要的 10 字节 [可能四舍五入为 16、32 或 64 字节]。并分别调用 new 和 delete 。是的,我肯定cout << buff会花费更多的周期,但这些在某种程度上是必要的。

要么使用:

for(int j = 0; j < m.rows; j++){
    char buff[10];
    sprintf(buff, "%5.1f ", m.at<float>(i,j));
    cout << buff;
}

或使用 C++ 样式格式:

for(int j = 0; j < m.rows; j++){
    cout << precision(1) << setw(5) << m.at<float>(i,j);
}

如果数组非常大,您可能更愿意将这些移出循环:

cout.precision(1); 
cout.setw(5);

我更喜欢最后一种方法——因为如果你的计算结果为 1210121281.9,它不会溢出[你的布局看起来有点滑稽]。

于 2013-01-12T21:01:45.127 回答