同じオブジェクトがC＃で何度もインスタンス化されています

Question

私はコーディングの初心者です。私は数日間、人気のモバイルゲーム「Crossy Road」を再作成しようとしていました。私はLevel Generationスクリプトを作成しようとしていましたが、問題が発生しました。私が直面している問題は、3つのオブジェクトをランダムにインスタンス化しようとすると（この場合はGrass、Road and Water）、草だけが生成されるということです。なぜ誰かが 'Grass'オブジェクトを何度もインスタンス化し続ける理由を教えてもらえれば、とても感謝しています。私はUnity 5 fyiでゲームを作り直しています。次のようにコードがある -

using UnityEngine; 
using System.Collections; 

public class LevelGenerationScript : MonoBehaviour { 

public GameObject Water; 
public GameObject Road; 
public GameObject Grass; 
int firstRand; 
int secondRand; 
int distPlayer = 10; 

Vector3 intPos = new Vector3(0,0,0); 

void Update() 
{ 
    if (Input.GetButtonDown("up")) 
    { 
     firstRand = Random.Range(1,4); 
     if(firstRand == 1) 
     { 
      secondRand = Random.Range(1,8); 
      for(int i = 0;i < secondRand; i++) 
      { 
       intPos = new Vector3(0,0,distPlayer); 
       distPlayer += 1; 
       GameObject GrassIns = Instantiate(Grass) as GameObject; 
       GrassIns.transform.position = intPos; 
      } 

      if(firstRand == 2) 
      { 
       secondRand = Random.Range(1,8); 
       for(int i = 0;i < secondRand; i++) 
       { 
        intPos = new Vector3(0,0,distPlayer); 
        distPlayer += 1; 
        GameObject RoadIns = Instantiate(Road) as GameObject; 
        RoadIns.transform.position = intPos; 
       } 

       if(firstRand == 3) 
       { 
        secondRand = Random.Range(1,8); 
        for(int i = 0;i < secondRand; i++) 
        { 
         intPos = new Vector3(0,0,distPlayer); 
         distPlayer += 1; 
         GameObject WaterIns = Instantiate(Water) as GameObject; 
         WaterIns.transform.position = intPos; 
        } 
       } 
      } 
     } 
    } 
} 
} 

誰かが私に間違いを伝えることができれば、私はそれを本当に感謝します。 ありがとう！

Answer 1

if (firstRand == 2)は、if (firstRand ==1)ステートメントに含まれているため、決して届きません。あなたが動作するはずです以下のコードを改善したい場合は

if (firstRand == 1) 
{ 
    ... 
} 

if (firstRand == 2) 
{ 
    ... 
} 

if (firstRand == 3) 
{ 
    ... 
} 

：

あなたはこのようなあなたのifステートメントを構築する必要が

そのコードに改善することができる多くのものがあります

firstRand = Random.Range(1,4); 
secondRand = Random.Range(1,8); 
GameObject instance = null; 

for (int i = 0; i < secondRand; i++) 
{ 
    intPos = new Vector3(0, 0, distPlayer); 
    distPlayer += 1; 

    switch (firstRand) 
    { 
     case 1: 
      instance = Instantiate(Grass) as GameObject; 
      break; 
     case 2: 
      instance = Instantiate(Road) as GameObject; 
      break; 
     case 3: 
      instance = Instantiate(Water) as GameObject; 
      break; 
    } 

    instance.transform.position = intPos; 
} 

Answer 2

firstRandが1のときに呼び出されるコードグループ内の他のオブジェクトのコードを配置しました。（FirstRand == 2）と（FirstRand == 3）はこの位置にないため、到達不能コード。