int型で受け取った値をchar型で出力したい

Question

以下のようなコードでint型で受け取った値をchar型の文字列として表示させたいのですが、うまくいきません。control reaches end of non-void function [-Wreturn-type]のようなwarningが表示されます。制御が非ボイド関数の終了に達した場合とは、どういうことでしょうか？

#include <stdio.h>

char* answer(int val) {
  val = (char)val;

  if ((0 <= val) && (val <= 5)) {
    printf("0~5を選択しました。\n");
  }
  else if ((6 <= val) && (val <= 10)) {
    printf("6~10を選択しました。\n");
  }
  return val;
}

int main(void) {
  int value;

  printf("0~10で好きな数字を入力してください\n");
  scanf("%d\n", &value);

  printf("%s\n", answer(value));

	return 0;
}

Comments

[maisumakun]

提示されたソースコードで動作確認を行おうとしたところ、「control reaches end of non-void function」ではなく別なエラーとなってコンパイルに失敗しました。

https://paiza.io/projects/s_KDHgvtKZEq5GNez8bAqg?language=c

Answer 1

val = (char)val;

これは整数を文字列にしようとしているのでしょうか？
こういうやり方を何処で覚えましたか？
キャストはこのような目的では使えません。
しかも、answer() の出口で val を返しています。
answer() は char * ですので、方が一致していません。

基本から復習したほうが良いように思います。

Comments

[gyoruo]

ご回答ありがとうございます！
もう一度しっかしと基本を学び直します。

Answer 2

x.c

#include <stdio.h>

char buffer[10];

char* answer(int val) {
  if ((0 <= val) && (val <= 5)) {
    printf("0~5を選択しました。\n");
  }
  else if ((6 <= val) && (val <= 10)) {
    printf("6~10を選択しました。\n");
  } else {
    printf("0~10以外を選択しました。\n");
  }
  sprintf(buffer, "%d", val);
  return buffer;
}

int main(void) {
  int value;

  printf("0~10で好きな数字を入力してください\n");
  scanf("%d", &value);
  printf("%s\n", answer(value));

  return 0;
}

実行例：

Answer 3

char型の文字列

char型は整数型の一種です。文字列では無いです。

プログラムからやりたいことを想像して書き直すと、

#include <stdio.h>

char *answer(int val) {
  if ((0 <= val) && (val <= 5)) {
    return "0~5を選択しました。\n";
  }
  else if ((6 <= val) && (val <= 10)) {
    return "6~10を選択しました。\n";
  }
  return "0~10以外を選択しました。\n";
}

int main(void) {
  int value;

  printf("0~10で好きな数字を入力してください\n");
  scanf("%d", &value);

  printf("%s\n", answer(value));

    return 0;
}

Comments

[gyoruo]

わかりやすいご回答、ありがとうございます！

Answer 4

#define  _CRT_SECURE_NO_WARNINGS
#include <stdio.h>
#include <string.h>
#include <stdlib.h>

/* 返値は必ずfreeすること */
char* answer(int val) {
    char buffer[16];
    char* result;

    sprintf(buffer, "%d", val);
    result = (char*)malloc(strlen(buffer)+1);
    if ( result != NULL ) {
      strcpy(result, buffer);
    }
    return result;
}

int main(void) {
    int value;
    char* result;

    printf("0~10で好きな数字を入力してください\n");
    scanf("%d", &value);

    printf("input : [%s]\n", result = answer(value));
    free(result); // わすれずに。
    return 0;
}

Comments

[gyoruo]

ご回答ありがとうございます！
メモリ開放を忘れずにします。

Answer 5

char* answer(int val) {

ポインタを返すようになってるのに、charを返してるのでワーニングが出ています
＊を抜きましょう

それと、

printf("%s\n", answer(value));

は意味不明です。ここで何をしたいのかよく考えよう

Comments

[gyoruo]

ご回答、ありがとうございます。
確かに、返り値の型が合っていないですね。。。