C#の入力の例外処理がうまくいかない

Question

入力された数字を処理するシステムを作りたいと考えています。この時入力されたものが数字かどうかを判断し、”hoge”などの数字以外が入力されたとき-1を返して止まる例外処理を作りたいです。そのためtry_catch文を使いました。入力lineをint型にキャストし、エラーFormatExceptionを吐いたら-1を返して止まるコードを作成しました。コメントアウトされているvar line = "hoge";で試してみるとうまく作動したのですが、var line = 1;の時
「引数 1: は 'int' から 'System.ReadOnlySpan<char>' へ変換することはできません」
とエラーを吐かれ実行できません。これも引数による型違いのエラーなためFormatExceptionの例外処理で止まると考えたのですが、なぜ実行できないのでしょうか？対策を教えてください。

        //var line = Console.ReadLine();
        //var line = "hoge";
        var line = 1;
        if (line == null)
        {
            Console.WriteLine(-1);
            return;
        }
        try
        {
            int int_test = int.Parse(line);//
        }
        catch (FormatException)
        {
            Console.WriteLine(-1);//hogehoge対策
            return;
        }

Comments

[退会済みユーザー]

質問者さん、無言ですが、回答したのでそれに対するフィードバックを書いてください。役に立った/立たなかったぐらいはすぐに返せるのでは？　役に立たなかったならどこがダメだったかを書くと、より期待するものに近い回答が出てくるかも。とにかく無言は NG です。

[退会済みユーザー]

「やってることがメチャクチャ」と言われて逆ギレして私の回答に低評価を付けたのかな？　議論する価値がないほどメチャクチャなコードを書いているにも関わらず、質問を読んでもらえて、かつ問題点と対策を書いてもらえたことに感謝すべきと思うけど。

Answer 1

やってることがメチャクチャという感じです。

コード (var line = 1;) を拝見すると line は int 型です。なので、まず、

if (line == null)

がダメですし、さらに、

int int_test = int.Parse(line);

というのもあり得ません。

いったい何をしたいのでしょう？

---

【追記】

コンソール入力なんですよね。コンソールから 1 と入力すると、

var line = Console.ReadLine();

で変数 line に取得できるのは文字列で "1" です。123 入力なら "123" になります。そこをよく考えてください。

あと、パースできるか否かは Int32.Parse と try -catch でやるのではなく、Int32.TryParse メソッドを使うことをお勧めします。

Int32.TryParse メソッド
https://docs.microsoft.com/ja-jp/dotnet/api/system.int32.tryparse?view=net-6.0

Comments

[退会済みユーザー]

「やってることがメチャクチャ」と言われて逆ギレして低評価を付けたのかな？　議論する価値がないほどメチャクチャなコードを書いているのにも関わらず、それを読んでもらえて、かつ問題点と対策を書いてもらえたことに感謝すべきと思うけど。

Answer 2

Consoleで受け取った入力をforeachで回して数字以外が入っていたら-1を返す関数を作ることで、入力の例外処理を作成しました。

var line = Console.ReadLine();
foreach (var t in line)
        {
            int test = ctoi(t);
            if (test == -1)
            {//数字でなかったら
                Console.WriteLine(-1);
                return;
            }
        }
 public static int ctoi(char c)
    {
        switch (c)
        {
            case '0': return 0;
            case '1': return 1;
            case '2': return 2;
            case '3': return 3;
            case '4': return 4;
            case '5': return 5;
            case '6': return 6;
            case '7': return 7;
            case '8': return 8;
            case '9': return 9;
            default: return -1;
        }
    }

Comments

[退会済みユーザー]

普通の意味で「解決」というにはあまりにひどい内容なのでマイナス評価。" -322 " とか "+4302" とかはどうするのかな？