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

**デフォルトの名無しさん** · 2023/03/16(木) 19:49:01.61

unity初めて１日。初心者です助けてわからん↓のどこかが間違ってるらしい
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);
}
}

**デフォルトの名無しさん** · 2023/03/16(木) 19:54:46.90

(x.0.z)

**デフォルトの名無しさん** · 2023/03/16(木) 19:56:52.87

まだあるっぽいです。

**デフォルトの名無しさん** · 2023/03/16(木) 20:31:42.04

寝ます

**デフォルトの名無しさん** · 2023/03/16(木) 20:35:08.10

書いといてくれ

**デフォルトの名無しさん** · 2023/03/16(木) 20:40:07.63

エラー内容
All compiler errors have to be fixed before you can enter playmode!
UnityEditor.SceneView:ShowCompileErrorNotification ()

**デフォルトの名無しさん** · 2023/03/16(木) 22:46:18.43

MAUIの臭い

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

MAUIの臭ってなんですか専門用語？ちょっとわからないです。

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

>>7
全然違うが？
ころすぞ？

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

アップデートに毎回GetConponentやってるのがだめなんじゃね？
Start()に入れてみろ

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

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

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

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(金) 12:56:09.41

aiに聞いてやったぞ合ってるか知らんけど
このコードで間違っているところは、以下の行です：

java
Copy code
Vector3 vec = new Vector3(x.0.z);
この行は構文エラーを引き起こします。正しい行は以下のようになります：

java
Copy code
Vector3 vec = new Vector3(x, 0, z);

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

わいもAI Code Reviewerというアプリとゆうかブラウザ？でやったけどロード遅すぎ２日まってるわｗ助かる

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

難しいなまだできないなんか手間かけてすまん。

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

カンマ
ドット

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

？

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

何を困ってんですか

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

すみませんカンマの意味を知らなくて(汗
ググりました

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

>>12
ちゃう
private void FixedUpdate()の上に
private void Start()
{
RigidBody ～
}
って感じ

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

ホントだドットになってるのか…

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

このカンマは区切りって意味だ
Vector3っていうベクトル情報を作成するメソッドにX方向の値、Y方向の値、Z方向の値を与えているという意味
Input.GetAxsisで横方向("Horizontal"Xの値)縦方向("Vertical"Zの値)を求めて
X方向にXの値、Y方向に0、Z方向にZの値を渡してベクトルを作ってるという意味

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

自分で考えるの苦手なので(汗↓言い訳
YouTubeで人のうつしてますその際にドットとかわかりにくいんですよ。すいません

**デフォルトの名無しさん** · 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()は最初の一回だけ呼び出されるから一回取得したらもうずっとこのコンポーネントを使いますよ～って感じで無駄な処理が走らなくなる