jquery hoverメソッドを機能させたい。

Question

前提・実現したいこと

下記コードが機能しません。原因及び解決方法を教えて頂きたいです。
ここに質問の内容を詳しく書いてください。
（例）PHP(CakePHP)で●●なシステムを作っています。
■■な機能を実装中に以下のエラーメッセージが発生しました。

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

エラ〜メッセージはないが、反応しない。

エラーメッセージ
エラーは出ません。
### 該当のソースコード
$(function() {
    $(document).ready(function() {
    $(window).scroll(function() {
　　　var aa = $(this).scrollTop();
     if (aa　> 0 ) {
        $('.menu-navi ui li a ').hover(function(){
          $('this').css('color','#3366FF');
        },function(){
          $('this').css('color','');
        });
        };
    });
  });
});
```ここに言語名を入力
ソースコード

jquery

試したこと

ここに問題に対して試したことを記載してください。

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

<script src="https://ajax.googleapis.com/ajax/libs/jquery/3.5.1/jquery.min.js"></script> <script> ここにより詳細な情報を記載してください。

Comments

[m.ts10806]

HTMLどうなってますか？
あと、同じ意味の記述が重なってたりhoverに付与する条件が厳しかったり
デバッグすれある程度見えてきそうに思います。

[Kokjfnvena]

ご連絡ありがとうございます。
HTML下記の記載となっております。
<div class = "menu-navi">
<ul>
<li>
<a href = "./home.html">HOME</a>
</li>
<li>
<a href = "./nextevent.html">NEXT EVENT</a>
</li>
<li>
<a href = "./backnumber.html">BACK NUMBER</a>
</li>
<li>
<a href = "./reservation.html">イベント予約はこちら</a>
</li>
<ul>
</div>
以上、宜しくお願い致します。

[m.ts10806]

質問は編集できますので。

[m.ts10806]

あと
$(function() {
と
$(document).ready(function() {
は同義ですし、windowでイベント発火したいのでしたら不要です。

[Kokjfnvena]

ご指摘ありがとうございます。
改善試みます。
また何かありましたら宜しくお願い致します。

[m.ts10806]

質問は編集できますので。 htmlを質問に追記してください。

Answer 1

コメントに書きましたが、イベントの条件が厳しい、というか滅茶苦茶になってます。

$(function() {                                        //ドキュメント読み込み時のイベント付与
    $(document).ready(function() {                    //ドキュメント読み込み時のイベント付与
    $(window).scroll(function() {                     //画面スクロールイベント付与
　　　var aa = $(this).scrollTop();                    //windowのスクロール位置取得
     if (aa　> 0 ) {                                  //windowのスクロール位置判定
        $('.menu-navi ui li a ').hover(function(){    //.menu-navi ui li aのhoverイベント付与
          $('this').css('color','#3366FF');  　　　　　//'this'に対するcss付与（''で囲むと文字列で、.も#もなければ「タグ」に対する指定）
        },function(){  　　　　　                      //.menu-navi ui li aからアウトしたときの処理
          $('this').css('color','');  　          　　//'this'に対するcss付与
        });
        };
    });
  });
});

かなりの渋滞具合が分かると思います。

同義のイベントを重ねる意味もあまりありません。
１つ１つのイベントを整理し、きちんと役割を明確にしましょう。

ロジック的にはこうあるべき。

• スクロール時に「現在のポイント」をグローバルに保存
• hover時に「現在のポイント」で判定させる。

未検証で、ちょっと冗長ですが、イメージはこういう感じ。

let point = 0;
$(window).scroll(function(e) {
    point = $(e.currentTarget).scrollTop();
});

$(function() {
    initNavi();

    $('.menu-navi ui li a ').hover(function(e){
        initNavi();
        if (point > 0 ) {
            $(e.currentTarget).css('color','#3366FF');
        }
    });
});

const initNavi = function(){
    $('.menu-navi ui li a ').css('color',''); //初期化
}

Answer 2

scrollするたびにhoverを追加するのが目的でなければ
実装方法を変えた方がいいでしょう