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

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

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

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

Unity3D

Unity3Dは、ゲームや対話式の3Dアプリケーション、トレーニングシュミレーション、そして医学的・建築学的な技術を可視化する、商業用の開発プラットフォームです。

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

Unity

Unityは、Unity Technologiesが開発・販売している、IDEを内蔵するゲームエンジンです。主にC#を用いたプログラミングでコンテンツの開発が可能です。

Q&A

1回答

2762閲覧

Unity:オブジェクトの固定位置からのランダム生成について

退会済みユーザー

退会済みユーザー

総合スコア0

C#

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

Unity3D

Unity3Dは、ゲームや対話式の3Dアプリケーション、トレーニングシュミレーション、そして医学的・建築学的な技術を可視化する、商業用の開発プラットフォームです。

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

Unity

Unityは、Unity Technologiesが開発・販売している、IDEを内蔵するゲームエンジンです。主にC#を用いたプログラミングでコンテンツの開発が可能です。

0グッド

1クリップ

投稿2017/12/16 14:25

###前提・実現したいこと
Unityの3Dゲームにて、固定の出現場所からの敵のランダム生成をしようと考えています。
実装したいこととしては、
・フィールド内のいくつかの固定位置から、ランダムに敵を生成する
・生成に関して上限を定める
固定位置に関しては、出現場所のオブジェクトを作り、それをコピーしてフィールド内に配置していこうと思っています。
こちらの記事を参考にしているのですが、Java→C#の変換で詰まっています。

###発生している問題・エラーメッセージ
翻訳サイトを使っているのですが、Appear と length にエラーが起きています。
また、サイトでの自動翻訳を使っているため、本当に合っているかも分からない状態です。

###該当のソースコード

C#

1using System.Collections; 2using System.Collections.Generic; 3using UnityEngine; 4 5public class Appear : MonoBehaviour 6{ 7 public GameObject obj; 8 private GameObject[] objs; 9 public int max; 10 public float appearTime; 11 public int total; 12 private int num; 13 private float currentTime; 14 15 16 void Start() 17 { 18 objs = new GameObject[max]; 19 currentTime = 0.0f; 20 num = 0; 21 } 22 23 void Update() 24 { 25 objs = new GameObject[max]; 26 currentTime = 0.0f; 27 num = 0; 28 29 if (num < total) 30 { 31 currentTime += Time.deltaTime; 32 if (currentTime > appearTime) 33 { 34 appear(); 35 } 36 } 37 { 38 if (num < total) 39 { 40 41 currentTime += Time.deltaTime; 42 43 if (currentTime > appearTime) 44 { 45 appear(); 46 } 47 } 48 } 49 50 void Appear() 51 { 52 for (int i = 0; i < objs.length; i++) 53 { 54 if (objs[i] == null) 55 { 56 objs[i] = GameObject.Instantiate(obj); 57 num++; 58 break; 59 } 60 } 61 currentTime = 0.0f; 62 { 63 for (int i = 0; i < objs.length; i++) 64 { 65 if (objs[i] == null) 66 { 67 objs[i] = GameObject.Instantiate(obj); 68 num++; 69 break; 70 } 71 } 72 currentTime = 0.0f; 73 } 74 } 75 } 76}

見づらくて申し訳ありません。
よろしくお願いいたします。

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

・Appear クラスに void Appear()メソッドがあるのでエラーが出ていると思います。
void Appear()メソッドを違う名前にするとエラーが解消されると思います。

・public int max;に1以上の数値が設定されていないのでlengthでエラーが出ていると思います。
public int max = 10;またはインスペクタ上で初期値を設定していただくとエラーが解消されると思います。

###Bongoさんにコメントで教えていただいた内容を追記ます。
・C#の場合、配列の要素数のプロパティ名は「Length」ではないか?
・「Appear」メソッドの定義位置が「Update」メソッドの内部になってしまっている(さらにIShikawanさんのご指摘通り、メソッド名も変えねばならないでしょう)。
・「Start」メソッドの内容と同じものが「Update」メソッドの先頭にも入っていたり、「Update」や「Appear」メソッドの内容が2回繰り返されていて奇妙に見える。元のサイトのコードはそうなっていないので、Javascript→C#の変換がうまくいっていないのではないか?

Bongoさんありがとうございます。

###おまけ
・JavascriptとJavaはまったくの別言語なのでJavascriptと言った方がいいです。略したい場合はJSです。細かいと思うかもしれませんが、相手を混乱させてしまうこともあるかもしれませんので今のうちに覚えておくことをおすすめします。

・敵という明確な役割があるのにobjやobjsは、後々誰かと共同で仕事をする際、嫌がられるので今のうちにEnemyやEnemysに直す癖をつけておくといいと思います。

・今回のコードではobjにGameObjectの参照が入ります。ですが、敵という役割がある以上、必要な要素がたくさんあると思います。例えば、敵のアニメーションやプレイヤーに何ダメージ与えるか、または何ダメージ与えられると死ぬかなど探せば色々出てくると思います。今回のようにGameObjectを直接参照すると、そういう敵ならではの要素を追加できません。なのでEnemyクラスを用意してPrefabにアタッチしそれを参照する形をとるといいと思います。

・参考コードはご本人に申し訳ないですがプログラムをあまり勉強されていない方のコードに見えます。外部に何を公開(public)すべきで何を隠蔽(private)すべきかしっかり考えて構築されるといいと思います。

以上です。偉そうに長文すみません。上記の中で納得できるものがありましたら実践して頂けますと幸いです。

投稿2017/12/16 15:01

編集2017/12/17 13:09
IShix

総合スコア1724

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

Bongo

2017/12/16 21:32

IShikawanさんのご回答に加えて、私の気になった点を述べさせていただきますと... ・C#の場合、配列の要素数のプロパティ名は「Length」ではないか? ・「Appear」メソッドの定義位置が「Update」メソッドの内部になってしまっている(さらにIShikawanさんのご指摘通り、メソッド名も変えねばならないでしょう)。 ・「Start」メソッドの内容と同じものが「Update」メソッドの先頭にも入っていたり、「Update」や「Appear」メソッドの内容が2回繰り返されていて奇妙に見える。元のサイトのコードはそうなっていないので、Javascript→C#の変換がうまくいっていないのではないか? サイトのコードはさほど複雑ではありませんので、このさいC#の学習も兼ねて自前でC#コードに書き換えてみるのも面白いんじゃないでしょうか。
IShix

2017/12/17 13:07

またまた教えていただきありがとうございます。感謝です。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

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

アカウントをお持ちの方は

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問