Question

Sup人，学习C并从事C编程任务，我将扩展给定的位图图像，我整天都在坚持这一点。这是我的代码到目前为止，但我得到一个段错误，无法弄清楚为什么。我整天都在寻找代码而且只是卡住了。这是我的扩展功能代码，任何帮助将不胜感激

int enlarge(PIXEL* original, int rows, int cols, int scale, 
    PIXEL** new, int* newrows, int* newcols) 
{
  int ncols, nrows;
  ncols = cols * scale;
  nrows = rows * scale;
  double xratio =(double) rows / nrows;
  double yratio =(double) cols / ncols;
  int px, py;
  int auxw, cnt;
  int i, j;

 *new = (PIXEL*)malloc(nrows * ncols * sizeof(PIXEL));
  for (i = 0; i < nrows; i++){
    auxw = 0;
    cnt = 0;
    int m = i * 3;
     for (j = 0; j < ncols; j++){
         px = (int)floor( j * xratio);
         py = (int)floor( i * yratio);

         PIXEL* o1 = original + ((py*rows + px) *3);
         PIXEL* n1 = (*new) + m*ncols + j + auxw;
         *n1 = *o1;

         PIXEL* o2 = original + ((py*rows + px) *3) + 1;
         PIXEL* n2 = (*new) + m*ncols + j + 1 + auxw;
         *n2 = *o2;

         PIXEL* o3 = original + ((py*rows + px) *3) + 2;
         PIXEL* n3 = (*new) + m*ncols + j + 2 + auxw;
         *n3 = *o3;

         auxw += 2;
         cnt++;
       }
     }
 return 0;
}

使用GDB，我得到以下内容：

Program received signal SIGSEGV, Segmentation fault.
0x00000000004013ff in enlarge (original=0x7ffff7f1e010, rows=512, cols=512, scale=2,                   new=0x7fffffffdeb8, 
newrows=0x7fffffffdfb0, newcols=0x0) at maind.c:53
53       *n3 = *o3;

然而，我无法理解究竟是什么问题

感谢

编辑：使用我们的教授为我们提供的代码，PIXEL定义如下：

typedef struct {
  unsigned char r;
  unsigned char g;
  unsigned char b;
} PIXEL;

根据我的理解，我有一个二维数组，其中该数组的每个元素都包含一个3元素的PIXEL数组。此外，当我在纸上追踪我的代码时，我添加了auxw逻辑以便向前推进阵列。它的工作方式与乘以3相同。

Answer 1

您的数组是cols X rows个PIXEL个对象数组 - 或者实际上是cols X rows X 3 PIXEL个original个数组，您称之为pixel实际上是一个像素的channel组件？你的代码不清楚。访问PIXEL* o1 = original + ((py*rows + px) *3);数组时，乘以3，建议使用3个通道的数组：

(*new)

但是当访问auxw数组时，没有乘以3，而是我不能跟PIXEL* n1 = (*new) + m*ncols + j + auxw; auxw += 2;一起使用的逻辑：

     *new = (PIXEL*)malloc(nrows * ncols * 3*sizeof(PIXEL));

无论如何，假设你所谓的像素实际上是一个通道，并且每个像素中有标准的3个RGB通道，你需要为你的阵列分配3倍的内存：

int* newrows

其他一些问题：

int* newcols和nrows永远不会被初始化。您可能希望将它们初始化为ncols和PIXEL
如果CHANNEL确实是#include "assert.h" PIXEL *get_channel(PIXEL *pixelArray, int nRows, int nCols, int nChannels, int iRow, int iCol, int iChannel) { if (iRow < 0 || iRow >= nRows) { assert(!(iRow < 0 || iRow >= nRows)); return NULL; } if (iCol < 0 || iCol >= nCols) { assert(!(iRow < 0 || iRow >= nRows)); return NULL; } if (iChannel < 0 || iChannel >= nChannels) { assert(!(iChannel < 0 || iChannel >= nChannels)); return NULL; } return pixelArray + (iRow * nCols + iCol) * nChannels + iChannel; }，请重命名以正确表达其含义。
不是在整个地方复制多维数组指针算法的逻辑，而是使用函数保护自己不要索引像素/通道/任何数组：
```
#define get_channel(pixelArray, nRows, nCols, nChannels, iRow, iCol, iChannel)\
    ((pixelArray) + ((iRow) * (nCols) + (iCol)) * (nChannels) + (iChannel))
```
稍后，一旦您的代码被完全调试，如果性能有问题，您可以在发布模式下用宏替换该函数：
```
get_channel()
```
使用标准PIXEL* o1 = original + ((py*rows + px) *3); PIXEL* n1 = (*new) + m*ncols + j + auxw;函数的另一个原因是你的指针算术不一致：
```
original
```
要访问array + iCol * nRows + iRow像素，请执行*new，这看起来不错。但要访问array + iCol * nCols + iRow数组，您会看PIXEL，这看起来不对。创建单个函数来访问任何像素阵列，对其进行调试并使用它。

<强>更新

鉴于你对PIXEL结构的定义，你没有必要加上那些+1和+2值允许我到达PIXEL结构的第二和第三个元素。＆＃ 34;由于PIXEL *p_oldPixel = get_pixel(old, nRowsOld, nColsOld, iRowOld, iColOld); PIXEL *p_newPixel = get_pixel(*p_new, nRowsNew, nColsNew, iRowNew, iColNew); p_newPixel->r = p_oldPixel->r; p_newPixel->g = p_oldPixel->g; p_newPixel->b = p_oldPixel->b;是一个结构，如果你有一个指针，你可以使用->运算符访问它的字段：

            *p_newPixel = *p_oldPixel;

或者，在这种情况下，您可以使用assignment operator to copy the struct：

PIXEL

对于PIXEL *数组的索引，因为你的指针被正确地声明为i，所以C编译器的算术将multiply offsets by the size of the struct。

另外，我建议使用明确且一致的命名约定来澄清您的代码：

为循环迭代器和边界使用一致的描述性名称。 i是一行还是一列？为什么在一个地方使用py而在另一个地方使用old？一致的命名约定有助于确保您永远不会混淆行和列。
通过预先添加＆＃34; p _＆＃34;来区分变量或结构的指针。或附加＆＃34; _ptr＆＃34;。明确区分指针的命名约定可以使引用传递的实例更加清晰，因此（例如）您不会忘记初始化输出参数。
对与旧位图和新位图对应的所有变量使用相同的音节。例如。如果您有名为nRowsOld，nColsOld和nColsOld的参数，则不太可能意外地将#include "assert.h" typedef struct _pixel { unsigned char r; unsigned char g; unsigned char b; } PIXEL; PIXEL *get_pixel(PIXEL *pixelArray, int nRows, int nCols, int iRow, int iCol) { if (iRow < 0 || iRow >= nRows) { assert(!(iRow < 0 || iRow >= nRows)); return NULL; } if (iCol < 0 || iCol >= nCols) { assert(!(iRow < 0 || iRow >= nRows)); return NULL; } return pixelArray + iRow * nCols + iCol; } int enlarge(PIXEL* old, int nRowsOld, int nColsOld, int scale, PIXEL **p_new, int *p_nRowsNew, int *p_nColsNew) { int nColsNew = nColsOld * scale; int nRowsNew = nRowsOld * scale; double xratio =(double) nRowsOld / nRowsNew; double yratio =(double) nColsOld / nColsNew; int iRowNew, iColNew; *p_new = malloc(nRowsNew * nColsNew * sizeof(PIXEL)); *p_nRowsNew = nRowsNew; *p_nColsNew = nColsNew; for (iRowNew = 0; iRowNew < nRowsNew; iRowNew++){ for (iColNew = 0; iColNew < nColsNew; iColNew++){ int iColOld = (int)floor( iColNew * xratio); int iRowOld = (int)floor( iRowNew * yratio); PIXEL *p_oldPixel = get_pixel(old, nRowsOld, nColsOld, iRowOld, iColOld); PIXEL *p_newPixel = get_pixel(*p_new, nRowsNew, nColsNew, iRowNew, iColNew); *p_newPixel = *p_oldPixel; } } return 0; }与新位图一起使用。

因此您的代码变为：

{{1}}

我还没有测试过这段代码，但是通过使用一致的命名约定，我们可以清楚地看到它正在做什么以及为什么它应该工作。

缩放位图图像得到段错误

1 个答案: