Files
Mimante/Mimante/Dialogs/StatsCleanupDialog.xaml.cs
T
Alby96andClaude Opus 5 1d578debe2 Togli dal percorso caldo la rilettura dell'intero storico
Sei modifiche chieste in questa sessione, di cui una pesa piu' di tutte le altre.

CompletedAuctionsStore rileggeva e rideserializzava l'intero file a ogni chiamata:
misurati 72 ms su 5876 record, 3,9 MB. Append pagava quel prezzo piu' la
riserializzazione e la riscrittura completa per cambiare un record solo, 120 ms
in tutto, e la chiamata arriva da un gestore di eventi di MainWindow, quindi dal
thread dell'interfaccia, che restava fermo a ogni asta conclusa. Il recupero
delle puntate dei vincitori lo faceva in un ciclo: 4675 aste erano 7,2 minuti di
solo rimescolamento JSON e 17,8 GB scritti sul disco.

Adesso c'e' una copia in memoria invalidata sulla data di modifica del file, cosi'
resta giusta anche se il file viene sostituito da fuori (ripristino di un backup),
e un AppendMany che applica molti record con un salvataggio solo, con un indice
per id perche' il caso per cui esiste non torni quadratico. Il recupero salva a
blocchi di cinquanta e deposita comunque quello che ha in mano quando esce, anche
per annullamento: le richieste gia' fatte sono state pagate al server.

Misurato sullo stesso file: 95 volte piu' veloce, 7,2 minuti diventano 4,5
secondi, 17,8 GB diventano 0,36 GB.

Le altre cinque:

- Il setup si chiama AutoBidder_<versione>.exe.
- I pulsanti di stop sono rossi, sia quello globale sia quelli per asta nella
  griglia: erano gli unici tre senza colore mentre avvia e osserva ce l'avevano.
- La pulizia dello storico sa togliere le aste seguite solo in parte. Non serve
  nessun campo nuovo: il prezzo finale in centesimi e' il totale delle puntate
  dell'asta, quindi il rapporto con i reset osservati dice quanta asta si e'
  vista, e la regola vale anche sullo storico gia' raccolto. Verificata sui dati
  veri: su 5876 aste nessuna supera copertura 1,02, che e' quanto ci si aspetta
  se l'identita' e' giusta.
- La griglia ricicla i contenitori di riga invece di ricrearli.
- Il costo puntata gia' esisteva col predefinito giusto: al suggerimento e' stato
  aggiunto perche' 0,20 e' il numero da tenere.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-08-06 14:48:54 +02:00

126 lines
4.9 KiB
C#

using System;
using System.Collections.Generic;
using System.Linq;
using System.Windows;
using AutoBidder.Models;
using AutoBidder.Utilities;
namespace AutoBidder.Dialogs
{
/// <summary>
/// Sceglie cosa considerare incompleto nello storico e lo toglie.
///
/// <para>Il conto si aggiorna a ogni spunta, <b>prima</b> di toccare qualunque cosa: una
/// pulizia è irreversibile, e l'unico modo di renderla una decisione informata è mostrare
/// quante righe cadrebbero e per quale motivo mentre si sceglie. La copia di sicurezza
/// prima di scrivere è la seconda rete: le statistiche sono mesi di raccolta che nessun
/// pulsante ha il diritto di far sparire per un clic distratto.</para>
/// </summary>
public partial class StatsCleanupDialog : Window
{
private readonly List<CompletedAuctionRecord> _records;
public StatsCleanupDialog(List<CompletedAuctionRecord> records)
{
InitializeComponent();
_records = records ?? new List<CompletedAuctionRecord>();
Loaded += (_, _) => Refresh();
}
/// <summary>Righe rimosse, dopo la conferma. 0 se l'utente ha annullato.</summary>
public int RemovedCount { get; private set; }
/// <summary>Dove è finita la copia di sicurezza.</summary>
public string? BackupPath { get; private set; }
private StatsMaintenance.CleanupOptions ReadOptions()
{
var options = new StatsMaintenance.CleanupOptions
{
RemoveWithoutPrice = ChkNoPrice.IsChecked == true,
RemoveInconsistentBids = ChkBadBids.IsChecked == true,
RemoveDuplicates = ChkDuplicates.IsChecked == true,
RemoveWithoutWinnerBids = ChkNoWinnerBids.IsChecked == true,
RemoveWithoutWinner = ChkNoWinner.IsChecked == true,
RemoveWithoutValue = ChkNoValue.IsChecked == true
};
// Una soglia illeggibile o fuori scala non deve diventare "togli tutto":
// il filtro resta spento finché il numero non ha senso.
if (ChkCoverage.IsChecked == true &&
double.TryParse(CoverageBox.Text.Trim().Replace(',', '.'),
System.Globalization.NumberStyles.Float,
System.Globalization.CultureInfo.InvariantCulture, out var percento) &&
percento is > 0 and <= 100)
{
options.MinObservedCoverage = percento / 100.0;
}
if (ChkOlderThan.IsChecked == true &&
int.TryParse(DaysBox.Text.Trim(), out var days) && days > 0)
{
options.OlderThan = DateTime.Now.AddDays(-days);
}
return options;
}
private void Option_Changed(object sender, RoutedEventArgs e) => Refresh();
private void Refresh()
{
if (SummaryText == null) return;
var report = StatsMaintenance.Preview(_records, ReadOptions());
SummaryText.Text = report.Removed == 0
? $"Nessuna delle {report.Total} aste verrebbe tolta."
: $"Verrebbero tolte {report.Removed} aste su {report.Total}. " +
$"Ne resterebbero {report.Kept}.";
DetailText.Text = report.ByReason.Count == 0
? ""
: string.Join("\n", report.ByReason
.OrderByDescending(kv => kv.Value)
.Select(kv => $"• {kv.Value} {kv.Key}"));
CleanButton.IsEnabled = report.Removed > 0;
}
private void Clean_Click(object sender, RoutedEventArgs e)
{
var options = ReadOptions();
var report = StatsMaintenance.Preview(_records, options);
if (report.Removed == 0) return;
var answer = MessageBox.Show(this,
$"Tolgo {report.Removed} aste su {report.Total}.\n\n" +
"L'operazione non è reversibile, ma prima viene salvata una copia " +
"dello storico nella cartella dei backup.\n\nProcedo?",
"Pulizia dello storico", MessageBoxButton.YesNo, MessageBoxImage.Warning);
if (answer != MessageBoxResult.Yes) return;
try
{
BackupPath = CompletedAuctionsStore.BackupNow();
// La stessa risposta usata per il conto: contare con una regola e
// cancellare con un'altra è il modo classico di far sparire dati che
// nessuno voleva perdere.
var kept = StatsMaintenance.Apply(_records, options);
CompletedAuctionsStore.ReplaceAll(kept);
RemovedCount = report.Removed;
DialogResult = true;
}
catch (Exception ex)
{
MessageBox.Show(this, $"Pulizia non riuscita: {ex.Message}",
"Pulizia dello storico", MessageBoxButton.OK, MessageBoxImage.Error);
}
}
}
}