スワイプ処理における不具合の原因がわからない

Question

スマホのスワイプ処理を作りましたが
横にフリックしたのにもかかわらず
稀に始点と終点の座標が全く同じになるという不具合が出てしまいます（１５０回スワイプして１回起きるぐらい
実機のハードウェア不具合という可能性も無きにしも非ずですがどうしてもわからず

このスクリプトだと横あるいは下にフリックして不具合が発生すると上にフリックすることになっています
mousePos.ContainsKeyは複数のタップ開始位置とfingeridを保存するために
処理を追加していますが、すこしややこしい処理になってしまったかもしれません

タップの開始点と終点の座標が全く同じになってしまう原因を探しています
よろしくお願いいたします。

スクリプトは

private Dictionary<int,Vector2> mousePos = new Dictionary<int,Vector2>();

void GetTouchInput()
	{
		if (Input.touchCount > 0)
			{
				for(int i = 0;i < Input.touchCount;i++){
					
					Touch touch = Input.GetTouch(i);

					if (touch.phase == TouchPhase.Began)
					{
						if (mousePos.ContainsKey(touch.fingerId))
						{
							mousePos.Remove(touch.fingerId);
						}

						mousePos.Add(touch.fingerId,touch.position);
						
					}
					else if (touch.phase == TouchPhase.Ended)
					{
						if (mousePos.ContainsKey(touch.fingerId))
						{
							Vector2 MouseDownScreenPos = mousePos[touch.fingerId];
							Vector2 MouseUpScreenPos = touch.position;

							SwipeCheck(MouseDownScreenPos, MouseUpScreenPos);
							mousePos.Remove(touch.fingerId);
						}
						
					}else if (touch.phase == TouchPhase.Canceled)
					{
						if (mousePos.ContainsKey(touch.fingerId))
						{
							mousePos.Remove(touch.fingerId);
						}
						
					}
				}
			}
	}

	void SwipeCheck(Vector2 MouseDownPos,Vector2 MouseUpPos)
	{
		
		float distanceX = MouseUpPos.x - MouseDownPos.x;
		float distanceY = MouseUpPos.y - MouseDownPos.y;

		if (Mathf.Abs(distanceX) > Mathf.Abs(distanceY))
        {
			if (MouseDownPos.x > MouseUpPos.x)
            {
				Debug.Log("Left");
			}
            else
            {
				Debug.Log("Right");
			}
        }
        else
        {
			if (MouseDownPos.y > MouseUpPos.y)
			{
				Debug.Log("Down");
			}
			else
			{
				Debug.Log("Up");
			}
		}
		
	}

Answer 1

デバッグを直接手伝うつもりはないのであくまでデバッグの考え方の紹介になりますが、

まずこのSwipe検出コードをテスト可能な形に切り出しましょう。

例えば、Input周りを抽象化した上で、Inputのデータを外部のコードから注入してSwipe検出処理の妥当性を検査できるようにします。人間が実際にタッチ操作せずとも入力操作をエミュレートしたデータを与えることが出来れば、Swipe検出処理をテスト可能な状態にできます。

テストすべき内容としては、Inputの値を与えるごとにSwipe処理内部の状態が変更される可能性がありますから、入力の値を与えた後の想定される状態と実際の状態を突合し、想定される状態と異なる地点でテストを失敗させれば、問題が起きた「状態」と「入力値」の組み合わせがわかるはずです。

処理の妥当性が担保されたら、次は入力データ自体を検証して、仮に妥当ではないデータが渡ってきている（意図せずFingerIdが変わったり、positionが飛ぶ等）ようなら値の補正を考えます。

Answer 2

こちらで動作確認したわけではなく
コードからの類推なのですが
positionオブジェクトのポインタの値が変わってしまうタイミングがあるのかなと感じました。
Dictionaryに保存するために下記の様に別途ポインタを作成させて上げると確実かなと

// 18行目
mousePos.Add(touch.fingerId,touch.position.Clone())

この場合、Dicityonaryから値を取り出す時はCASTしないとだめかも知れません。

// 25行目
Vector2 MouseDownScreenPos = (Position)mousePos[touch.fingerId];