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

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

ただいまの
回答率

91.26%

  • C#

    5003questions

    C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

  • Unity

    2465questions

    Unityは、ユニティテクノロジーが開発したゲームエンジンです。 主にモバイルやブラウザ向けのゲーム製作に利用されていましたが、3Dの重力付きゲームが簡単に作成できることから需要が増え、現在はマルチプラットフォームに対応しています。 言語はC言語/C++で書かれていますが、C#、JavaScript、Booで書かれたコードにも対応しています。

  • Arduino

    366questions

    Arduinoは、AVRマイコン、単純なI/O(入出力)ポートを備えた基板、C言語を元としたArduinoのプログラム言語と、それを実装した統合開発環境から構成されたシステムです。

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

受付中

回答 2

投稿

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

moyas

score 0

前提・実現したいこと

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/ツール等のバージョンなど)

  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

+2

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

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 が取得できなかったので、それ相応の処理
}

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

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/12/27 13:14

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

    キャンセル

0

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


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

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/12/26 18:48

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

    キャンセル

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

ただいまの回答率

91.26%

関連した質問

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

  • C#

    5003questions

    C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

  • Unity

    2465questions

    Unityは、ユニティテクノロジーが開発したゲームエンジンです。 主にモバイルやブラウザ向けのゲーム製作に利用されていましたが、3Dの重力付きゲームが簡単に作成できることから需要が増え、現在はマルチプラットフォームに対応しています。 言語はC言語/C++で書かれていますが、C#、JavaScript、Booで書かれたコードにも対応しています。

  • Arduino

    366questions

    Arduinoは、AVRマイコン、単純なI/O(入出力)ポートを備えた基板、C言語を元としたArduinoのプログラム言語と、それを実装した統合開発環境から構成されたシステムです。