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

**デフォルトの名無しさん** · 2023/03/17(金) 15:34:22.48

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);
}
}
現状

**デフォルトの名無しさん** · 2023/03/17(金) 15:36:48.88

こうですかね？

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);
}
}

**デフォルトの名無しさん** · 2023/03/17(金) 15:39:18.40

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

**デフォルトの名無しさん** · 2023/03/17(金) 15:40:27.35

>>25
FixdUpdateのRigidbodyをStartの中のやつに入れ替える感じ

**デフォルトの名無しさん** · 2023/03/17(金) 15:41:10.91

ちな参考
https://www.youtube.com/watch?v=loz4_lb-JsM&t=530s

**デフォルトの名無しさん** · 2023/03/17(金) 15:47:46.83

これが正解

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);
}
}

**デフォルトの名無しさん** · 2023/03/17(金) 15:49:59.38

ありがとうございますやってみます

**デフォルトの名無しさん** · 2023/03/17(金) 15:50:27.32

いやたしかに調べたら色んなところでFixdUpdateの中にGetConponent<>が来てるの見つけたわ
例題的に簡単なやつとして用意したんだろうが回り回ってクソコードが蔓延っちゃったった感じだろうな
GetComponentはStart();{}の中に入れておくのが正解

**デフォルトの名無しさん** · 2023/03/17(金) 15:52:12.69

コピペできるように/　/やってくれるなんてあざす

**デフォルトの名無しさん** · 2023/03/17(金) 15:53:01.82

>>32
👍

**デフォルトの名無しさん** · 2023/03/17(金) 15:53:48.76

なぜだろう矢印キーで動かない実行はできだお
(´・ω・`)

**デフォルトの名無しさん** · 2023/03/17(金) 15:54:53.90

Unityの公式で移動の基礎みたいなことやってるから参考にしてみるといい

https://youtu.be/VqS1dTiVLFA

**デフォルトの名無しさん** · 2023/03/17(金) 15:59:38.79

rigidbodyって物理エンジンになってるから物体が重いと動かないのかもしれん
Vector3 vec =new Vector3(x,0,y)
のところを
Vector3 vec =new Vector3(x*1000,0,z*1000)とかに変えてみて？
ちなみにこれは今の力に1000をかけてるという形

**デフォルトの名無しさん** · 2023/03/17(金) 16:03:41.63

すまんタイプミスしてたわ
最後のrigidbodyはrigidBodyで大文字じゃないといけない

**デフォルトの名無しさん** · 2023/03/17(金) 16:10:29.71

主書けなくなったんで別垢でやりまーす

**デフォルトの名無しさん** · 2023/03/17(金) 16:11:17.30

これでいいですか？
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);
}
}

**デフォルトの名無しさん** · 2023/03/17(金) 16:12:16.42

ちな本アカでやるとこうなる
RROR: あなたはこのスレッドにはもう書けません。🐙
問い合わせID： mev/7a936c76de153547
ホスト：M014011163064.v4.enabler.ne.jp
名前：デフォルトの名無しさん
E-mail：
内容：

**デフォルトの名無しさん** · 2023/03/17(金) 16:24:33.62

>>40
いやID変わってないけど

**デフォルトの名無しさん** · 2023/03/17(金) 16:25:13.90

>>39
これでとりあえずやってみよう

**デフォルトの名無しさん** · 2023/03/17(金) 16:53:44.44

ちな使ってるPCは同じでGooGleのアカウントを切り替えただけです。

**デフォルトの名無しさん** · 2023/03/17(金) 16:55:30.26

どうあがいても矢印キーで動かないヽ(≧Д≦)ノｳﾜｧｧﾝ!!

**デフォルトの名無しさん** · 2023/03/17(金) 17:13:15.00

助けてくれ。ﾟ(ﾟ∩´﹏`∩ﾟ)ﾟ。

**デフォルトの名無しさん** · 2023/03/17(金) 17:13:15.56

助けてくれ。ﾟ(ﾟ∩´﹏`∩ﾟ)ﾟ。

**デフォルトの名無しさん** · 2023/03/17(金) 17:20:43.86

なんかまたエラーでた
All compiler errors have to be fixed before you can enter playmode!
UnityEditor.SceneView:ShowCompileErrorNotification ()

**デフォルトの名無しさん** · 2023/03/17(金) 17:21:54.81

ぷログラム
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);
}
}

**デフォルトの名無しさん** · 2023/03/17(金) 17:27:10.01

ちょい７時まで放置

**デフォルトの名無しさん** · 2023/03/17(金) 17:28:38.48

なんとかプログラムかんせいないかな？

**デフォルトの名無しさん** · 2023/03/17(金) 17:29:19.78

↑し入れ忘れた
なんとかプログラムかんせいしないかな？

**デフォルトの名無しさん** · 2023/03/17(金) 20:02:09.89

すまんいろいろあって遅れた

**デフォルトの名無しさん** · 2023/03/17(金) 20:03:01.61

できない

**デフォルトの名無しさん** · 2023/03/18(土) 08:14:27.84

頭悪いだけでなくガチのキチガイですやん
病院に入れてお薬飲ませてずっと寝かせとけ

**デフォルトの名無しさん** · 2023/03/18(土) 08:40:42.92

物体にスクリプトアタッチしてる？

**デフォルトの名無しさん** · 2023/03/18(土) 08:41:58.64

VisualStudioでコード書いてるってことでいいよね？
そのUnityの画面とVisualStudioの画面スクショ撮って見せてくれないか？

**デフォルトの名無しさん** · 2023/03/18(土) 09:54:32.69

とりあえずYouTubeうつしてるコマンドは修正していってます

**デフォルトの名無しさん** · 2023/03/18(土) 09:57:39.87

chromebookのスクショならやり方わかるけどこのPCなんの機種だったっけか？
スクショのやりかたわからん

**デフォルトの名無しさん** · 2023/03/18(土) 10:20:47.66

paizaとかideoneつかってコード全部を上げるのが早いよ
このままじゃあラチが開かない

**デフォルトの名無しさん** · 2023/03/18(土) 10:22:07.40

ちな前にわいの参考動画出したけど動画と同じで動かすのはCapsuleだから

**デフォルトの名無しさん** · 2023/03/18(土) 10:22:44.72

paizaとかideoneってなんですか？

**デフォルトの名無しさん** · 2023/03/18(土) 10:30:40.22

何って言われても……
パソコンで検索して全部調べていけばいいんじゃねえの

**デフォルトの名無しさん** · 2023/03/18(土) 10:41:04.27

paiza意味検索したけどITエンジニアしか出なかった

**デフォルトの名無しさん** · 2023/03/18(土) 10:42:59.77

ideoneすごいですね

**デフォルトの名無しさん** · 2023/03/18(土) 10:44:51.60

使い方わからんけど

**デフォルトの名無しさん** · 2023/03/18(土) 13:58:45.47

>>58
WindowsならShift+Alt+Sだ
何ならスマホで写真撮ったの送ってくれてもいいぞ

**デフォルトの名無しさん** · 2023/03/18(土) 16:19:11.43

ありがとうございますでもできなかったです。でも主はスマホ持ってないだお(´・ω・`)

**デフォルトの名無しさん** · 2023/03/18(土) 16:42:49.89

>>67
持ってないのか…
君いくつ？

**デフォルトの名無しさん** · 2023/03/18(土) 16:56:04.92

ここで質問する前にChatGPTに質問しろよ

**デフォルトの名無しさん** · 2023/03/18(土) 17:43:54.41

年齢わ控えます察してください。
スマホは持っていますが検索アプリが消されています。

**デフォルトの名無しさん** · 2023/03/19(日) 00:21:52.29

>>70
なるほど小学生か

**デフォルトの名無しさん** · 2023/03/19(日) 00:23:05.32

画面がどうなってるのか見せてくれないと対処の仕様が無いな…

**デフォルトの名無しさん** · 2023/03/19(日) 07:53:21.02

小学生でわないです

**デフォルトの名無しさん** · 2023/03/19(日) 07:54:49.63

そもそ画像のうpしかたも知らん

**デフォルトの名無しさん** · 2023/03/19(日) 08:25:15.26

スレ初です。数少ない友にunityわかんないと言ったら5chに聞けばと言われたので何それと思いやりました。

**デフォルトの名無しさん** · 2023/03/19(日) 10:04:28.99

隔離スレへようそこ

**デフォルトの名無しさん** · 2023/03/19(日) 10:04:33.65

いつまでかかってんだ
幼稚園か０歳児か知らんが５ｃｈで単発質問スレは原則禁止だ

**デフォルトの名無しさん** · 2023/03/19(日) 12:31:14.99

画像はimugurってところにアップロードしてそのURLを貼り付けたらこっちで見れるようになるよ

**デフォルトの名無しさん** · 2023/03/19(日) 14:34:59.83

imugurを検索したんですが変な画像が出できただけで終わりました。画像を出すのは難しいですね

**デフォルトの名無しさん** · 2023/03/19(日) 14:58:57.33

難しいか…
なら無理だな…
画像の通りにやったら動いたならもうそれでいい気がする

**デフォルトの名無しさん** · 2023/03/19(日) 17:10:50.43

>>80
どういうこと？

**デフォルトの名無しさん** · 2023/03/20(月) 06:29:13.76

>>81
画像じゃなくて動画だ

**デフォルトの名無しさん** · 2023/03/20(月) 18:39:22.68

どうやっても動かない

**デフォルトの名無しさん** · 2023/03/20(月) 18:50:02.44

クソスレ上げんなしね低能

**デフォルトの名無しさん** · 2023/03/20(月) 19:13:33.45

すいませんね^^

**デフォルトの名無しさん** · 2023/03/20(月) 20:14:03.44

動かねぇ。最初からプログラム書きなおします出来たら書き込みします

**デフォルトの名無しさん** · 2023/03/20(月) 20:17:49.69

たぶん明日書き込みます。タイピング遅いんで。ちな寿司打お手軽コース最高記録が60円お得の雑魚なんで

**デフォルトの名無しさん** · 2023/03/20(月) 20:36:38.17

コードをアタッチしてないことはないよね？

**デフォルトの名無しさん** · 2023/03/20(月) 20:48:27.57

>>88
なにそれちゃんとunityにコード読み込ませてますよあの青いぐるぐる

**デフォルトの名無しさん** · 2023/03/20(月) 23:01:13.42

paiza意味検索したけど
検索アプリが消されています

でも主はスマホ持ってないだお
スマホは持っていますが

支離滅裂なんだけど、こんな奴にかまってるやつもおかしいよ
とりあえず下げろよ。板の荒らしかよ

**デフォルトの名無しさん** · 2023/03/21(火) 00:17:13.74

>>90
無視すればよくね？

**デフォルトの名無しさん** · 2023/03/21(火) 00:18:10.66

>>89
インスペクターのところにスクリプトがちゃんと入ってるかどうか

**デフォルトの名無しさん** · 2023/03/21(火) 00:20:23.59

とりあえず最初はひろはすって人のブロック崩しの動画でもやってみたらアタッチがどうとかの基本的な話が出てくるからやってみるといいよ

**デフォルトの名無しさん** · 2023/03/21(火) 09:16:49.20

ひろはすさんの動画を見てきます。

**デフォルトの名無しさん** · 2023/03/21(火) 09:20:25.41

>>90
混乱させてしまってすいません。私はスマホに検索アプリが入ってません。今はPCでやっています。

**デフォルトの名無しさん** · 2023/03/21(火) 09:21:31.99

まあ、スマホじゃなくてタブレットだけど。どっちでもいいか。

**デフォルトの名無しさん** · 2023/03/22(水) 15:32:14.46

このスレ作って自分がどれだけプログラム構文が下手なのかわかりました本屋で本買ってまた現れます。
ねちがえて湿布はって寝てました

**デフォルトの名無しさん** · 2023/03/22(水) 16:17:00.60

朝から首がいたい

**デフォルトの名無しさん** · 2023/03/23(木) 11:00:26.84

99記念
https://eater.net/quaternions

**デフォルトの名無しさん** · 2023/03/23(木) 11:33:14.56

>>97
がんばれー

**デフォルトの名無しさん** · 2023/03/25(土) 08:15:40.37

今までありがとうございます。最初から学習します。学習し終わったらまた新しいスレ作ります。
そん時はよろb。ちなまだ首痛い。

**デフォルトの名無しさん** · 2023/03/25(土) 08:16:57.14

優しく接していたきありがとうございます。

**デフォルトの名無しさん** · 2023/03/25(土) 08:17:23.91

さいなら

**デフォルトの名無しさん** · 2023/03/25(土) 18:18:33.91

>>101
👍

**デフォルトの名無しさん** · 2023/10/21(土) 12:22:14.00

ここのコードをレビューしてみたい
https://edu.monaca.io/puyoschool