Question

我练习使用OpenMP，为此，我做了一个非常简单的并行打印程序。

我有一个2d char*数组，我用以下函数填充：

void addItem( char * arr[4][4], int row, int col ) {
    for ( int i=0; i< row; i++ ) {
        for ( int j=0; j< col; j++ ) {
            arr[i][j] = new char [10];
            strcpy( arr[i][j], "input");

            char * tmp = new char [5];
            *tmp = ( i + '0' );
            strcat( arr[i][j],tmp );
            delete [] tmp;
        }
    }
}

在我的主要功能中，我有以下内容：

int threadNum =4;
omp_set_num_threads(threadNum);
#pragma omp parallel
{
    #pragma omp for
    for ( int i=0; i<threadNum; i++ ) {
        cout << omp_get_thread_num() << endl;
        #pragma omp critical
        {
            printItem ( input, row, col );
        }
    }
}

printItem()函数的作用是打印数组的元素

void printItem (char * arr[4][4], int row, int col) {
    for ( int i=0; i<row; i++ ) {
        cout << " row " << i << endl;
        for ( int j=0; j<col; j++ ) {
           cout << arr[i][j] << " ";
        }
        cout << endl;
    }
}

输出

01
row 0
input0 input0 input0 input0
row 1
input1 input1 input1 input1r_savings
row 2
input2 input2 input2 input2
row 3
input3 input3 input3 input3
32
row 0
input0 input0 input0 input0
row 1
input1 input1 input1 input1r_savings
row 2
input2 input2 input2 input2
row 3
input3 input3 input3 input3

.......

正如你所看到的，在每个线程的工作中，都有“r_savings”，我没有在我的函数中添加任何内容。

你能告诉我它来自哪里吗？我做错了吗？

Answer 1

您的问题与OpenMP或并行性无关，但与C字符串及其终止有关。

在以下代码中

arr[i][j] = new char [10];
strcpy( arr[i][j], "input");

char * tmp = new char [5];
*tmp = ( i + '0' );
strcat( arr[i][j],tmp );
delete [] tmp;

您分配一个包含10个字符的字符串，然后在其中复制"input"。你的字符串现在是{ 'i', 'n', 'p', 'u', 't', '\0', ... }（AFAIK，最后4个元素有未定义的值）。
然后你分配一个5个字符的字符串tmp，你可以将第一个元素设置为对应于索引i的值的字符。现在tmp（例如i等于零）{ '0', ... }。
您是否注意到NULL字符串中没有tmp终结符？这就是问题的原因。因为现在，当您使用strcat()作为第二个参数调用tmp时，该函数不知道停止复制的位置。更具体地说，该函数将继续复制，直到它在NULL指向的内存中找到tmp字符，这可能需要一段时间，并且结束很糟糕。

我将以您选择的样式快速修复您的代码，但由于您使用的是C ++，我建议您使用std::string而不是char*来处理字符串。

所以这应该＆＃34;修复＆＃34;这部分代码：

arr[i][j] = new char[10];
sprintf( arr[i][j], "input%d", i );

（openmp）我得到奇怪的字符串，“r_saving”

1 个答案: