宣言した配列の上書き方法

Question

前提・実現したいこと

宣言した配列の中身を上書きする方法が知りたいです。
int[] ply = new int[] { 9, 9, 9, 9, 9, 9, 9, 9, 9 };
この中身を、上書きしたいです。
LogScriptで上書きした配列を使用したいです。

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

LogScriptの表示結果が　9,9,9 となってしまう。

該当のソースコード

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

public class BCheck : MonoBehaviour
{
public GameObject[] buttons;

bool aji;

int i;
int[] ply = new int[] { 9, 9, 9, 9, 9, 9, 9, 9, 9 };

// Start is called before the first frame update
void Start()
{       
    aji = true;
}
// Update is called once per frame
void Update()
{
           
    //プレイヤー配列作成---------------------------------
    if (aji)
    {
        if (Input.GetKeyDown(KeyCode.Alpha0))
        {
            ply[i] = 0;
            i = i + 1;
           
        }
        if (Input.GetKeyDown(KeyCode.Alpha1))
        {
            ply[i] = 1;
            i = i + 1;
        }
        if (Input.GetKeyDown(KeyCode.Alpha2))
        {
            ply[i] = 2;
            i = i + 1;
        }
        if (Input.GetKeyDown(KeyCode.Alpha3))
        {
            ply[i] = 3;
            i = i + 1;
        }
        if (Input.GetKeyDown(KeyCode.Alpha4))
        {
            ply[i] = 4;
            i = i + 1;
        }
        if (Input.GetKeyDown(KeyCode.Alpha5))
        {
            ply[i] = 5;
            i = i + 1;
        }
        if (Input.GetKeyDown(KeyCode.Alpha6))
        {
            ply[i] = 6;
            i = i + 1;
        }
        if (Input.GetKeyDown(KeyCode.Alpha7))
        {
            ply[i] = 7;
            i = i + 1;
        }
        if (Input.GetKeyDown(KeyCode.Alpha8))
        {
            ply[i] = 8;
            i = i + 1;
        }
        if (i == 8)
        {            
            aji = false;
            
        }



    }

    //プレイヤー配列作成ここまで------------------------------------   
    

}
}

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using System.IO;

public class LogScript : MonoBehaviour
{
ButtonMane script;

int[] log = new int[] { 9, 9, 9, 9, 9, 9, 9, 9, 9 };

string datetimeStr1 = System.DateTime.Now.ToString("yyyy年MM月dd日");
string datetimeStr2 = System.DateTime.Now.ToString("yyyy年MM月dd日HH時mm分ss秒");

bool ikr;
private void Start()
{
    ikr = true;
    for (int i = 0; i < 9; i++)
    {
        log[i] = BCheck.ply[i];
    }

private void LateUpdate()
{
if (ikr)
{
for (int i = 0; i < 5; i++)
{
pro[i] = ButtonMane.ply[i];
}

        if (File.Exists("log") == false)
        {
            Directory.CreateDirectory("log");
        }

        if (File.Exists("log/" + datetimeStr1 + "") == false)
        {
            Directory.CreateDirectory("log/" + datetimeStr1 + "");
        }
        StreamWriter sw;
        sw = new StreamWriter("log/" + datetimeStr1 + "/" + datetimeStr2 + ".csv", true);
        sw.WriteLine("test" + 100);
        for (int i = 0; i < 9; i++)
        {
            sw.WriteLine(log[i]);
        }
       
        sw.Close();
        ikr = false;
    }

}

C#

試したこと

ネットで調べてみましたが、わかりませんでした。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Comments

[y_waiwai]

ふつーに上書きするのでいけませんか？
問題点は何でしょうか

[退会済みユーザー]

他のスクリプトで配列を使用するときに、定義した順番になってしまうことが問題点です。

[y_waiwai]

定義した順番とは？
質問を編集してそこらへんのことを詳しく説明しましょう

[退会済みユーザー]

言葉足らずで申し訳ないです。

[k.matsuda]

新しい質問を投稿されたようですので、こちらは解決済みにしておいて下さいね。

Answer 1

いまいち問題がなにかわかってませんが、、
はやいはなし、LateUpdate関数が呼ばれる前にWriteLog関数を呼んでるだけなのでは。

VisualStudioを使ってるなら、ブレークポイントを設定して実行を止め、ワンステップづつ変数の内容をモニタしながら実行できるので、そうやって動作を確認してはどうでしょうか

Comments

[退会済みユーザー]

ありがとうございます。

Answer 2

log[0] = BCheck.ply[0];
log[1] = BCheck.ply[1];
log[2] = BCheck.ply[2]; 

aryじゃなくてplyでは？

追記
他にも色々問題ありそうな気がしますが・・・。

追記

void Start()
{
    aji = true;
    int i = 0;
    int[] ply = new int[] { 9, 9, 9, 9, 9, 9, 9, 9, 9 }; 
}

こちらで初期化してみるのはどうでしょうか？
あと、i の初期化も必要ですね。

Comments

[退会済みユーザー]

すみません。そうですね。ありがとうございます。

[k.matsuda]

BCheckが、実行されているか確認してみてください。

さらに、キーが押されたとき、それぞれのif文が実行されているか確認してください。

[退会済みユーザー]

BCheckが実行されていることが、確認出来ました。
キーを押したif文も実行されていることが、確認出来ました。

[k.matsuda]

色々試行錯誤されているようですが、
キーが押されたら、ログを記録するスクリプトを呼び出す。
という方法のほうがいい気がします。

[退会済みユーザー]

確かにそっちのほうが良さそうですね。

Answer 3

ply = new int[] { 8, 8, 9, 9, 9, 9, 9, 9, 9 };

Comments

[Zuishin]

この回答で解決しないなら質問が間違っているのでしょう。書き直してください。