Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor: [Client][Watch/Header/clock] 時間表示方法を改善 #120

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

makeding
Copy link

@makeding makeding commented Jan 29, 2025

変更の種類

  • 不具合の修正
  • 新しい機能の追加
  • 改善・リファクタリング(機能は追加されないが、動作やコードを改善する破壊的でない変更)

チェックリスト:

  • 開発資料データベース設計 は読みましたか?
    • もともと自分用のメモですが、このプロジェクトの開発方針や設計などが記載されています。開発時の参考にしてください。
  • Git のコミットメッセージは開発資料に記載のフォーマットになっていますか?(重要)
    • このプロジェクト上のコミットメッセージは、基本的に全てこのフォーマットに従って記述されています(文字数は問いません)。
    • 正しいフォーマットになっていない場合、force push で必ずコミットメッセージを変更してから送信してください。
  • このプロジェクトのコーディング規約に従ったコードになっていますか?
    • コーディング規約は開発資料に記載されています。
    • そもそもコーディング規約と言えるほど大層なものではありませんが、一度目を通しておいてください。
  • プルリクエストは master ブランチに送信されていますか?
    • release ブランチはリリースした時にしか更新されません。必ず master ブランチに送信してください。

説明

Refactor: [Client][Watch/Header/clock] 時間表示方法を改善。

動機とコンテキスト

この変更は、setInterval を使用することによるパフォーマンス低下を防ぐために行いました。
0.1秒間隔での定期的な更新は無駄な処理を引き起こす可能性があり、スマートフォンやリソースが限られたデバイスでのパフォーマンスに影響を与えることがあります。
setTimeout を利用することで、無駄な処理を削減することができます。


私のMacでページが初期化される際 (created())、ページのレンダリングと時間表示の更新が競合し、時間が跳ねる問題が発生することがありました。
そして一秒のsetTimeoutをさせることで、ページの描画が安定してから時間更新を開始できるようにしています。
多分将来 requestAnimationFrame を活用することで、さらにスムーズなパフォーマンスを実現しました。

:)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant