unityのコードレビューしてくれ。

レス数: 106

概要: unity初めて1日。初心者です助けてわからん↓のどこかが間違ってるらしい using System.Collections; using System.Collections.Generic; using UnityEngine; public class PlayerController : MonoBehav...
No.1
unity初めて1日。初心者です助けてわからん↓のどこかが間違ってるらしい
using System.Collections;
using System.Collections.Generic;
using UnityEngine;
public class PlayerController : MonoBehaviour
{
private void FixedUpdate()
{
//Rigdboodyを取得
Rigidbody rigidbody = GetComponent<Rigidbody>();
//方向キーの入力
float x = Input.GetAxis("Horizontal"); //横軸
float z = Input.GetAxis("Vertical"); //縦軸
//ベクトルに変換
Vector3 vec = new Vector3(x.0.z);
//力を加える
rigidbody.AddForce(vec);
}
}
No.2
(x.0.z)
No.3
まだあるっぽいです。
No.4
寝ます
No.5
書いといてくれ
No.6
エラー内容
All compiler errors have to be fixed before you can enter playmode!
UnityEditor.SceneView:ShowCompileErrorNotification ()
No.7
MAUIの臭い
No.8
MAUIの臭ってなんですか専門用語?ちょっとわからないです。
No.9
>>7

全然違うが?
ころすぞ?
No.10
アップデートに毎回GetConponentやってるのがだめなんじゃね?
Start()に入れてみろ
No.11
ありがとうございますやってみます。
No.12
public class PlayerController : MonoBehaviour
{
private void FixedUpdate()
{
//Rigdboodyを取得
Rigidbody rigidbody = GetComponent<Rigidbody>(Start);
//方向キーの入力
float x = Input.GetAxis("Horizontal"); //横軸
float z = Input.GetAxis("Vertical"); //縦軸
//ベクトルに変換
Vector3 vec = new Vector3(x. 0. z);
//力を加える
rigidbody.AddForce(vec);
}
}
これでいいですか?
No.13
aiに聞いてやったぞ 合ってるか知らんけど
このコードで間違っているところは、以下の行です:
java
Copy code
Vector3 vec = new Vector3(x.0.z);
この行は構文エラーを引き起こします。正しい行は以下のようになります:
java
Copy code
Vector3 vec = new Vector3(x, 0, z);
No.14
わいもAI Code Reviewerというアプリとゆうかブラウザ?でやったけどロード遅すぎ2日まってるわw助かる
No.15
難しいなまだできないなんか手間かけてすまん。
No.16
カンマ
ドット
No.17
No.18
何を困ってんですか
No.19
すみませんカンマの意味を知らなくて(汗
ググりました
No.20
>>12

ちゃう
private void FixedUpdate()の上に
private void Start()
{
RigidBody ~
}
って感じ
No.21
ホントだドットになってるのか…
No.22
このカンマは区切りって意味だ
Vector3っていうベクトル情報を作成するメソッドにX方向の値、Y方向の値、Z方向の値を与えているという意味
Input.GetAxsisで横方向("Horizontal"Xの値)縦方向("Vertical"Zの値)を求めて
X方向にXの値、Y方向に0、Z方向にZの値を渡してベクトルを作ってるという意味
No.23
自分で考えるの苦手なので(汗↓言い訳
YouTubeで人のうつしてますその際にドットとかわかりにくいんですよ。すいません
No.24
public class PlayerController : MonoBehaviour
{
private void FixedUpdate()
{
//Rigdboodyを取得
Rigidbody rigidbody = GetComponent<Rigidbody>(Start);
//方向キーの入力
float x = Input.GetAxis("Horizontal"); //横軸
float z = Input.GetAxis("Vertical"); //縦軸
//ベクトルに変換
Vector3 vec = new Vector3(x, 0, z);
//力を加える
rigidbody.AddForce(vec);
}
}
現状
No.25
こうですかね?
public class PlayerController : MonoBehaviour
{
private void Start()
{
RigidBody 〜
}
private void FixedUpdate()
{
//Rigdboodyを取得
Rigidbody rigidbody = GetComponent<Rigidbody>(Start);
//方向キーの入力
float x = Input.GetAxis("Horizontal"); //横軸
float z = Input.GetAxis("Vertical"); //縦軸
//ベクトルに変換
Vector3 vec = new Vector3(x, 0, z);
//力を加える
rigidbody.AddForce(vec);
}
}
No.26
>>23

いや初心者ならよくあることだと思うから別に問題はない
頑張って身につけてくれ
ちなみにGetConponent<>をなぜFixdUpdateの中に入れるとダメなのかというとこのFixdUpdateってのは毎フレームに実行されるからすでにGetConponent<>でコンポーネントを取得していても次のフレームでもまた取得してきて、そのまた次のフレームでも取得してきて~ってのを繰り返しちゃってめちゃくちゃ無駄が多くなっちゃうから
Start()は最初の一回だけ呼び出されるから一回取得したらもうずっとこのコンポーネントを使いますよ~って感じで無駄な処理が走らなくなる
No.27
>>25

FixdUpdateのRigidbodyをStartの中のやつに入れ替える感じ
No.29
これが正解
using System.Collections;
using UnityEngine;
public class MakeRigidAddForceChara: MonoBehaviour {
private Rigidbody rigidBody;//ここでrigidbodyの宣言をしておく
void Start()
{
rigidBody = GetComponent<Rigidbody>();//Start()でコンポーネントを1回コッキリで取得
}
void FixedUpdate()
{
float x = Input.GetAxis("Horizontal"); //横軸
float z = Input.GetAxis("Vertical"); //縦軸
//ベクトルに変換
Vector3 vec = new Vector3(x, 0, z);
//力を加える
rigidbody.AddForce(vec);
}
}
No.30
ありがとうございますやってみます
No.31
いやたしかに調べたら色んなところでFixdUpdateの中にGetConponent<>が来てるの見つけたわ
例題的に簡単なやつとして用意したんだろうが回り回ってクソコードが蔓延っちゃったった感じだろうな
GetComponentはStart();{}の中に入れておくのが正解
No.32
コピペできるように/ /やってくれるなんてあざす
No.33
>>32

👍
No.34
なぜだろう矢印キーで動かない実行はできだお
(´・ω・`)
No.35
Unityの公式で移動の基礎みたいなことやってるから参考にしてみるといい
https://youtu.be/VqS1dTiVLFA
No.36
rigidbodyって物理エンジンになってるから物体が重いと動かないのかもしれん
Vector3 vec =new Vector3(x,0,y)
のところを
Vector3 vec =new Vector3(x*1000,0,z*1000)とかに変えてみて?
ちなみにこれは今の力に1000をかけてるという形
No.37
すまんタイプミスしてたわ
最後のrigidbodyはrigidBodyで大文字じゃないといけない
No.38
主書けなくなったんで別垢でやりまーす
No.39
これでいいですか?
using System.Collections;
using UnityEngine;
public class MakeRigidAddForceChara: MonoBehaviour {
private Rigidbody rigidBody;//ここでrigidbodyの宣言をしておく
void Start()
{
rigidBody = GetComponent<Rigidbody>();//Start()でコンポーネントを1回コッキリで取得
}
void FixedUpdate()
{
float x = Input.GetAxis("Horizontal"); //横軸
float z = Input.GetAxis("Vertical"); //縦軸
//ベクトルに変換
Vector3 vec = new Vector3(x, 0, z);
//力を加える
rigidBody.AddForce(vec);
}
}
No.40
ちな本アカでやるとこうなる
RROR: あなたはこのスレッドにはもう書けません。🐙
問い合わせID: mev/7a936c76de153547
ホスト:M014011163064.v4.enabler.ne.jp
名前: デフォルトの名無しさん
E-mail:
内容:
No.41
>>40

いやID変わってないけど
No.42
>>39

これでとりあえずやってみよう
No.43
ちな使ってるPCは同じでGooGleのアカウントを切り替えただけです。
No.44
どうあがいても矢印キーで動かないヽ(≧Д≦)ノウワァァン!!
No.45
助けてくれ。゚(゚∩´﹏`∩゚)゚。
No.46
助けてくれ。゚(゚∩´﹏`∩゚)゚。
No.47
なんかまたエラーでた
All compiler errors have to be fixed before you can enter playmode!
UnityEditor.SceneView:ShowCompileErrorNotification ()
No.48
ぷログラム
using System.Collections;
using UnityEngine;
public class MakeRigidAddForceChara: MonoBehaviour {
private Rigidbody rigidBody;//ここでrigidbodyの宣言をしておく
void Start()
{
rigidBody = GetComponent<Rigidbody>();//Start()でコンポーネントを1回コッキリで取得
}
void FixedUpdate()
{
float x = Input.GetAxis("Horizontal"); //横軸
float z = Input.GetAxis("Vertical"); //縦軸
//ベクトルに変換
Vector3 vec = new Vector3(x, 0, z);
//力を加える
rigidBody.AddForce(vec);
}
}
No.49
ちょい7時まで放置
No.50
なんとかプログラムかんせいないかな?
No.51
↑し入れ忘れた
なんとかプログラムかんせいしないかな?
No.52
すまんいろいろあって遅れた
No.53
できない
No.54
頭悪いだけでなくガチのキチガイですやん
病院に入れてお薬飲ませてずっと寝かせとけ
No.55
物体にスクリプトアタッチしてる?
No.56
VisualStudioでコード書いてるってことでいいよね?
そのUnityの画面とVisualStudioの画面スクショ撮って見せてくれないか?
No.57
とりあえずYouTubeうつしてるコマンドは修正していってます
No.58
chromebookのスクショならやり方わかるけどこのPCなんの機種だったっけか?
スクショのやりかたわからん
No.59
paizaとかideoneつかってコード全部を上げるのが早いよ
このままじゃあラチが開かない
No.60
ちな前にわいの参考動画出したけど動画と同じで動かすのはCapsuleだから
No.61
paizaとかideoneってなんですか?
No.62
何って言われても……
パソコンで検索して全部調べていけばいいんじゃねえの
No.63
paiza意味検索したけどITエンジニアしか出なかった
No.64
ideoneすごいですね
No.65
使い方わからんけど
No.66
>>58

WindowsならShift+Alt+Sだ
何ならスマホで写真撮ったの送ってくれてもいいぞ
No.67
ありがとうございますでもできなかったです。でも主はスマホ持ってないだお(´・ω・`)
No.68
>>67

持ってないのか…
君いくつ?
No.69
ここで質問する前にChatGPTに質問しろよ
No.70
年齢わ控えます察してください。
スマホは持っていますが検索アプリが消されています。
No.71
>>70

なるほど小学生か
No.72
画面がどうなってるのか見せてくれないと対処の仕様が無いな…
No.73
小学生でわないです
No.74
そもそ画像のうpしかたも知らん
No.75
スレ初です。数少ない友にunityわかんないと言ったら5chに聞けばと言われたので何それと思いやりました。
No.76
隔離スレへようそこ
No.77
いつまでかかってんだ
幼稚園か0歳児か知らんが5chで単発質問スレは原則禁止だ
No.78
画像はimugurってところにアップロードしてそのURLを貼り付けたらこっちで見れるようになるよ
No.79
imugurを検索したんですが変な画像が出できただけで終わりました。画像を出すのは難しいですね
No.80
難しいか…
なら無理だな…
画像の通りにやったら動いたならもうそれでいい気がする
No.81
>>80

どういうこと?
No.82
>>81

画像じゃなくて動画だ
No.83
どうやっても動かない
No.84
クソスレ上げんなしね低能
No.85
すいませんね^^
No.86
動かねぇ。最初からプログラム書きなおします出来たら書き込みします
No.87
たぶん明日書き込みます。タイピング遅いんで。ちな寿司打お手軽コース最高記録が60円お得の雑魚なんで
No.88
コードをアタッチしてないことはないよね?
No.89
>>88

なにそれちゃんとunityにコード読み込ませてますよあの青いぐるぐる
No.90
paiza意味検索したけど
検索アプリが消されています
でも主はスマホ持ってないだお
スマホは持っていますが
支離滅裂なんだけど、こんな奴にかまってるやつもおかしいよ
とりあえず下げろよ。板の荒らしかよ
No.91
>>90

無視すればよくね?
No.92
>>89

インスペクターのところにスクリプトがちゃんと入ってるかどうか
No.93
とりあえず最初はひろはすって人のブロック崩しの動画でもやってみたらアタッチがどうとかの基本的な話が出てくるからやってみるといいよ
No.94
ひろはすさんの動画を見てきます。
No.95
>>90

混乱させてしまってすいません。私はスマホに検索アプリが入ってません。今はPCでやっています。
No.96
まあ、スマホじゃなくてタブレットだけど。どっちでもいいか。
No.97
このスレ作って自分がどれだけプログラム構文が下手なのかわかりました本屋で本買ってまた現れます。
ねちがえて湿布はって寝てました
No.98
朝から首がいたい
No.100
>>97

がんばれー
No.101
今までありがとうございます。最初から学習します。学習し終わったらまた新しいスレ作ります。
そん時はよろb。ちなまだ首痛い。
No.102
優しく接していたきありがとうございます。
No.103
さいなら
No.104
>>101

👍
No.106
コード糞すぎ
v=4lVm8YqFtvw