質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
87.20%
C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

Unity

Unityは、ユニティテクノロジーが開発したゲームエンジンです。 主にモバイルやブラウザ向けのゲーム製作に利用されていましたが、3Dの重力付きゲームが簡単に作成できることから需要が増え、現在はマルチプラットフォームに対応しています。 言語はC言語/C++で書かれていますが、C#、JavaScript、Booで書かれたコードにも対応しています。

解決済

スクリプトがif文ばかりになる

lkiuxc
lkiuxc

総合スコア29

C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

Unity

Unityは、ユニティテクノロジーが開発したゲームエンジンです。 主にモバイルやブラウザ向けのゲーム製作に利用されていましたが、3Dの重力付きゲームが簡単に作成できることから需要が増え、現在はマルチプラットフォームに対応しています。 言語はC言語/C++で書かれていますが、C#、JavaScript、Booで書かれたコードにも対応しています。

3回答

4評価

8クリップ

7241閲覧

投稿2018/10/04 11:24

こんにちは。まだラムダ式などが良く解らず勉強中なのですが、趣味でUnityを使いゲームを作っています。

早速本題に。スクリプトを使いプレイヤーを動かしたりしているのですが、if文がとても多いんです。例えば

using System.Collections; using System.Collections.Generic; using UnityEngine; public class EnemyTestRoutine : MonoBehaviour { [SerializeField] GameObject enemyBullet; [SerializeField] GameObject player; [SerializeField] Rigidbody rb; private float shotTime = 0.05f; private bool shot = true; private float specialShotTime = 40; private bool specialShot = true; private float power = 80; private float speed = 30; Vector3 playerPoint; private float move = 10; int movePoint; private Vector3 escapeSky; private float downLimit = 5; private Vector3 down; private float capriciousDown = 3; private Vector3 capricious; private int Life = 10; // Use this for initialization void Start () { specialShot = false; move = 10; movePoint = Random.Range(0, 4); escapeSky = Vector3.zero; capricious = Vector3.zero; } Vector3 EnemyMove(int move) { switch (move) { case 0: return rb.velocity = transform.right * speed; case 1: return rb.velocity = -transform.right * speed; case 2: return rb.velocity = (-transform.forward / 2 + transform.right * 2) / 2 * speed; case 3: return rb.velocity = (-transform.forward / 2 + -transform.right * 2) / 2 * speed; } Debug.Log("EnemyMoveError"); return rb.velocity = Vector3.zero; } void SpecialShot() { Quaternion nin = Quaternion.LookRotation(player.transform.position - transform.position); for (int red = -40; red <= 40; red += 6) { Quaternion r = nin * Quaternion.Euler(0, red, 0); GameObject bullets = Instantiate(enemyBullet, gameObject.transform.position, r); bullets.GetComponent<Rigidbody>().AddForce(bullets.transform.forward * power, ForceMode.Impulse); } } void Shot() { Quaternion nin = Quaternion.LookRotation(player.transform.position - transform.position); GameObject bullets = Instantiate(enemyBullet, gameObject.transform.position, nin); bullets.GetComponent<Rigidbody>().AddForce(transform.forward * power, ForceMode.Impulse); } private void OnTriggerEnter(Collider other) { if (other.gameObject.CompareTag("Player1Bullet")) { Debug.Log("hit"); } } // Update is called once per frame void Update () { transform.LookAt(player.transform.position); if (GameStart.gameStart == 1) { shotTime -= Time.deltaTime; if (shotTime <= 0) { shot = true; } if (shot) { Shot(); shot = false; shotTime = 0.05f; } specialShotTime -= Time.deltaTime; if (specialShotTime <= 0) { specialShot = true; } if (specialShot) { SpecialShot(); specialShot = false; specialShotTime = 1; } move -= Time.deltaTime; if (move <= 0) { movePoint = Random.Range(0, 4); move = 5; } if (gameObject.transform.position.y >= 30) { escapeSky = -transform.up * speed; } else { escapeSky = Vector3.zero; } if (gameObject.transform.position.y >= 29) { downLimit -= Time.deltaTime; if (downLimit <= 0) { down = -transform.up * speed; } } if (gameObject.transform.position.y <= 0) { Debug.Log("a"); down = Vector3.zero; downLimit = 5; } capriciousDown -= Time.deltaTime; if (capriciousDown <= 0) { switch (Random.Range(0, 15)) { case 0: capricious = -transform.up * speed; break; case 1: capricious = transform.up * speed; break; default: capricious = Vector3.zero; break; } capriciousDown = 1; } if (Life <= 0) { Destroy(gameObject); } } } private void FixedUpdate() { if (GameStart.gameStart == 1) { rb.velocity = EnemyMove(movePoint) + escapeSky + down + capricious; } } }

このように。これは良い事なのでしょうか。何かとても悪い事のように考えてしまって、色々調べてみたのですが「ネストが深いのは良くない」とか、「スパゲティ死すべし」等書いてあっただけでして・・・・善悪だけでも構いません、教えていただけないでしょうか。

よろしくお願いします。

良い質問の評価を上げる

以下のような質問は評価を上げましょう

  • 質問内容が明確
  • 自分も答えを知りたい
  • 質問者以外のユーザにも役立つ

評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

気になる質問をクリップする

クリップした質問は、後からいつでもマイページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

  • プログラミングに関係のない質問
  • やってほしいことだけを記載した丸投げの質問
  • 問題・課題が含まれていない質問
  • 意図的に内容が抹消された質問
  • 過去に投稿した質問と同じ内容の質問
  • 広告と受け取られるような投稿

評価を下げると、トップページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

まだ回答がついていません

会員登録して回答してみよう

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
87.20%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問

同じタグがついた質問を見る

C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

Unity

Unityは、ユニティテクノロジーが開発したゲームエンジンです。 主にモバイルやブラウザ向けのゲーム製作に利用されていましたが、3Dの重力付きゲームが簡単に作成できることから需要が増え、現在はマルチプラットフォームに対応しています。 言語はC言語/C++で書かれていますが、C#、JavaScript、Booで書かれたコードにも対応しています。