unity ボタンを押したら、、、

Question

ボタンがクリックされたら、csvに書き込むようにしたいのですができなくて困っています。
ここ↓　と２か所書いた場所でできると思っていたのですが、なにがおかしいでしょうか？それとも、unityのボタンのほうに原因があるのでしょうか？

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using Enflux.SDK.Core;
using UnityEngine.UI;
using System;
using System.Linq;
using Enflux.Common.DataTypes;
using Enflux.SDK.Core.Devices;
using Enflux.SDK.DataTypes;
using Enflux.SDK.Extensions;
using Enflux.SDK.Utils;
using Enflux.Shim.Utils;
using System.IO;

public class math : MonoBehaviour
{
    // Assign these in the editor.
    public EnfluxManager EnfluxManager;
    public Humanoid Humanoid;
    // Retrieve limb orientations as Unity local space quaternions.
    private void GetLocalSpaceOrientations()
    {
        var chest = Humanoid.LocalOrientations.Chest.Orientation;
        var leftUpperArm = Humanoid.LocalOrientations.LeftUpperArm.Orientation;
        var leftLowerArm = Humanoid.LocalOrientations.LeftLowerArm.Orientation;
        var rightUpperArm = Humanoid.LocalOrientations.RightUpperArm.Orientation;
        var rightLowerArm = Humanoid.LocalOrientations.RightLowerArm.Orientation;
        var pelvis = Humanoid.LocalOrientations.Pelvis.Orientation;
        var leftUpperLeg = Humanoid.LocalOrientations.LeftUpperLeg.Orientation;
        var leftLowerLeg = Humanoid.LocalOrientations.LeftLowerLeg.Orientation;
        var rightUpperLeg = Humanoid.LocalOrientations.RightUpperLeg.Orientation;
        var rightLowerLeg = Humanoid.LocalOrientations.RightLowerLeg.Orientation;
    }

    // Retrieve limb orientations as real world NED (North-East-Down) space quaternions.
    private string GetNedSpaceOrientations()
    {

        var chest = EnfluxManager.NedOrientations.Chest.Orientation;
        var leftUpperArm = EnfluxManager.NedOrientations.LeftUpperArm.Orientation;
        var leftLowerArm = EnfluxManager.NedOrientations.LeftLowerArm.Orientation;
        var rightUpperArm = EnfluxManager.NedOrientations.RightUpperArm.Orientation;
        var rightLowerArm = EnfluxManager.NedOrientations.RightLowerArm.Orientation;
        var pelvis = EnfluxManager.NedOrientations.Pelvis.Orientation;
        var leftUpperLeg = EnfluxManager.NedOrientations.LeftUpperLeg.Orientation;
        var leftLowerLeg = EnfluxManager.NedOrientations.LeftLowerLeg.Orientation;
        var rightUpperLeg = EnfluxManager.NedOrientations.RightUpperLeg.Orientation;
        var rightLowerLeg = EnfluxManager.NedOrientations.RightLowerLeg.Orientation;


        return "胸 " + chest + "\n左肘 " + leftUpperArm + "\n左手首 " + leftLowerArm + "\n右肘 " + rightUpperArm + "\n右手首 " + rightLowerArm +
            "\n腰 " + pelvis + "\n左膝 " + leftUpperLeg + "\n左足首 " + leftLowerLeg + "\n右膝 " + rightUpperLeg + "\n右足首 " + rightLowerLeg + "\n\n";
    }

    //時間
    void FindSundayOfTheWeek()
    {
        System.DateTime work = System.DateTime.Now;
        Debug.Log(work.ToString());
    }

    //書き出し準備
    private int i = 1;
    StreamWriter sw;
    FileInfo fi;


    bool aa;
    public void Push()
    {
        aa = true;
    }
    public void noPush()
    {
        aa = false;
    }

    void Start()
    {

        FindSundayOfTheWeek();

        //ファイル準備
        fi = new FileInfo(Application.dataPath + "/" + DateTime.Now.ToString("yyyyMMddHHmmss") + ".csv");
        sw = fi.AppendText();

    }

    private float timeleft;
    int log;

    void Update()
    {

        if (i > 10000)
        {
            sw.Close();
            return;
        }
        timeleft -= Time.deltaTime;
        if (timeleft <= 0.0)
        {
            timeleft = 1.0f;

            //処理
            Debug.Log(GetNedSpaceOrientations());
            this.GetComponent<Text>().text = GetNedSpaceOrientations();

            //書き出す

            sw.WriteLine(GetNedSpaceOrientations());
            sw.Flush();

            i++;
        }
    }
}

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using Enflux.SDK.Core;
using UnityEngine.UI;
using System;
using System.Linq;
using Enflux.Common.DataTypes;
using Enflux.SDK.Core.Devices;
using Enflux.SDK.DataTypes;
using Enflux.SDK.Extensions;
using Enflux.SDK.Utils;
using Enflux.Shim.Utils;
using System.IO;

public class math : MonoBehaviour
{
    // Assign these in the editor.
    public EnfluxManager EnfluxManager;
    public Humanoid Humanoid;
    // Retrieve limb orientations as Unity local space quaternions.
    private void GetLocalSpaceOrientations()
    {
        var chest = Humanoid.LocalOrientations.Chest.Orientation;
        var leftUpperArm = Humanoid.LocalOrientations.LeftUpperArm.Orientation;
        var leftLowerArm = Humanoid.LocalOrientations.LeftLowerArm.Orientation;
        var rightUpperArm = Humanoid.LocalOrientations.RightUpperArm.Orientation;
        var rightLowerArm = Humanoid.LocalOrientations.RightLowerArm.Orientation;
        var pelvis = Humanoid.LocalOrientations.Pelvis.Orientation;
        var leftUpperLeg = Humanoid.LocalOrientations.LeftUpperLeg.Orientation;
        var leftLowerLeg = Humanoid.LocalOrientations.LeftLowerLeg.Orientation;
        var rightUpperLeg = Humanoid.LocalOrientations.RightUpperLeg.Orientation;
        var rightLowerLeg = Humanoid.LocalOrientations.RightLowerLeg.Orientation;
    }

    // Retrieve limb orientations as real world NED (North-East-Down) space quaternions.
    private string GetNedSpaceOrientations()
    {
        var chest = EnfluxManager.NedOrientations.Chest.Orientation;
        var leftUpperArm = EnfluxManager.NedOrientations.LeftUpperArm.Orientation;
        var leftLowerArm = EnfluxManager.NedOrientations.LeftLowerArm.Orientation;
        var rightUpperArm = EnfluxManager.NedOrientations.RightUpperArm.Orientation;
        var rightLowerArm = EnfluxManager.NedOrientations.RightLowerArm.Orientation;
        var pelvis = EnfluxManager.NedOrientations.Pelvis.Orientation;
        var leftUpperLeg = EnfluxManager.NedOrientations.LeftUpperLeg.Orientation;
        var leftLowerLeg = EnfluxManager.NedOrientations.LeftLowerLeg.Orientation;
        var rightUpperLeg = EnfluxManager.NedOrientations.RightUpperLeg.Orientation;
        var rightLowerLeg = EnfluxManager.NedOrientations.RightLowerLeg.Orientation;


        return "胸 " + chest + "\n左肘 " + leftUpperArm + "\n左手首 " + leftLowerArm + "\n右肘 " + rightUpperArm + "\n右手首 " + rightLowerArm +
            "\n腰 " + pelvis + "\n左膝 " + leftUpperLeg + "\n左足首 " + leftLowerLeg + "\n右膝 " + rightUpperLeg + "\n右足首 " + rightLowerLeg + "\n\n";
    }

    //時間
    void FindSundayOfTheWeek()
    {
        System.DateTime work = System.DateTime.Now;
        Debug.Log(work.ToString());
    }

    //書き出し準備
    private int i = 1;
    StreamWriter sw;
    FileInfo fi;

    int x = 0;
    public void OnClick()
    {
        x++;
        Debug.Log(x);
        Debug.Log("Button click!");
    }

void Start()
    {

        FindSundayOfTheWeek();

        //ファイル準備
        fi = new FileInfo(Application.dataPath + "/" + DateTime.Now.ToString("yyyyMMddHHmmss") + ".csv");
        sw = fi.AppendText();

    }

    private float timeleft;
    int log;

    void Update()
    {
            if (i > 10000)
            {
                sw.Close();
                return;
            }
            timeleft -= Time.deltaTime;
            if (timeleft <= 0.0)
            {
                timeleft = 1.0f;

                //処理
                Debug.Log(GetNedSpaceOrientations());
                this.GetComponent<Text>().text = GetNedSpaceOrientations();

                //書き出す

                sw.WriteLine(GetNedSpaceOrientations());
                sw.Flush();

                i++;
            }
        }
    }

Comments

[moscow3]

あまりにコードが見にくいので、マークダウンでコード形式にして書くことをおすすめします。

[daityann]

編集しました。できれば回答お願いします。

[Zuishin]

エラーメッセージがありません。

[daityann]

エラーメッセージは出てないのですが。

[Zuishin]

出ていないなら出ていないと書きましょう。「できません」で済ませていいのははおじいさんおばあさんです。具体的にどうなるのかという記述がありません。以上を「エラーメッセージがありません」の一言で理解できると思っていました。

[daityann]

すみません。現在ボタンをクリックした際に書き込みをしたいのですが、On Click時に書き込みをしたいと考えています。しかし、ボタンを押しても変化がないのと、下記で指摘していただいているno functionのところでつまずいています。

Answer 1

現在のボタン設定がスクリーンショットの通りだとすると、何のメソッドも呼ばれません。
（No Function＝何も登録されていない）
なのでコードが正しくても動かないのは当然、というのが１点。

更にコードの方、Update()内ですが、
iの値が初期値(1)のまま変更されていないので、if (i > 10000)以降に入りません。
また、入ったとしても2行後にreturnしているので、書き込み処理に絶対到達しないようになっています。

もっというとこの構成だと、１フレーム毎に書き出しされる可能性がある・Close処理が恐らく正しく行われない・負荷が高い状態です。
if文の組み方を考え直すか、Update()ではなくPush()メソッド内に書き出し処理をそのまま記載すればいいのでは？

---

2017/12/20 13:00　コメントを受けて追記

    public void Push()
    {
            //処理
            Debug.Log(GetNedSpaceOrientations());
            this.GetComponent<Text>().text = GetNedSpaceOrientations();

            //書き出す
            sw.WriteLine(GetNedSpaceOrientations());
            sw.Flush();
    }

上と同じことを書きますが、こんな感じでPushメソッド内に書き出し処理を入れて
ボタンのOnClickイベント（今No Functionになっているところ）で「Push」メソッドを選択すれば動作するのではないかと思います。
（ファイルを閉じる処理をどこで行うべきかは要件や仕様によるので入れていないです）

まずはボタンの基本的な使い方を学習した方がいいかもしれません。
【uGUI】Buttonの使い方 - Qiita

ファイル書き出しのソースコードは精査していませんが、強いて言うならCSVなのに改行区切り？と思います。空行も含むようになっていますし。
ログとして出したいだけ、テストとして今はこう書いている、というなら問題無いのですが少し気になりました。

Comments

[daityann]

回答ありがとうございます。
No Functionの件ですが赤いボタンを押したときに書き込まれるようにしたいのですが、何を入れるのがいいのかよくわからなくて。

あと、プログラムの書き込み処理の件ですが、こちらは解決したと思いますが、プログラム改善するなら何処を直したらいいですか？

[sakura_hana]

回答を編集しました。

[daityann]

編集しましたがエラーが起こっています。エラー部は写真の通りです

[sakura_hana]

＞ボタンについて
Debug.Log("Button click!");は表示されていますか？
もしされていないなら、リンク先のページを見ながらもう一度設定を行ってください。

＞エラーについて
エラーが出た時はエラー文をそのまま検索しましょう。
「IOException: Sharing violation on path」で検索すると以下のような情報がヒットします。
http://kamikuzu-tips.blogspot.jp/2015/10/csv.html
もし改善しない場合は自分で調べてください。

２番目のエラーは
this.GetComponent<Text>().text = GetNedSpaceOrientations();
に問題があります。「このスクリプトが付いているGameObjectにTextコンポーネントが付いていない」ことが原因です。この行をコメントアウトするか、Textコンポーネントを付けてください。

３番目のエラーは
sw.WriteLine(GetNedSpaceOrientations());
に問題があります。
Updateは最初の１回はStartよりも早く発動する為、Start内の「sw = fi.AppendText();」が呼ばれるより前に「sw.WriteLine〜」が呼ばれているものと思われます。
if (sw != null) {
sw.WriteLine(GetNedSpaceOrientations());
sw.Flush();
}
こうすれば恐らくエラーは消えます。

＞ソースコードについて
自分で書いたスクリプトが何をしているのか、理解していますか？
このコードだと「１秒毎に書き出し、1万回書き出ししたらファイルを閉じる」
「ボタンを押す度xが加算される（そのxは他のどこにも使われていない）」という処理になります。

ちなみに私が回答として追記したコードを使えば、Update内の処理は全て不要になります。