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

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

ただいまの
回答率

87.49%

【C#】文字列を入力するとエラーの表示が出るようにしたい

受付中

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 833

score 1

文字列を入力するとエラーの表示が出るようにしたいのですが、どのようにすれば良いでしょうか。

負の数が入力されるまで、キーボードから繰り返し数値を入力するプログラムを作成中です。
以下が現在のコードです。負の数が入力されると終了するようになっています。空白はエラー表示が出るようになっています。

その他、修正推奨箇所がございましたら、よろしければ教えてください。

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace ConsoleApp
{
    class Program
    {
        static void Main(string[] args)
        {
            var num = 0;
            while(num >= 0)
            {
                Console.Write("負数を入力してください:");
                var line = Console.ReadLine();
                if (line == "")
                {
                    Console.WriteLine("入力結果:エラー");
                    continue;
                }
                num = int.Parse(line);
                Console.WriteLine($"入力結果;{num}");
            }
            Console.WriteLine("負数が入力されました");
        }
    }
}
  • 気になる質問をクリップする

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • 退会済みユーザー

    2021/01/22 00:32

    複数のユーザーから「問題・課題が含まれていない質問」という意見がありました
    teratailでは、漠然とした興味から票を募るような質問や、意見の主張をすることを目的とした投稿は推奨していません。
    「編集」ボタンから編集を行い、質問の意図や解決したい課題を明確に記述していただくと回答が得られやすくなります。

  • Zuishin

    2021/01/22 10:27 編集

    int.Parse に失敗したら数値ではないというのは非常に具体的では?
    どこが抽象的なのかわからないんですが。

    喧嘩とか不快とかなぜそのような言葉が出てくるのかも意味が分かりません。

    キャンセル

  • funabashi

    2021/01/22 10:57

    int.Parseに失敗したらは非常にわかりやすいご回答でした。誤解を招いてしまい大変申し訳ございません。
    すみません、後ほど修正させていただきます。皆様ありがとうございました。今後は気をつけます。

    キャンセル

回答 1

-1

出来るだけ元のコードを変えずに回答いたします。

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace ConsoleApp
{
    class Program
    {
        static void Main(string[] args)
        {
            var num = 0;
            while (num >= 0)
            {
                var line = 0;
                try
                {
                    Console.Write("負数を入力してください:");
                    line = int.Parse(Console.ReadLine());
                }
                catch
                {
                    Console.WriteLine("入力結果:エラー");
                    continue;
                }

                num = line;
                Console.WriteLine($"入力結果;{num}");
            }
            Console.WriteLine("負数が入力されました");
        }
    }
}

try-catchというものがあるので、これを使えばエラー判定が簡単にできます。
tryの中でエラーが起きたら、catchの中に入る、そんな処理です。

line = int.Parse(Console.ReadLine());で「数字のみ入れることが出来るよ!」みたいな処理にします。

数字以外、要するに空白や、文字列が入ってきたときにcatchに入ることでエラーを出します。

こんな感じでいかがでしょうか。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2021/01/22 13:27

    「失敗した理由が(ある程度?)知りたい → Parse 側を使う」とたら,その選択はどの程度不適切でしょうか?

    キャンセル

  • 2021/01/22 13:45 編集

    昔はこういうコードは普通でしたね。公式サイトにも多くのサンプルがあります。

    https://docs.microsoft.com/ja-jp/dotnet/csharp/programming-guide/types/how-to-convert-a-string-to-a-number
    https://docs.microsoft.com/ja-jp/dotnet/standard/base-types/parsing-numeric

    C 言語ではエラーハンドリングがネストの山になっていたので、それを解消することは例外という機構が作られた最大の目的だと思います。
    つまりこのように例外的な入力をはじいて主なるロジックをシンプルにするために例外を使うというのは、例外の本来の使い方であったと認識しています。
    言うなれば例外機構は Maybe モナドのような使い方をされていたわけですね。
    重いというのはありますが、ユーザーの入力に限って言うなら入力の方が重いので無視できます。

    しかし時代が進んでこのような方法を使わなくてもよくなったので、TryParse やタプルとパターンマッチングを使ったモダンな方法が今は推奨されます。

    キャンセル

  • 2021/01/22 15:30

    .NET アプリの例外処置について、自分的に一般的と思うことを書きます。

    (1) 予測可能で正しい業務フローに戻すことができる「業務エラー」(例:ユーザーの入力間違い)と、予測できないもしくは予測はできても何の対応もできない「例外」(例:DB サーバーダウン)を区別して対処。

    (2) 「例外」はランタイムに拾わせてアプリケーションを停止させる。無かったことにして、ユーザが作業を続けられるようにすると、大事なデータを壊したりと、強制的に停止させるより好ましからざる状況に陥るかも。

    (3) よほどのことがない限り try-catch は書かない。

    (4) キャッチせざるを得ない場合でも Execption はキャッチしない。キャッチせざるを得ないとしても範囲を絞る。例えば DB 関係の例外が予測されるなら SqlException に限定して catch し、Number プロパティなどでエラーの内容を調べて対処するとか。

    (5) 間違って補足してしまった例外は throw する。(注:catch ブロックでキャッチした例外を throw するとスタックトレースが途切れるので単に throw と書く)

    (6) ユーザーへの通知が必要なら、集約的例外処置を利用する。

    それから、.NET 4 からは破損状態例外は catch できなくなっているそうですが、「それでも Catch (Exception e) を使用するのはよくない」ということについては以下の記事を見てください。

    破損状態例外を処理する
    https://docs.microsoft.com/ja-jp/archive/msdn-magazine/2009/february/clr-inside-out-handling-corrupted-state-exceptions

    自分が持っている Microsoft の本に、上に書いた原則に則ったサンプルが載っていたので紹介しておきます。

    データベースに INSERT する際に PK 制約違反で発生する例外のみ catch して「業務エラー」(2 重登録)としてユーザーに再入力を促し、その他は再 throw してランタイムに拾わせてアプリケーションを停止させるというものです。


    public bool InsertAuthors()
    {
    var connection = new SqlConnection("接続文字列");
    var command = new SqlCommand("INSERT INTO authors VALUES ('172-32-1176', 'White', ...)", connection);
    try
    {
    connection.Open();
    try
    {
    command.ExecuteNonQuery();
    }
    catch(SqlException sqle)
    {
    if (sqle.Number == 2627)
    {
    return false;
    }
    else
    {
    throw;
    }
    }
    }
    finally
    {
    connection.Close();
    }

    return true;
    }

    以下の記事も参考になると思います。

    例外の推奨事項
    https://docs.microsoft.com/ja-jp/dotnet/standard/exceptions/best-practices-for-exceptions

    キャンセル

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

  • ただいまの回答率 87.49%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る