アルファベット文字列をstrを使わずに昇順にソート、表示する。

Question

前提・実現したいこと

C言語です
char s[][10] = { "JAY","FARY","RCH","AIL","MY"
,"JE","JY","AST","SER","OCER","NER","DER" }, tmp[12];という初期値が与えられていて、この文字列をstrcpyを使わずにアルファベット順にソート、表示したいです。

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

エラーは出ず、３つの文字列だけ昇順に表示されました。

エラーメッセージ

該当のソースコード

#include<stdio.h>
#define TUKI 11

int main(void) {

	int i, j, k;

	char s[][10] = { "JAY","FARY","RCH","AIL","MY"
		,"JE","JY","AST","SER","OCER","NER","DER" }, tmp[12];

	for (i = 0; i < TUKI; ++i) {
		for (j = i + 1; j < TUKI; ++j) {
			if (s[i][0] > s[j][0]) {
				for (k = 0; k < 10; k++) {
					tmp[k] = s[j][k];
					s[i][k] = s[j][k];
					s[j][k] = tmp[k];
				}
			}
		}
	}
	for (i = 0; i < TUKI; ++i)
		printf("%s\n", s[i]);
}

試したこと

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

visual studio2019
ここにより詳細な情報を記載してください。

Comments

[1T2R3M4]

debugしていないのですか。

[nbvcc]

ビルド実行してエラーメッセージは出ませんでした

[dodox86]

> ビルド実行してエラーメッセージは出ませんでした

それはデバッグとは言わないのでは。

少なくとも
if ([i][0] > s[j][0])) {
の部分で、頭の1文字しか比較していませんし、TUKIは11だけど、char s[][10]の要素は12個あったりします。

Answer 1

こんなんで。

#include <stdio.h>
#include <stdlib.h>

int comp(const void* x, const void* y) {
  const char* sx = (const char*)x;
  const char* sy = (const char*)y;
  while ( *sx != '\0' && *sy !='\0' && *sx == *sy ) {
    ++sx;
    ++sy;
  }
  return *sx - *sy;
}

int main() {
  char s[][10] = { "JAY", "FARY", "RCH", "AIL",  "MY",  "JE",
                   "JY",  "AST",  "SER", "OCER", "NER", "DER" };
    qsort(s,12,10,&comp);
    for ( int i = 0; i < 12; ++i ) {
      printf("[%s] ", s[i]);
    }
    return 0;
}

Answer 2

先頭文字だけ比較するのではなく、先頭から順番に比較していき、
k 番目が異なると分かった時に逆順なら交換すればよいでしょう。

#include <stdio.h>

#define TUKI (sizeof s / sizeof s[0])

int main(void)
{
    int i, j, k;

    char s[][10] = { "JAY","FARY","RCH","AIL","MY","JE",
		"JY","AST","SER","OCER","NER","DER" }, tmp;

    for (i = 0; i < TUKI; ++i) {
        for (j = i + 1; j < TUKI; ++j) {
			for (k = 0; s[i][k] == s[j][k]; k++)
				if (s[i][k] == '\0') break;
			if (s[i][k] > s[j][k]) {
				for ( ; k < 10; ++k) {
					tmp = s[i][k];
					s[i][k] = s[j][k];
					s[j][k] = tmp;
				}
			}
        }
    }
    for (i = 0; i < TUKI; ++i)
        printf("%s\n", s[i]);
}

Answer 3

文字列へのポインタの配列を作って、 C標準のqsort()を使うのはどうでしょう。
ソートアルゴリズムを実装しなくてよくなります