Feature/start turn1 gui

[MartinStenhagen]

Summary by CodeRabbit

Release Notes

• Performance
  • Optimized turn tracking and HUD display refresh system to improve responsiveness and reduce latency during active gameplay

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

This PR introduces per-traveler turn caching in TravelGameController by adding a map to store last turn numbers. A new refreshTurnCache method syncs data from JourneyService before HUD rendering, and a corresponding service method fetches max turn numbers for multiple travelers in a single query.

Changes

Cohort / File(s)	Summary
Turn Caching Infrastructure src/main/java/org/example/TravelGameController.java, src/main/java/org/example/service/JourneyService.java	Added lastTurnByTravelerId cache field and refreshTurnCache() method in controller; integrated cache refresh into syncHudAndMap() workflow. New service method getLastTurnNumbersForPlayers() batch-fetches max turn numbers per traveler, returning empty map for null/empty input and defaulting missing travelers to turn 0. HUD label now derives current turn value from cache.

Sequence Diagram

sequenceDiagram
    participant UI as UI/Event
    participant Controller as TravelGameController
    participant Cache as lastTurnByTravelerId
    participant Service as JourneyService
    participant DB as Database

    UI->>Controller: syncHudAndMap()
    Controller->>Controller: refreshTurnCache()
    Controller->>Service: getLastTurnNumbersForPlayers(travelerIds)
    Service->>DB: Query MAX(turnNumber) per traveler
    DB-->>Service: Turn data
    Service-->>Controller: Map<Long, Integer>
    Controller->>Cache: Store results
    Controller->>UI: Update currentTurnLabel from cache

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• fixed the gui part after merge #21: Introduces syncHudAndMap() method which this PR hooks the new refreshTurnCache() call into for turn data synchronization.

Poem

🐰 A cache springs forth with hoppy delight,
Per traveler turns now stored just right,
The HUD syncs smooth, no queries in sight,
Batch-fetching turns keeps the game light,
From database hops to UI so bright! 🎮

✨ Finishing touches

• 📝 Generate docstrings

---

---

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 9cea17c and 73bd4ec.

📒 Files selected for processing (2)

• src/main/java/org/example/TravelGameController.java
• src/main/java/org/example/service/JourneyService.java

_{✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}