文字型ポインタの配列について[C]

Question

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

int main(void)
{
    char *key[] = {"一,二,三,四,五,六,七,八,九"};
    int c;
    int number = 19;
    int min = 0;
    
    printf("数字[1-19]:");
    scanf("%d", &c);
    
    if (key[c - 1]){
        printf( "%dは漢字で%sと書きます\n", c, *key );
    }else if(key > number){
        printf("1から19の間の数値を入力して下さい。");
    }else if(key == min){
        printf("1から19の間の数値を入力して下さい。");
    }
    return 0;
}

やりたいこと

数字[1-19]:1
1は漢字で一と書きます。
数字[1-19]:19
19は漢字で十九と書きます。
数字[1-19]:30
1から19の間の数値を入力して下さい。
数字[1-19]:0
1から19の間の数値を入力して下さい。

このように入力した数字に対して漢数字で表示させたい。
if文で条件分岐しているが思うようにならなくて困っています。

Answer 1

こんにちは、
入力した数字が1-19以外の数字の場合の再入力をdo~while文でやりました。

また、scanf関数は色んな意味で危険な関数なのでfgetsを使いました。

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

#define BUFSIZE 256

int main(void)
{
　char buf[BUFSIZE];
  char key[] = "一二三四五六七八九";
  int c;

  printf("数字[1-19]:");
    	
	do{
		fgets(buf,sizeof(buf),stdin);
		c = atoi(buf);
		if(c < 1 || 19 < c){
			printf("1-19の数字を入力してください。\n");
		}
	}while(c < 1 || 19 < c);
  
  if(c < 10){
  	printf( "%dは漢字で%c%cと書きます\n", c,key[c * 2 - 2],key[c * 2 - 1]);
  }
	else if(c <= 19){
		c = c % 10;
		printf( "%dは漢字で十%c%cと書きます\n", c,key[c * 2 - 2],key[c * 2 - 1]);
	}
  return 0;

}

Comments

[退会済みユーザー]

投稿完了したらすでに解決済みになった....

Answer 2

keyの定義はカンマで区切りましょう。
入力範囲のエラーチェックは最初に行わないとダメですね。
などを踏まえて書き直すとこんな感じになりました。
10のときだけはそのまま出力しています。

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

int main(void)
{
    char *key[] = {"一", "二", "三", "四", "五", "六", "七", "八", "九"};
    int c;
    int number = 19;
    int min = 1;

    printf("数字[1-19]:");
    scanf("%d", &c);

    if(c < min || c > number){
        printf("1から19の間の数値を入力して下さい。");
    }else {
		if(c < 10){
        	printf( "%dは漢字で%sと書きます\n", c, key[(c - 1)] );
		} else if(c == 10) {
			printf( "%dは漢字で十と書きます\n", c );
		} else {
        	printf( "%dは漢字で十%sと書きます\n", c, key[(c % 10 - 1)]);
		}
	}
    return 0;
}

Comments

[退会済みユーザー]

ありがとうございました。解決出来ました。

Answer 3

多分

char *key[] = {"一,二,三,四,五,六,七,八,九"};

じゃなくって

const char * const key[] = {"一","二","三","四","五","六","七","八","九"};

それから条件文はやりたいことが意味不明すぎる。例えば

if (key[c - 1])

はcが1-9でない場合その時点で不正なメモリーアクセス。その後のelse ifは多分cが1-9を保証したかったんだろうけどkey[c - 1]する前に判定しないと意味がない。

Comments

[退会済みユーザー]

すいません、条件文に関しては正直分からない状態で適当に作ったので仰る通りです。