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

unityのコードレビュ� ..

2:デフォルトの名無しさん
23/03/16 19:54:46.90 vHdvcKrM.net
(x.0.z)

3:デフォルトの名無しさん
23/03/16 19:56:52.87 2Ni9Md6U.net
まだあるっぽいです。

4:デフォルトの名無しさん
23/03/16 20:31:42.04 2Ni9Md6U.net
寝ます

5:デフォルトの名無しさん
23/03/16 20:35:08.10 2Ni9Md6U.net
書いといてくれ

6:デフォルトの名無しさん
23/03/16 20:40:07.63 2Ni9Md6U.net
エラー内容
All compiler errors have to be fixed before you can enter playmode!
UnityEditor.SceneView:ShowCompileErrorNotification ()

7:デフォルトの名無しさん
23/03/16 22:46:18.43 Tb2JmuPY.net
MAUIの臭い

8:デフォルトの名無しさん
23/03/17 06:28:35.43 F/5O1GME.net
MAUIの臭ってなんですか専門用語？ちょっとわからないです。

9:デフォルトの名無しさん
23/03/17 07:40:46.47 aeVIJ/KU.net
>>7
全然違うが？
ころすぞ？

10:デフォルトの名無しさん
23/03/17 07:42:27.72 aeVIJ/KU.net
アップデートに毎回GetConponentやってるのがだめなんじゃね？
Start()に入れてみろ

11:デフォルトの名無しさん
23/03/17 12:20:01.14 F/5O1GME.net
ありがとうございますやってみます。

12:デフォルトの名無しさん
23/03/17 12:22:59.49 F/5O1GME.net
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);
}
}
これでいいですか？

13:デフォルトの名無しさん
23/03/17 12:56:09.41 XOCAaSM/.net
aiに聞いてやったぞ合ってるか知らんけど
このコードで間違っているところは、以下の行です：

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

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

14:デフォルトの名無しさん
23/03/17 13:02:04.29 F/5O1GME.net
わいもAI Code Reviewerというアプリとゆうかブラウザ？でやったけどロード遅すぎ２日まってるわｗ助かる

15:デフォルトの名無しさん
23/03/17 13:04:58.96 F/5O1GME.net
難しいなまだできないなんか手間かけてすまん。

16:デフォルトの名無しさん
23/03/17 13:12:01.66 +Hvt/gNN.net
カンマ
ドット

17:デフォルトの名無しさん
23/03/17 14:25:35.15 F/5O1GME.net
？

18:デフォルトの名無しさん
23/03/17 14:38:01.41 2zEmq53G.net
何を困ってんですか

19:デフォルトの名無しさん
23/03/17 15:04:21.00 F/5O1GME.net
すみませんカンマの意味を知らなくて(汗
ググりました

20:デフォルトの名無しさん
23/03/17 15:24:52.91 aeVIJ/KU.net
>>12
ちゃう
private void FixedUpdate()の上に
private void Start()
{
RigidBody ～
}
って感じ

21:デフォルトの名無しさん
23/03/17 15:25:39.75 aeVIJ/KU.net
ホントだドットになってるのか…

22:デフォルトの名無しさん
23/03/17 15:30:41.59 aeVIJ/KU.net
このカンマは区切りって意味だ
Vector3っていうベクトル情報を作成するメソッドにX方向の値、Y方向の値、Z方向の値を与えているという意味
Input.GetAxsisで横方向("Horizontal"Xの値)縦方向("Vertical"Zの値)を求めて
X方向にXの値、Y方向に0、Z方向にZの値を渡してベクトルを作ってるという意味

23:デフォルトの名無しさん
23/03/17 15:32:31.00 F/5O1GME.net
自分で考えるの苦手なので(汗↓言い訳
YouTubeで人のうつしてますその際にドットとかわかりにくいんですよ。すいません

24:デフォルトの名無しさん
23/03/17 15:34:22.48 F/5O1GME.net
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);
}
}
現状

25:デフォルトの名無しさん
23/03/17 15:36:48.88 F/5O1GME.net
こうですかね？
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);
}
}

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

27:デフォルトの名無しさん
23/03/17 15:40:27.35 aeVIJ/KU.net
>>25
FixdUpdateのRigidbodyをStartの中のやつに入れ替える感じ

28:デフォルトの名無しさん
23/03/17 15:41:10.91 F/5O1GME.net
ちな参考
URLﾘﾝｸ(www.youtube.com)

29:デフォルトの名無しさん
23/03/17 15:47:46.83 aeVIJ/KU.net
これが正解
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);
}
}

30:デフォルトの名無しさん
23/03/17 15:49:59.38 F/5O1GME.net
ありがとうございますやってみます

31:デフォルトの名無しさん
23/03/17 15:50:27.32 aeVIJ/KU.net
いやたしかに調べたら色んなところでFixdUpdateの中にGetConponent<>が来てるの見つけたわ
例題的に簡単なやつとして用意したんだろうが回り回ってクソコードが蔓延っちゃったった感じだろうな
GetComponentはStart();{}の中に入れておくのが正解

32:デフォルトの名無しさん
23/03/17 15:52:12.69 F/5O1GME.net
コピペできるように/　/やってくれるなんてあざす

33:デフォルトの名無しさん
23/03/17 15:53:01.82 aeVIJ/KU.net
>>32
👍

34:デフォルトの名無しさん
23/03/17 15:53:48.76 F/5O1GME.net
なぜだろう矢印キーで動かない実行はできだお
(´・ω・`)

35:デフォルトの名無しさん
23/03/17 15:54:53.90 aeVIJ/KU.net
Unityの公式で移動の基礎みたいなことやってるから参考にしてみるといい
URLﾘﾝｸ(youtu.be)

36:デフォルトの名無しさん
23/03/17 15:59:38.79 aeVIJ/KU.net
rigidbodyって物理エンジンになってるから物体が重いと動かないのかもしれん
Vector3 vec =new Vector3(x,0,y)
のところを
Vector3 vec =new Vector3(x*1000,0,z*1000)とかに変えてみて？
ちなみにこれは今の力に1000をかけてるという形

37:デフォルトの名無しさん
23/03/17 16:03:41.63 aeVIJ/KU.net
すまんタイプミスしてたわ
最後のrigidbodyはrigidBodyで大文字じゃないといけない

38:デフォルトの名無しさん
23/03/17 16:10:29.71 F/5O1GME.net
主書けなくなったんで別垢でやりまーす

39:デフォルトの名無しさん
23/03/17 16:11:17.30 F/5O1GME.net
これでいいですか？
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);
}
}

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

41:デフォルトの名無しさん
23/03/17 16:24:33.62 aeVIJ/KU.net
>>40
いやID変わってないけど

42:デフォルトの名無しさん
23/03/17 16:25:13.90 aeVIJ/KU.net
>>39
これでとりあえずやってみよう

43:デフォルトの名無しさん
23/03/17 16:53:44.44 F/5O1GME.net
ちな使ってるPCは同じでGooGleのアカウントを切り替えただけです。

44:デフォルトの名無しさん
23/03/17 16:55:30.26 F/5O1GME.net
どうあがいても矢印キーで動かないヽ(≧Д≦)ノｳﾜｧｧﾝ!!

45:デフォルトの名無しさん
23/03/17 17:13:15.00 F/5O1GME.net
助けてくれ。ﾟ(ﾟ∩´﹏`∩ﾟ)ﾟ。

46:デフォルトの名無しさん
23/03/17 17:13:15.56 F/5O1GME.net
助けてくれ。ﾟ(ﾟ∩´﹏`∩ﾟ)ﾟ。

47:デフォルトの名無しさん
23/03/17 17:20:43.86 F/5O1GME.net
なんかまたエラーでた
All compiler errors have to be fixed before you can enter playmode!
UnityEditor.SceneView:ShowCompileErrorNotification ()

48:デフォルトの名無しさん
23/03/17 17:21:54.81 F/5O1GME.net
ぷログラム
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);
}
}

49:デフォルトの名無しさん
23/03/17 17:27:10.01 F/5O1GME.net
ちょい７時まで放置

50:デフォルトの名無しさん
23/03/17 17:28:38.48 F/5O1GME.net
なんとかプログラムかんせいないかな？

51:デフォルトの名無しさん
23/03/17 17:29:19.78 F/5O1GME.net
↑し入れ忘れた
なんとかプログラムかんせいしないかな？

52:デフォルトの名無しさん
23/03/17 20:02:09.89 F/5O1GME.net
すまんいろいろあって遅れた

53:デフォルトの名無しさん
23/03/17 20:03:01.61 F/5O1GME.net
できない

54:デフォルトの名無しさん
23/03/18 08:14:27.84 zWXXmVze.net
頭悪いだけでなくガチのキチガイですやん
病院に入れてお薬飲ませてずっと寝かせとけ

55:デフォルトの名無しさん
23/03/18 08:40:42.92 lSv7hS4s.net
物体にスクリプトアタッチしてる？

56:デフォルトの名無しさん
23/03/18 08:41:58.64 lSv7hS4s.net
VisualStudioでコード書いてるってことでいいよね？
そのUnityの画面とVisualStudioの画面スクショ撮って見せてくれないか？

57:デフォルトの名無しさん
23/03/18 09:54:32.69 pKPxuuze.net
とりあえずYouTubeうつしてるコマンドは修正していってます

58:デフォルトの名無しさん
23/03/18 09:57:39.87 pKPxuuze.net
chromebookのスクショならやり方わかるけどこのPCなんの機種だったっけか？
スクショのやりかたわからん

59:デフォルトの名無しさん
23/03/18 10:20:47.66 aOZz6AJ9.net
paizaとかideoneつかってコード全部を上げるのが早いよ
このままじゃあラチが開かない

60:デフォルトの名無しさん
23/03/18 10:22:07.40 pKPxuuze.net
ちな前にわいの参考動画出したけど動画と同じで動かすのはCapsuleだから

61:デフォルトの名無しさん
23/03/18 10:22:44.72 pKPxuuze.net
paizaとかideoneってなんですか？

62:デフォルトの名無しさん
23/03/18 10:30:40.22 aOZz6AJ9.net
何って言われても……
パソコンで検索して全部調べていけばいいんじゃねえの

63:デフォルトの名無しさん
23/03/18 10:41:04.27 pKPxuuze.net
paiza意味検索したけどITエンジニアしか出なかった

64:デフォルトの名無しさん
23/03/18 10:42:59.77 pKPxuuze.net
ideoneすごいですね

65:デフォルトの名無しさん
23/03/18 10:44:51.60 pKPxuuze.net
使い方わからんけど

66:デフォルトの名無しさん
23/03/18 13:58:45.47 lSv7hS4s.net
>>58
WindowsならShift+Alt+Sだ
何ならスマホで写真撮ったの送ってくれてもいいぞ

67:デフォルトの名無しさん
23/03/18 16:19:11.43 pKPxuuze.net
ありがとうございますでもできなかったです。でも主はスマホ持ってないだお(´・ω・`)

68:デフォルトの名無しさん
23/03/18 16:42:49.89 lSv7hS4s.net
>>67
持ってないのか…
君いくつ？

69:デフォルトの名無しさん
23/03/18 16:56:04.92 jcrb+kJK.net
ここで質問する前にChatGPTに質問しろよ

70:デフォルトの名無しさん
23/03/18 17:43:54.41 pKPxuuze.net
年齢わ控えます察してください。
スマホは持っていますが検索アプリが消されています。

71:デフォルトの名無しさん
23/03/19 00:21:52.29 yY/KWpLm.net
>>70
なるほど小学生か

72:デフォルトの名無しさん
23/03/19 00:23:05.32 yY/KWpLm.net
画面がどうなってるのか見せてくれないと対処の仕様が無いな…

73:デフォルトの名無しさん
23/03/19 07:53:21.02 awbwu7tX.net
小学生でわないです

74:デフォルトの名無しさん
23/03/19 07:54:49.63 awbwu7tX.net
そもそ画像のうpしかたも知らん

75:デフォルトの名無しさん
23/03/19 08:25:15.26 awbwu7tX.net
スレ初です。数少ない友にunityわかんないと言ったら5chに聞けばと言われたので何それと思いやりました。

76:デフォルトの名無しさん
23/03/19 10:04:28.99 fPDrKYk/.net
隔離スレへようそこ

77:デフォルトの名無しさん
23/03/19 10:04:33.65 fBd6o36j.net
いつまでかかってんだ
幼稚園か０歳児か知らんが５ｃｈで単発質問スレは原則禁止だ

78:デフォルトの名無しさん
23/03/19 12:31:14.99 yY/KWpLm.net
画像はimugurってところにアップロードしてそのURLを貼り付けたらこっちで見れるようになるよ

79:デフォルトの名無しさん
23/03/19 14:34:59.83 awbwu7tX.net
imugurを検索したんですが変な画像が出できただけで終わりました。画像を出すのは難しいですね

80:デフォルトの名無しさん
23/03/19 14:58:57.33 yY/KWpLm.net
難しいか…
なら無理だな…
画像の通りにやったら動いたならもうそれでいい気がする

81:デフォルトの名無しさん
23/03/19 17:10:50.43 awbwu7tX.net
>>80
どういうこと？

82:デフォルトの名無しさん
23/03/20 06:29:13.76 r9gDoaWx.net
>>81
画像じゃなくて動画だ

83:デフォルトの名無しさん
23/03/20 18:39:22.68 meQKyq+v.net
どうやっても動かない

84:デフォルトの名無しさん
23/03/20 18:50:02.44 YF6r0iTp.net
クソスレ上げんなしね低能

85:デフォルトの名無しさん
23/03/20 19:13:33.45 meQKyq+v.net
すいませんね^^

86:デフォルトの名無しさん
23/03/20 20:14:03.44 meQKyq+v.net
動かねぇ。最初からプログラム書きなおします出来たら書き込みします

87:デフォルトの名無しさん
23/03/20 20:17:49.69 meQKyq+v.net
たぶん明日書き込みます。タイピング遅いんで。ちな寿司打お手軽コース最高記録が60円お得の雑魚なんで

88:デフォルトの名無しさん
23/03/20 20:36:38.17 r9gDoaWx.net
コードをアタッチしてないことはないよね？

89:デフォルトの名無しさん
23/03/20 20:48:27.57 meQKyq+v.net
>>88
なにそれちゃんとunityにコード読み込ませてますよあの青いぐるぐる

90:デフォルトの名無しさん
23/03/20 23:01:13.42 dpT4FG92.net
paiza意味検索したけど
検索アプリが消されています
でも主はスマホ持ってないだお
スマホは持っていますが
支離滅裂なんだけど、こんな奴にかまってるやつもおかしいよ
とりあえず下げろよ。板の荒らしかよ

91:デフォルトの名無しさん
23/03/21 00:17:13.74 4irMO5jk.net
>>90
無視すればよくね？

92:デフォルトの名無しさん
23/03/21 00:18:10.66 4irMO5jk.net
>>89
インスペクターのところにスクリプトがちゃんと入ってるかどうか

93:デフォルトの名無しさん
23/03/21 00:20:23.59 4irMO5jk.net
とりあえず最初はひろはすって人のブロック崩しの動画でもやってみたらアタッチがどうとかの基本的な話が出てくるからやってみるといいよ

94:デフォルトの名無しさん
23/03/21 09:16:49.20 /SnQEqhC.net
ひろはすさんの動画を見てきます。

95:デフォルトの名無しさん
23/03/21 09:20:25.41 /SnQEqhC.net
>>90
混乱させてしまってすいません。私はスマホに検索アプリが入ってません。今はPCでやっています。

96:デフォルトの名無しさん
23/03/21 09:21:31.99 /SnQEqhC.net
まあ、スマホじゃなくてタブレットだけど。どっちでもいいか。

97:デフォルトの名無しさん
23/03/22 15:32:14.46 CNhnHWJa.net
このスレ作って自分がどれだけプログラム構文が下手なのかわかりました本屋で本買ってまた現れます。
ねちがえて湿布はって寝てました

98:デフォルトの名無しさん
23/03/22 16:17:00.60 CNhnHWJa.net
朝から首がいたい

99:デフォルトの名無しさん
23/03/23 11:00:26.84 AQHpwrnP.net
99記念
URLﾘﾝｸ(eater.net)

100:デフォルトの名無しさん
23/03/23 11:33:14.56 4E7FceMl.net
>>97
がんばれー

101:デフォルトの名無しさん
23/03/25 08:15:40.37 Sb6rBF5P.net
今までありがとうございます。最初から学習します。学習し終わったらまた新しいスレ作ります。
そん時はよろb。ちなまだ首痛い。

102:デフォルトの名無しさん
23/03/25 08:16:57.14 Sb6rBF5P.net
優しく接していたきありがとうございます。

103:デフォルトの名無しさん
23/03/25 08:17:23.91 Sb6rBF5P.net
さいなら

104:デフォルトの名無しさん
23/03/25 18:18:33.91 4oBR44jO.net
>>101
👍