为什么我的 for 循环没有循环我指定的次数？

Question

我一直在尝试在 C 中编写一个程序，但我不知道它是否正确，因为我for循环在开始时仅在 3 次循环后结束，而它应该在 10 次后结束。如果有人能告诉我问题出在哪里，我会欣赏它。 这是程序。

#include <stdio.h>
#define size 10
int main(int argc, char *argv[]){
    int array[size],sum[size],k,sum1,i,j;
    sum1=0;

    printf("give 10 integers:\n");
    for(i=0; i<size; i++){
        scanf("%d", array[i]);
        sum[i]=0;
        }
    
    for(i=1; i<size; i++)
       {
        for(j=10; j>i; j--){
            if(array[j-1]>array[j]){
                k=array[j-1];
                array[j-1]=array[j];
                array[j]=array[j-1];

            }
        }
    }
    for(i=1; i<size; i++){
            if(array[i]=array[i+1])
            sum[i]=sum[i]+1;
            }
    for(i=1; i<size; i++){
            sum1=sum1+sum[i];
            }

 printf("%d", sum1);
}

我尝试删除其他循环，但只删除一次 for 循环。

Answer 1

无效的 scanf 第二个参数：

scanf("%d", array[i]);

array[i] 是一个 integer，scanf 需要一个指针，所以：

scanf("%d", &array[i]);
// or
scanf("%d", array + i);

做作的条件：

if (array[i] = array[i+1])

应该

if (array[i] == array[i+1])

设置了变量 k，但从未使用过。

数组 'sum' 可以在声明时用 0 填充。

int sum[size] = { 0 };

从索引 1 而不是 0 开始迭代数组。

索引 'j' 初始化为 10，而最大索引为 9。

将变量设置为自身，您的意思是使用“k”交换变量吗？

array[j - 1] = array[j]; // both cells now have same value
array[j] = array[j - 1]; // so this means array[j] = array[j]

稍微改进了代码，但无法命名其他循环，我不知道您要实现什么。 代码编译没有错误，只留下语义错误。

#include <stdio.h>
#include <stdlib.h>

#define NUMBERS_COUNT 10

static void promptNumbers(int numbers[], int numbersCount)
{
    int readNumbers;

    printf("Give %d integers:\n", numbersCount);

    for (readNumbers = 0; readNumbers < numbersCount; readNumbers++)
        scanf("%d", numbers + readNumbers);
}

int main(void)
{
    int numbers[NUMBERS_COUNT];
    int sum[NUMBERS_COUNT] = { 0 };
    int i, j, sum1;

    sum1 = 0;

    promptNumbers(numbers, NUMBERS_COUNT);

    for (i = 0; i < NUMBERS_COUNT; i++)
    {
        for (j = 9; j > i; j--)
        {
            if (numbers[j - 1] > numbers[j])
            {
                numbers[j - 1] = numbers[j];
                numbers[j] = numbers[j - 1];
            }
        }
    }

    for (i = 0; i < NUMBERS_COUNT - 1; i++)
    {
        if (numbers[i] == numbers[i + 1])
            sum[i] = sum[i] + 1;
    }

    for (i = 0; i < NUMBERS_COUNT; i++)
        sum1 = sum1 + sum[i];

    printf("%d", sum1);

    return EXIT_SUCCESS;
}

gcc -ansi -pedantic -Wall -Wextra -Werror temp.c -o temp
echo '1 2 3 4 5 6 7 8 9 10' | ./temp

编辑：第二次迭代，仍然不清楚你在用数组“sum”做什么，计算你有多少重复项？

#include <stdio.h>
#include <stdlib.h>

#define NUMBERS_COUNT 10

static void promptNumbers(int numbers[], int numbersCount)
{
    int readNumbers;

    printf("Give %d integers:\n", numbersCount);

    for (readNumbers = 0; readNumbers < numbersCount; readNumbers++)
        scanf("%d", numbers + readNumbers);
}

static void swapNumbers(int * const first, int * const second)
{
    int swapBuffer = * first;
    * first = * second;
    * second = swapBuffer;
}

static void sortNumbers(int numbers[], int numbersCount)
{
    int i, j;

    for (i = 0; i < numbersCount; i++)
    {
        for (j = 9; j > i; j--)
        {
            if (numbers[j - 1] > numbers[j])
                swapNumbers(& numbers[j], & numbers[j - 1]);
        }
    }
}

int main(void)
{
    int numbers[NUMBERS_COUNT];
    int sum[NUMBERS_COUNT] = { 0 };
    int i, sum1;

    sum1 = 0;

    promptNumbers(numbers, NUMBERS_COUNT);
    sortNumbers(numbers, NUMBERS_COUNT);

    for (i = 0; i < NUMBERS_COUNT; i++)
    {
        if (numbers[i] == numbers[i + 1])
            sum[i] = sum[i] + 1;
    }

    for (i = 0; i < NUMBERS_COUNT; i++)
        sum1 = sum1 + sum[i];

    printf("%d", sum1);

    return EXIT_SUCCESS;
}

gcc -ansi -pedantic -Wall -Wextra -Werror temp.c -o temp
echo '10 2 3 7 1 1 8 8 8 8' | ./temp
echo '10 2 3 7 4 1 9 6 5 8' | ./temp

Answer 2

这里的问题是你的循环必须从 0 开始。使用：

for(i = 0; i < size; i++) {
    ...
}

看，循环将从i = 0开始，并且只要i < size就会一直运行。 当i正好等于size时，它将不再运行。

要了解这一点，请运行代码：

#include <stdio.h>

int main() {
    int i, size = 10;

    for(i = 0; i < size; i++) {
        printf("The value of i is: %d\n", i);
    }

    return 0;
}

这将告诉您循环永远不会运行i = 10 ，因为当i为10时i < size为 False，因此循环条件为 false。