質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.47%
C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

Xcode

Xcodeはソフトウェア開発のための、Appleの統合開発環境です。Mac OSXに付随するかたちで配布されています。

Q&A

解決済

2回答

226閲覧

”Variable 'num' may be uninitialized when used here”の警告の意図について

kihochi

総合スコア2

C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

Xcode

Xcodeはソフトウェア開発のための、Appleの統合開発環境です。Mac OSXに付随するかたちで配布されています。

0グッド

0クリップ

投稿2024/03/13 06:20

実現したいこと

数当てゲームを作成したい。

発生している問題・分からないこと

18行目に警告が出てくる。

1if (num != ans){

エラーメッセージ

error

1Variable 'num' may be uninitialized when used here 2Initialize the variable 'num' to silence this warning

該当のソースコード

Xcode

1#include <stdio.h> 2 3int main (void){ 4 int ans = 55; 5 int num; 6 printf("数値を当ててください。\n"); 7 for (int i = 0; i < 5; i++) { 8 scanf("%d", &num); 9 if (num < ans) { 10 printf("もっと大きい数字です。\n"); 11 } else if (num > ans){ 12 printf("もっと小さい数字です。\n"); 13 } else { 14 printf("正解\n"); 15 break; 16 } 17 } 18 if (num != ans){ 19 printf("ゲームオーバー\n"); 20 } 21 return 0; 22}

試したこと・調べたこと

  • teratailやGoogle等で検索した
  • ソースコードを自分なりに変更した
  • 知人に聞いた
  • その他
上記の詳細・結果

初期化されていないということなので、5行目を"int num = 0;"したら警告がなくなりました。
しかし、for文で入力値を与えるため、初期化は不要ではないかと考えます。
その点について、ご教授いただけると幸いです。

補足

特になし

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

jimbe

2024/03/13 07:18

コードのマークダウン(```)の右側に書くのは言語名( c とか)で、「18行目に警告が出てくる。」とか「Xcode」とかはちょっと違うかなと思います。
kihochi

2024/03/13 07:27

言語を入力とか例が出てくるのにガン無視してました💦 以後気をつけます!
guest

回答2

0

しかし、for文で入力値を与えるため、初期化は不要ではないかと考えます。

はい、そのとおりなのですが、それをコンパイラが知るにはある程度コードの動きを予測する必要があります。
for文の中に入るかを確認して
numに値が入る前にfor文を途中で抜けたりしないか判断して...と
そんなに賢くするよりは機械的に判断して警告を出しておこうということです。

投稿2024/03/13 06:36

ozwk

総合スコア13528

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

kihochi

2024/03/13 06:55

警告の必要性が理解できました。もう少しコンパイラの気持ちを理解できるようにならないといけませんね…ありがとうございました。
guest

0

ベストアンサー

コンパイラ的には

scanf("%d", &num);

の部分で 「num に何か値が入る」ということまでは 判断/理解 していないかもしれないので,警告を無視して良いとあなたが判断するのであれば,そうすれば良いでしょう.
ただし,コンパイル時に警告が出てくる状況はできれば解消しておきたいところなので,適当に初期値を与えても特に困ることが無いならば,コンパイラ様がおっしゃるように解消しておけば良いのではないでしょうか.

それはそれとして,この scanf は失敗するかもしれない(結果として num には一度も入力値が入らないかもしれない)ので,結果としては未初期化の状態で使う状況は起き得るかもしれませんよね.

投稿2024/03/13 06:39

編集2024/03/13 06:39
fana

総合スコア11658

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

kihochi

2024/03/13 06:51

”適当に初期値を与えても特に困ることが無いならば,コンパイラ様がおっしゃるように解消しておけば良い” →初期化せず中身がわからないままよりも何かしら数字を入れて整えておく方が利口だな、と納得いたしました。より詳細にご説明いただいたこちらをベストアンサーにいたします。お二人とも、ありがとうございました。
fana

2024/03/13 07:36

「この警告は放置しても大丈夫」と判断できるとしても, コンパイル結果を確認する度に(特に複数個出てくる場合とか)「この警告は大丈夫なハズだから無視してOK」とかいちいち判断しなければならないのも負担になりますし, これが他人の書いたコードだったりすると「何か警告出てるけど大丈夫なの?」とか不安になりますので, 適切に対処できるものはしておくのが良いです.
kihochi

2024/03/14 04:09

おっしゃる通りですね。実務的な観点からご教授いただき大変勉強になります。ありがとうございます!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.47%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問