C语言数组求最大值出错：ageProgMax得29而非预期60求助

你的问题出在条件判断的逻辑运算符使用错误，导致程序执行了不符合预期的赋值操作，最终得到了错误的最大值29。

错误原因分析 #

你写的判断条件是：

if (poste[i] =='P' || age[i] > ageProgMax)

这个条件的意思是：只要当前人员是程序员，或者当前年龄比记录的最大值大，就更新最大值。这会引发两个问题：

1. 非程序员的人员如果年龄更大，会错误地更新最大值（比如索引2的非程序员年龄27，会把初始的0替换成27）；
2. 最关键的是——只要是程序员，不管他的年龄比当前最大值小还是大，都会执行ageProgMax = age[i]。比如最后一个程序员（索引7）的年龄是29，虽然比之前找到的60小，但因为他是程序员，条件成立，ageProgMax被直接替换成29，这就是你得到错误结果的核心原因。

修正方案 #

正确的逻辑应该是：只针对程序员，再判断他们的年龄是否大于当前记录的最大值。你需要把逻辑或||改成逻辑与&&，或者拆分判断让代码更易读：

方案1：修正逻辑运算符

int main() { 
    char poste[] ={'P', 'P', 'O', 'A', 'P', 'A', 'P', 'P'}; 
    int nbCafe[] ={5, 1, 3, 0, 5, 1, 0, 2}; 
    int age[] ={25, 19, 27, 22, 49, 24, 60, 29}; 
    int nbPers = sizeof(age) / sizeof(int); 
    int i; 
    int ageProgMax = 0; 
    for (i = 0; i < nbPers; i++) {
        // 仅当是程序员且年龄更大时，更新最大值
        if (poste[i] =='P' && age[i] > ageProgMax) { 
            ageProgMax = age[i]; 
        }
    } 
    printf ("Max age of programmers : %d\n", ageProgMax); 
    return 0; 
}

方案2：拆分判断（可读性更强）

把“筛选程序员”和“比较年龄”拆分成两个嵌套的判断，逻辑更清晰：

for (i = 0; i < nbPers; i++) {
    if (poste[i] =='P') { // 先筛选出程序员
        if (age[i] > ageProgMax) { // 再判断是否是更大的年龄
            ageProgMax = age[i]; 
        }
    }
}

额外优化建议 #

初始值ageProgMax = 0在实际场景中没问题，但从代码健壮性考虑，如果所有程序员的年龄都小于0（极端情况），这个初始值会导致错误。可以改用<limits.h>中的INT_MIN作为初始值：

#include <limits.h>
// ...
int ageProgMax = INT_MIN;

至于你之前求分析师喝咖啡最小数量时可行，应该是当时的逻辑用对了运算符（比如用了&&来同时判断身份和数量更小），或者数据情况刚好没触发错误的赋值逻辑，而这次的逻辑因为误用||，导致了错误的覆盖。

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