C - 浮点异常(核心转储)

时间:2017-05-08 16:49:14

标签: c pointers pixel cycle floating-point-exceptions

这个函数应该逐像素地模糊图像,方法是在2n + 1"半径"中将每个像素的颜色转换为周围颜色的平均值。

(它跳到下一个像素的部分已经实现,不用担心)。

我成功编译了这段代码:

void
blur_pixels(image *img, pixel *p, size_t i, size_t j)
{
  //i = current height of pixel, j = current width of pixel
  int side = 2*blurRate+1;
  int total = 0;
  int leftRight = i-blurRate;
  int upDown = j-blurRate;
  int tmpHr = 0, tmpHg = 0, tmpHb = 0;

  for(; upDown < j+blurRate; upDown++) {
    if(upDown >= 0 && upDown < img->height) {
      for(; leftRight < i+blurRate; leftRight++) {
        if(leftRight >= 0 && leftRight < img->width) {
          tmpHr += (p+leftRight)->r;
          tmpHg += (p+leftRight)->g;
          tmpHb += (p+leftRight)->b;
          total++;
        }
      }
    }
  }
  p->r=tmpHr/total;
  p->g=tmpHg/total;
  p->b=tmpHb/total;
}

但是当我运行代码时,我得到以下异常:

Floating point exception

有谁知道为什么?

1 个答案:

答案 0 :(得分:2)

代码正在使用p->r=tmpHr/total;

执行除以0

total可能为零,因为未打开编译器警告,显示for()循环的混合有符号/无符号数学运算。打开所有编译器警告。

比较upDown < j+blurRate和其他代码是使用无符号数学完成的,可能不是OP所期望的,内部total++;也不会发生。如果upDown < 0,则upDown中的upDown < j+blurRate将成为较大的无符号值。然后比较是假的。

size_t j  // an unsigned type
...
int upDown = j-blurRate;
...
for(; upDown < j+blurRate; upDown++) {  // not firing

一种解决方案是仅使用int个变量。更强大的解决方案将使用无符号数学,但需要更多更高级别的代码才能获得良好的答案。

类似的东西:

blur_pixels(image *img, pixel *p, size_t i, size_t j) {
  //i = current height of pixel, j = current width of pixel
  size_t side = 2u*blurRate+1;
  size_t total = 0;
  size_t leftRight = (i > blurRate) ? i-blurRate : 0;
  size_t upDown = (j > blurRate) ? j-blurRate : 0;
  int tmpHr = 0, tmpHg = 0, tmpHb = 0;

  for(; upDown < j+blurRate; upDown++) {
    if (upDown < img->height) {
      // I suspect leftRight needs to be set here each iteration
      size_t leftRight = (i > blurRate) ? i-blurRate : 0;
      for(; leftRight < i+blurRate; leftRight++) {
        if (leftRight < img->width) {
          tmpHr += (p+leftRight)->r;
          tmpHg += (p+leftRight)->g;
          tmpHb += (p+leftRight)->b;
          total++;
        }
      }
    }
  }
  if (total) {
    p->r = tmpHr/total;
    p->g = tmpHg/total;
    p->b = tmpHb/total;
  } else {
    p->r = p->g = p->b = 0;
  }
}