C语言数组找最大元素程序问题：负数场景结果错误

你的程序在处理负数时出错，核心是最大值查找逻辑完全混乱，具体问题如下：

问题点拆解 #

• 每次循环重置larger：循环内第一行larger = arr[y]会覆盖之前的比较结果，导致永远只能和当前元素对比，无法追踪全局最大值
• 混淆元素值与数组索引：你既把larger存成元素值，又赋值成数组索引y，最后arr[larger]会用元素值当作索引，直接引发访问错误
• 错误的正负分支逻辑：负数分支里用arr[y] < larger，这是在找更小的数，完全背离“找最大值”的目标，而且根本不需要分正负判断——找最大值的逻辑对正负是统一的

修复后的代码 #

#include <stdio.h>

int main()
{
    int ele;
    printf("Enter the required number of elements:");
    scanf("%d", &ele);
    int arr[ele];
    for (int i = 0; i < ele; i++)
    {
        printf("Enter value for the array:\n");
        scanf("%d", &arr[i]);
    }
    // 打印数组
    int len = ele; // 直接用输入的ele作为数组长度，无需sizeof计算
    for (int z = 0; z < len; z++)
    {
        printf("Array element is %d\t", arr[z]);
    }
    printf("\n"); // 换行提升可读性
    
    // 查找数组最大值
    int max_index = 0; // 用索引追踪最大值位置，避免值与索引混淆
    for (int y = 1; y < len; y++) // 从第二个元素开始比较
    {
        if (arr[y] > arr[max_index])
        {
            max_index = y;
        }
    }
    printf("The largest element in the array is: %d", arr[max_index]);
    return 0;
}

关键修改说明 #

• 用索引追踪最大值：max_index初始化为第一个元素的索引，全程只存储索引，彻底避免值与索引的混淆
• 统一比较逻辑：不管元素是正还是负，只要当前元素比arr[max_index]大，就更新max_index，逻辑简单且通用
• 简化长度计算：直接使用输入的ele作为数组长度，比sizeof(arr)/sizeof(arr[0])更直接（变长数组的sizeof虽能生效，但此处ele已是已知长度）
• 修复格式问题：补全输入提示的换行符，打印数组后添加换行，优化输入输出的可读性

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