我的 for 循环增加了 +1 多余,我不知道为什么

My for loop is adding +1 excess and i do not know why

提问人:casual 提问时间:10/28/2021 更新时间:10/28/2021 访问量:496

问:

基本上,我正在尝试制作一个遍历给定数组的程序,并检查右侧元素是否比左侧元素大 2 倍,如果 true 在中间插入这两个元素的平均值。之后,它打印出带有插入元素的数组,然后再次遍历数组,计算某个数字出现的次数。我用笔和纸成功地编码了所有这些,并将问题写成更小的块,然后用 C 编码,但问题是当我输入 100 个零(一百个零)时。程序打印出数字 0 重复 200 次而不是 199 次。我不知道为什么。对不起,代码很糟糕,我目前的任务是善于用笔和纸解决问题,在我变得体面并发展我的逻辑之后,我会尝试让代码更简单。

Input sample: 
Enter the number of elements: 100
Enter the array: 0000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000
After adding middle element: 00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000002.33412e-310 
The number is repeated 200 time/s

我的代码

#include <math.h>
#include <stdio.h>

#define EPSILON 0.0001

int main() {
  int n, i, j, k, digit, length = 0, digit_array[10] = {0};
  double array[200], temp;
  do {
    printf("Enter number of elements: ");
    scanf("%d", &n);
  } while (n <= 0 || n >= 101);

  printf("Enter elements: ");

  length = n;

  for (i = 0; i < length; i++)
    scanf("%lf", &array[i]);

  for (i = 0; i < length; i++) {
    temp = array[i] + array[i];
    if (fabs(temp - array[i + 1]) < EPSILON) {
      for (j = length; j > i + 1; j--)
        array[j] = array[j - 1];
      array[i + 1] = (array[i] + array[i + 1]) / 2.;
      i++;
      length++;
    }
  }

  printf("After adding middle element: \n");
  for (i = 0; i < length; i++)
    printf("%g ", array[i]);

  for (i = 0; i < length; i++) {
    temp = array[i];
    digit = ((int)(temp * 10)) % 10;
    digit_array[digit]++;
  }
  printf("\n");

  for (i = 0; i < 10; i++) {
    if (digit_array[i] != 0)
      printf("Number %d is repeated %d time/s.\n", i, digit_array[i]);
  }

  return 0;
}
C FOR 循环 epsilon

评论

0赞 yano 10/28/2021
我希望这能解释为什么链表比数组更适合插入中间。
0赞 casual 10/28/2021
@yano我还在学习 C,想先熟悉数组。
1赞 user3386109 10/28/2021
代码将超过数组的末尾。在循环的最后一次迭代中等于 。这意味着表达式正在读取 ,但已超过数组的末尾。ilength-1temp - array[i+1]array[length]array[length]
1赞 user3386109 10/28/2021
尚未完全审查代码,但可能会起作用。for (i = 0; i < length-1; i++)
1赞 yano 10/28/2021
@user3386109在我看来是正确的:godbolt.org/z/j1sPcGc6x

答:

2赞 ikegami 10/28/2021 #1

与其不断移动阵列,不如使用两个阵列更容易、更快捷。您所需要的只是这个:

// Inputs:
//   n: The number of inputs.
//   a: An array of at least n doubles containing the inputs.
//   b: An array of at least n*2-1 doubles that will containing the outputs.

// Outputs:
//   m: The number of outputs.
//   b: An array of at least m doubles containing the outputs.

size_t i = 0;
size_t j = 0; 

double prev = b[j++] = a[i++];

while (i < n) {
   double next = a[i];
   if (fabs(prev*2 - next) < EPSILON) {   // If a[i-1] exactly equal a[i]*2.
      b[j++] = next / 2.0 + prev / 2.0;   // Or: b[j++] = prev * 1.5;
   }

   prev = b[j++] = a[i++];
}

size_t m = j;

关于:prev * 1.5

average(next, prev)
= ( next + prev ) / 2
= ( prev * 2 + prev ) / 2
= ( prev * 3 ) / 2
= prev * 1.5

包含在适当的功能中:

int f(double *a, size_t n, double **b_ptr, size_t *m_ptr) {
   double b = malloc( (n*2-1) * sizeof(double) );  // We need up to this much.
   if (b == NULL) {
      *b_ptr = NULL;
      return 0;
   }

   size_t i = 0;
   size_t j = 0; 

   double prev = b[j++] = a[i++];

   while (i < n) {
      double next = a[i];
      if (fabs(prev*2 - next) < EPSILON) {   // If a[i-1] exactly equal a[i]*2.
         b[j++] = next / 2.0 + prev / 2.0;   // Or: b[j++] = prev * 1.5;
      }

      prev = b[j++] = a[i++];
   }

   b = realloc(b, j * sizeof(double));  // Free the excess. (Optional)
   size_t m = j;

   *b_ptr = b;
   *m_ptr = m;
   return 1;
}

评论

0赞 casual 10/28/2021
正如我所写的,我仍然只是在学习初学者任务,我不理解你写的代码的一半,我认为我只是复制粘贴代码对自己没有任何好处:/
0赞 ikegami 10/28/2021
除了使用 malloc 而不是硬编码数组的大小之外,这里没有使用任何内容。只需忽略循环之前和之后的所有内容即可。查看更新int i = 0;
0赞 casual 10/28/2021
好的,我理解了上一个 * 1.5。非常感谢你们的澄清。由于我还是 stackoverflow 的新手,我不知道如何代表你们 +1,但我认为无论如何你都不需要我。非常感谢你@ikegami
0赞 ikegami 10/28/2021
您点击了帖子旁边的向上箭头。这可能有最低限度的声誉吗?
0赞 casual 10/28/2021
是的,先生,15 次