Verbesserung: autoSaveSessions() ist Dead Code und sollte entfernt werden #14

New Issue

Jannis · 2026-06-10T11:46:28Z

Jannis commented

2026-06-10 11:46:28 +00:00

Code-Qualität / Verbesserung

Beschreibung

Die Funktion autoSaveSessions() in main.go existiert, wird aber nirgends aufgerufen. Das Auto-Saving läuft bereits über den Cron-Job. Die Funktion ist dadurch toter Code (dead code) und verursacht Verwirrung.

Problem

func autoSaveSessions() {
    ticker := time.NewTicker(5 * time.Minute)
    defer ticker.Stop()
    for range ticker.C {
        fmt.Println("Auto-Save")
        for uid, sess := range sessions {
            processAndSave(uid, sess)
        }
    }
}

Diese Funktion:

Wird nie aufgerufen (kein go autoSaveSessions() in main())
Dupliziert die Logik des Cron-Jobs
Würde dieselbe Race Condition haben wenn aufgerufen
Trägt zur unnötigen Komplexität des Codes bei

Empfehlung

Die Funktion komplett entfernen. Das Auto-Saving ist bereits durch den robfig/cron-Job abgedeckt.

Alternativ: Falls man den Ticker-Ansatz bevorzugt, die Cron-Dependency entfernen und autoSaveSessions() stattdessen als go autoSaveSessions() in main() starten.

## Code-Qualität / Verbesserung ### Beschreibung Die Funktion `autoSaveSessions()` in `main.go` existiert, wird aber **nirgends aufgerufen**. Das Auto-Saving läuft bereits über den Cron-Job. Die Funktion ist dadurch toter Code (dead code) und verursacht Verwirrung. ### Problem ```go func autoSaveSessions() { ticker := time.NewTicker(5 * time.Minute) defer ticker.Stop() for range ticker.C { fmt.Println("Auto-Save") for uid, sess := range sessions { processAndSave(uid, sess) } } } ``` Diese Funktion: - Wird nie aufgerufen (kein `go autoSaveSessions()` in `main()`) - Dupliziert die Logik des Cron-Jobs - Würde dieselbe Race Condition haben wenn aufgerufen - Trägt zur unnötigen Komplexität des Codes bei ### Empfehlung Die Funktion komplett entfernen. Das Auto-Saving ist bereits durch den `robfig/cron`-Job abgedeckt. Alternativ: Falls man den Ticker-Ansatz bevorzugt, die Cron-Dependency entfernen und `autoSaveSessions()` stattdessen als `go autoSaveSessions()` in `main()` starten.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starfour/discord-go-bot#14