Unityで指定した位置からキャラクターをランダムに生成

Question

前提・実現したいこと

Unityで指定した位置からキャラクターをランダムに生成したいのですが、
以下の2つのエラーが発生しています。

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

Assets\Script\List.cs(23,66): error CS1061: 'Vector3' does not contain a definition for 'Count' and no accessible extension method 'Count' accepting a first argument of type 'Vector3' could be found (are you missing a using directive or an assembly reference?)

Assets\Script\List.cs(23,32): error CS0165: Use of unassigned local variable 'position'

該当のソースコード

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

public class List : MonoBehaviour
{
    public GameObject obj;

    public List<Transform> positions;

    // 経過時間
    private float time;

    // Update is called once per frame
    void Update()
    {
        // 前フレームからの時間を加算していく
        time = time + Time.deltaTime;

        // 約1秒置きにランダムに生成されるようにする。
        if (time > 1.0f)
        {
            Vector3 position = positions[Random.Range(0, position.Count)].position;
            Instantiate(obj, position, Quaternion.identity);

            // 経過時間リセット
            time = 0f;
        }
    }

Comments

[1T2R3M4]

質問は何でしょうか。
また、
調べたこと、試したこと、デバッグした内容を質問に追記していただけませんか。

[such22]

現状、
CS1061「存在しないメソッドを呼び出そうとしたときや、存在しないクラス メンバーにアクセスしようとしたときに発生する」
CS0165「未割り当てのローカル変数の使用」というところまでは調べている段階です。

Answer 1

Vector3 position = positions[Random.Range(0, position.Count)].position;

この部分ですが

Vector3 position = positions[Random.Range(0, positions.Count - 1)].position;

としてはどうでしょうか？
修正点

• positionをpositionsに
• indexの最大値 = 要素数 - 1なので　position.Count - 1　にする

追記

    [SerializeField]private GameObject obj;

    //public List<Transform> positions;
    [SerializeField]private List<Vector3> positions = new List<Vector3>(){
        //初期値を設定
        new Vector3(0f,0f,0f),new Vector3(1f,0f,0f),new Vector3(0f,1f,0f),new Vector3(0f,0f,1f)
    };
    private float time;

    void Update()
    {
        //time = time + Time.deltaTime;
        time += Time.deltaTime;
        
        float limitTime = 1.0f;
        if (time >　limitTime)
        {
            //Vector3 position = positions[Random.Range(0, positions.Count -1)];
            Vector3 position = positions[Random.Range(0, positions.Count)];
            GameObject target = Instantiate(obj, position, Quaternion.identity);
            time = 0f;
        }
    }

修正点
もしインスペクター上に入力項目を表示したいだけならpublic修飾子を付けるのではなく[SerialisedField]を使ったほうがいいと思います(スコープはprivateのままでinspector上に表示する)

objの生成場所のみを指定したいのならTransform型を使うのではなくVector3型を使うだけで十分かと思います。(Transform.positionと同じ型)

リストに初期値を一応設定していますがinspector上で値を入力するなら消しても構わないです({}の部分)script上で記入する場合は逆にinspector上で変更されないように[SerializedField]を消してください

timeの部分はこちらのほうが見やすいと思ってこうしました。やってることは同じです。

1.0fと記入するよりlimitTimeという変数を用いたほうが可読性が高くなると思います

[訂正] Random.Rangeのint型を対象としたものは010の範囲を設定したとき実際には09の中の整数を選択するようなのでRandom.Range(0, positions.Count)が正しいです。すみません...

一応Instantiateの部分「GameObject target =」を付け加えました。今後obj(正確にはobjのクローン体)を削除したいときはこいつをうまく使ってください。

Comments

[such22]

回答いただきありがとうございます。
ご教示いただいた通りにすると、エラーは消えたのですが、
キャラクターは生成されないままです。
public List<Transform> positions;
の下に何か記述する必要があるのでしょうか？

[sphim13]

回答を修正しました。これでどうでしょうか？

[such22]

回答いただきありがとうございます。

GameObject[] target = Instantiate(obj, position, Quaternion.identity);
の部分に以下のようなエラーが現れました。

Assets\Script\Appear.cs(31,35): error CS0029: Cannot implicitly convert type 'UnityEngine.GameObject' to 'UnityEngine.GameObject[]'

[sphim13]

あ、すみません。間違えました。型はGameObjectです。しかし、これだと何回も内容を上書をしてしまいます。もしリストや配列を使って管理したいならList＜GameObject＞やGameObject[]をつかってください。

[such22]

ありがとうございます！
無事できました！

Answer 2

Listっていうクラス名はやめましょう。
ソースコード內で利用している通り、Listクラスは既に存在しているので重複します。