要求是fun函数的功能：利用指针数组对形参ss所指字符串数组中的字符串按由长到短顺序排列

Question

为什么我代码输出结果是什么都没变呢？输入顺序是red,green,blue,yellow,black，输出的顺序还是red,green,blue,yellow,black

#include<stdio.h>
#include<string.h>
#define N 5
#define M 8
void fun(char (*ss)[M])
{char *ps[N],*tp; int i,j,k;
for(i=0;i<N;i++)ps[i]=ss[i];
for(i=0;i<N-1;i++){
k=i;
for(j=i+1;j<N;j++)
if(strlen(ps[k])<strlen(ps[i]))k=j;
tp=ps[i];ps[i]=ps[k];ps[k]=tp;
}
printf("\nThe string after sorting by length:\n\n");
for(i=0;i<N;i++) puts(ps[i]);
}
main()
{char ch[N][M]={"red","green","blue","yellow","black"};
int i;
printf("\nThe original string\n\n");
for(i=0;i<N;i++)puts(ch[i]); printf("\n");
fun(ch);
}

Answer 1

你是使用了选择排序法，所以k的值一定要给准确
for( j = i + 1; j < N ; j++)
if( strlen(ps[k]) < strlen(ps[i])) // 这永远不成立，因为k == i。（改： 不是i 应该是j）
k = j;
改为:if(strlen(ps[k])<strlen(ps[j])) k=j;

Answer 2

if语句里面里面的判断，应该是*ps[k]与*ps[i]的比较.掉了*

Answer 3

for(j=i+1;j<N;j++)
if(strlen(ps[k])<strlen(ps[i]))k=j;
tp=ps[i];ps[i]=ps[k];ps[k]=tp这里的i应该是j吧，而且变量k好像有点多余，别处于无问题还没看。