Card1とCard2のY座標が690の位置に移動したら、seikaicounterを１増やしたい

Question

前提

Unityで神経衰弱みたいなゲームを作ろうとしています。

実現したいこと

Card1とCard2のY座標が690の位置に移動したら、seikaicounterを１増やしたい。

発生している問題・エラーメッセージ

Card1とCard2のY座標が690になっても、seikaicounterが増えない。

該当のソースコード

using UnityEngine;

public class GManager : MonoBehaviour
{
    public GameObject Card1;
    public GameObject Card2;
    public GameObject Card3;
    public GameObject Card4;
    public GameObject Card5;
    public GameObject Card6;
    public GameObject Card7;
    public GameObject Card8;
    [SerializeField] private int selectcounter = 0;
    [SerializeField] private int seikaicounter = 0;

    // Start is called before the first frame update
    void Start()
    {
        
    }

    // Update is called once per frame
    void Update()
    {
        Transform Card1transform = Card1.transform;
        Vector3 Card1pos = Card1transform.position;

        Transform Card2transform = Card2.transform;
        Vector3 Card2pos = Card2transform.position;

        Transform Card3transform = Card3.transform;
        Vector3 Card3pos = Card3transform.position;
        
        Transform Card4transform = Card4.transform;
        Vector3 Card4pos = Card4transform.position;

        Transform Card5transform = Card5.transform;
        Vector3 Card5pos = Card5transform.position;

        Transform Card6transform = Card6.transform;
        Vector3 Card6pos = Card6transform.position;

        Transform Card7transform = Card7.transform;
        Vector3 Card7pos = Card7transform.position;
        
        Transform Card8transform = Card8.transform;
        Vector3 Card8pos = Card8transform.position;

        if(Card1pos.y == 690 && Card2pos.y == 690)
        {
            Debug.Log("nnn");
            seikaicounter += 1;
        }
        
        if(selectcounter == 3)
        {
            selectcounter = 0;
        }
    }

    public void Card1Click()
    {
        selectcounter += 1;
        Transform Card1transform = Card1.transform; //transformを取得
        Vector3 Card1pos = Card1transform.position; //座標を取得
        Card1pos.y += 10;
        Card1transform.position = Card1pos; //座標を設定
    }

    public void Card2Click()
    {
        selectcounter += 1;
        Transform Card2transform = Card2.transform;
        Vector3 Card2pos = Card2transform.position;
        Card2pos.y += 10;
        Card2transform.position = Card2pos;
    }

    public void Card3Click()
    {
        selectcounter += 1;
        Transform Card3transform = Card3.transform;
        Vector3 Card3pos = Card3transform.position;
        Card3pos.y += 10;
        Card3transform.position = Card3pos;
    }

    public void Card4Click()
    {
        selectcounter += 1;
        Transform Card4transform = Card4.transform; //transformを取得
        Vector3 Card4pos = Card4transform.position; //座標を取得
        Card4pos.y += 10;
        Card4transform.position = Card4pos;
    }

    public void Card5Click()
    {
        selectcounter += 1;
        Transform Card5transform = Card5.transform; //transformを取得
        Vector3 Card5pos = Card5transform.position; //座標を取得
        Card5pos.y -= 10;
        Card5transform.position = Card5pos;
    }

    public void Card6Click()
    {
        selectcounter += 1;
        Transform Card6transform = Card6.transform; //transformを取得
        Vector3 Card6pos = Card6transform.position; //座標を取得
        Card6pos.y -= 10;
        Card6transform.position = Card6pos;
    }

    public void Card7Click()
    {
        selectcounter += 1;
        Transform Card7transform = Card7.transform; //transformを取得
        Vector3 Card7pos = Card7transform.position; //座標を取得
        Card7pos.y -= 10;
        Card7transform.position = Card7pos;
    }

    public void Card8Click()
    {
        selectcounter += 1;
        Transform Card8transform = Card8.transform; //transformを取得
        Vector3 Card8pos = Card8transform.position; //座標を取得
        Card8pos.y -= 10;
        Card8transform.position = Card8pos;
    }
}

試したこと

if文が原因だと思い、調べてみましたが分かりませんでした。

補足情報（FW/ツールのバージョンなど）

2020.3.20f

Comments

[退会済みユーザー]

Unity のタグをつけてください。

[Kerupim]

一応補足ですが、浮動小数点数値型でも整数だけ扱う場合は、誤差が生じることはありません。
また浮動小数点同士の計算で発生する誤差は Mathf.Approximately を使って比較したほうが良いかと思います。
※何百回と計算するのであれば別の対応も必要ですが。

とりあえず成功時にデバッグログ確認を行うということをしているのであれば、
各Click関数内にもデバッグログを追加して座標がどうなっているかも見たほうがいいかと思います。

[退会済みユーザー]

Unity のタグをつけてください。Unity に興味のある人の目を引いて回答を得やするなるというあなたにとってのメリット、自分のように興味のない人はタグを見ただけでスルーできるというメリットがあります。

Answer 1

Vector3の各値はfloat型ですが、そのような浮動小数点型で690ジャストになることはほとんどありません。
あるとしたら、690を直接代入した時くらいで、それ以外のケースだと何らかの誤差でずれることがほとんどです。

この場合、689.9～690.1のようにある程度範囲を設けておくのが一般的です。

カードを扱っているのであれば、そのカードの大きさ分の範囲を設定するのがいいでしょう。

Comments

[Ito]

コメントありがとうございます。

if(Card1pos.y >= 689.1 && Card1pos.y <= 690.9)
{
Debug.Log("nnn");
seikaicounter += 1;
}

このようにしてみたのですが、うまく動かず、調べてみたのですが分かりませんでした。問題点があればコメントお願いします。

[fiveHundred]

申し訳ないのですが、解決していないのであれば、ベストアンサーを設定しないでください。
他の人からは解決済みに見えてしまうため、回答をほとんど得られなくなります。
解除できたはずなので、それを行ってください。

Answer 2

Card1,Card2のYの初期値は0か、あるいは10の倍数でなければ条件に合致することは無い。
インスペクタに表示される座標値はローカル座標であって、positionはワールド座標を扱う為、インスペクタで0だからといってposition.yが0である保証は無い。

690ドット移動したら終了、としたいのか座標値として690が絶対の基準で、そこに達したらなのかでまた判定も変わるけど、座標値を絶対の基準として扱うなら単純に

if(Card1pos.y >= 689.1)
{
Debug.Log("nnn");
seikaicounter += 1;
}

で判定すれば良い

Comments

[Ito]

ありがとうございます。解決できました。