Unity C＃ - このコードを修正するにはどうすればよいですか？

Question

私はからのコードをコピー：https://forum.unity3d.com/threads/clean-est-way-to-find-nearest-object-of-many-c.44315/

しかし、このコードを：

Transform GetClosestCube (Transform[] cubes); 

    void GetClosestCube() 
    { 
     { 
      Transform bestTarget = null; 
      float closestDistanceSqr = Mathf.Infinity; 
      Vector3 currentPosition = transform.position; 
      foreach(Transform potentialTarget in cubes) 
      { 
       Vector3 directionToTarget = potentialTarget.position - currentPosition; 
       float dSqrToTarget = directionToTarget.sqrMagnitude; 
       if(dSqrToTarget < closestDistanceSqr) 
       { 
        closestDistanceSqr = dSqrToTarget; 
        bestTarget = potentialTarget; 
       } 
      } 

      return bestTarget; 
     } 
    } 

それはエラーを与える：

Assets/Player.cs(29,12): error CS0501: `Player.GetClosestCube(Transform[])' must have a body because it is not marked abstract, extern, or partial

そして、私はそれがスレッドにあった正確にどのようにコードを使用する場合：

void GetClosestCube() 
    { 
     { 
      Transform GetClosestCube (Transform[] cubes) 
      { 
      Transform bestTarget = null; 
      float closestDistanceSqr = Mathf.Infinity; 
      Vector3 currentPosition = transform.position; 
      foreach(Transform potentialTarget in cubes) 
      { 
       Vector3 directionToTarget = potentialTarget.position - currentPosition; 
       float dSqrToTarget = directionToTarget.sqrMagnitude; 
       if(dSqrToTarget < closestDistanceSqr) 
       { 
        closestDistanceSqr = dSqrToTarget; 
        bestTarget = potentialTarget; 
       } 
      } 

      return bestTarget; 
     } 
     } 
    } 

Error: Assets/Player.cs(34,28): error CS1525: Unexpected symbol (', expecting ,', ;', or ='

Answer 1

Unity Frameworkを使用してプログラムしたことがないので、私はおそらくここに投稿するべきではないでしょう。私はこれが有効なC＃コードのようには見えません。

 Transform GetClosestCube (Transform[] cubes) 
     { 
      Transform bestTarget = null; 
      float closestDistanceSqr = Mathf.Infinity; 
      Vector3 currentPosition = transform.position; 
      foreach (Transform potentialTarget in cubes) 
      { 
       Vector3 directionToTarget = potentialTarget.position - currentPosition; 
       float dSqrToTarget = directionToTarget.sqrMagnitude; 
       if (dSqrToTarget < closestDistanceSqr) 
       { 
        closestDistanceSqr = dSqrToTarget; 
        bestTarget = potentialTarget; 
       } 
      } 

      return bestTarget; 
     } 

もっと正確に見えます。このメソッドは、最も近いキューブを取得しようとしています。つまり、あなたはそれが無効になる（または何も返されない）ことを望みません。あなたは、あなたのメソッドがあなたが配列で送信したキューブの1つ、好ましくは最も近いものを返すようにします。そのため、戻り型のTransform（キューブなので）があります。

私はこのメソッドをキューブと関係のあるクラスに含めたいと思うでしょう。

これはReadyPlayerOne（悪いクラス名、良い本）と呼ばれるクラスにあって、ジョブはあなたのコードがReadyPlayerOneのインスタンスに最も近いキューブを教えてくれたとしましょう。

public class ReadyPlayerOne { 
    Transform GetClosestCube (Transform[] cubes) 
    { 
     Transform bestTarget = null; 
     float closestDistanceSqr = Mathf.Infinity; 
     Vector3 currentPosition = transform.position; 
     foreach (Transform potentialTarget in cubes) 
     { 
      Vector3 directionToTarget = potentialTarget.position - currentPosition; 
      float dSqrToTarget = directionToTarget.sqrMagnitude; 
      if (dSqrToTarget < closestDistanceSqr) 
      { 
       closestDistanceSqr = dSqrToTarget; 
       bestTarget = potentialTarget; 
      } 
     } 

     return bestTarget; 
    } 
} 

次に、インスタンスを新規作成して呼び出すことができます。

var playerone = new ReadyPlayerOne(); 
var closestCube = playerone.GetClosestCube(cubes); 

私はこれが役に立てば幸い:)