ObserveEveryValueChangedがStartメソッドを通った時点で実行されてしまう

Question

DOTweenAnimationをUniRxのObserveEveryValueChangedを利用して動かしたい

下記のスクリプトをDOtweenAnimationがついたオブジェクトに張り付けて、PlayerScript内のDOtweenAnimationをDORestart();で動かそうとしています。
**DORestart()**自体は__ObserveEveryValueChanged__でplayerHPの変化を検知して実行したいです。

しかし、下記の画像の通りスタートメソッドを通った時点でなぜかDORestart()を実行してしまっています。

これがなぜ起きるのか教えていただきたいです。

該当のソースコード

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UniRx;
using DG.Tweening;
public class PlayerHPShaker : MonoBehaviour
{
    private int playerHP=0;
    private DOTweenAnimation dot;
    void Awake()
    {
        playerHP=PlayerScript.Instance.Player_HP; 
        Debug.Log("a"+playerHP);   
    }
    void Start()
    {
        Debug.Log("s"+playerHP);
        dot = GetComponent<DOTweenAnimation>();
        dot.autoPlay=false;
        this.ObserveEveryValueChanged(x => x.playerHP)
            .Where(_ => _ > 0)
            .Subscribe(_ => PlayHPDOTween()); 
    }
    void Update()
    {
        Debug.Log("u"+playerHP);
        playerHP=PlayerScript.Instance.Player_HP;//ここで変数が変化
    }
    void PlayHPDOTween(){
        Debug.Log("揺らす");
        dot.DORestart();
    }
}

試したこと

・上記にある通りDebug.Logで確認したところAnimationのAutoPlayが原因ではないです。
・whereの部分を.Where(x => x),.Where(x => x>0)に変えてもだめでした。

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

unity2019.4.8

Answer 1

UniRxのコードを見ると初期値を必ず発行するようなので、不要な場合はSkip(1)で読み飛ばせばいいと思います。

this.ObserveEveryValueChanged(x => x.playerHP)
+   .Skip(1)
    .Where(_ => _ > 0)
    .Subscribe(_ => PlayHPDOTween()); 

参考：ObserveEveryValueChangedのソースコード
https://github.com/neuecc/UniRx/blob/284d5c50d3f1ddd9fa7df3d382ea904732a9c2ff/Assets/Plugins/UniRx/Scripts/UnityEngineBridge/ObserveExtensions.cs#L67

Comments

[Beat-san]

おっしゃる通り、該当部分を書き加えたところ想定した通りの挙動をするようになりました。ありがとうございました。

[tor4kichi]

一点補足ですが、`ObserveEveryValueChanged`はフレーム毎に値を読み取る動作をするため、基本的にパフォーマンスの改善余地がある書き方です。

`ReactivePropertySlim<T>`を利用すると変更通知可能なプロパティを作れます。ObserveEveryValueChangedよりも軽量に動作しますので、試作段階で`ObserveEveryValueChanged`使う程度で、なるたけ`ReactivePropertySlim<T>`の利用をお勧めします。

[Beat-san]

そうなんですね！てっきりObserveEveryValueChangedが優れたものとばかりに思って調べていたので、目から鱗です。ご親切に教えてくださり、ありがとうございます！