テキストボックスに数値を入れてエンターキーを押すとエラーが出る

Question

unityで計算ツールを作っています。
いくつかのテキストボックスがあり、それらに数値を入力するとテキストに表示されるような感じです
実行してテキストボックスに数値を入力してエンターキーを押すと以下のエラーメッセージが発生しました。

発生している問題・エラーメッセージ

FormatException: Input string was not in a correct format.
System.Number.ParseSingle (System.String value, System.Globalization.NumberStyles options, System.Globalization.NumberFormatInfo numfmt)

エラーメッセージ

該当のソースコード

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.UI;

public class kkk : MonoBehaviour
{


    public Text Fomula;

     InputField hougubairitu;
     InputField classaisyou;
     InputField tentijin;
     InputField boost;
     InputField classhosei;
     InputField cardhosei;
     InputField ATK;
     InputField reisouATK;
     InputField kougeki;
     InputField bougyo;
     InputField iro;
     InputField irodebahu;
     InputField hougu;
     InputField tokkoubahu;
     InputField tokkouhougu;
     InputField koteidamage;
     InputField tokusyutaisei;


    // Start is called before the first frame update
    void Start()
    {
        hougubairitu = GameObject.Find("hougubairitu").GetComponent<InputField>();
        classaisyou = GameObject.Find("classaisyou").GetComponent<InputField>();
        tentijin = GameObject.Find("tentijin").GetComponent<InputField>();
        boost = GameObject.Find("boost").GetComponent<InputField>();
        classhosei = GameObject.Find("classhosei").GetComponent<InputField>();
        cardhosei = GameObject.Find("cardhosei").GetComponent<InputField>();
        ATK = GameObject.Find("ATK").GetComponent<InputField>();
        reisouATK = GameObject.Find("reisouATK").GetComponent<InputField>();
        kougeki = GameObject.Find("kougeki").GetComponent<InputField>();
        bougyo = GameObject.Find("bougyo").GetComponent<InputField>();
        iro = GameObject.Find("iro").GetComponent<InputField>();
        irodebahu = GameObject.Find("irodebahu").GetComponent<InputField>();
        hougu = GameObject.Find("hougu").GetComponent<InputField>();
        tokkoubahu = GameObject.Find("tokkoubahu").GetComponent<InputField>();
        tokkouhougu = GameObject.Find("tokkouhougu").GetComponent<InputField>();
        koteidamage = GameObject.Find("koteidamage").GetComponent<InputField>();
        tokusyutaisei = GameObject.Find("tokusyutaisei").GetComponent<InputField>();


        Fomula = GameObject.Find("Fomula").GetComponent<Text>();
    }

   


    public void CalcTotal()
    {
        float input1 = float.Parse(hougubairitu.text);
        float input2 = float.Parse(classaisyou.text);
        float input3 = float.Parse(tentijin.text);
        float input4 = float.Parse(boost.text);
        float input5 = float.Parse(classhosei.text);
        float input6 = float.Parse(cardhosei.text);
        float input7 = float.Parse(ATK.text);
        float input8 = float.Parse(reisouATK.text);
        float input9 = float.Parse(kougeki.text);
        float input10 = float.Parse(bougyo.text);
        float input11 = float.Parse(iro.text);
        float input12 = float.Parse(irodebahu.text);
        float input13 = float.Parse(hougu.text);
        float input14 = float.Parse(tokkoubahu.text);
        float input15 = float.Parse(tokkouhougu.text);
        float input16 = float.Parse(koteidamage.text);
        float input17 = float.Parse(tokusyutaisei.text);


        
        float Total = (input7 + input8 * (input1/100) * input6 * (1.0f +(input11/100)+(input12/100)) * input5 * input2 * input3 * 1 * 0.23f * (1.0f + (input9/100)+(input10/100))*(1.0f - (input17/100))*(1.0f + (input4/100)+(input14/100)+(input13/100)) * (input15/100) * input16);

       


        Fomula.text = Total.ToString();

Answer 1

テキストボックスに入力した値は本当にすべて数値でしたか？float.Parse(string s)は文字列sをfloat型に変換するメソッドですが、floatは数値を扱う型ですので、sが123aとかだと数値に変換することができず、エラーとなってしまいます。FormatExceptionは書式が正確でないときに発生するエラーです。

Answer 2

        float input1 = float.Parse(hougubairitu.text);
        float input2 = float.Parse(classaisyou.text);
        float input3 = float.Parse(tentijin.text);
        float input4 = float.Parse(boost.text);
        float input5 = float.Parse(classhosei.text);
        float input6 = float.Parse(cardhosei.text);
        float input7 = float.Parse(ATK.text);
        float input8 = float.Parse(reisouATK.text);
        float input9 = float.Parse(kougeki.text);
        float input10 = float.Parse(bougyo.text);
        float input11 = float.Parse(iro.text);
        float input12 = float.Parse(irodebahu.text);
        float input13 = float.Parse(hougu.text);
        float input14 = float.Parse(tokkoubahu.text);
        float input15 = float.Parse(tokkouhougu.text);
        float input16 = float.Parse(koteidamage.text);
        float input17 = float.Parse(tokusyutaisei.text);

上記、全てに数値が入っていないと、入れていない箇所のfloat.Parse()が行えず、例外（エラー）が発生します。

Comments

[hayamoe]

ありがとうございます!

Answer 3

エラー内容が全てのInputFieldに値が入力されていないがゆえに起きてるものでした
正常に動きました。
皆さま方ありがとうございました。

Answer 4

public static float ToFloat(string s) {
    float f;
    if (float.TryParse(s, out f)) {
        return f;
    } else {
        return 0f;
    }
}

こんな関数を作って、float.Parse と差し替え、float への変換が失敗したらゼロを返し、とりあえず動くようにしてはどうでしょうか？

Comments

[hayamoe]

float.Parseは何に置き換えれば良いのでしょうか？

[KOZ6.0]

作った関数です。

[hayamoe]

すみません初心者なので何もわかりません...
どうか教えていただけますでしょうか。

[KOZ6.0]

関数というからわからないんでしょうか？初心者でもメソッドを作ったりしますよね？
float.Parse だと、変換できない文字が入ると異常終了するので、同じ機能（文字を float に変換する）をもった、異常終了しないメソッドを作って、float.Parse の代わりに使うということです。
私が提示したコードは、そのメソッドの一例です。

[hayamoe]

つまり「s」の部分を変えればいいってことですかね...?

[KOZ6.0]

どう変えるんですか？

[fiveHundred]

> KOZ6.0さん
横やりで失礼しますが、「数値が入っていない文字列」や「数値ではない文字列」が存在している場合のことを考慮していますか？
現状では「エラーが出ないからfloat.TryParse()を使う」という意図しか見えず、本当にそうなのであれば「変換に失敗しているのに無理やり動かしている」ということになり不適切です。
値fが「関数呼び出し前と変わらない」「強制的に0になる」「不定」のどれにあたるかは経験者でもドキュメント見ないと分からないレベルです。
「失敗したらCalcTotal()そのものを終了させる」という処理が一番だと思いますが、そうでなくとも「失敗したら0にする」ことが明確なコードにしたほうが適切です。

[KOZ6.0]

out 引数は言語仕様的に「関数呼び出し前と変わらない」「不定」とかありえないですが、ゼロが返ってくるというのはたしかにマニュアルを読まないとわからないので確かに不親切ですね。回答を書き換えておきます。