[過去ログ] 【ゲームエンジン】Unity初心者質問スレBuild1 (1002レス)
上下前次1-新
このスレッドは過去ログ倉庫に格納されています。
次スレ検索 歴削→次スレ 栞削→次スレ 過去ログメニュー
98(1): 名前は開発中のものです。 [sage] 2018/06/28(木) 14:28:13.80 ID:9SECcI4o(2/2) AAS
ついでに、dirこそUpdate関数内ローカルでいい。
int dir=tfTriangle.position.x<limit?1:-1;
って書いとけばcheckrightのなかでdirをゼロにする必要もない。
99(1): 弟子 ◆rZPPjYuxmQ [sage] 2018/06/28(木) 14:29:20.15 ID:/UzN8HYz(2/9) AAS
>>9797(1): 名前は開発中のものです。 [sage] 2018/06/28(木) 14:20:39.58 ID:9SECcI4o(1/2) AAS
んじゃ、せっかくだからケチつけてみようかな。
サンプルなんだから分かりやすく書いたほうがいいよ
ってことで、v2posのスコープが広すぎる。Update関数内のローカル変数でいい。ってかv2pos.xしか使ってないんだからfloat pxにしとけばいいだろ。理解するのに30秒かかったわ。(変数生成負荷を嫌ったんだろうけど、見やすさは重要だと思うんだ)
てかlimitって名前が分かりにくい。x座標の目的地なんだからtargetXとかでいいんでね?30秒(略)
これ悪い例ね
pxにして直接potisionに代入できないから、たぶんこの人は作ったこと無くて覚え立てのスコープとか使ってわざわざ自分の才能のなさを書きにきてるのね
そんな時間あるなら自分流のサンプル作って出せばいいのに
100(2): 名前は開発中のものです。 [sage] 2018/06/28(木) 14:39:42.06 ID:UPpeKgHO(1/6) AAS
つーかbasicからやってるような人間がリファクタリングして他人に見せる用として提示するコードがこれなの?
変数大文字から始めるわ、ハンガリアンつけるわでびっくりなんだけど
初学者のため思うならこそちゃんとしたコード見せてあげるべきなのに
101(1): 名前は開発中のものです。 [sage] 2018/06/28(木) 14:53:19.52 ID:/+kZ1wVw(3/9) AAS
>>100
ハンガリアンもスネークも人それぞれだろ。初心者スレでそんな細かいことで煽るなよ。
こういう記法がいいと思うってアドバイスだけでいいじゃん。
ちなみに俺はローワーキャメルが好きだ。
記法についてはここな。
外部リンク:www.google.co.jp
102: 名前は開発中のものです。 [sage] 2018/06/28(木) 14:56:43.12 ID:UPpeKgHO(2/6) AAS
vector2.zero知ってるならrightやleftも使おうよ
なんすか、dir=1って
なんでv2pos.xに代入したあとcheck関数呼んでからv2posを代入してんの?
check関数でなにかしてるわけでもないならそいつらあとでいいよね?
check関数切り替えてるけど切り分け方がよくわからん
キー入力、移動先計算、移動方向判定、移動
移動方向判定だけなんで関数化したの?checkって名前もどうかと思うけど、まぁ命名はいいや
103(1): 名前は開発中のものです。 [sage] 2018/06/28(木) 14:57:38.47 ID:/+kZ1wVw(4/9) AAS
ハンガリアンに否定的なサイトしか見つからなかったが、ハンガリアンにも間違ったコードが間違って見えるって長所があるからな
104: 名前は開発中のものです。 [sage] 2018/06/28(木) 14:59:54.67 ID:UPpeKgHO(3/6) AAS
>>101
人それぞれだけどサンプルとして提示するならせめて公式に合わせるべきでは?
このコードを参考にしてください、って初心者に提示してるんだよ?
if使うんですか?ってレベルの初心者に
105: 名前は開発中のものです。 [sage] 2018/06/28(木) 15:01:01.11 ID:UPpeKgHO(4/6) AAS
>>103
システムハンガリアンとかでググればハンガリアンの本来の意味を解説したとこ見つかるよ
間違ってもGameObjectだからgoってつけようなんて発想にはならん
106(1): 名前は開発中のものです。 [sage] 2018/06/28(木) 15:11:47.18 ID:h3xp0Kjr(1) AAS
>>99
変数生成負荷とか言ってるんだからfloat型pxをxにしたnew Vector2を生成しろってことだろ
107: 名前は開発中のものです。 [sage] 2018/06/28(木) 15:17:43.19 ID:K46AO26G(1) AAS
俺、駄目なハンガリアンの使い方してるわw
勉強になるなぁ
108(1): 弟子 ◆rZPPjYuxmQ [sage] 2018/06/28(木) 15:20:04.03 ID:/UzN8HYz(3/9) AAS
大漁だなw
サンプル作れないけど文句しか言えない人がこんなに居るんだねー
一応ここは素人が素人に答えるスレなので、素人にも分かるようにみなさんもサンプル提示すればいいと思うよ
きっと頭いいから見やすくて分かりやすいサンプル提示してくれるんだろうなぁ
だよねー
まさか、文句だけ言ってプログラム書けないとかないよね?
109: 名前は開発中のものです。 [sage] 2018/06/28(木) 16:01:20.93 ID:qauh58MA(1) AAS
上のサンプルで充分だろ。
揚げ足とってウダウダ言っても、質問者の為にならんぞ。ほんとケチだけは立派なんて頭悪すぎるから辞めなよ。
110(1): 名前は開発中のものです。 [sage] 2018/06/28(木) 16:07:14.65 ID:/+kZ1wVw(5/9) AAS
>>108
俺のスパゲティーコードでいいならのせるぞ
111(1): 弟子 ◆rZPPjYuxmQ [sage] 2018/06/28(木) 16:07:56.12 ID:/UzN8HYz(4/9) AAS
最後のサンプル
Unityという事でアニメにして動かしてみた
まぁこういう事も出来るという事で…
画像リンク
画像リンク
画像リンク
つまり何が言いたいかというと、一つのやりたいことを行うのに沢山のやり方があって
そのどれをやってもいいしコードの書き方も色々とあるわけさ
まぁ仕事として他人と共同で作成とかになるとコード規約とか出てくるけど
まぁこんなとこでホビーとしてやる分には、まず動かしてみる事が一番大事
口だけで文句言うのが一番進歩もしないって事よ
112: 弟子 ◆rZPPjYuxmQ [sage] 2018/06/28(木) 16:11:42.18 ID:/UzN8HYz(5/9) AAS
>>110
是非是非、他人のコードはいい勉強になりますので見て見たいです
自分では思ってもみない「気づき」が必ずあるので
>>111
public Transform trTriangle;
private void Awake()
{
trTriangle = GetComponent<Transform>();
}
これ使ってないので外すの忘れました^^;
113: 名前は開発中のものです。 [sage] 2018/06/28(木) 16:42:26.57 ID:UPpeKgHO(5/6) AAS
111が初心者向けとしてのサンプルとしては一番良いと思う
なんでactiveの切り替えなんてトリッキーとも取れるような手法を一番に持ってきたんだろう
animatorだと他の要素を加えるときに制約になる可能性があるから、別手法として96みたいにすることもできるって流れが普通だと思うけど
114: 名前は開発中のものです。 [sage] 2018/06/28(木) 17:06:10.17 ID:/+kZ1wVw(6/9) AAS
とりあえずてきとうに作った。
自分でもクソコード書いてるかわからんからどんどん指摘してくれ。よくない癖とかついてそうで怖いからな。
画面内に収まるようにもしてみた。
画像リンク
115(2): 名前は開発中のものです。 [sage] 2018/06/28(木) 17:06:14.63 ID:/+kZ1wVw(7/9) AAS
とりあえずてきとうに作った。
自分でもクソコード書いてるかわからんからどんどん指摘してくれ。よくない癖とかついてそうで怖いからな。
画面内に収まるようにもしてみた。
画像リンク
116: 名前は開発中のものです。 [sage] 2018/06/28(木) 17:07:01.22 ID:/+kZ1wVw(8/9) AAS
うわ、連投すまん。
117: 名前は開発中のものです。 [sage] 2018/06/28(木) 17:17:38.71 ID:UPpeKgHO(6/6) AAS
>>115
特に気になるとこ無い気がする
min maxは画面サイズが変更されたときに再計算すればいいじゃないとか思ったけど、この程度の処理気にするもんでも無いし
118(2): 弟子 ◆rZPPjYuxmQ [sage] 2018/06/28(木) 17:42:03.28 ID:/UzN8HYz(6/9) AAS
>>115
GetAxis便利ですよねー
ただ今回の最初の要求って>>7979(3): 名前は開発中のものです。 [sage] 2018/06/27(水) 15:12:21.69 ID:07W788OP(1/3) AAS
C#を使用して2Dゲーム制作をしている初心者です
左右の方向キーを使い、特定の位置五ヶ所にキャラクターを移動させるにはどのようにしたらいいのでしょうか?
・移動箇所
☆キャラクター
・ ・ ☆ ・ ・
なので上下いらないかなと思うのです
それと、位置移動なんですが、これって指定位置で止まるんでしょうか?
ちなみに自分はSerializeFieldの使い方知らないのですけど
TransformがPrivateなのですが、これってどこで指定するんでしょうか?
インスペクタじゃ出来ないですよね?
119: 弟子 ◆rZPPjYuxmQ [sage] 2018/06/28(木) 17:44:41.51 ID:/UzN8HYz(7/9) AAS
あ、あとspeedもSerializeFieldになってますけど
ただこちらも初期値入ってないけど、もしかしてSerializeFieldにするとどっかで初期化されるというか
アタッチしたオブジェクトの何かを持ってきてくれるのでしょうか?
ちょっと自分にはわからない事が多いかもしれません><
120(1): 名前は開発中のものです。 [sage] 2018/06/28(木) 17:55:42.78 ID:/+kZ1wVw(9/9) AAS
>>118
SerializeFieldはprivateでもインスペクタで値を設定できるようにする属性だよ。
publicにしたらどこから値が変えられるかわからないからprivateにしてるんだけど、それじゃインスペクタで参照できないからこの属性で強制的にシリアライズしてる。
121: 名前は開発中のものです。 [sage] 2018/06/28(木) 18:57:50.31 ID:H6a6imyl(1) AAS
>>118
止まるというより2点で指定された長方形区画から外れると枠上に移動するようになってないか
122(1): 弟子 ◆rZPPjYuxmQ [sage] 2018/06/28(木) 19:22:46.38 ID:/UzN8HYz(8/9) AAS
>>120
なるほど、これは便利な機能ですね!勉強ななりました!
ありがとう
上下前次1-新書関写板覧索設栞歴
あと 880 レスあります
スレ情報 赤レス抽出 画像レス抽出 歴の未読スレ AAサムネイル
ぬこの手 ぬこTOP 0.024s