do while循环处理矩阵信号时末尾越界问题排查及goto优化咨询

你的代码核心错误 #

问题出在检测到matrix[i][j] == 1时的内部循环条件：你只判断了j < m + 3，却没有限制j不能超过矩阵的最大列数MAXTIME（也就是你说的10列）。举个例子，如果信号出现在第8列（假设列索引从0开始），m + 3就是11，而你的矩阵只有0-9共10列，这时候循环会继续让j增加到11，导致访问newmatrix[i][10]甚至newmatrix[i][11]——这属于数组越界访问，看起来像是“新增了列”，但实际上是非法访问了矩阵之外的内存，会导致不可预料的问题。

另外，你的代码里存在冗余的j++和continue：外层do-while的最后还有一个j++，但两个if分支里已经执行了j++并continue，这个末尾的j++永远不会被执行，可以删掉。

修正后的代码 #

这里我调整了逻辑，同时加入了越界保护：

for (int i = 0; i < rows; i++) {
    int j = 0;
    do {
        if (matrix[i][j] == 1) {
            int m = j;
            // 同时限制j不超过MAXTIME，避免越界
            while (j < m + 3 && j < MAXTIME) {
                newmatrix[i][j] = 10;
                printf("newmatrix[%i][%i] = %f\n", i, j, newmatrix[i][j]);
                j++;
            }
        } else {
            newmatrix[i][j] = 0;
            printf("newmatrix[%i][%i] = %f\n", i, j, newmatrix[i][j]);
            j++;
        }
    } while (j < MAXTIME);
}

这样修改后，当信号出现在列末尾附近时，循环会在j达到MAXTIME时停止，不会再越界访问。

关于使用goto实现的可行性 #

确实可以用goto来简化逻辑，但其实上面的代码已经很简洁了。现代C编程中，goto通常只用于跳出多层循环或者错误处理，过度使用会让代码可读性下降。

不过如果你想试试，这里有个示例版本：

for (int i = 0; i < rows; i++) {
    int j = 0;
    do {
        if (matrix[i][j] == 1) {
            int m = j;
            while (j < m + 3 && j < MAXTIME) {
                newmatrix[i][j] = 10;
                goto PRINT_AND_INCREMENT;
            }
            // 跳出内部循环后直接进入下一次外层循环
            continue;
        }
        newmatrix[i][j] = 0;
PRINT_AND_INCREMENT:
        printf("newmatrix[%i][%i] = %f\n", i, j, newmatrix[i][j]);
        j++;
    } while (j < MAXTIME);
}

这个版本用goto把重复的打印和j++逻辑合并了，但并没有比之前的版本简洁多少，反而增加了跳转的认知成本，所以还是推荐第一个修正后的版本。

内容的提问来源于stack exchange，提问作者newbie_new