C#:while文で何回割ると100以下になるか

Question

Visual Studio2017で、自分の力で書きました。問題は「while文を使って10000を2で何回割ると100以下になるかを調べるプログラムを作ってください。」です。文法通りですか？どこか改善した方がいいですか？アドバイスをください。

     static void Main(string args[])
    {
        int num = 10000;
        int time = 0;   

        while(num >= 100)
        {
            time++;
            num = num / 2;
        }
        Console.WriteLine(time);
    }   

Comments

[退会済みユーザー]

「コードをください・デバッグしてください等の丸投げの質問」は推奨されません。困ってることを書いてください

[maisumakun]

文法が正しいかはコンパイラがいちばん良く知っています。その面については、投げれば結論は出ます。

[退会済みユーザー]

前にあなたが立てたスレッドをきちんとクローズして下さい。きちんとクローズしないでスレッドを乱立するのは迷惑です。

[len_souko]

前回の添削依頼で指摘しても無視して繰り返していますので通報しておきました。サービスを利用する際にはそのサービスの目的に合わせた使い方をしてください。あなたの目的とサービスの方針が異なる場合は別のサービスを利用してください。

Answer 1

文法通りですか？

意地の悪い返答をしてしまえば、「わかりません」ですね。

どこか改善した方がいいですか？

改善するなら、それは質問者さんのコードではなく質問者ご自身です。
今回のケースにおける改善できる点は、**「質問をしない」**ことです。

では、「どうすれば質問をしなくて済むか？」をお答え致します。
（先日お答えした別の方への回答と内容が被ってしまいますが…）

#####デバッグをする
質問者さんはVisual Studio 2017を使われているとのことなので
Visual Studioのデバッガーを使いましょう。

デバッガーとはデバッグを支援するツールです。
また、デバッグとは端的に言うと「バグを取り除く作業」のことです。

例を挙げると、
「ブレークポイント」を使うことでプログラムを一時停止することができる。
一時停止中は、変数やフィールドの中身を見ることができる。
ということが出来たりします。（他にも勿論あります）

なぜデバッグの使用を質問者さんに勧めたかというと、
書いたコードが「文法通りか」・「意図通りに動いているか」は、人に聞くべき内容ではないから
です。

結局、コードが文法通りかはコンパイラ次第で
意図通りに動いているかはプログラム次第です。

コードは嘘をつきません。書いた通りに動きます。
なので、まずは実行して動作確認をしてみてください。
動くにしろ動かないにしろ、その結果が真実です。

話が長くなってしまい申し訳ないです。
改めて回答しますと、

文法通りですか？

コンパイルが通ることを確認してください。
通れば合ってます。

どこか改善した方がいいですか？

それを確認する手段の一つとしてデバッグが存在します。
まずはデバッグの使い方を覚えることで改善点があるかを判断してください。

例えば、今回でいう「num」や「time」がループの中で意図通りの値になっているか？を
しっかり自身の目で確認されると良いです。

#####Visual Studioにおけるデバッガーの使い方
Microsoftのドキュメントにて使い方が記載されているので、参考にしてみてください。

• ブレークポイントの使い方
• 変数の確認方法1
• 変数の確認方法2

何か分からない点などあれば、お気軽に返信してください。

Answer 2

これだと、１００未満になったときの回数、になってますねー

Comments

[Koopa]

…直したみたいだね