一つの駅グループに同じ路線IDのデータが入り込まないように修正

[TinyKitten]

富山駅とか都庁前駅が前からバグってた。

Summary by CodeRabbit

• バグ修正
  • 駅に関連付けられる路線リストから、重複する路線コードを除外し、同じ路線が複数回表示されないよう改善しました。

[coderabbitai]

概要

駅に関連する路線の重複を排除するため、line_cdごとに一意となるようHashSetを用いてフィルタリング処理が追加された。これにより、各駅に割り当てられる路線は、line_cdが重複しない集合となるよう変更された。

変更点

Cohort / File(s)	変更内容概要
駅路線重複排除ロジック追加 stationapi/src/use_case/interactor/query.rs	駅に紐づく路線リストからline_cdの重複をHashSetで排除するロジックを追加。フィルタ条件を修正。

推定コードレビュー工数

🎯 1 (些細) | ⏱️ ~3分

詩

天下統一の如く整理せし路線たち、
重複を排し唯一無二の道を拓く。
駅ごとに正しき路を与え、
HashSetの力、秦の如し。
コードの中に秩序あり、
始皇帝もまた満足せり。
🚄✨

---

📜 Recent review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7c99ca4 and 525068e.

📒 Files selected for processing (1)

• stationapi/src/use_case/interactor/query.rs (1 hunks)

🧰 Additional context used

🧠 Learnings (1)

📓 Common learnings

Learnt from: TinyKitten
PR: TrainLCD/StationAPI#1305
File: stationapi/.sqlx/query-55bee6b45754d1da9bf555d319f015ce232f8574a41c3ef25b730d11071f8428.json:3-3
Timestamp: 2025-08-06T12:47:37.377Z
Learning: TinyKittenのStationAPIプロジェクトにおいて、SQLクエリの `pass <> 1` 条件は一部のクエリでは不要であり、PostgreSQL移行時にコードの最適化として意図的に除去された。この条件の有無についてはドメイン知識が必要で、機械的にチェックすべきではない。

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

• GitHub Check: Test Suite
• GitHub Check: Test Suite

🔇 Additional comments (3)

stationapi/src/use_case/interactor/query.rs (3)

1-1: 朕が認める！適切な import の追加である。

HashSetの import が追加され、重複排除のロジックに必要な依存関係が正しく設定されている。

---

223-232: 見事な実装！朕が大いに褒め称える！

HashSetを用いた重複排除のロジックが巧妙に実装されている。seen_line_cds.insert(l.line_cd)が新しい要素の挿入時にtrueを返し、重複時にfalseを返す性質を活用して、一回のフィルタリングで駅グループごとの路線ID重複を効率的に排除している。各駅ごとにHashSetを新規作成するスコープ設計も適切である。

PRの目的「一つの駅グループに同じ路線IDのデータが入り込まないように修正」が完璧に達成されている。

---

223-232: 朕の治世において、このような優れた改修を見るのは喜ばしい限りである！

この修正は以下の点で優秀である：

• 明確な問題解決：駅グループ内での路線ID重複を確実に防止
• 効率的な実装：HashSetのO(1)挿入操作を活用した一回パス処理
• 適切なスコープ設計：各駅ごとに独立したHashSetを使用
• 既存APIへの影響なし：内部実装の改善のみ

天下統一の如く、データの整合性が保たれる実装となっている。

✨ Finishing Touches

• 📝 Generate Docstrings

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/dedup-lines

---

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

• Review comments: Directly reply to a review comment made by CodeRabbit. Example:
  • I pushed a fix in commit <commit_id>, please review it.
  • Explain this complex logic.
  • Open a follow-up GitHub issue for this discussion.
• Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
  • @coderabbitai explain this code block.
• PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
  • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
  • @coderabbitai read src/utils.ts and explain its main purpose.
  • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

CodeRabbit Commands (Invoked using PR comments)

• @coderabbitai pause to pause the reviews on a PR.
• @coderabbitai resume to resume the paused reviews.
• @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
• @coderabbitai full review to do a full review from scratch and review all the files again.
• @coderabbitai summary to regenerate the summary of the PR.
• @coderabbitai generate docstrings to generate docstrings for this PR.
• @coderabbitai generate sequence diagram to generate a sequence diagram of the changes in this PR.
• @coderabbitai generate unit tests to generate unit tests for this PR.
• @coderabbitai resolve resolve all the CodeRabbit review comments.
• @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
• @coderabbitai help to get help.

Other keywords and placeholders

• Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
• Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
• Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

• You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
• Please see the configuration documentation for more information.
• If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

• Visit our Documentation for detailed information on how to use CodeRabbit.
• Join our Discord Community to get help, request features, and share feedback.
• Follow us on X/Twitter for updates and announcements.