どうnullチェックしていいか分かりません

Question

###前提・実現したいこと

Arduinoの値を使用してUnityでじゃんけんを実装するシステムを作っているのですが、
その実装中に以下の2つのエラーメッセージが発生しました。
これにより、debug.logにnullが表示されているためそのnull表示を消したい思っております。
nullチェックをこのコードに組み込めればいいと思うのですが、どう組み込めばいいか全くわかりません。
どなたかご教授いただけると幸いです。
###発生している問題・エラーメッセージ

NullReferenceException: Object reference not set to an instance of an object
jankenn.Update () (at Assets/janken/jankenn.cs:46)

ArgumentNullException: Argument cannot be null.
Parameter name: s
System.Int32.Parse (System.String s) (at /Users/builduser/buildslave/mono/build/mcs/class/corlib/System/Int32.cs:629)
jankenn.Update () (at Assets/janken/jankenn.cs:41)

###該当のソースコード
using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.UI;

public class jankenn : MonoBehaviour {
public Text jankenLabel;

public static int janken; 
private string key = "JANKEN"; 

public static int getJanken()
{
	return janken;
}


void Start()
{		
	janken = PlayerPrefs.GetInt(key, 0);	
}


public void Update()
{

	string mageatai = SerialTest.serial.GetData();
	var intmageatai = int.Parse(mageatai); 


	if (intmageatai >= 800)
	{
		jankenLabel.text = "パー".ToString();
	}
		

	if (intmageatai >= 600 && intmageatai < 800)
	{
		jankenLabel.text = "チョキ".ToString();
	}
	if (intmageatai < 600 )
	{
		jankenLabel.text = "グー".ToString();
	}




}

}
###試したこと

###補足情報(言語/FW/ツール等のバージョンなど)

Answer 1

※ソースはコードブロックで囲んでください

nullチェックをこのコードに組み込めればいいと思うのですが、どう組み込めばいいか全くわかりません。

int.Parse は引数が null だけでなく他にも失敗する場合があります（ string.Empty や int の範囲を超える値の場合など）ので、null をチェックするだけでは足りません。
int.TryParse のほうが良いでしょう。

Int32.TryParse メソッド (String, Int32)

---

string mageatai = SerialTest.serial.GetData();
var intmageatai = int.Parse(mageatai);
if (intmageatai >= 800)
// 以下略

ここを、

string mageatai = SerialTest.serial.GetData();
int intmageatai = 0;
if (int.TryParse(mageatai, out intmageatai))
{
    // 正常に intmageatai が取得できたので、手の判断
    if (intmageatai >= 800)
    // 以下略
}
else
{
    // intmageatai が取得できなかったので、それ相応の処理
}

こんな感じにしてみてはどうでしょうか。

Comments

[moyas]

回答ありがとうございます！

Answer 2

Arduinoは詳しくないのですが、nullチェックをするには、if (mageatai == null)とかif (mageatai != null)のように、単純にnullと比較すればよいです。
そしてエラーを回避するためには、この方法を使って、nullで処理できない処理を実行しないようにすればよいです。

---

ソースコードは「```」×2で囲うと見やすくなります。
また、この内容だとどこでエラーが起こっているのか分からないので、記載するようにしてください。

Comments

[moyas]

回答ありがとうございます！
その方法で試してみたいと思います。
後、ご指摘ありがとうございます。今後そうします。
今回質問初めてしましてやり方あまり分からなかったので助かります。