Abort trap:6、Thread 1: signal SIGABRTの解決策

Question

前提・実現したいこと

初めての質問なので説明等で至らない点があるかと思いますが、何卒よろしくお願いします。
C言語で文字列を選択ソートせよ。という問題です。
文字列としてはAZ、az、0~9を用いるものとします。
１）文字列の長さが短い方が小さい
２）長さが同じ時は先頭から文字列の大小を比較し文字の小さい方が文字列として小さいとする
なお、文字の大小は0<1<2<・・・<a<・・・<z<A<・・・<Z

発生している問題・エラーメッセージ

対象を比較後交換する際にstrcpy() を用いるところでエラーが出ます。xcodeによるとstrcpy（str[i],str[min]）;部分でThread 1: signal SIGABRTが表示されます。
ターミナルを用いて実行した場合Abort trap:6が表示されます。

```Thread 1: signal SIGABRT
　　Abort  trap:6

該当のソースコード

strcpy(str[i],str[min]);

ソースコード

#include <stdio.h>
#include <string.h>
#define n 10
void selectSort(char str[n][32])
{
char temp[31];
int min,i,j;
for (i=0;i<n;i++){
min=i;
for (j=i+1;j<n;j++){
if (strlen(str[j])<strlen(str[min])){
min=j;
}else if (strlen(str[j])==strlen(str[min])){
if (strcmp(str[min],str[j])>0){
min=j;
}
}else{
min=i;
}
}

    strcpy(temp,str[i]);
    strcpy(str[i],str[min]);  /*Xcodeの場合ここでThread 1: signal SIGABRTが表示されます*/
    strcpy(str[min],temp);
    
}

for (int k=0;k<n;k++){
    printf("%s,",str[k]);
}

}

int main()
{
char str[10][32]={"abc","2020","osaka","aiu","5","314","tokyo","kyoto","ikeda","abcd1234"};
selectSort(str);

return 0;

}

試したこと

str[I] とstr[min]のメモリサイズがあっていないのかと思い、min=j;の直後にstrcpy(str[min],str[j]);を入れてみるなどしました。

補足情報（FW/ツールのバージョンなど）

xcode Version 11.2 (11B52)

Answer 1

strcpy はコピー元とコピー先が重なっていた場合の動作は未定義です。よって、i==minのときはstrcpyしないようにしなければいけません。
また、下の部分は余計です。これでは最小値がiに戻ってしまいます。

}else{
min=i;
}

Comments

[VanS]

修正したら正しく動きました！
拙い説明なのにわかりやすい説明ありがとうございました！