レビュー依頼

[lemontea619]

countのやつ。Goliveから飛ぶとうまく動くけど、ファイルから飛ぶとなぜかJSが反応してくれない。

[yui-szk]

1. ファイルからhtmlを開くとmain.jsが機能しない

開発者ツールのコンソールを開くと、以下のように表示される

Access to script at 'file:"main.jsのパス"' from origin 'null' has been blocked
by CORS policy: Cross origin requests are only supported for protocol schemes:
http, data, isolated-app,chrome-extension, chrome-untrusted, https, edge.

CORS（Cross-Origin Resource Sharing）エラー ブラウザのセキュリティの問題でJavaScriptファイルがローカルファイルにアクセスしようとすると制限される。 ブラウザの拡張機能を使用したり、サーバー上で設定したりすると解決できるらしいが、コードに問題があるわけではないので大丈夫

2. main.js：演算処理部分

https://github.com/lemontea619/app/blob/1cfa671ef1b6475c92a0732b4d970c04e32bd116/main.js#L7 https://github.com/lemontea619/app/blob/1cfa671ef1b6475c92a0732b4d970c04e32bd116/main.js#L11

以下に変更した方が処理速度は速くなる

counter ++; counter --;

でも少しだからそこまで気にしなくてもいいはず

3. main.js：各Buttonの定義を使う直前にもってくる

https://github.com/lemontea619/app/blob/1cfa671ef1b6475c92a0732b4d970c04e32bd116/main.js#L1-L3

それぞれ使う直前にした方が読みやすい

例：

const counterPlusButton = document.getElementById("countUp");
counterPlusButton.addEventListener("click", () => {
  counter +=1;
  count.textContent = counter;
});
 ...

4. index.html：不要なコメントアウトは削除する

総評

おつかれ！！めっちゃよくできてて勉強になりました！ 初レビューなので色々あったらごめんねー