文字列のアドレス渡しができません。

Question

文字列resultのアドレスを関数に渡して、その値を変更したいのですが、文字列が変わらず、”初期文字列”のままです。
文字列のアドレス渡しは、できないのでしょうか？

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

int hantei(int ran, int iti, int hani, char *result);
int main()
{
	srand((unsigned)time(NULL));
	int ran = rand() % 100 + 1;
	int i, iti, hani, flg = 0;
	char *result = "初期文字列";
	
	puts("☆レーダー☆");
	puts("ターゲットとして 0～100 のうち１つの数が設定されます。探索する位置と範囲を入力してターゲットを見つけてください。");
	puts("ターゲットと探索位置の差が探索範囲以下であればヒットです。探索範囲より大きければ、はずれです。");
	puts("探索位置とターゲットが一致すれば正解です。");
	puts("----------------------------------------------");
	puts("ゲームを開始します。");
	puts("ターゲットが設定されました。");

	for(i = 1; flg == 0; i++)
	{
		printf("位置は？ ");
		scanf("%d", &iti);
		printf("範囲は？ ");
		scanf("%d", &hani);
		flg = hantei(ran, iti, hani, &result[0]);
		printf("%s\n\n", result);
	}
	printf("おめでとう！ %d回目で正解です！");
	
	return 0;
}

int hantei(int ran, int iti, int hani, char *result)
{
	if(ran == iti) return 1;
	else if(hani >= iti < ran ? ran - iti : iti - ran)
	{
		*result = "ヒットしました。";
		return 0;
	}
	else
	{
		*result = "はずれました。";
		return 0;
	}
}

Comments

[episteme]

これ、コンパイルエラーにはならんですか?

[thkana]

Cは甘々だから警告止まりですね

[episteme]

質問主は警告をガン無視したんじゃろか...

Answer 1

文字列のアドレス渡しは、できないのでしょうか？

できますよ。

#include <stdio.h>

/*
 * *dst に src をセットし、元(セット前)の*dstを返す。
 */
const char* swap_str(const char** dst, const char* src) {
  const char* old = *dst;
  *dst = src;
  return old;
}

int main() {
  const char* str = "apple";
  const char* old = swap_str(&str, "りんご");
  printf("[%s] を [%s] に変更しました\n", old, str);
  return 0;
}

実行結果:

[apple] を [りんご] に変更しました

Answer 2

解決したようなので削除

Answer 3

VS Community 2019で、コンパイル(clコマンド)したら、エラーにはなりませんでしたが、警告が複数。

printf("おめでとう！ %d回目で正解です！");

引数の数が足りません。 --> %dに対応する引数が無いです。

   *result = "ヒットしました。";
   *result = "はずれました。";

間接参照のレベルが合いません。 ('char *とchar` に代入。

さて、問題の代入の件ですが、
簡単には、
flg = hantei(ran, iti, hani, &result[0]);
--> flg = hantei(ran, iti, hani, &result);

int hantei(int ran, int iti, int hani, char *result)
--> int hantei(int ran, int iti, int hani, char **result)

でしょうか。
ポインタの使い方を間違えているようですね。

Answer 4

ここで問題なのは、変更しようとしてるところが、

     *result = "ヒットしました。";

ってことです。
resultってのはなんの型か考えましょう

*result ってすると、char型に"ヒットしました。"を代入しようとしてるってことになります。
これではダメですね

＃まあ、他にもいろいろダメなとこはありますが

Answer 5

処理の内容は見ていませんが、以下のようにしてください。
これでresultの内容が変更されます。

#include <stdio.h>
#include <stdlib.h>
#include <string.h>		//追加
#include <time.h>

int hantei(int ran, int iti, int hani, char *result);
int main()
{
    srand((unsigned)time(NULL));
    int ran = rand() % 100 + 1;
    int i, iti, hani, flg = 0;
    char result[100] = "初期文字列";	//変更

    puts("☆レーダー☆");
    puts("ターゲットとして 0～100 のうち１つの数が設定されます。探索する位置と範囲を入力してターゲットを見つけてください。");
    puts("ターゲットと探索位置の差が探索範囲以下であればヒットです。探索範囲より大きければ、はずれです。");
    puts("探索位置とターゲットが一致すれば正解です。");
    puts("----------------------------------------------");
    puts("ゲームを開始します。");
    puts("ターゲットが設定されました。");

    for(i = 1; flg == 0; i++)
    {
        printf("位置は？ ");
        scanf("%d", &iti);
        printf("範囲は？ ");
        scanf("%d", &hani);
        flg = hantei(ran, iti, hani, &result[0]);
        printf("%s\n\n", result);
    }
    printf("おめでとう！ %d回目で正解です！");

    return 0;
}

int hantei(int ran, int iti, int hani, char *result)
{
    if(ran == iti) return 1;
    else if(hani >= iti < ran ? ran - iti : iti - ran)
    {
        strcpy(result,"ヒットしました。");	//変更
        return 0;
    }
    else
    {
        strcpy(result,"はずれました。");	//変更
        return 0;
    }
}

Comments

[episteme]

「resultに確保された領域が小さい場合を考慮しなくていい」
ならコレでOKってことですよね?

[tatsu99]

はい。そうです。そのようなことにまで、質問者のかたがきずくかどうかはわかりませんが、
まずは、質問者のかたが成功体験をするのを優先して、そのように回答しました。