質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

Unity

Unityは、Unity Technologiesが開発・販売している、IDEを内蔵するゲームエンジンです。主にC#を用いたプログラミングでコンテンツの開発が可能です。

Q&A

解決済

2回答

3603閲覧

if文で反応しないところがあります。

manmanJ

総合スコア19

C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

Unity

Unityは、Unity Technologiesが開発・販売している、IDEを内蔵するゲームエンジンです。主にC#を用いたプログラミングでコンテンツの開発が可能です。

0グッド

0クリップ

投稿2018/07/25 12:59

編集2018/07/26 08:30
void Update() { if (Game_01 == 0)//これが原因で { if (Game_03 == 0) { Hide02(); }else{ Show02(); } } private void Show02() { if (Game_03 == 1) { } if (Game_03 == 2) { } if (Game_01 == 2) { //ここが反応しません。 } } private void Hide02() {//ここで非表示} }

public intでここに変数2は代入できてましたが最初のifが邪魔して反応してくれません。
if文でなくてもいいので訂正お願いします。
途中経過

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

papinianus

2018/07/26 02:56

途中経過が見れません
guest

回答2

0

ベストアンサー

エスパーします。こういうことではないでしょうか?

  • Game_01が0 または Game_03が0だったら、隠しアクション(Hide02)をする
  • それ以外のときは、見せアクション(Show02)をする

csharp

1void Update() 2{ 3 if (Game_01 == 0 || Game_03 == 0) 4 { 5 Hide02(); 6 } 7 else 8 { 9 Show02(); 10 } 11 private void Show02() 12 { 13 14 if (Game_03 == 1) 15 { } 16 if (Game_03 == 2) 17 { } 18 if (Game_01 == 2) 19 { //ここが反応しません。 20 } 21 } 22 private void Hide02() 23 { 24 //ここで非表示 25 } 26}

質問と関係ない補足ですが、Game_01とか、Show02とかっている名前はやめたほうがいいです。変数や関数には意味を表現する名前を付けたほうがいいです。
teratailに質問するからそういう名前にしているのだとしたら方針が間違っています。全体の実装を知らない助言者には(名前で)意図を伝えるべきです。

投稿2018/07/26 03:04

papinianus

総合スコア12705

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

manmanJ

2018/07/26 08:27

ありがとうございます。 if (Game_03 >= 1 || Game_01 >= 1) { Show02(); } else { Hide02(); } でいけました。 名前はまだ大まかに作っていて決めてないので単純な物にしてました。 if (Game_01 == 0 || Game_03 == 0)の || はorの役割として考えても大丈夫でしょうか?
papinianus

2018/07/26 09:03

`||`はorの役割です。 大まかでも何でもこれから何かを作ろうとしているイメージがあるのですよね。GetGetData()みたいな無意味な名前でその中でSetもしているゴミの山をメンテナンスした経験から言いますが、単純な名前をつけていると、歯止めがかからずにぐっちゃぐちゃになります。 書き始めて気付くことも色々あるとは思いますが、最初の時点で適切な名前をつけておかないと、本来予定していたこととはちがう処理をまぜてスパゲッティ化していることに気付けません。 もし、ざっくりのイメージすら持ててないなら、イメージをもってからコード化したほうがいいと思います。
manmanJ

2018/07/27 04:25 編集

そうですね、今少しこんがらがってました。 一応イメージができてきましたからこれからちょっとずつ明確化させていきたいと思います! ありがとうございました! if (Game_01 == 0||Tree_01 == 0) {if (Reality_01== 0||Tree_01 == 0) { Show02(); } } もしこうゆうTree_01 == 0の場合の無条件の文章は簡略できませんか?
papinianus

2018/07/27 06:05

条件が分からないのですが、 if(Game_01 == 0 || Reality_01 == 0 || Tree_01 == 0) { Show02(); } でしょうか(どれか1つが0ならShow02()を実行ですか)?
guest

0

C#

1void Update() 2{ 3 if (Game_01 == 0)//これが原因で 4 { 5 if (Game_03 == 0) 6 { 7 Hide02(); 8 }else{ Show02(); } 9 } 10 private void Show02() 11 { 12 13 if (Game_03 == 1) 14 { } 15 if (Game_03 == 2) 16 { } 17 if (Game_01 == 2) 18 { //ここが反応しません。 19 } 20 } 21 private void Hide02() 22 {} 23} 24

あなたのソースのインデントをきちんと整形してみました。
で、これを見て、あなたの言ってる問題はなにかを考えてみてください

投稿2018/07/25 13:04

y_waiwai

総合スコア87719

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

manmanJ

2018/07/25 13:19

最初のif (Game_01 == 0)のせいでif (Game_01 == 2)が反応しないんだと思うけど if文消すとShow02,Hide02が反応しなくなってしまいました。 Hide02はGame_03の方は反応できていたので大丈夫だと思いますが、 if以外でもこのShow02,Hide02にが反応する方法があればおしえてもらいたいです。
y_waiwai

2018/07/25 13:24

あなたのいう「ここが反応しません」、は、Show02関数の中にあります(関数内関数) で、そのShow02関数は、Game_01 == 0 のときにしか実行されません。 それをどうしたいんでしょうか
manmanJ

2018/07/25 13:38

Hide02の中で2つのテキストを非表示にしてるんですけど、 それはGame_01かGame_03が0の状態なのでif文でまとめれたんですけれど そうするとif (Game_01 == 2)が反応してくれなくて、これが反応するやり方があったら教えてもらいたいです!
y_waiwai

2018/07/25 13:41

そりゃGame_01が0の状態のときに、Game_01 == 2 は成立するはずもありません。 また、Game_01が2の状態のときには、Hide02関数は呼び出されませんねー
manmanJ

2018/07/25 14:01

途中経過をのせたんですけど、みれますか? if以外でも訂正できるのであればおねがいます!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問