如何在c程序中找到最长和最短的字符串？

Question

我尝试编写一个程序来使用函数和数组来查找最短和最长的字符串，但程序不起作用。程序不显示订购的功能。 这是我的代码：

#include<stdio.h>
#include<string.h>

void Max(char x[][1000], int n);
void Min(char x[][1000],int n);

void Max(char x[][1000], int n){
    int i,Max,len1,c;
    Max=strlen(x[0]);
    for(i=1;i<n;i++){
        len1=strlen(x[i]);
        if(len1>Max)
        {
            c=i;
            Max=len1;
        }
    }printf("\nthe longest string among all is \"%s\" \n \n",x[c]);
}
void Min(char x[][1000],int n){
    int i,min,len2,d;
    min=strlen(x[0]);
    for(i=1;i<n;i++){
        len2=strlen(x[i]);
        if(len2<min)
        {
            d=i;
            min=len2;
        }
    }
    printf("\n the shortest string among all is \"%s\" \n \n",x[d]);
}
int main(){
    
    int i,jmlh=0,n,z;
    printf("How many name to accept: ");
    scanf("%d",&n);
    char x[n][1000];
    printf("\nEnter %d words: \n");
    for(i=0;i<=n;i++){
        gets(x[i]);
    }
    Max(x,n);
    Min(x,n);
    
    return 0;
}

输入是

3

然后输入：

robin van persie
lionel messi
ronaldo

应该是这样的输出：

the longest string among all is "robin van persie".
the shortest string among all is "ronaldo".

也许有人想帮助我修复这个程序，我真的需要你的意见。谢谢

Answer 1

变量 c 和变量 d 均未在函数 Max 和 Min 中初始化。

int i,Max,len1,c;

和

int i,min,len2,d;

至少像这样改变这些声明

int i,Max,len1,c = 0;

和

int i,min,len2,d = 0;

你在这个循环中输入了 n + 1 个元素

for(i=0;i<=n;i++){

但只将值 n 而不是 n + 1 传递给函数。

请注意函数gets 非常不安全，C 标准不再支持它。相反，请使用函数fgets。

Answer 2

试试这个代码：

#include<stdio.h>
#include<string.h>

void Max(char x[][1000], int n);
void Min(char x[][1000],int n);

void Max(char x[][1000], int n){
 int i,Max,len1,c=0; // c=0 (in case x[0] is the max) otherwise x[c] is undefined
 Max=strlen(x[0]);
 for(i=1;i<n;i++){ // i<n because I starts from 0
    len1=strlen(x[i]);
    if(len1>Max)
    {
        c=i;
        Max=len1;
    }
 }printf("\nthe longest string among all is \"%s\" \n \n",x[c]);
}
void Min(char x[][1000],int n){
 int i,min,len2,d=0; // d=0 otherwise if min=x[0], x[d]is undefined
 min=strlen(x[0]);
 for(i=1;i<n;i++){
    len2=strlen(x[i]);
    if(len2<min)
    {
        d=i;
        min=len2;
    }
}
printf("\n the shortest string among all is \"%s\" \n \n",x[d]);
}
int main(){

 int i,n,c;
 printf("How many name to accept: ");
 scanf("%d",&n);
 char x[n][1000];
 printf("\nEnter %d words: \n",n);
 while ((c = getchar()) != '\n' && c != EOF) { }
 for(i=0;i<n;i++){ // i<n because i starts from 0 (and not from 1)
    fgets(x[i],sizeof x[i],stdin); // fgets is more secure than gets
    

 }

 Max(x,n);
 Min(x,n);

 return 0;
}