11 Commits
Author SHA1 Message Date
Alby96andClaude Opus 5 4e835e220a Togli da sole le aste concluse, ma trattieni quelle costate qualcosa
Con l'aggiunta automatica accesa le aste concluse si accumulano a centinaia e
seppelliscono quelle vive, che e' il contrario di quello che serve guardando un
cruscotto. Il pulsante manuale "rimuovi le aste concluse" c'era gia', ma toglieva
tutto senza distinguere.

La rimozione automatica e' accesa di serie e toglie solo le aste che non hanno
piu' niente da dire. Restano invece, ciascuna con la sua opzione:

- quelle su cui si e' puntato, perche' sono le uniche che vale la pena riguardare
  e i soldi erano veri;
- quelle vinte, perche' c'e' da confermare l'acquisto su Bidoo e una vittoria
  sparita dall'elenco e' il modo piu' semplice di dimenticarsene;
- quelle di cui non si e' vista la fine, perche' sono esattamente quelle da
  controllare.

L'ordine dei controlli conta: prima le ragioni per trattenere, cosi' il motivo
scritto nel registro e' il piu' importante e non l'ultimo incontrato. Una vinta su
cui si e' anche puntato resta "vinta", che e' quella che richiede un'azione. Il
motivo finisce nel registro anche quando l'asta resta: senza, un'asta che sparisce
e una che non sparisce sarebbero entrambe inspiegabili.

La rimozione avviene come ultimo passo, dopo che storico, dossier e statistiche
del prodotto sono gia' scritti: a quel punto togliere l'asta dall'elenco non perde
niente.

Corretto anche un difetto introdotto ieri con la fascia oraria: la rigiocata sui
dossier usava le impostazioni vere, fascia compresa, e la fascia guarda l'orologio
di parete. Una rigiocata ripercorre aste gia' concluse e l'ora in cui la si lancia
non c'entra nulla con l'ora in cui quelle aste correvano: lanciata di notte
avrebbe rifiutato ogni puntata e riportato zero, che e' il tipo di risultato che
sembra vero e non lo e'. Adesso la rigiocata lavora su una copia delle
impostazioni con la fascia spenta, senza mutare quelle dell'utente.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-08-18 00:39:00 +02:00
Alby96andClaude Opus 5 480e423124 Togli la pubblicazione dalla obj/ condivisa: rompeva la compilazione di debug
Sintomo: una raffica di CS2001 "file di origine MainWindow.g.cs non trovato" su
otto file .g.cs, lanciando la normale compilazione di debug da VS Code, senza che
il codice fosse cambiato. Il classico guasto intermittente della compilazione WPF.

Causa: il target Pubblica usava la sola opzione -o, che sposta il risultato ma
lascia gli INTERMEDI nella obj/ del progetto, cioe' la stessa cartella che usa la
compilazione di debug. La compilazione WPF genera un progetto temporaneo
_wpftmp.csproj a ogni giro, e quando ci trova dentro lo stato di un'altra
configurazione smette di produrre le classi parziali dello XAML. Bastava
pubblicare mentre si compilava in VS Code, o subito prima, per rompere tutto.

Era gia' il rimedio adottato per i test, dove artifacts-path era stato messo
proprio per questo; il target Pubblica era rimasto indietro. Le due opzioni
convivono: artifacts-path sposta gli intermedi, -o continua a decidere dove
finisce l'eseguibile.

Verificato ripetendo tre volte la sequenza che rompeva - pubblica, poi compila in
debug - con la obj/ svuotata all'inizio: zero errori tutte e tre le volte, e la
pubblicazione non crea piu' obj/Release.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-08-07 16:27:40 +02:00
Alby96andClaude Opus 5 48096c7fc7 Porta avanti le impostazioni vecchie, o le correzioni di oggi non arrivano
Trovato provando davvero l'applicazione, non dai test. Il settings.json esistente
conteneva MaxRequestsPerSecond 40 e DefaultBidBeforeDeadlineMs 1000, e il valore
su disco vince sempre sui predefiniti: installando la versione nuova, le due
correzioni che pesano di piu' - tetto delle richieste tolto e anticipo a mezzo
secondo - sarebbero state annullate dal file dell'utente, che avrebbe concluso
che non funzionano.

Che il file vinca e' giusto: sono scelte di chi usa il programma. Ma quei due
valori non erano scelte, erano conseguenze di misure sbagliate. Erano stati
tarati quando il tetto stesso accodava le chiamate e gonfiava il ping a 444 ms al
p99,9; senza tetto il ping misurato sta fra 47 e 78 ms.

Migrazione una tantum, con un numero di schema nel file perche' giri una volta
sola. Tocca solo chi ha ancora esattamente il vecchio valore: chi lo aveva gia'
cambiato di suo se lo tiene, perche' quella e' una scelta e non spetta a un
aggiornamento ribaltarla. Il salvataggio avviene dentro Load, che di norma non
tocca il disco, ma una migrazione capita una volta nella vita del file e
lasciarla non salvata la farebbe ripetere a ogni avvio.

Verificato dal vivo sull'eseguibile pubblicato: 40 diventa 0, 1000 diventa 500,
lo schema passa da assente a 1, e le impostazioni nuove (fascia oraria, ritiro
dai duelli) compaiono coi valori giusti.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-08-07 16:16:54 +02:00
Alby96andClaude Opus 5 8b9691d692 Sospendi le puntate nella fascia oraria in cui costano il doppio
Su 6060 aste concluse con prezzo e valore noti, la stessa asta non costa uguale a
tutte le ore. Prezzo di chiusura mediano in percentuale del valore del prodotto:
4,6% alle 12 e 5,3% fra le 10 e le 13, contro 8,6% a mezzanotte e 9,7% alle 9. Le
puntate necessarie per vincere seguono la stessa curva: 21-24 nelle ore buone,
26-29 in quelle cattive. Quasi il doppio del costo per lo stesso oggetto.

Il motivo e' che di notte Bidoo sospende le puntate: restano poche aste in corso e
la concorrenza si concentra li'. Le ore di confine, mezzanotte e le nove, sono le
peggiori, perche' c'e' ancora qualcosa da giocare e ci sono tutti addosso.

La fascia predefinita e' 00:00-10:00. Sospende le puntate, NON il monitoraggio:
l'asta resta in stato Attiva, continua a essere seguita e riprende a puntare da
sola alla fine della fascia. E' voluto: le aste lunghe attraversano la fascia, e
fermarle davvero significherebbe perderle invece che risparmiare. Ogni puntata
saltata scrive nel registro il motivo e l'orario di ripresa.

Tre livelli di override, dal piu' specifico al piu' generale: la singola asta
scavalca il prodotto, il prodotto scavalca le impostazioni generali. Un null non
vuol dire "no" ma "non ho un'opinione", ed e' la differenza che permette di
spegnere il filtro su una sola asta senza toccare il resto.

Una fascia a cavallo di mezzanotte (22-6) funziona; inizio uguale a fine vuol dire
filtro spento, che e' l'unica lettura sensata visto che l'alternativa sarebbe non
puntare mai.

Backtest della regola sul duello, completato su tutte le soglie. Sui dossier veri
in cui il bot ha puntato: 13 aste, 366 puntate, zero vittorie. Con soglia 5 le
puntate sarebbero state 49, risparmiate 317 (86,6%), nessuna vittoria persa. Il
risultato tiene da soglia 2 a soglia 10, che e' quanto ci si puo' aspettare da un
campione in cui non si era vinto niente.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-08-07 15:50:06 +02:00
Alby96andClaude Opus 5 5bac29b0af Densita' della griglia e suggerimento dell'anticipo allineato ai dati nuovi
Il pannello della singola asta consigliava di tenere l'anticipo fra 600 e 2000 ms
e proponeva 800: numeri nati da misure fatte quando il tetto di 40 richieste al
secondo accodava le chiamate e gonfiava il ping. Adesso dice quello che i dati
dicono davvero: tre quarti delle puntate avversarie sono automatiche e scattano a
2 secondi, sotto quella soglia resta solo chi punta a mano, e il predefinito e'
500 ms perche' e' circa nove volte il ping misurato.

Righe della griglia da 28 a 22 px con carattere piccolo: su un cruscotto che
serve a tenere d'occhio molte aste insieme l'altezza e' la risorsa scarsa, e sei
pixel di aria per riga costano una decina di aste sullo schermo.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-08-07 15:31:36 +02:00
Alby96andClaude Opus 5 9b94e2d405 Ritirati dai duelli con l'autopuntata, e punta a mezzo secondo
La strategia A, dai numeri dei dossier: 5410 aste, 2,08 milioni di puntate.

Bidoo dichiara per ogni puntata se e' automatica o manuale. Le automatiche sono
il 75,3% del totale ma vincono il 46,0% delle aste; le manuali sono il 24,7% e ne
vincono il 53,9%. Una puntata manuale vale 3,6 volte una automatica: 178 puntate
per vittoria contro 634.

Il meccanismo del duello e' stato ricostruito esattamente. L'intervallo mediano
fra due puntate consecutive e' 6,0 s su 2.799.681 intervalli: il timer si azzera
a ~8 s e l'autopuntata scatta a 2 s, quindi 8-2=6. Il ciclo da 13,0 s osservato
nelle prove reali era la somma di 7 s di attesa nostra e 6 s della sua risposta.
Puntare tardi non evita l'autopuntata avversaria: la sveglia, perche' si riarma
quando il suo padrone perde la testa della classifica, non a un orario fisso.

Contro una macchina cosi' non esiste attrito vincibile. Adesso il motore conta le
risposte con la firma dei 6+/-2,5 s e dopo cinque di fila smette di puntare su
quell'asta. Una risposta fuori firma azzera il conteggio invece di scalarlo:
l'errore costoso e' ritirarsi da un'asta che si poteva vincere, non spendere
cinque puntate di troppo.

Rigiocato sui dossier veri in cui il bot ha puntato: 366 puntate spese, 13 aste,
zero vittorie. Con la regola a soglia 3 sarebbero state 31 puntate, 335
risparmiate (91,5%), nessuna vittoria persa.

L'anticipo passa da 1000 a 500 ms. Le due ragioni per stare larghi sono cadute:
il p99,9 del ping a 444 ms era misurato col tetto di 40 richieste al secondo che
accodava le chiamate, e senza tetto il ping sta fra 47 e 78 ms; e la concorrenza
sotto i due secondi non e' fitta ma rada, visto che tre quarti degli avversari
sono autopuntate che hanno gia' sparato.

Il tipo di puntata non viene piu' inventato. In due punti il motore ricostruiva
una puntata da un cambio di prezzo fra due interrogazioni e la marcava "Auto"
comunque: adesso quelle portano un trattino, perche' il tipo non lo ha detto
nessuno. La colonna in cronologia mostra quindi solo tipi dichiarati dal server.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-08-07 15:20:34 +02:00
Alby96andClaude Opus 5 20434ca273 Riscatta aprendo il collegamento, e fai scorrere il conto alla rovescia
Due difetti che i registri di tre giorni mostravano chiaramente.

Il riscatto dei buoni giudicava la risposta del sito, e la giudicava male: ogni
singolo tentativo risultava "sessione scaduta: risposta la pagina di accesso" —
6 trovati, 0 riscossi, 6 non riusciti, giro dopo giro per tre giorni — mentre le
puntate arrivavano lo stesso. Il riconoscimento della pagina di accesso scattava
su pagine che di accesso non erano, e il servizio dichiarava di non funzionare
mentre funzionava.

Adesso aprire il collegamento e' il riscatto: si apre, si scrive che e' stato
aperto, e il numero vero lo da' la differenza di saldo, che e' l'unica misura che
non mente. Resta un solo fallimento possibile, l'errore di trasporto, dove il
collegamento non e' stato aperto affatto. Un premio si riscuote una volta sola:
fra i due errori, insistere su un premio gia' preso non costa niente, mentre
dichiarare falliti dei riscatti riusciti riempie il registro di allarmi inutili.
Il giudizio vecchio non e' stato cancellato, e' in GiudicaRisposta.

Il conto alla rovescia in griglia restava fermo fra un poll e l'altro. Il valore
era gia' giusto — si ricava dalla scadenza ancorata all'orologio del server, non
dall'ultimo poll — ma nessuno lo richiedeva finche' non arrivava una risposta, e
nelle prove reali fra un poll e l'altro passava anche un secondo e mezzo:
sembrava un'applicazione bloccata. Un battito da cinque volte al secondo notifica
ora le tre sole proprieta' che dipendono dal tempo.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-08-06 21:29:05 +02:00
Alby96andClaude Opus 5 feb39ee2de Togli il tetto alle richieste e l'attesa fra i poll: era il collo di bottiglia
Misurato sui registri di tre giorni di prove reali (886 dossier, 4,86 milioni di
interrogazioni). Il tetto globale di 40 richieste al secondo veniva spartito fra
tutte le aste seguite, fino a 31 insieme: la distribuzione risultava tagliata
netta sul limite (p90 39, p99 41, saturo nell'8% dei secondi) e ogni singola asta
finiva interrogata ogni ~596 ms invece dei 220 configurati per la finestra
critica. Con il timer a un secondo restava un solo sguardo prima della scadenza,
e la puntata partiva su una stima della scadenza vecchia di mezzo secondo.

Le quattro soglie di cadenza (lontano/medio/vicino/critico) non descrivevano
quindi niente di reale: qualunque numero ci si scrivesse, il tetto decideva al
posto loro. Adesso non c'e' nessuna attesa fra un poll e il successivo: si
riparte appena la risposta e' arrivata, cioe' ~17 interrogazioni al secondo per
asta con il ping misurato di 57 ms. Resta la sola cadenza allargata per le aste
non ancora cominciate, che non e' una scelta di ritmo: fino all'apertura il
server non ha nulla di diverso da dire.

Senza tetto fisso serviva pero' qualcosa che reagisse, altrimenti l'unico segnale
di "troppe richieste" sarebbe stato un blocco dell'account. Il freno ora lo detta
il server: un 429 o un 503 mette in pausa il polling per un secondo, raddoppiando
fino a otto se insistono, e la pausa si azzera al primo giro riuscito. Le puntate
hanno priorita' Critical e non passano dal freno: se il momento giusto e' adesso,
una pausa preventiva costerebbe l'asta.

Il tetto resta configurabile per chi lo vuole, con 0 = nessun limite come nuovo
predefinito.

Scelta dell'utente, dopo avergli mostrato il rischio: niente tetto, il massimo
che regge la rete.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-08-06 18:28:02 +02:00
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
Alby96andClaude Opus 5 5adc4a6526 Passa le note di rilascio dall'ambiente: -p: le spezzava sulle virgole
MSBuild divide il valore di una proprieta' sulle virgole: -p:Note=uno, due
diventa la proprieta' Note=uno piu' un'opzione ` due`, e si finisce su MSB1006
`proprieta' non valida`. Una nota di rilascio in italiano contiene quasi sempre
una virgola, quindi l'attivita' di rilascio si sarebbe rotta al primo uso reale
del prompt delle note.

Le note arrivano ora in AUTOBIDDER_NOTE, che MSBuild legge come proprieta' senza
passare dal parsing della riga di comando. Provato con virgole, apostrofi, due
punti e virgolette: arrivano integri. -p:Note= resta accettato per chi lo passa
a mano senza virgole.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-08-04 23:02:18 +02:00
Alby96andClaude Opus 5 66e3af043c Riformatta tasks.json come lo scrive il formattatore di VS Code
Solo impaginazione: gli oggetti group e presentation passano da una riga a piu'
righe. Nessuna differenza di contenuto (git diff --ignore-all-space e' vuoto).
Committato per non lasciare l'albero sporco a ogni salvataggio del file.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-08-04 23:00:08 +02:00
37 changed files with 1848 additions and 111 deletions
+36 -8
View File
@@ -17,7 +17,10 @@
"/property:GenerateFullPaths=true",
"/consoleloggerparameters:NoSummary"
],
"group": { "kind": "build", "isDefault": true },
"group": {
"kind": "build",
"isDefault": true
},
"problemMatcher": "$msCompile"
},
{
@@ -32,7 +35,10 @@
"-nologo",
"-v:m"
],
"group": { "kind": "test", "isDefault": true },
"group": {
"kind": "test",
"isDefault": true
},
"problemMatcher": "$msCompile"
},
{
@@ -47,7 +53,10 @@
"-nologo",
"-v:m"
],
"presentation": { "reveal": "always", "panel": "dedicated" },
"presentation": {
"reveal": "always",
"panel": "dedicated"
},
"problemMatcher": []
},
{
@@ -63,7 +72,10 @@
"-nologo",
"-v:m"
],
"presentation": { "reveal": "always", "panel": "dedicated" },
"presentation": {
"reveal": "always",
"panel": "dedicated"
},
"problemMatcher": []
},
{
@@ -79,7 +91,10 @@
"-nologo",
"-v:m"
],
"presentation": { "reveal": "always", "panel": "dedicated" },
"presentation": {
"reveal": "always",
"panel": "dedicated"
},
"problemMatcher": "$msCompile"
},
{
@@ -96,7 +111,10 @@
"-nologo",
"-v:m"
],
"presentation": { "reveal": "always", "panel": "dedicated" },
"presentation": {
"reveal": "always",
"panel": "dedicated"
},
"problemMatcher": "$msCompile"
},
{
@@ -109,11 +127,21 @@
"${workspaceFolder}/build/Release.proj",
"-t:Rilascia",
"-p:Versione=${input:versione}",
"-p:Note=${input:note}",
"-nologo",
"-v:m"
],
"presentation": { "reveal": "always", "panel": "dedicated" },
// Le note passano dall'ambiente, non da -p:. MSBuild spezza il valore di
// una proprietà sulle virgole e una nota in italiano ne ha quasi sempre
// una: si otterrebbe MSB1006 «proprietà non valida». Vedi Release.proj.
"options": {
"env": {
"AUTOBIDDER_NOTE": "${input:note}"
}
},
"presentation": {
"reveal": "always",
"panel": "dedicated"
},
"problemMatcher": "$msCompile"
}
],
+10 -4
View File
@@ -1,4 +1,4 @@
<UserControl x:Class="AutoBidder.Controls.AuctionMonitorControl"
<UserControl x:Class="AutoBidder.Controls.AuctionMonitorControl"
xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
@@ -60,6 +60,7 @@
<TextBlock Style="{StaticResource Glyph}" Text="&#xE769;"/>
</Button>
<Button x:Name="StopButton" Style="{StaticResource IconButton}"
Foreground="{DynamicResource Brush.Danger}"
ToolTip="Ferma tutte: nessuna richiesta"
Click="StopButton_Click">
<TextBlock Style="{StaticResource Glyph}" Text="&#xE71A;"/>
@@ -390,6 +391,7 @@
<TextBlock FontFamily="Segoe MDL2 Assets" Text="&#xE769;"/>
</Button>
<Button Style="{StaticResource MiniIconButton}"
Foreground="{DynamicResource Brush.Danger}"
ToolTip="Ferma"
IsEnabled="{Binding CanStop}"
Command="{Binding DataContext.GridStopCommand, RelativeSource={RelativeSource AncestorType=UserControl}}"
@@ -524,14 +526,18 @@
<TextBlock Grid.Row="0" Grid.Column="0" Text="Anticipo puntata (ms)"
ToolTip="Quanti millisecondi prima della scadenza parte la puntata. È il parametro che decide quanto costa vincere: il motore punta solo nei cicli che arrivano fin qui senza che nessun altro abbia puntato.
Dalla rigiocata di 2.095 aste concluse: sotto il secondo basta in mediana 1 puntata per asta, sotto i 2 s ne servono 15, sotto i 3 s 46. Ma sotto i 600 ms un picco di rete fa arrivare la puntata a giochi chiusi. Tienilo fra 600 e 2000 ms, e comunque sopra il tuo ping medio."
Il 75,3% delle puntate avversarie e' automatica e scatta a 2 secondi esatti: sotto quella soglia resta solo chi punta a mano, e una puntata manuale vince 3,6 volte piu' spesso di una automatica (178 puntate per vittoria contro 634, su 5410 aste).
Predefinito 500 ms: circa nove volte il ping misurato (47-78 ms). Piu' in basso non conviene, perche' Bidoo dichiara la scadenza al secondo intero."
Style="{StaticResource FieldLabel}"/>
<TextBox Grid.Row="1" Grid.Column="0" x:Name="SelectedBidBeforeDeadlineMs"
Margin="0,0,10,2" Text="800"
Margin="0,0,10,2" Text="500"
TextChanged="SelectedBidBeforeDeadlineMs_TextChanged"
ToolTip="Quanti millisecondi prima della scadenza parte la puntata. È il parametro che decide quanto costa vincere: il motore punta solo nei cicli che arrivano fin qui senza che nessun altro abbia puntato.
Dalla rigiocata di 2.095 aste concluse: sotto il secondo basta in mediana 1 puntata per asta, sotto i 2 s ne servono 15, sotto i 3 s 46. Ma sotto i 600 ms un picco di rete fa arrivare la puntata a giochi chiusi. Tienilo fra 600 e 2000 ms, e comunque sopra il tuo ping medio."/>
Il 75,3% delle puntate avversarie e' automatica e scatta a 2 secondi esatti: sotto quella soglia resta solo chi punta a mano, e una puntata manuale vince 3,6 volte piu' spesso di una automatica (178 puntate per vittoria contro 634, su 5410 aste).
Predefinito 500 ms: circa nove volte il ping misurato (47-78 ms). Piu' in basso non conviene, perche' Bidoo dichiara la scadenza al secondo intero."/>
<TextBlock Grid.Row="0" Grid.Column="1" Text="Prezzo minimo €"
ToolTip="Sotto questo prezzo non puntare. Serve a lasciar salire un'asta prima di entrare. 0 = nessun limite."
+1 -1
View File
@@ -158,7 +158,7 @@
<TextBlock Grid.Row="1" Grid.Column="0" Text="Costo medio di una puntata (€)"
Foreground="{DynamicResource Brush.Text}" Margin="0,10" VerticalAlignment="Center"
ToolTip="Quanto ti costa una puntata. Serve a calcolare il tetto di spesa: valore del prodotto meno le puntate massime moltiplicate per questo costo."/>
ToolTip="Quanto ti costa una puntata. Serve a calcolare il tetto di spesa: valore del prodotto meno le puntate massime moltiplicate per questo costo.&#10;&#10;Il valore predefinito è 0,20 €, cioè il pacchetto meno vantaggioso in vendita su Bidoo: tenerlo così fa stimare per eccesso, e una stima per eccesso al massimo ti fa rinunciare a un affare, mentre una per difetto ti fa spendere più del valore del prodotto.&#10;&#10;Se compri pacchetti più grandi, abbassalo al costo che paghi davvero."/>
<TextBox Grid.Row="1" Grid.Column="1" x:Name="AverageBidCostTextBox"
Text="0,20" Margin="10,10"/>
</Grid>
@@ -97,6 +97,12 @@ namespace AutoBidder
{
LoadStatistics();
}
// Ultimo passo, e non prima: lo storico è già scritto, il dossier
// chiuso, le statistiche del prodotto aggiornate. Togliere l'asta
// dall'elenco a questo punto non perde niente — tutto ciò che aveva
// da dire è già altrove.
RimuoviSeConclusaEDaTogliere(auction, won, settings);
});
}
catch (Exception ex)
@@ -105,6 +111,56 @@ namespace AutoBidder
}
}
/// <summary>
/// Toglie dall'elenco l'asta appena conclusa, se la regola lo consente.
///
/// <para>Con l'aggiunta automatica accesa le aste concluse si accumulano a centinaia
/// e seppelliscono quelle vive, che è il contrario di quello che serve guardando un
/// cruscotto. Restano però quelle su cui si è speso: sono le uniche che vale la pena
/// riguardare, e i soldi erano veri. Vedi <see cref="FinishedAuctionCleanup"/>.</para>
///
/// <para>Il motivo finisce sempre nel registro, anche quando l'asta resta: senza,
/// un'asta che sparisce o che non sparisce sarebbero entrambe inspiegabili.</para>
/// </summary>
private void RimuoviSeConclusaEDaTogliere(AuctionInfo auction, bool won, AppSettings settings)
{
try
{
var verdict = FinishedAuctionCleanup.Decide(auction, won, new FinishedAuctionCleanup.Options
{
Enabled = settings.AutoRemoveFinished,
KeepWithMyBids = settings.AutoRemoveKeepWithMyBids,
KeepWon = settings.AutoRemoveKeepWon,
KeepUnclear = settings.AutoRemoveKeepUnclear
});
if (!verdict.Remove)
{
if (settings.AutoRemoveFinished)
Log($"[ELENCO] '{auction.Name}' resta in elenco: {verdict.Reason}", LogLevel.Info);
return;
}
// Stesso giro della rimozione a mano: prima il motore, poi la griglia,
// poi il salvataggio. Toglierla solo dalla griglia lascerebbe il runner
// a interrogare un'asta che non esiste piu' per chi guarda.
_auctionMonitor.RemoveAuction(auction.AuctionId);
var vm = _auctionViewModels.FirstOrDefault(x => x.AuctionId == auction.AuctionId);
if (vm != null) _auctionViewModels.Remove(vm);
SaveAuctions();
UpdateTotalCount();
UpdateGlobalControlButtons();
Log($"[ELENCO] '{auction.Name}' tolta dall'elenco: {verdict.Reason}", LogLevel.Info);
}
catch (Exception ex)
{
Console.WriteLine($"[ELENCO ERROR] {ex.Message}");
}
}
/// <summary>
/// Quando l'asta è finita davvero, in ordine di attendibilità: l'istante fissato dal
/// monitor quando ha visto la conclusione dal vivo; la scadenza dichiarata dal
+27
View File
@@ -18,6 +18,7 @@ namespace AutoBidder
public partial class MainWindow
{
private DispatcherTimer? _headerTimer;
private DispatcherTimer? _timerTicker;
private void StartMonitorHeaderTimer()
{
@@ -25,9 +26,35 @@ namespace AutoBidder
_headerTimer.Tick += (_, _) => RefreshMonitorHeader();
_headerTimer.Start();
// ── Il conto alla rovescia scorre da solo ────────────────────────
//
// Il tempo mancante non ha bisogno di una risposta dal server per essere
// ricalcolato: la scadenza è ancorata all'orologio di Bidoo e il resto è
// sottrazione. Prima però veniva notificato solo all'arrivo di un poll, e
// nelle prove reali fra un poll e l'altro passava anche un secondo e mezzo:
// il numero restava fermo e sembrava tutto bloccato.
//
// Cinque battiti al secondo: la colonna mostra i decimi sotto il minuto,
// quindi si vede scorrere davvero, e restano un ventesimo delle notifiche
// che servirebbero per seguire il decimo esatto.
_timerTicker = new DispatcherTimer { Interval = TimeSpan.FromMilliseconds(200) };
_timerTicker.Tick += (_, _) => TickTimers();
_timerTicker.Start();
RefreshMonitorHeader();
}
private void TickTimers()
{
var vms = _auctionViewModels;
for (var i = 0; i < vms.Count; i++)
{
try { vms[i].RefreshTimeDisplay(); }
catch { /* una riga che non si aggiorna non deve fermare le altre */ }
}
}
private void RefreshMonitorHeader()
{
try
+11
View File
@@ -46,6 +46,17 @@
ToolTip="Senza il «Compra Subito» non si può calcolare il risparmio, ma il prezzo e le puntate restano validi."
Checked="Option_Changed" Unchecked="Option_Changed"/>
<StackPanel Orientation="Horizontal" Margin="0,10,0,0">
<CheckBox x:Name="ChkCoverage" VerticalAlignment="Center"
Content="Aste seguite per meno del"
ToolTip="Toglie le aste agganciate a metà. Sono il caso peggiore: i numeri sono veri ma parziali, quindi sembrano buone e abbassano le medie senza dare segnali.&#10;&#10;Quanta asta si è vista si ricava dal prezzo finale, senza bisogno di alcun dato in più: ogni puntata vale un centesimo, quindi il prezzo in centesimi è il totale delle puntate dell'asta, e i reset osservati dicono quanti se ne sono visti.&#10;&#10;Sul tuo storico, su 5876 aste: al 75% ne resterebbero circa 78 su 100, al 90% circa 64, al 95% circa 54, al 99% circa 34."
Checked="Option_Changed" Unchecked="Option_Changed"/>
<TextBox x:Name="CoverageBox" Text="90" Width="50" Margin="8,0"
VerticalAlignment="Center" TextChanged="Option_Changed"/>
<TextBlock Text="% della loro durata" VerticalAlignment="Center"
Foreground="{DynamicResource Brush.Text}"/>
</StackPanel>
<StackPanel Orientation="Horizontal" Margin="0,10,0,0">
<CheckBox x:Name="ChkOlderThan" VerticalAlignment="Center"
Content="Aste chiuse da più di"
@@ -45,6 +45,17 @@ namespace AutoBidder.Dialogs
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)
{
+19 -13
View File
@@ -164,22 +164,28 @@ namespace AutoBidder.Engine
private int NextPollDelay(AppSettings settings)
{
// Un'asta non ancora cominciata ha una cadenza tutta sua: vedi NotStartedDelay.
// È l'unico caso rimasto in cui si aspetta, e non è una scelta di ritmo: fino
// all'apertura il server non ha niente di diverso da dire, e interrogarlo
// seicento volte al minuto per tre ore non anticipa l'inizio di un istante.
var notStarted = NotStartedDelay(settings);
if (notStarted > 0) return notStarted;
var remaining = _auction.EstimatedRemainingMs();
if (remaining == double.MaxValue) return settings.PollIntervalFarMs;
// In sola osservazione non c'è puntata da azzeccare: si risparmiano chiamate.
var critical = _auction.State == RunState.Active
? settings.PollIntervalCriticalMs
: settings.PollIntervalNearMs;
if (remaining <= settings.CriticalWindowMs) return critical;
if (remaining <= 10_000) return settings.PollIntervalNearMs;
if (remaining <= 60_000) return settings.PollIntervalMidMs;
return settings.PollIntervalFarMs;
// ── Nessuna attesa: la cadenza la detta la rete ──────────────────────
//
// Prima qui c'erano quattro soglie configurabili (lontano / medio / vicino /
// critico). Sui registri di tre giorni non hanno mai funzionato come scritto:
// con 40 richieste al secondo di tetto globale spartite fra tutte le aste
// seguite, i 220 ms della finestra critica diventavano 596 ms misurati, e con
// il timer a un secondo restava un solo sguardo prima della scadenza.
//
// Il ritmo adesso è quello del giro di andata e ritorno: si riparte appena la
// risposta è arrivata. Con ping ~57 ms misurato sono circa diciassette
// interrogazioni al secondo per asta.
//
// Il freno non è più un numero scelto a tavolino ma la risposta del server:
// vedi la reazione a 429 e 503 in PollLoopAsync. Se Bidoo dice di rallentare,
// si rallenta; finché non lo dice, si va al massimo che regge la rete.
return 0;
}
/// <summary>
+34 -4
View File
@@ -31,7 +31,37 @@ namespace AutoBidder.Engine.Backtest
int LeadMs,
AppSettings Settings,
string Username = "",
double BidCostEuro = 0.20);
double BidCostEuro = 0.20)
{
/// <summary>
/// Le impostazioni come le vede la rigiocata.
///
/// <para>Spegne la sospensione a fascia oraria, che guarda l'orologio di parete:
/// una rigiocata ripercorre aste già concluse, e l'ora in cui viene lanciata non
/// c'entra nulla con l'ora in cui quelle aste correvano. Lasciarla accesa
/// renderebbe il risultato dipendente da quando si preme il pulsante — di notte
/// il motore rifiuterebbe ogni puntata e la rigiocata direbbe zero, che è
/// esattamente il tipo di risultato che sembra vero e non lo è.</para>
///
/// <para>Vale anche per la suite di test, che gira a qualunque ora.</para>
/// </summary>
public AppSettings EffectiveSettings { get; } = Clona(Settings);
private static AppSettings Clona(AppSettings origine)
{
var copia = origine ?? new AppSettings();
// Copia superficiale via serializzazione: le impostazioni sono un sacco di
// proprietà semplici, e mutare quella dell'utente sarebbe un effetto
// collaterale che nessuno si aspetta da una rigiocata.
var json = System.Text.Json.JsonSerializer.Serialize(copia);
var clone = System.Text.Json.JsonSerializer.Deserialize<AppSettings>(json)
?? new AppSettings();
clone.QuietHoursEnabled = false;
return clone;
}
}
/// <summary>Esito su una singola asta.</summary>
public sealed class Result
@@ -139,7 +169,7 @@ namespace AutoBidder.Engine.Backtest
ShippingCost: null,
MaxBids: auction.MaxClicks,
MaxTotalSpendEuro: auction.MaxTotalSpendEuro,
MinSavingsPercentage: options.Settings.MinSavingsPercentage,
MinSavingsPercentage: options.EffectiveSettings.MinSavingsPercentage,
StopAtBreakEven: auction.StopAtBreakEven));
if (!budget.CanBid)
@@ -151,8 +181,8 @@ namespace AutoBidder.Engine.Backtest
continue;
}
strategy.UpdateHeatMetric(auction, options.Settings, options.Username);
var decision = strategy.ShouldPlaceBid(auction, state, options.Settings, options.Username);
strategy.UpdateHeatMetric(auction, options.EffectiveSettings, options.Username);
var decision = strategy.ShouldPlaceBid(auction, state, options.EffectiveSettings, options.Username);
if (!decision.ShouldBid)
{
+48
View File
@@ -565,6 +565,47 @@ namespace AutoBidder.Models
/// </summary>
[JsonIgnore]
public bool IsInSoftRetreat { get; set; } = false;
// ── Duello con l'autopuntata avversaria ──────────────────────────
// Vedi Utilities/AutoBidDuel per il perche' e per i numeri misurati.
/// <summary>Istante dell'ultima puntata nostra, per misurare quanto tarda la risposta.</summary>
[JsonIgnore]
public DateTime? LastMyBidAtUtc { get; set; }
/// <summary>
/// Risposte avversarie di fila con la firma dell'autopuntata (~6 s). Oltre la
/// soglia si smette di puntare su quest'asta: vedi <see cref="AutoBidDuelDetected"/>.
/// </summary>
[JsonIgnore]
public int AutoResponsesInARow { get; set; }
/// <summary>Riconosciuta un'autopuntata armata dall'altra parte.</summary>
[JsonIgnore]
public bool AutoBidDuelDetected { get; set; }
/// <summary>
/// Registra la risposta avversaria arrivata adesso e aggiorna il conteggio.
/// Restituisce true se proprio con questa risposta il duello viene riconosciuto,
/// cosi' chi chiama puo' scriverlo nel registro una volta sola.
/// </summary>
public bool NoteOpponentResponse(DateTime nowUtc, int soglia)
{
if (LastMyBidAtUtc is not { } miaUltima) return false;
var ritardo = (nowUtc - miaUltima).TotalSeconds;
// Consumata: la prossima risposta va misurata dalla prossima puntata nostra.
LastMyBidAtUtc = null;
AutoResponsesInARow = Utilities.AutoBidDuel.Aggiorna(AutoResponsesInARow, ritardo);
if (AutoBidDuelDetected) return false;
if (!Utilities.AutoBidDuel.DuelloRiconosciuto(AutoResponsesInARow, soglia)) return false;
AutoBidDuelDetected = true;
return true;
}
/// <summary>
/// Contatore puntate effettuate in questa sessione su questa asta
@@ -662,6 +703,13 @@ namespace AutoBidder.Models
/// Override: abilita/disabilita soft retreat per questa asta
/// </summary>
public bool? SoftRetreatEnabledOverride { get; set; }
/// <summary>
/// Override per questa asta della sospensione a fascia oraria.
/// <c>null</c> = come dice il prodotto, e se il prodotto tace come dicono le
/// impostazioni globali. Vedi <see cref="Utilities.BiddingHours"/>.
/// </summary>
public bool? QuietHoursEnabledOverride { get; set; }
/// <summary>
/// Override: limite puntate per questa asta
+29 -3
View File
@@ -14,11 +14,37 @@ namespace AutoBidder.Models
[JsonPropertyName("Price")]
public decimal Price { get; set; }
/// <summary>Puntata fatta dall'autopuntata del sito: scatta a 2 secondi dalla fine.</summary>
public const string TipoAuto = "Auto";
/// <summary>Puntata fatta a mano. Vale 3,6 volte una automatica: vedi <see cref="Utilities.AutoBidDuel"/>.</summary>
public const string TipoManuale = "Manuale";
/// <summary>
/// Tipo di puntata (Auto/Manuale)
/// Il tipo non lo ha detto il server.
///
/// <para>Serve perché non tutte le puntate in elenco vengono dallo storico di
/// <c>data.php</c>: quando il motore si accorge di un cambio di prezzo fra due
/// interrogazioni ne ricostruisce una da sé, e di quella il tipo non lo sa. Prima
/// venivano marcate "Auto" comunque, il che faceva contare come automatiche delle
/// puntate su cui nessuno aveva detto niente.</para>
/// </summary>
public const string TipoSconosciuto = "—";
/// <summary>
/// Tipo di puntata: <see cref="TipoAuto"/>, <see cref="TipoManuale"/> oppure
/// <see cref="TipoSconosciuto"/>. Lo dichiara Bidoo nello storico dell'asta.
/// </summary>
[JsonPropertyName("BidType")]
public string BidType { get; set; } = "Auto";
public string BidType { get; set; } = TipoSconosciuto;
/// <summary>Vero solo se il server ha davvero detto che era automatica.</summary>
[JsonIgnore]
public bool IsAuto => string.Equals(BidType, TipoAuto, StringComparison.OrdinalIgnoreCase);
/// <summary>Vero solo se il server ha davvero detto che era manuale.</summary>
[JsonIgnore]
public bool IsManual => string.Equals(BidType, TipoManuale, StringComparison.OrdinalIgnoreCase);
/// <summary>
/// Timestamp della puntata (Unix timestamp)
@@ -55,7 +81,7 @@ namespace AutoBidder.Models
}
/// <summary>
/// Indica se la puntata è stata fatta dall'utente corrente
/// Indica se la puntata stata fatta dall'utente corrente
/// </summary>
[JsonIgnore]
public bool IsMyBid { get; set; }
+7
View File
@@ -32,6 +32,13 @@ namespace AutoBidder.Models
/// <summary>Prezzo minimo sotto il quale non puntare. <c>null</c> = predefinito.</summary>
public double? MinPrice { get; set; }
/// <summary>
/// Sospendere le puntate nella fascia oraria. <c>null</c> = come le impostazioni
/// generali. Un'asta può a sua volta scavalcare questo valore.
/// Vedi <see cref="Utilities.BiddingHours"/>.
/// </summary>
public bool? QuietHoursEnabled { get; set; }
/// <summary>Prezzo oltre il quale lasciar perdere l'asta. <c>null</c> = predefinito.</summary>
public double? MaxPrice { get; set; }
+2 -2
View File
@@ -1,2 +1,2 @@
Cose da fare:
- Nulla per ora
Ottimo! Ora altre modifiche:
- Aggiungi l'opzione di default già attiva che mi permette di cancellare dall'elenco un'asta terminata. Aggiungere altre opzioni per non eliminare quelle su cui ho puntato ma che sono terminate in modo da verificarle a mano
+70 -3
View File
@@ -111,6 +111,53 @@ namespace AutoBidder.Net
/// <summary>Aggiorna i limiti di ritmo a caldo, senza ricreare il trasporto.</summary>
public void Configure(double requestsPerSecond) => _limiter.SetRate(requestsPerSecond);
// ── Freno dettato dal server ─────────────────────────────────────────
//
// Senza tetto fisso serve qualcosa che reagisca, altrimenti l'unico segnale di
// "troppe richieste" sarebbe un blocco dell'account. Un 429 o un 503 mettono in
// pausa il polling per un secondo, raddoppiando fino a otto se insistono, e la
// pausa si azzera al primo giro andato bene.
//
// Le puntate hanno priorità Critical e non passano da qui: se il momento giusto
// è adesso, un freno preventivo costerebbe l'asta.
private const int FrenoMinimoMs = 1000;
private const int FrenoMassimoMs = 8000;
private long _frenoFinoA;
private int _frenoCorrenteMs = FrenoMinimoMs;
/// <summary>Millisecondi di pausa ancora da scontare. Zero quando si va liberi.</summary>
public int FrenoResiduoMs
{
get
{
var mancano = (Interlocked.Read(ref _frenoFinoA) - Stopwatch.GetTimestamp())
* 1000 / Stopwatch.Frequency;
return mancano > 0 ? (int)mancano : 0;
}
}
private void Frena()
{
var durata = Volatile.Read(ref _frenoCorrenteMs);
Interlocked.Exchange(ref _frenoFinoA,
Stopwatch.GetTimestamp() + durata * Stopwatch.Frequency / 1000);
Volatile.Write(ref _frenoCorrenteMs, Math.Min(FrenoMassimoMs, durata * 2));
}
private void Rilascia() => Volatile.Write(ref _frenoCorrenteMs, FrenoMinimoMs);
private ValueTask RispettaFrenoAsync(CancellationToken ct)
{
var mancano = FrenoResiduoMs;
return mancano <= 0
? ValueTask.CompletedTask
: new ValueTask(Task.Delay(mancano, ct));
}
/// <summary>Costruisce una GET già completa di header di autenticazione e di contesto.</summary>
public HttpRequestMessage BuildGet(string url, string? referer = null, bool ajax = true)
{
@@ -206,6 +253,7 @@ namespace AutoBidder.Net
if (!critical)
{
await RispettaFrenoAsync(ct).ConfigureAwait(false);
await _limiter.WaitAsync(ct).ConfigureAwait(false);
await _concurrency.WaitAsync(ct).ConfigureAwait(false);
}
@@ -224,6 +272,12 @@ namespace AutoBidder.Net
if (!response.IsSuccessStatusCode) Interlocked.Increment(ref _requestsFailed);
// Tolto il tetto fisso, il freno lo dà il server. 429 e 503 sono il modo in
// cui dice "stai chiedendo troppo": si smette per un attimo invece di
// insistere finché non arriva un blocco vero. Le puntate non passano di qui.
if ((int)response.StatusCode is 429 or 503) Frena();
else if (response.IsSuccessStatusCode) Rilascia();
return new HttpOutcome
{
Success = response.IsSuccessStatusCode,
@@ -304,9 +358,20 @@ namespace AutoBidder.Net
private double _tokens;
private long _lastTicks;
/// <summary>
/// Zero o negativo = nessun limite: il secchiello si toglie di mezzo del tutto.
///
/// <para>Serve perché il tetto globale era il vero collo di bottiglia. Misurato sui
/// registri: con 24 aste seguite insieme e 40 richieste al secondo di tetto, la
/// distribuzione delle richieste era tagliata netta sul limite (p90 39, p99 41) e
/// ogni asta finiva interrogata ogni ~596 ms invece dei 220 configurati — proprio
/// negli ultimi secondi, dove quel ritardo costa l'asta.</para>
/// </summary>
private bool Illimitato => _ratePerSecond <= 0;
public TokenBucket(double ratePerSecond, int capacity)
{
_ratePerSecond = Math.Max(1, ratePerSecond);
_ratePerSecond = ratePerSecond;
_capacity = Math.Max(1, capacity);
_tokens = _capacity;
_lastTicks = Stopwatch.GetTimestamp();
@@ -316,8 +381,8 @@ namespace AutoBidder.Net
{
lock (_sync)
{
_ratePerSecond = Math.Max(1, ratePerSecond);
_capacity = Math.Max(8, _ratePerSecond);
_ratePerSecond = ratePerSecond;
_capacity = Math.Max(8, Math.Abs(_ratePerSecond));
if (_tokens > _capacity) _tokens = _capacity;
}
}
@@ -328,6 +393,8 @@ namespace AutoBidder.Net
lock (_sync)
{
if (Illimitato) return ValueTask.CompletedTask;
var now = Stopwatch.GetTimestamp();
var elapsedSeconds = (now - _lastTicks) / (double)Stopwatch.Frequency;
_lastTicks = now;
+13 -3
View File
@@ -1,4 +1,4 @@
using System;
using System;
using System.Collections.Concurrent;
using System.Collections.Generic;
using System.Linq;
@@ -620,7 +620,7 @@ namespace AutoBidder.Services
Username = state.LastBidder,
Price = statePrice,
Timestamp = DateTimeOffset.UtcNow.ToUnixTimeSeconds(),
BidType = "Auto"
BidType = BidHistoryEntry.TipoSconosciuto
});
}
}
@@ -1193,10 +1193,20 @@ namespace AutoBidder.Services
Username = currentBidder,
Price = statePrice,
Timestamp = DateTimeOffset.UtcNow.ToUnixTimeSeconds(),
BidType = "Auto"
BidType = BidHistoryEntry.TipoSconosciuto
});
}
// Una puntata avversaria: se arriva col ritardo tipico dell'autopuntata
// (~6 s dalla nostra) alimenta il conteggio del duello.
var impostazioni = SettingsManager.Load();
if (impostazioni.AutoBidDuelWithdrawEnabled &&
auction.NoteOpponentResponse(DateTime.UtcNow, impostazioni.AutoBidDuelResponses))
{
auction.AddLog(AutoBidDuel.Spiegazione(auction.AutoResponsesInARow),
AuctionLogLevel.Warning, AuctionLogCategory.Strategy);
}
// I conteggi per utente restano a carico di UpdateBidderStatsFromRecentBids,
// unico punto che li tocca: contarli anche qui li raddoppierebbe.
}
+66 -1
View File
@@ -169,7 +169,47 @@ namespace AutoBidder.Services
{
return decision;
}
// 0. DUELLO CON L'AUTOPUNTATA — il primo controllo, perché è quello che
// costa di più sbagliare.
//
// L'autopuntata del sito scatta a 2 secondi e si riarma ogni volta che il suo
// padrone perde la testa: risponde sempre, risponde in ~6 secondi, e non si
// stanca. Contro di lei non esiste attrito che si possa vincere — si
// scambiano puntate una a una finché uno dei due esaurisce il credito.
//
// Nelle prove reali il motore ci ha lasciato 191 puntate su tre aste per zero
// vittorie, con un ciclo regolare da 13,0 secondi (7 di attesa nostra più 6
// della sua risposta). Rigiocando i dossier con questa regola, quelle aste
// sarebbero costate una manciata di puntate senza perdere nessuna vittoria,
// perché vittorie non ce n'erano.
if (settings.AutoBidDuelWithdrawEnabled && auction.AutoBidDuelDetected)
{
decision.ShouldBid = false;
decision.Reason = Utilities.AutoBidDuel.Spiegazione(auction.AutoResponsesInARow);
return decision;
}
// 0-bis. FASCIA ORARIA SOSPESA.
//
// L'asta NON viene fermata: resta Attiva, continua a essere seguita e
// riprende a puntare da sola alla fine della fascia. È voluto — le aste
// lunghe attraversano la fascia, e fermarle davvero significherebbe
// perderle invece che risparmiare.
//
// Su 6060 aste concluse: chiusura mediana al 4,6% del valore alle 12 e al
// 5,3% fra le 10 e le 13, contro l'8,6% a mezzanotte e il 9,7% alle 9, con
// 26-29 puntate per vincere invece di 21-24. Quasi il doppio del costo per
// lo stesso oggetto.
if (QuietHoursApply(auction, settings) &&
Utilities.BiddingHours.IsQuiet(DateTime.Now, settings.QuietHoursStart, settings.QuietHoursEnd))
{
decision.ShouldBid = false;
decision.Reason = Utilities.BiddingHours.Spiegazione(
settings.QuietHoursStart, settings.QuietHoursEnd);
return decision;
}
// ? RIMOSSO: Entry Point - Era sbagliato!
// I limiti MinPrice/MaxPrice impostati dall'utente sono RIGIDI.
// Se l'utente imposta MaxPrice=2, vuole puntare FINO A 2, non fino al 70%!
@@ -466,6 +506,26 @@ namespace AutoBidder.Services
/// <summary>
/// Registra una puntata effettuata (per tracking)
/// </summary>
/// <summary>
/// La sospensione a fascia oraria vale per quest'asta?
///
/// <para>Tre livelli, dal più specifico al più generale: l'asta scavalca il
/// prodotto, il prodotto scavalca le impostazioni generali. Un <c>null</c> non
/// significa "no", significa "non ho un'opinione": è la differenza che permette di
/// spegnere il filtro su una singola asta senza toccare il resto.</para>
/// </summary>
public static bool QuietHoursApply(AuctionInfo auction, AppSettings settings)
{
if (auction?.QuietHoursEnabledOverride is { } perAsta) return perAsta;
// Le aste del monitor non portano con sé l'id prodotto di Bidoo: la scheda si
// ritrova dal nome, che è la via prevista da WatchedProductsStore.
var scheda = WatchedProductsStore.FindByName(auction?.Name);
if (scheda?.QuietHoursEnabled is { } perProdotto) return perProdotto;
return settings.QuietHoursEnabled;
}
public void RecordBidAttempt(AuctionInfo auction, bool success, bool collision = false)
{
auction.SessionBidCount++;
@@ -475,6 +535,11 @@ namespace AutoBidder.Services
{
auction.SuccessfulBidCount++;
auction.ConsecutiveCollisions = 0;
// Da qui si misura quanto tarda la risposta avversaria: e' l'unico punto
// attraversato sia dal motore dal vivo sia dalla rigiocata sui dossier,
// quindi la regola del duello vede le stesse cose in entrambi.
auction.LastMyBidAtUtc = DateTime.UtcNow;
}
else
{
+4 -2
View File
@@ -192,8 +192,10 @@ namespace AutoBidder.Services
var settings = config.PromoHarvest;
var text = body ?? "";
if (FreeBidsPageParser.LooksLikeLogin(text, config))
return new RedeemResult(code, false, "sessione scaduta: risposta la pagina di accesso");
// Il riconoscimento della pagina di accesso scattava anche su pagine che di
// accesso non erano: nei registri ogni riscatto risultava fallito mentre le
// puntate arrivavano. Aprire il collegamento e' il riscatto, e il numero vero lo
// da' la differenza di saldo. Vedi FreeBidsResponseValidator.Validate.
var success = settings.SuccessMarkers
.FirstOrDefault(m => !string.IsNullOrWhiteSpace(m) &&
@@ -35,10 +35,51 @@ namespace AutoBidder.Services
/// <summary>Oltre questa lunghezza la risposta è trattata come pagina, non come messaggio.</summary>
private const int FragmentMaxLength = 2000;
/// <summary>
/// Aprire il collegamento <b>è</b> il riscatto: non si giudica cosa risponde.
///
/// <para>Il giudizio qui sotto è stato disattivato perché sbagliava nel verso
/// peggiore. Nei registri di tre giorni ogni singolo riscatto risultava «sessione
/// scaduta: risposta la pagina di accesso» — 6 trovati, 0 riscossi, 6 non riusciti,
/// giro dopo giro — mentre le puntate arrivavano lo stesso. Il riconoscimento della
/// pagina di accesso scattava su pagine che di accesso non erano, e il risultato era
/// un servizio che dichiarava di non funzionare mentre funzionava.</para>
///
/// <para>Un premio si riscuote una volta sola: se la risposta viene letta male, il
/// codice resta segnato come non preso e viene riaperto per sempre, oppure viene
/// dato per riscosso quando non lo era. Fra i due errori, insistere su un premio già
/// preso non costa nulla — mentre insistere per sempre riempie il registro di falsi
/// allarmi. Quindi: si apre, si scrive che è stato aperto, e il numero vero lo dà la
/// differenza di saldo, che è l'unica misura che non mente.</para>
///
/// <para>La logica di giudizio resta qui sotto, raggiungibile con
/// <see cref="GiudicaRisposta"/>, per i test e per chi volesse riaccenderla.</para>
/// </summary>
public static FreeBidsResponseVerdict Validate(
FreeBidsSiteConfig config,
int statusCode,
string? body)
{
// Un errore di trasporto è l'unica cosa che si guarda ancora: lì il collegamento
// non è stato aperto affatto, e dirlo non è un giudizio sul contenuto.
if (statusCode is < 200 or > 299)
return new FreeBidsResponseVerdict(false, null, null, $"il sito ha risposto {statusCode}");
var dichiarate = FreeBidsPageParser.ReadGainedBids(body ?? "", (config ?? new FreeBidsSiteConfig()).Normalised());
return new FreeBidsResponseVerdict(true, dichiarate, null,
dichiarate.HasValue ? $"collegamento aperto (+{dichiarate} puntate dichiarate)"
: "collegamento aperto");
}
/// <summary>
/// Il vecchio giudizio sul contenuto della risposta. Non più sul percorso normale:
/// vedi <see cref="Validate"/> per il perché.
/// </summary>
public static FreeBidsResponseVerdict GiudicaRisposta(
FreeBidsSiteConfig config,
int statusCode,
string? body)
{
config = (config ?? new FreeBidsSiteConfig()).Normalised();
+69 -37
View File
@@ -56,57 +56,89 @@ namespace AutoBidder.Services
var failed = 0;
var skipped = 0;
for (var i = 0; i < missing.Count; i++)
// ── Perché si salva a blocchi ────────────────────────────────
// Un Append per asta rilegge e riscrive l'intero storico ogni volta:
// su un recupero completo (4675 aste) erano minuti di solo rimescolamento
// JSON e svariati gigabyte scritti sul disco. Un blocco ogni cinquanta
// taglia i salvataggi di cinquanta volte senza mettere a rischio più di
// cinquanta richieste già pagate: quello che non è ancora stato salvato
// viene comunque scritto all'uscita, annullamento compreso.
const int DimensioneBlocco = 50;
var daSalvare = new List<CompletedAuctionRecord>(DimensioneBlocco);
void Deposita()
{
if (ct.IsCancellationRequested) break;
if (daSalvare.Count == 0) return;
var record = missing[i];
CompletedAuctionsStore.AppendMany(daSalvare);
daSalvare.Clear();
}
try
try
{
for (var i = 0; i < missing.Count; i++)
{
var state = await FetchAsync(record.AuctionId, ct).ConfigureAwait(false);
if (ct.IsCancellationRequested) break;
if (state?.WinnerBidsTotal is null)
{
skipped++;
}
else
{
record.WinnerBidsPaid = state.WinnerBidsPaid;
record.WinnerBidsFree = state.WinnerBidsFree;
var record = missing[i];
var check = AuctionIntegrity.CheckWinnerBids(record);
if (check.IsTrustworthy)
try
{
var state = await FetchAsync(record.AuctionId, ct).ConfigureAwait(false);
if (state?.WinnerBidsTotal is null)
{
// Si riscrive solo il record toccato: Append sostituisce
// quello con lo stesso id.
CompletedAuctionsStore.Append(record);
updated++;
skipped++;
}
else
{
// Il valore c'è ma non regge il confronto col prezzo: non si
// salva, e si dice perché. Meglio un buco che una media storta.
record.WinnerBidsPaid = null;
record.WinnerBidsFree = null;
skipped++;
OnLog?.Invoke($"[RECUPERO] {record.Name}: scartato ({check.Reason})");
record.WinnerBidsPaid = state.WinnerBidsPaid;
record.WinnerBidsFree = state.WinnerBidsFree;
var check = AuctionIntegrity.CheckWinnerBids(record);
if (check.IsTrustworthy)
{
// Va nel blocco: AppendMany sostituisce i record con lo
// stesso id, quindi il risultato è identico a un Append
// per ciascuno, con un salvataggio invece di cinquanta.
daSalvare.Add(record);
if (daSalvare.Count >= DimensioneBlocco) Deposita();
updated++;
}
else
{
// Il valore c'è ma non regge il confronto col prezzo: non
// si salva, e si dice perché. Meglio un buco che una media
// storta.
record.WinnerBidsPaid = null;
record.WinnerBidsFree = null;
skipped++;
OnLog?.Invoke($"[RECUPERO] {record.Name}: scartato ({check.Reason})");
}
}
}
}
catch (OperationCanceledException)
{
break;
}
catch (Exception ex)
{
failed++;
OnLog?.Invoke($"[RECUPERO] {record.AuctionId}: {ex.Message}");
}
catch (OperationCanceledException)
{
break;
}
catch (Exception ex)
{
failed++;
OnLog?.Invoke($"[RECUPERO] {record.AuctionId}: {ex.Message}");
}
OnProgress?.Invoke(i + 1, missing.Count, updated);
OnProgress?.Invoke(i + 1, missing.Count, updated);
if (!await Wait.DelayAsync(DelayBetweenRequestsMs, ct).ConfigureAwait(false)) break;
if (!await Wait.DelayAsync(DelayBetweenRequestsMs, ct).ConfigureAwait(false)) break;
}
}
finally
{
// Anche se si esce per annullamento o per un errore: le richieste già
// fatte sono state pagate al server, buttarne i risultati significa
// rifarle al giro dopo.
Deposita();
}
return new Result(missing.Count, updated, failed, skipped);
+136
View File
@@ -0,0 +1,136 @@
using System;
using AutoBidder.Models;
using AutoBidder.Utilities;
using Xunit;
namespace AutoBidder.Tests;
/// <summary>
/// Riconoscimento dell'autopuntata avversaria.
///
/// <para>La proprieta' da difendere non e' "riconosce le macchine": e' che <b>non si ritira
/// da un'asta che si poteva vincere</b>. Sbagliare in quel verso costa un'asta; sbagliare
/// nell'altro costa cinque puntate. Da qui la scelta di azzerare il conteggio a ogni
/// risposta fuori firma invece di scalarlo.</para>
///
/// <para>I numeri vengono dai dossier: 2.799.681 intervalli fra puntate consecutive in 5037
/// aste, mediana 6,0 s, quartili 5,3 e 7,1 s.</para>
/// </summary>
public class AutoBidDuelTests
{
[Theory]
[InlineData(6.0, true)] // la mediana misurata
[InlineData(5.3, true)] // primo quartile
[InlineData(7.1, true)] // terzo quartile
[InlineData(3.4, false)] // troppo svelta: qualcuno che stava guardando
[InlineData(13.0, false)] // troppo lenta
[InlineData(0.0, false)] // nessun ritardo: non e' una risposta
[InlineData(-2.0, false)] // prima della nostra puntata: non e' una risposta
public void La_firma_e_sei_secondi_piu_o_meno_due_e_mezzo(double ritardo, bool atteso)
{
Assert.Equal(atteso, AutoBidDuel.ERispostaAutomatica(ritardo));
}
[Fact]
public void Le_risposte_in_firma_si_sommano()
{
var n = 0;
for (var i = 0; i < 5; i++) n = AutoBidDuel.Aggiorna(n, 6.0);
Assert.Equal(5, n);
Assert.True(AutoBidDuel.DuelloRiconosciuto(n));
}
[Fact]
public void Una_risposta_fuori_firma_azzera_tutto()
{
// Il punto della regola: quattro indizi piu' uno contrario non fanno quattro
// indizi. Serve una sequenza pulita, perche' ritirarsi per sbaglio costa l'asta.
var n = 0;
for (var i = 0; i < 4; i++) n = AutoBidDuel.Aggiorna(n, 6.0);
Assert.Equal(4, n);
n = AutoBidDuel.Aggiorna(n, 2.0);
Assert.Equal(0, n);
Assert.False(AutoBidDuel.DuelloRiconosciuto(n));
}
[Fact]
public void Sotto_la_soglia_non_ci_si_ritira()
{
var n = 0;
for (var i = 0; i < 4; i++) n = AutoBidDuel.Aggiorna(n, 6.0);
Assert.False(AutoBidDuel.DuelloRiconosciuto(n));
}
// ── Lo stato sull'asta ───────────────────────────────────────────────
private static AuctionInfo Asta() => new() { AuctionId = "1", Name = "prova" };
[Fact]
public void Il_duello_si_riconosce_una_volta_sola()
{
var a = Asta();
var t = DateTime.UtcNow;
var riconosciuto = 0;
for (var i = 0; i < 8; i++)
{
a.LastMyBidAtUtc = t;
t = t.AddSeconds(6);
if (a.NoteOpponentResponse(t, 5)) riconosciuto++;
t = t.AddSeconds(7);
}
Assert.True(a.AutoBidDuelDetected);
Assert.Equal(1, riconosciuto); // non una volta per ogni risposta successiva
}
[Fact]
public void Senza_una_nostra_puntata_non_si_misura_niente()
{
// Una risposta avversaria che non risponde a noi non dice nulla sulla loro
// autopuntata: puo' essere un terzo che punta per conto suo.
var a = Asta();
Assert.False(a.NoteOpponentResponse(DateTime.UtcNow, 5));
Assert.Equal(0, a.AutoResponsesInARow);
Assert.False(a.AutoBidDuelDetected);
}
[Fact]
public void Ogni_risposta_si_misura_dalla_puntata_sua()
{
// Consumato il riferimento, una seconda risposta di fila non deve contare due
// volte lo stesso intervallo.
var a = Asta();
var t = DateTime.UtcNow;
a.LastMyBidAtUtc = t;
a.NoteOpponentResponse(t.AddSeconds(6), 5);
Assert.Equal(1, a.AutoResponsesInARow);
a.NoteOpponentResponse(t.AddSeconds(12), 5);
Assert.Equal(1, a.AutoResponsesInARow);
}
[Fact]
public void Un_avversario_umano_irregolare_non_viene_scambiato_per_macchina()
{
var a = Asta();
var t = DateTime.UtcNow;
// Ritardi veri, presi dalla distribuzione misurata fuori dalla firma.
foreach (var ritardo in new[] { 6.0, 3.1, 6.2, 9.4, 5.9, 2.2, 6.0 })
{
a.LastMyBidAtUtc = t;
t = t.AddSeconds(ritardo);
a.NoteOpponentResponse(t, 5);
t = t.AddSeconds(7);
}
Assert.False(a.AutoBidDuelDetected);
}
}
+69
View File
@@ -309,4 +309,73 @@ public class StatsMaintenanceTests
Assert.Equal(report.Kept, kept.Count);
Assert.Equal(report.Removed, list.Count - kept.Count);
}
// ── Aste seguite solo in parte ───────────────────────────────────────
//
// Il prezzo finale in centesimi e' il totale delle puntate dell'asta: confrontarlo
// con i reset osservati dice quanta asta si e' vista, senza bisogno di registrare
// nulla di nuovo. Vale quindi anche sullo storico raccolto prima della regola.
private static CompletedAuctionRecord Seguita(string id, double price, int resets)
{
var r = R(id: id, price: price, bidsFree: 1);
r.TotalResets = resets;
return r;
}
[Theory]
[InlineData(1.00, 100, 1.0)] // 1,00 EUR = 100 puntate, viste tutte
[InlineData(1.00, 50, 0.5)] // agganciata a meta'
[InlineData(1.00, 0, 0.0)] // vista solo la fine
public void La_copertura_e_i_reset_sul_totale_di_puntate(
double price, int resets, double attesa)
{
Assert.Equal(attesa, AuctionIntegrity.ObservedCoverage(price, resets)!.Value, 3);
}
[Fact]
public void Senza_prezzo_la_copertura_non_si_puo_dire()
{
Assert.Null(AuctionIntegrity.ObservedCoverage(0, 42));
}
[Fact]
public void La_copertura_non_supera_mai_uno()
{
// L'arrotondamento del prezzo puo' produrre qualche reset di troppo: non e'
// un'asta osservata piu' del possibile, e non deve diventare 103%.
Assert.Equal(1.0, AuctionIntegrity.ObservedCoverage(1.00, 103)!.Value, 3);
}
[Fact]
public void Toglie_le_aste_seguite_sotto_la_soglia()
{
var list = new List<CompletedAuctionRecord>
{
Seguita("intera", 1.00, 100),
Seguita("quasi", 1.00, 95),
Seguita("meta", 1.00, 50),
};
var options = new StatsMaintenance.CleanupOptions
{
MinObservedCoverage = 0.90
};
var kept = StatsMaintenance.Apply(list, options);
Assert.Equal(2, kept.Count);
Assert.DoesNotContain(kept, r => r.AuctionId == "meta");
}
[Fact]
public void Spenta_la_soglia_non_toglie_nulla_per_copertura()
{
var list = new List<CompletedAuctionRecord> { Seguita("meta", 1.00, 1) };
var report = StatsMaintenance.Preview(list,
new StatsMaintenance.CleanupOptions { MinObservedCoverage = null });
Assert.DoesNotContain("seguita solo in parte", report.ByReason.Keys);
}
}
+92
View File
@@ -0,0 +1,92 @@
using System;
using AutoBidder.Utilities;
using Xunit;
namespace AutoBidder.Tests;
/// <summary>
/// Fascia oraria sospesa.
///
/// <para>La proprieta' da difendere e' che la fascia <b>non fermi l'asta</b>: salta la
/// puntata e basta, perche' le aste lunghe attraversano la fascia e fermarle davvero
/// significherebbe perderle. Qui si prova il calcolo della fascia; che lo stato resti
/// Attiva e' garantito da come la regola e' collegata, non da questo file.</para>
///
/// <para>I numeri che motivano il predefinito 0-10: su 6060 aste concluse la chiusura
/// mediana e' il 4,6% del valore alle 12 e il 5,3% fra le 10 e le 13, contro l'8,6% a
/// mezzanotte e il 9,7% alle 9.</para>
/// </summary>
public class BiddingHoursTests
{
[Theory]
[InlineData(0, true)] // mezzanotte: la peggiore, 8,6%
[InlineData(3, true)]
[InlineData(9, true)] // ultima ora dentro: 9,7%
[InlineData(10, false)] // si riparte: 4,9%
[InlineData(12, false)] // la migliore: 4,6%
[InlineData(23, false)]
public void La_fascia_predefinita_va_da_mezzanotte_alle_dieci(int ora, bool sospesa)
{
Assert.Equal(sospesa,
BiddingHours.IsQuiet(ora, BiddingHours.DefaultStart, BiddingHours.DefaultEnd));
}
[Fact]
public void Inizio_incluso_e_fine_esclusa()
{
Assert.True(BiddingHours.IsQuiet(1, 1, 5));
Assert.True(BiddingHours.IsQuiet(4, 1, 5));
Assert.False(BiddingHours.IsQuiet(5, 1, 5));
}
[Theory]
[InlineData(22, true)]
[InlineData(23, true)]
[InlineData(0, true)]
[InlineData(5, true)]
[InlineData(6, false)]
[InlineData(12, false)]
public void Una_fascia_a_cavallo_di_mezzanotte_funziona(int ora, bool sospesa)
{
Assert.Equal(sospesa, BiddingHours.IsQuiet(ora, 22, 6));
}
[Fact]
public void Inizio_uguale_a_fine_significa_filtro_spento()
{
// L'alternativa sarebbe non puntare mai, che non e' quello che intende nessuno
// scrivendo due volte la stessa ora.
for (var ora = 0; ora < 24; ora++)
Assert.False(BiddingHours.IsQuiet(ora, 3, 3));
}
[Theory]
[InlineData(24, 0)]
[InlineData(25, 1)]
[InlineData(-1, 23)]
public void Le_ore_fuori_scala_si_riportano_dentro(int fuori, int atteso)
{
// Un valore scritto a mano nelle impostazioni non deve produrre una fascia che
// non sospende mai o che sospende sempre.
Assert.Equal(BiddingHours.IsQuiet(atteso, 0, 10), BiddingHours.IsQuiet(fuori, 0, 10));
}
[Fact]
public void La_spiegazione_dice_la_fascia_e_che_riprende_da_sola()
{
var testo = BiddingHours.Spiegazione(0, 10);
Assert.Contains("00:00-10:00", testo);
Assert.Contains("resta attiva", testo);
}
[Fact]
public void La_versione_con_data_usa_solo_l_ora()
{
var dentro = new DateTime(2026, 8, 7, 3, 45, 0);
var fuori = new DateTime(2026, 8, 7, 15, 45, 0);
Assert.True(BiddingHours.IsQuiet(dentro, 0, 10));
Assert.False(BiddingHours.IsQuiet(fuori, 0, 10));
}
}
@@ -0,0 +1,137 @@
using System;
using System.Collections.Generic;
using System.IO;
using System.Linq;
using System.Text.Json;
using AutoBidder.Models;
using AutoBidder.Utilities;
using Xunit;
namespace AutoBidder.Tests;
/// <summary>
/// Archivio delle aste concluse: copia in memoria e salvataggio a blocchi.
///
/// <para>Sono le due modifiche che hanno tolto dal percorso caldo la rilettura dell'intero
/// file — ~72 ms su 5876 record, pagati sul thread dell'interfaccia a ogni asta che finisce.
/// La proprieta' da difendere e' che andare piu' veloce non cambi <b>cosa</b> viene salvato:
/// AppendMany deve dare lo stesso risultato di tanti Append, e la copia in memoria non deve
/// mai servire un contenuto che sul disco non c'e' piu'.</para>
/// </summary>
public class CompletedAuctionsStoreTests
{
private static CompletedAuctionRecord R(string id, double price = 1.00, string? nome = null) =>
new()
{
AuctionId = id,
Name = nome ?? $"Asta {id}",
ProductKey = "prova",
FinalPrice = price,
Winner = "tizio",
EndedAt = DateTime.Now
};
private static void Svuota() => CompletedAuctionsStore.ReplaceAll(new List<CompletedAuctionRecord>());
[Fact]
public void Append_sostituisce_quella_con_lo_stesso_id()
{
Svuota();
CompletedAuctionsStore.Append(R("100", price: 1.00));
CompletedAuctionsStore.Append(R("100", price: 2.50));
var tutte = CompletedAuctionsStore.LoadAll();
Assert.Single(tutte);
Assert.Equal(2.50, tutte[0].FinalPrice);
}
[Fact]
public void AppendMany_da_lo_stesso_risultato_di_tanti_Append()
{
var lotto = new[] { R("a", 1), R("b", 2), R("c", 3) };
Svuota();
foreach (var r in lotto) CompletedAuctionsStore.Append(r);
var unoAllaVolta = CompletedAuctionsStore.LoadAll()
.OrderBy(r => r.AuctionId).Select(r => (r.AuctionId, r.FinalPrice)).ToList();
Svuota();
CompletedAuctionsStore.AppendMany(lotto);
var inBlocco = CompletedAuctionsStore.LoadAll()
.OrderBy(r => r.AuctionId).Select(r => (r.AuctionId, r.FinalPrice)).ToList();
Assert.Equal(unoAllaVolta, inBlocco);
}
[Fact]
public void AppendMany_sostituisce_e_aggiunge_nello_stesso_giro()
{
Svuota();
CompletedAuctionsStore.AppendMany(new[] { R("x", 1), R("y", 2) });
CompletedAuctionsStore.AppendMany(new[] { R("y", 9), R("z", 3) });
var tutte = CompletedAuctionsStore.LoadAll().ToDictionary(r => r.AuctionId, r => r.FinalPrice);
Assert.Equal(3, tutte.Count);
Assert.Equal(9, tutte["y"]);
Assert.Equal(1, tutte["x"]);
Assert.Equal(3, tutte["z"]);
}
[Fact]
public void AppendMany_senza_niente_da_fare_non_esplode()
{
Svuota();
CompletedAuctionsStore.AppendMany(Array.Empty<CompletedAuctionRecord>());
Assert.Empty(CompletedAuctionsStore.LoadAll());
}
[Fact]
public void I_record_senza_id_restano_tutti()
{
// Un id vuoto non identifica niente: due record cosi' non sono lo stesso record,
// e accorparli farebbe sparire un'asta davvero conclusa.
Svuota();
CompletedAuctionsStore.AppendMany(new[] { R(""), R("") });
Assert.Equal(2, CompletedAuctionsStore.LoadAll().Count);
}
[Fact]
public void La_copia_in_memoria_si_accorge_se_il_file_cambia_da_fuori()
{
Svuota();
CompletedAuctionsStore.Append(R("uno"));
Assert.Single(CompletedAuctionsStore.LoadAll());
// Ripristino di un backup o modifica a mano: la copia non deve continuare a
// servire il contenuto vecchio. La data si forza per non dipendere dalla
// granularita' con cui il file system aggiorna l'ora di scrittura.
File.WriteAllText(CompletedAuctionsStore.StorePath,
JsonSerializer.Serialize(new List<CompletedAuctionRecord> { R("due"), R("tre") }));
File.SetLastWriteTimeUtc(CompletedAuctionsStore.StorePath, DateTime.UtcNow.AddSeconds(5));
var dopo = CompletedAuctionsStore.LoadAll();
Assert.Equal(2, dopo.Count);
Assert.DoesNotContain(dopo, r => r.AuctionId == "uno");
}
[Fact]
public void Dopo_ReplaceAll_si_legge_quello_nuovo()
{
Svuota();
CompletedAuctionsStore.AppendMany(new[] { R("vecchia") });
CompletedAuctionsStore.ReplaceAll(new List<CompletedAuctionRecord> { R("nuova") });
var tutte = CompletedAuctionsStore.LoadAll();
Assert.Single(tutte);
Assert.Equal("nuova", tutte[0].AuctionId);
}
}
@@ -0,0 +1,111 @@
using AutoBidder.Utilities;
using Xunit;
namespace AutoBidder.Tests;
/// <summary>
/// Rimozione automatica delle aste concluse dall'elenco del monitor.
///
/// <para>La proprieta' da difendere e' asimmetrica, e va nel verso della prudenza: togliere
/// un'asta solo osservata non costa niente, perche' lo storico la racconta meglio
/// dell'elenco; togliere un'asta su cui si e' speso significa perdere l'occasione di
/// controllare dove sono finiti dei soldi veri. Nel dubbio si trattiene.</para>
/// </summary>
public class FinishedAuctionCleanupTests
{
private static FinishedAuctionCleanup.Verdict Decidi(
bool conclusa = true, bool vinta = false, int puntate = 0, bool fineVista = true,
FinishedAuctionCleanup.Options? opzioni = null) =>
FinishedAuctionCleanup.Decide(conclusa, vinta, puntate, fineVista,
opzioni ?? new FinishedAuctionCleanup.Options());
[Fact]
public void Un_asta_persa_senza_mie_puntate_si_toglie()
{
Assert.True(Decidi().Remove);
}
[Fact]
public void Un_asta_ancora_in_corso_non_si_tocca()
{
var v = Decidi(conclusa: false);
Assert.False(v.Remove);
Assert.Contains("non conclusa", v.Reason);
}
[Fact]
public void Le_aste_su_cui_ho_puntato_restano()
{
// Sono quelle costate qualcosa, ed e' l'unico elenco in cui si vedono accanto ai
// loro dati: farle sparire da sole significa perdere il controllo a mano.
var v = Decidi(puntate: 12);
Assert.False(v.Remove);
Assert.Contains("12 puntate", v.Reason);
}
[Fact]
public void Le_vinte_restano_perche_vanno_confermate_su_Bidoo()
{
var v = Decidi(vinta: true);
Assert.False(v.Remove);
Assert.Contains("conferma", v.Reason);
}
[Fact]
public void Se_la_fine_non_e_stata_vista_l_asta_resta()
{
var v = Decidi(fineVista: false);
Assert.False(v.Remove);
Assert.Contains("fine non", v.Reason);
}
[Fact]
public void Una_vinta_su_cui_ho_puntato_resta_per_il_motivo_piu_importante()
{
// Due ragioni per trattenerla: quella che finisce nel registro deve essere la
// vittoria, non le puntate, perche' e' quella che richiede un'azione.
var v = Decidi(vinta: true, puntate: 30);
Assert.False(v.Remove);
Assert.Contains("vinta", v.Reason.ToLowerInvariant());
}
[Fact]
public void Spegnendo_la_rimozione_non_si_toglie_piu_niente()
{
var v = Decidi(opzioni: new FinishedAuctionCleanup.Options { Enabled = false });
Assert.False(v.Remove);
Assert.Contains("spenta", v.Reason);
}
[Fact]
public void Ogni_eccezione_si_puo_spegnere_da_sola()
{
var solo = new FinishedAuctionCleanup.Options
{
KeepWithMyBids = false,
KeepWon = false,
KeepUnclear = false
};
Assert.True(Decidi(puntate: 40, opzioni: solo).Remove);
Assert.True(Decidi(vinta: true, opzioni: solo).Remove);
Assert.True(Decidi(fineVista: false, opzioni: solo).Remove);
}
[Fact]
public void I_predefiniti_tolgono_le_osservate_e_trattengono_il_resto()
{
var o = new FinishedAuctionCleanup.Options();
Assert.True(o.Enabled);
Assert.True(o.KeepWithMyBids);
Assert.True(o.KeepWon);
Assert.True(o.KeepUnclear);
}
}
+66 -9
View File
@@ -464,7 +464,7 @@ public class FreeBidsResponseValidatorTests
[Fact]
public void Un_errore_http_non_e_un_riscatto()
{
var verdict = FreeBidsResponseValidator.Validate(Config, 500, "");
var verdict = FreeBidsResponseValidator.GiudicaRisposta(Config, 500, "");
Assert.False(verdict.Accepted);
Assert.Contains("500", verdict.Reason);
@@ -473,7 +473,7 @@ public class FreeBidsResponseValidatorTests
[Fact]
public void Il_json_del_sito_ha_lultima_parola()
{
var verdict = FreeBidsResponseValidator.Validate(Config, 200,
var verdict = FreeBidsResponseValidator.GiudicaRisposta(Config, 200,
"""{"success": true, "bids": 5, "message": "Hai riscattato 5 puntate!"}""");
Assert.True(verdict.Accepted);
@@ -484,7 +484,7 @@ public class FreeBidsResponseValidatorTests
[Fact]
public void Un_rifiuto_con_200_resta_un_rifiuto()
{
var verdict = FreeBidsResponseValidator.Validate(Config, 200,
var verdict = FreeBidsResponseValidator.GiudicaRisposta(Config, 200,
"""{"success": false, "message": "Premio già riscattato"}""");
Assert.False(verdict.Accepted);
@@ -496,7 +496,7 @@ public class FreeBidsResponseValidatorTests
{
// Chiedere all'utente di indovinare il percorso completo sarebbe un modo per
// fargli sbagliare la configurazione.
var verdict = FreeBidsResponseValidator.Validate(Config, 200,
var verdict = FreeBidsResponseValidator.GiudicaRisposta(Config, 200,
"""{"status": 200, "data": {"success": "true", "bids": "3"}}""");
Assert.True(verdict.Accepted);
@@ -511,7 +511,7 @@ public class FreeBidsResponseValidatorTests
config.ResponseValidation.SuccessValue = "ok";
config.ResponseValidation.BidsCountKey = "bids_added";
var verdict = FreeBidsResponseValidator.Validate(config, 200,
var verdict = FreeBidsResponseValidator.GiudicaRisposta(config, 200,
"""{"esito": "ok", "bids_added": 12}""");
Assert.True(verdict.Accepted);
@@ -523,7 +523,7 @@ public class FreeBidsResponseValidatorTests
{
// È il caso peggiore: sessione scaduta, HTTP 200, e ogni giro conterebbe premi
// che non sono mai stati presi.
var verdict = FreeBidsResponseValidator.Validate(Config, 200,
var verdict = FreeBidsResponseValidator.GiudicaRisposta(Config, 200,
"<html><body><form action=\"/login.php\"><input name=\"password\"></form></body></html>");
Assert.False(verdict.Accepted);
@@ -533,7 +533,7 @@ public class FreeBidsResponseValidatorTests
[Fact]
public void Una_pagina_intera_vale_come_accettata_e_le_puntate_si_leggono_dal_testo()
{
var verdict = FreeBidsResponseValidator.Validate(Config, 200,
var verdict = FreeBidsResponseValidator.GiudicaRisposta(Config, 200,
"<html><body><h1>Complimenti! Hai ricevuto 3 puntate</h1></body></html>");
Assert.True(verdict.Accepted);
@@ -543,7 +543,7 @@ public class FreeBidsResponseValidatorTests
[Fact]
public void In_un_messaggio_breve_le_parole_di_rifiuto_contano()
{
var verdict = FreeBidsResponseValidator.Validate(Config, 200, "Premio già riscattato oggi");
var verdict = FreeBidsResponseValidator.GiudicaRisposta(Config, 200, "Premio già riscattato oggi");
Assert.False(verdict.Accepted);
Assert.Contains("rifiutato", verdict.Reason);
@@ -552,7 +552,7 @@ public class FreeBidsResponseValidatorTests
[Fact]
public void Una_risposta_vuota_si_accetta_e_decide_il_saldo()
{
var verdict = FreeBidsResponseValidator.Validate(Config, 200, "");
var verdict = FreeBidsResponseValidator.GiudicaRisposta(Config, 200, "");
Assert.True(verdict.Accepted);
Assert.Null(verdict.BidsDeclared);
@@ -794,3 +794,60 @@ public class FreeBidsStatsTests
Assert.Equal(7, FreeBidsStats.Read().TotalBids);
}
}
/// <summary>
/// La politica nuova: aprire il collegamento <b>e'</b> il riscatto, e non si giudica cosa
/// risponde il sito.
///
/// <para>Nasce da tre giorni di registri in cui ogni singolo riscatto risultava fallito con
/// "sessione scaduta: risposta la pagina di accesso" — 6 trovati, 0 riscossi, 6 non
/// riusciti, giro dopo giro — mentre le puntate arrivavano lo stesso. Un premio si riscuote
/// una volta sola: fra i due errori possibili, insistere su un premio gia' preso non costa
/// niente, mentre dichiarare falliti dei riscatti riusciti riempie il registro di allarmi
/// che non servono a nessuno.</para>
/// </summary>
public class FreeBidsPoliticaApertaTests
{
private static FreeBidsSiteConfig Config() => new FreeBidsSiteConfig().Normalised();
[Fact]
public void La_pagina_di_accesso_non_fa_piu_fallire_il_riscatto()
{
// Esattamente il caso che nei registri falliva sempre.
var login = "<html><body><form action=\"/login.php\">" +
new string('x', 500) + "<input name=\"password\"></form></body></html>";
var esito = FreeBidsResponseValidator.Validate(Config(), 200, login);
Assert.True(esito.Accepted);
Assert.Contains("aperto", esito.Reason);
}
[Fact]
public void Una_risposta_di_rifiuto_non_fa_piu_fallire_il_riscatto()
{
var esito = FreeBidsResponseValidator.Validate(Config(), 200, "{\"success\":false}");
Assert.True(esito.Accepted);
}
[Fact]
public void Un_errore_di_trasporto_resta_un_fallimento()
{
// Qui il collegamento non e' stato aperto affatto: dirlo non e' un giudizio sul
// contenuto, e riprovare ha senso.
var esito = FreeBidsResponseValidator.Validate(Config(), 500, "");
Assert.False(esito.Accepted);
Assert.Contains("500", esito.Reason);
}
[Fact]
public void Il_giudizio_vecchio_resta_disponibile()
{
// Non e' stato cancellato: serve ai test e a chi volesse riaccenderlo.
var esito = FreeBidsResponseValidator.GiudicaRisposta(Config(), 200, "{\"success\":false}");
Assert.False(esito.Accepted);
}
}
+90
View File
@@ -0,0 +1,90 @@
using AutoBidder.Utilities;
using Xunit;
namespace AutoBidder.Tests;
/// <summary>
/// Migrazione delle impostazioni salvate da una versione precedente.
///
/// <para>La proprieta' da difendere: un predefinito nuovo <b>non arriva</b> a chi ha gia' un
/// file salvato, perche' il valore su disco vince. Va bene per le scelte dell'utente, non
/// per due valori che erano conseguenze di misure sbagliate — il tetto di 40 richieste al
/// secondo e l'anticipo di 1000 ms erano stati tarati quando il tetto stesso accodava le
/// chiamate e gonfiava il ping. Senza migrazione l'aggiornamento sarebbe arrivato con le
/// correzioni gia' annullate dal file dell'utente.</para>
///
/// <para>L'altra proprieta', altrettanto importante: chi quei valori li aveva gia' cambiati
/// di suo se li tiene. Quella e' una scelta, e non spetta a un aggiornamento ribaltarla.</para>
/// </summary>
public class SettingsMigrationTests
{
private static AppSettings Vecchie() => new()
{
SettingsSchemaVersion = 0,
MaxRequestsPerSecond = 40,
DefaultBidBeforeDeadlineMs = 1000
};
[Fact]
public void I_valori_vecchi_vengono_portati_avanti()
{
var s = Vecchie();
Assert.True(SettingsManager.Migra(s));
Assert.Equal(0, s.MaxRequestsPerSecond); // nessun tetto
Assert.Equal(500, s.DefaultBidBeforeDeadlineMs);
Assert.Equal(SettingsManager.SchemaCorrente, s.SettingsSchemaVersion);
}
[Fact]
public void Una_scelta_dell_utente_non_viene_ribaltata()
{
var s = new AppSettings
{
SettingsSchemaVersion = 0,
MaxRequestsPerSecond = 15, // scelto apposta, non il vecchio predefinito
DefaultBidBeforeDeadlineMs = 2000 // idem
};
SettingsManager.Migra(s);
Assert.Equal(15, s.MaxRequestsPerSecond);
Assert.Equal(2000, s.DefaultBidBeforeDeadlineMs);
}
[Fact]
public void Gira_una_volta_sola()
{
var s = Vecchie();
Assert.True(SettingsManager.Migra(s));
Assert.False(SettingsManager.Migra(s));
// E se poi l'utente rimette 40 di suo, non glielo si toglie al giro dopo.
s.MaxRequestsPerSecond = 40;
Assert.False(SettingsManager.Migra(s));
Assert.Equal(40, s.MaxRequestsPerSecond);
}
[Fact]
public void Impostazioni_nuove_di_zecca_sono_gia_allo_schema_corrente()
{
// Un file che non esiste ancora non e' un file vecchio: nasce gia' aggiornato,
// e non deve far scattare nessuna migrazione.
var s = new AppSettings { SettingsSchemaVersion = SettingsManager.SchemaCorrente };
Assert.False(SettingsManager.Migra(s));
}
[Fact]
public void I_predefiniti_di_fabbrica_sono_quelli_misurati_oggi()
{
var s = new AppSettings();
Assert.Equal(0, s.MaxRequestsPerSecond);
Assert.Equal(500, s.DefaultBidBeforeDeadlineMs);
Assert.True(s.QuietHoursEnabled);
Assert.True(s.AutoBidDuelWithdrawEnabled);
}
}
+13 -3
View File
@@ -449,14 +449,24 @@
<Setter Property="AlternatingRowBackground" Value="{DynamicResource Brush.Surface}"/>
<Setter Property="GridLinesVisibility" Value="None"/>
<Setter Property="HeadersVisibility" Value="Column"/>
<Setter Property="RowHeight" Value="28"/>
<Setter Property="ColumnHeaderHeight" Value="30"/>
<!-- Densità: 22 px invece di 28 fanno entrare circa un quarto di righe in più a
parità di altezza. Su un cruscotto che serve a tenere d'occhio molte aste
insieme, l'altezza è la risorsa scarsa, e sei pixel di aria per riga costano
una decina di aste sullo schermo. -->
<Setter Property="RowHeight" Value="22"/>
<Setter Property="ColumnHeaderHeight" Value="26"/>
<Setter Property="FontFamily" Value="{StaticResource Font.UI}"/>
<Setter Property="FontSize" Value="{StaticResource Font.Size.Base}"/>
<Setter Property="FontSize" Value="{StaticResource Font.Size.Sm}"/>
<Setter Property="CanUserAddRows" Value="False"/>
<Setter Property="AutoGenerateColumns" Value="False"/>
<Setter Property="SelectionMode" Value="Single"/>
<Setter Property="EnableRowVirtualization" Value="True"/>
<!-- Il riciclo riusa i contenitori di riga invece di ricrearli a ogni giro di
scorrimento. Con la griglia del monitor, che si aggiorna piu' volte al
secondo mentre le aste corrono, e' la differenza fra allocare righe nuove
di continuo e non allocarne quasi nessuna. Il valore predefinito di WPF e'
"Standard", cioe' senza riciclo. -->
<Setter Property="VirtualizingPanel.VirtualizationMode" Value="Recycling"/>
<Setter Property="VerticalGridLinesBrush" Value="Transparent"/>
<Setter Property="HorizontalGridLinesBrush" Value="{DynamicResource Brush.Border}"/>
</Style>
+28
View File
@@ -63,5 +63,33 @@ namespace AutoBidder.Utilities
/// </summary>
public static int? TrustedWinnerBids(CompletedAuctionRecord record) =>
CheckWinnerBids(record).IsTrustworthy ? record.WinnerBidsUsed : null;
/// <summary>
/// Quanta parte dell'asta è stata davvero osservata, da 0 a 1. <c>null</c> se non
/// si può dire (prezzo finale mancante).
///
/// <para>Sfrutta la stessa identità del controllo qui sopra: il prezzo finale in
/// centesimi è il numero totale di puntate dell'asta. Confrontarlo con i reset
/// visti dice, senza bisogno di registrare nient'altro, se l'asta è stata seguita
/// dall'inizio o presa per strada. Un'asta agganciata a metà ha numeri veri ma
/// parziali — è il caso peggiore, perché sembra buona.</para>
///
/// <para>Il valore è tagliato a 1: sui dati reali (5876 aste) nessun record supera
/// 1,02, e il poco che sfora è arrotondamento del prezzo, non un'asta osservata
/// più del possibile.</para>
/// </summary>
public static double? ObservedCoverage(double finalPrice, int totalResets)
{
if (finalPrice <= 0) return null;
var totalBids = Math.Round(finalPrice * 100);
if (totalBids <= 0) return null;
return Math.Min(1.0, Math.Max(0, totalResets) / totalBids);
}
/// <summary>Comodità: la copertura di un'asta già registrata.</summary>
public static double? ObservedCoverage(CompletedAuctionRecord record) =>
record == null ? null : ObservedCoverage(record.FinalPrice, record.TotalResets);
}
}
+80
View File
@@ -0,0 +1,80 @@
using System;
namespace AutoBidder.Utilities
{
/// <summary>
/// Riconosce l'avversario che ha l'autopuntata di Bidoo armata, e dice quando smettere.
///
/// <para><b>Il meccanismo.</b> L'autopuntata del sito scatta a 2 secondi dalla chiusura,
/// e si riarma ogni volta che il suo padrone non è più in testa. Il timer si azzera
/// intorno agli 8 secondi, quindi la risposta arriva circa 6 secondi dopo la puntata che
/// l'ha svegliata. Misurato su 2.799.681 intervalli fra puntate consecutive in 5037 aste:
/// mediana <b>6,0 s</b>, primo quartile 5,3 s, terzo quartile 7,1 s.</para>
///
/// <para><b>Perché serve ritirarsi.</b> Contro un'autopuntata armata non esiste attrito
/// che si possa vincere: risponde sempre, risponde in sei secondi, non dorme e non si
/// distrae. Puntare di nuovo significa comprare un altro giro di sei secondi al prezzo
/// di una puntata a testa, all'infinito, finché uno dei due esaurisce il credito. Nelle
/// prove reali sono state 191 puntate su tre aste per zero vittorie, con un ciclo
/// perfettamente regolare da 13,0 secondi: 7 di attesa nostra più 6 della loro risposta.</para>
///
/// <para><b>Perché non basta puntare tardi.</b> Puntare a un secondo dalla chiusura non
/// evita l'autopuntata avversaria: la sveglia. Lei scatta quando il suo padrone perde la
/// testa della classifica, non a un orario fisso, quindi ogni nostra puntata tardiva è
/// esattamente ciò che la fa ripartire.</para>
///
/// <para>Classe pura, senza stato e senza orologio: prende i numeri e restituisce un
/// giudizio. Così la stessa identica logica serve il motore dal vivo e la rigiocata sui
/// dossier, senza due versioni che divergono.</para>
/// </summary>
public static class AutoBidDuel
{
/// <summary>Ritardo tipico della risposta automatica, in secondi.</summary>
public const double RispostaTipicaSecondi = 6.0;
/// <summary>
/// Quanto ci si allontana dai 6 secondi continuando a chiamarla automatica.
/// Coincide grosso modo con l'intervallo fra primo e terzo quartile misurati
/// (5,37,1 s), allargato per la latenza di rete e per l'imprecisione del polling.
/// </summary>
public const double TolleranzaSecondi = 2.5;
/// <summary>
/// Risposte automatiche di fila oltre le quali si smette. Cinque perché è il punto in
/// cui la coincidenza smette di essere credibile: cinque intervalli indipendenti
/// dentro la finestra dei 6±2,5 s non capitano per caso, e cinque puntate sono un
/// prezzo che si può pagare per avere la certezza.
/// </summary>
public const int RispostePerConcludere = 5;
/// <summary>
/// Questo intervallo ha la firma della risposta automatica?
/// </summary>
public static bool ERispostaAutomatica(double secondiDopoLaMiaPuntata) =>
secondiDopoLaMiaPuntata > 0 &&
Math.Abs(secondiDopoLaMiaPuntata - RispostaTipicaSecondi) <= TolleranzaSecondi;
/// <summary>
/// Il conteggio aggiornato dopo aver visto una risposta avversaria.
///
/// <para>Una risposta che <b>non</b> ha la firma azzera il conteggio invece di
/// scalarlo: serve una sequenza pulita per concludere che c'è una macchina
/// dall'altra parte, e un solo intervallo umano in mezzo basta a rimettere tutto in
/// discussione. È la scelta prudente, perché l'errore costoso è ritirarsi da
/// un'asta che si poteva vincere.</para>
/// </summary>
public static int Aggiorna(int consecutiveFinora, double secondiDopoLaMiaPuntata) =>
ERispostaAutomatica(secondiDopoLaMiaPuntata) ? consecutiveFinora + 1 : 0;
/// <summary>
/// Si è capito che dall'altra parte c'è un'autopuntata armata?
/// </summary>
public static bool DuelloRiconosciuto(int consecutive, int soglia = RispostePerConcludere) =>
consecutive >= Math.Max(1, soglia);
/// <summary>Come spiegarlo nel registro, in una riga.</summary>
public static string Spiegazione(int consecutive) =>
$"autopuntata avversaria riconosciuta: {consecutive} risposte di fila a ~{RispostaTipicaSecondi:N0}s. " +
"Continuare significa scambiare puntate una a una con una macchina che non si stanca.";
}
}
+77
View File
@@ -0,0 +1,77 @@
using System;
namespace AutoBidder.Utilities
{
/// <summary>
/// Fascia oraria in cui non si punta.
///
/// <para><b>Perché.</b> La stessa asta non costa uguale a tutte le ore. Su 6060 aste
/// concluse con prezzo e valore noti, il prezzo di chiusura mediano in percentuale del
/// valore del prodotto è 4,6% alle 12 e 5,3% fra le 10 e le 13, ma sale a <b>8,6% a
/// mezzanotte</b> e <b>9,7% alle 9</b>. Le puntate che servono per vincere seguono la
/// stessa curva: 21-24 nelle ore buone, 26-29 in quelle cattive. Quasi il doppio del
/// costo per lo stesso oggetto.</para>
///
/// <para>Il motivo è che di notte Bidoo sospende le puntate — restano poche aste in
/// corso e la concorrenza si concentra su quelle. Le ore di confine, mezzanotte e le
/// nove, sono le peggiori: c'è ancora qualcosa da giocare e ci sono tutti addosso.</para>
///
/// <para><b>Cosa fa.</b> Sospende le puntate, non il monitoraggio: l'asta resta in stato
/// Attiva, continua a essere seguita, e riprende a puntare da sola appena la fascia
/// finisce. È fatto apposta per le aste lunghe che attraversano la fascia — fermarle
/// davvero significherebbe perderle.</para>
///
/// <para>Classe pura, l'ora arriva da fuori: così i test non dipendono da quando
/// vengono eseguiti.</para>
/// </summary>
public static class BiddingHours
{
/// <summary>Prima ora della fascia sospesa, inclusa. Predefinito: mezzanotte.</summary>
public const int DefaultStart = 0;
/// <summary>Prima ora in cui si torna a puntare. Predefinito: le 10.</summary>
public const int DefaultEnd = 10;
/// <summary>
/// L'ora indicata cade nella fascia sospesa?
///
/// <para>Estremi: inizio incluso, fine esclusa — 0-10 significa "da mezzanotte fino
/// alle 9:59". Una fascia che scavalca la mezzanotte (22-6) funziona. Inizio uguale
/// a fine vuol dire fascia vuota, cioè filtro spento: è l'unica lettura sensata,
/// perché l'alternativa sarebbe non puntare mai.</para>
/// </summary>
public static bool IsQuiet(int hour, int start, int end)
{
hour = Normalize(hour);
start = Normalize(start);
end = Normalize(end);
if (start == end) return false;
return start < end
? hour >= start && hour < end
: hour >= start || hour < end; // a cavallo di mezzanotte
}
/// <summary>Comodità: la fascia applicata a un istante.</summary>
public static bool IsQuiet(DateTime when, int start, int end) =>
IsQuiet(when.Hour, start, end);
/// <summary>
/// Quando riprende, per poterlo scrivere nel registro. Restituisce l'ora di ripresa
/// (0-23); chi legge sa già che è la prossima volta che scocca.
/// </summary>
public static int ResumesAt(int end) => Normalize(end);
/// <summary>Riga per il registro: dice cosa succede e quando finisce.</summary>
public static string Spiegazione(int start, int end) =>
$"puntata saltata: fascia oraria sospesa ({Normalize(start):00}:00-{Normalize(end):00}:00). " +
"L'asta resta attiva e riprende a puntare da sola alla fine della fascia.";
private static int Normalize(int hour)
{
var h = hour % 24;
return h < 0 ? h + 24 : h;
}
}
}
+84 -8
View File
@@ -23,6 +23,25 @@ namespace AutoBidder.Utilities
private static readonly JsonSerializerOptions JsonOptions = new() { WriteIndented = true };
// ── Copia in memoria ─────────────────────────────────────────────
//
// Il file cresce di un record per asta e arriva in fretta a qualche megabyte:
// a 5876 aste, leggerlo e deserializzarlo costa ~72 ms misurati. Senza copia
// in memoria quel prezzo si paga a ogni apertura delle statistiche e a ogni
// asta che finisce — e quella chiamata arriva da un gestore di eventi, cioe'
// dal thread dell'interfaccia, che resta fermo per tutto il tempo.
//
// L'invalidazione guarda la data di modifica del file, non un tempo di
// scadenza: cosi' la copia resta giusta anche se il file viene sostituito da
// fuori — ripristino di un backup, modifica a mano — che qui capita davvero.
//
// Il prezzo da sapere: i record restituiti sono gli STESSI oggetti che stanno
// nella copia. Chi ne modifica uno sta modificando anche la copia, prima e
// indipendentemente dal salvataggio. Repair e il recupero puntate contano su
// questo; chi non lo vuole si faccia una copia del record.
private static List<CompletedAuctionRecord>? _cache;
private static DateTime _cacheStamp;
/// <summary>Percorso del file storico (per mostrarlo/aprirlo nella UI).</summary>
public static string StorePath => FilePath;
@@ -34,16 +53,50 @@ namespace AutoBidder.Utilities
public static void Append(CompletedAuctionRecord record)
{
if (record == null) return;
AppendMany(new[] { record });
}
/// <summary>
/// Aggiunge o sostituisce piu' aste con <b>un solo</b> salvataggio.
///
/// <para>Serve a chi ne tocca tante di fila, come il recupero delle puntate dei
/// vincitori. Chiamare <see cref="Append"/> in un ciclo rilegge e riscrive tutto
/// il file a ogni giro: sulle 4675 aste di un recupero completo erano minuti di
/// solo rimescolamento JSON e svariati gigabyte scritti sul disco per nulla.</para>
/// </summary>
public static void AppendMany(IReadOnlyCollection<CompletedAuctionRecord> records)
{
if (records == null || records.Count == 0) return;
lock (_lock)
{
var list = LoadInternal();
var existing = list.FindIndex(r =>
!string.IsNullOrEmpty(r.AuctionId) && r.AuctionId == record.AuctionId);
// Indice per id: cercare ogni record con un FindIndex renderebbe
// quadratico proprio il caso per cui questo metodo esiste.
var byId = new Dictionary<string, int>(list.Count, StringComparer.Ordinal);
for (var i = 0; i < list.Count; i++)
{
var id = list[i].AuctionId;
if (!string.IsNullOrEmpty(id)) byId[id] = i;
}
if (existing >= 0) list[existing] = record;
else list.Add(record);
foreach (var record in records)
{
if (record == null) continue;
var id = record.AuctionId;
if (!string.IsNullOrEmpty(id) && byId.TryGetValue(id, out var at))
{
list[at] = record;
}
else
{
if (!string.IsNullOrEmpty(id)) byId[id] = list.Count;
list.Add(record);
}
}
SaveInternal(list);
}
@@ -119,10 +172,12 @@ namespace AutoBidder.Utilities
try
{
if (File.Exists(FilePath)) File.Delete(FilePath);
_cache = null;
}
catch (Exception ex)
{
Console.WriteLine($"[STATS] Errore pulizia storico: {ex.Message}");
_cache = null;
}
}
}
@@ -221,18 +276,33 @@ namespace AutoBidder.Utilities
try
{
if (!File.Exists(FilePath))
{
_cache = null;
return new List<CompletedAuctionRecord>();
}
// Una lettura di data di modifica costa microsecondi; la lettura piu'
// la deserializzazione che evita ne costa decine di millisecondi.
var stamp = File.GetLastWriteTimeUtc(FilePath);
if (_cache != null && stamp == _cacheStamp) return _cache;
var json = File.ReadAllText(FilePath);
if (string.IsNullOrWhiteSpace(json))
return new List<CompletedAuctionRecord>();
return JsonSerializer.Deserialize<List<CompletedAuctionRecord>>(json)
?? new List<CompletedAuctionRecord>();
var list = string.IsNullOrWhiteSpace(json)
? new List<CompletedAuctionRecord>()
: JsonSerializer.Deserialize<List<CompletedAuctionRecord>>(json)
?? new List<CompletedAuctionRecord>();
_cache = list;
_cacheStamp = stamp;
return list;
}
catch (Exception ex)
{
Console.WriteLine($"[STATS] Errore lettura storico: {ex.Message}");
// Non si sa cosa ci sia sul disco: la prossima volta si rilegge.
_cache = null;
return new List<CompletedAuctionRecord>();
}
}
@@ -246,10 +316,16 @@ namespace AutoBidder.Utilities
var json = JsonSerializer.Serialize(list, JsonOptions);
File.WriteAllText(FilePath, json);
// La copia in memoria e' esattamente quello che si e' appena scritto:
// tenerla evita di rileggere il file al primo uso successivo.
_cache = list;
_cacheStamp = File.GetLastWriteTimeUtc(FilePath);
}
catch (Exception ex)
{
Console.WriteLine($"[STATS] Errore salvataggio storico: {ex.Message}");
_cache = null;
}
}
}
+106
View File
@@ -0,0 +1,106 @@
using AutoBidder.Models;
namespace AutoBidder.Utilities
{
/// <summary>
/// Decide se un'asta conclusa può sparire da sola dall'elenco del monitor.
///
/// <para><b>Perché serve.</b> Le aste concluse restano nel monitor finché non le si
/// toglie a mano, e con l'aggiunta automatica accesa se ne accumulano centinaia: quelle
/// vive finiscono sepolte sotto quelle finite, che è il contrario di quello che serve
/// guardando un cruscotto.</para>
///
/// <para><b>Perché non basta togliere tutto.</b> Un'asta su cui si è puntato è l'unica
/// che vale la pena riguardare: quanto è costata, se il ritiro dal duello è scattato al
/// momento giusto, se il prezzo finale sta dove ci si aspettava. Farla sparire da sola
/// significa perdere l'occasione di controllarla, e i soldi erano veri. Le altre — quelle
/// solo osservate — non hanno niente da dire che lo storico non dica meglio.</para>
///
/// <para>Da qui la forma della regola: si toglie di serie, si trattiene ciò su cui c'è
/// stato un esborso. Classe pura: nessuno stato, nessun orologio, nessun disco.</para>
/// </summary>
public static class FinishedAuctionCleanup
{
/// <summary>Cosa fare delle aste concluse, e cosa risparmiare.</summary>
public sealed class Options
{
/// <summary>Togliere da sole le aste concluse. Acceso di serie.</summary>
public bool Enabled { get; set; } = true;
/// <summary>
/// Trattenere le aste su cui si è puntato, per poterle controllare a mano.
/// Acceso di serie: sono quelle costate qualcosa.
/// </summary>
public bool KeepWithMyBids { get; set; } = true;
/// <summary>
/// Trattenere le aste vinte. Acceso di serie: c'è da confermare l'acquisto su
/// Bidoo, e una vittoria sparita dall'elenco è il modo più semplice di
/// dimenticarsene.
/// </summary>
public bool KeepWon { get; set; } = true;
/// <summary>
/// Trattenere le aste finite con un errore o senza esito leggibile: sono
/// esattamente quelle da guardare. Acceso di serie.
/// </summary>
public bool KeepUnclear { get; set; } = true;
}
/// <summary>Esito della decisione, col motivo per il registro.</summary>
public readonly record struct Verdict(bool Remove, string Reason);
/// <summary>
/// Quest'asta conclusa si può togliere dall'elenco?
///
/// <para>L'ordine dei controlli è quello che conta: prima le ragioni per
/// <b>trattenere</b>, poi la rimozione. Un'asta vinta su cui si è anche puntato deve
/// restare per il primo motivo che si incontra, non per l'ultimo, così il messaggio
/// nel registro dice la cosa più importante.</para>
/// </summary>
public static Verdict Decide(bool conclusa, bool vinta, int puntateMie, bool fineVista, Options options)
{
options ??= new Options();
if (!options.Enabled)
return new Verdict(false, "rimozione automatica spenta");
if (!conclusa)
return new Verdict(false, "asta non conclusa");
if (options.KeepWon && vinta)
return new Verdict(false, "vinta: resta in elenco per la conferma su Bidoo");
if (options.KeepWithMyBids && puntateMie > 0)
return new Verdict(false,
$"trattenuta: ci hai speso {puntateMie} puntate, controllala a mano");
if (options.KeepUnclear && !fineVista)
return new Verdict(false, "trattenuta: la fine non è stata vista, esito da controllare");
return new Verdict(true, "conclusa senza puntate mie: tolta dall'elenco");
}
/// <summary>
/// Comodità per il motore: legge dall'asta i quattro numeri che servono.
///
/// <para>Le puntate si prendono dal massimo fra il contatore dell'asta e quello di
/// sessione: uno dei due può essere a zero a seconda di come l'asta è entrata nel
/// monitor, e trattenere per eccesso è l'errore giusto da fare.</para>
/// </summary>
public static Verdict Decide(AuctionInfo auction, bool vinta, Options options)
{
if (auction == null) return new Verdict(false, "nessuna asta");
var perAsta = auction.BidsUsedOnThisAuction ?? 0;
var puntate = perAsta > auction.SessionBidCount ? perAsta : auction.SessionBidCount;
return Decide(
conclusa: !string.IsNullOrWhiteSpace(auction.ConcludedOutcome),
vinta: vinta,
puntateMie: puntate,
fineVista: auction.ObservedToEnd,
options);
}
}
}
+132 -3
View File
@@ -38,7 +38,21 @@ namespace AutoBidder.Utilities
/// dossier non possono dire di piu', perche' Bidoo dichiara la scadenza al secondo
/// intero.</para>
/// </summary>
public int DefaultBidBeforeDeadlineMs { get; set; } = 1000;
///
/// <para><b>Aggiornato a 500 ms.</b> Le due ragioni per stare larghi sono cadute
/// entrambe. La prima — il p99,9 del ping a 444 ms — era misurata quando il tetto di
/// 40 richieste al secondo faceva accodare le chiamate: con il tetto tolto il ping
/// misurato sui dossier nuovi sta fra 47 e 78 ms, mediana 55. La seconda era la
/// paura della concorrenza fitta sotto i due secondi, ma i tipi di puntata dicono
/// il contrario: il 75,3% delle puntate avversarie è automatica e scatta a 2 s
/// esatti, quindi sotto quella soglia resta solo chi punta a mano.</para>
///
/// <para>A 500 ms la puntata parte con circa nove volte il ping di margine, e cade
/// nella metà di secondo in cui l'autopuntata del sito ha già sparato. Più in là non
/// conviene andare: Bidoo dichiara la scadenza al secondo intero, quindi sotto i
/// 500 ms si starebbe scommettendo sull'arrotondamento.</para>
/// </summary>
public int DefaultBidBeforeDeadlineMs { get; set; } = 500;
public double DefaultMinPrice { get; set; } = 0;
public double DefaultMaxPrice { get; set; } = 0;
public int DefaultMaxClicks { get; set; } = 0;
@@ -82,7 +96,18 @@ namespace AutoBidder.Utilities
/// Tetto complessivo di richieste al secondo verso Bidoo (le puntate non sono soggette
/// al limite: hanno corsia preferenziale). Default: 40.
/// </summary>
public double MaxRequestsPerSecond { get; set; } = 40;
/// <summary>
/// Tetto di richieste al secondo verso Bidoo. <b>0 = nessun tetto</b>, ed è il
/// predefinito: il ritmo lo detta la rete, e a frenare è il server con 429/503.
///
/// <para>Il vecchio predefinito era 40, ed era il vero collo di bottiglia. Misurato
/// sui registri di tre giorni: con fino a 31 aste seguite insieme la distribuzione
/// delle richieste era tagliata netta sul tetto (p90 39, p99 41), e ogni asta
/// veniva interrogata ogni ~596 ms invece dei 220 configurati per la finestra
/// critica. Con il timer a un secondo restava un solo sguardo prima della
/// scadenza: da lì le puntate perse e il tempo che non scorreva.</para>
/// </summary>
public double MaxRequestsPerSecond { get; set; } = 0;
/// <summary>
/// Porta la granularità del timer di sistema a 1 ms mentre il monitor è in funzione.
@@ -535,6 +560,62 @@ namespace AutoBidder.Utilities
/// Default: false (DISABILITATO - non blocca mai)
/// </summary>
public bool HardcodedAntiCollisionEnabled { get; set; } = false;
// ── Fascia oraria sospesa ────────────────────────────────────────
// Vedi BiddingHours per i numeri: alle 0 e alle 9 la stessa asta costa quasi il
// doppio che fra le 10 e le 13.
// ── Rimozione automatica delle aste concluse ─────────────────────
// Vedi FinishedAuctionCleanup: si toglie di serie, si trattiene cio' su cui c'e'
// stato un esborso.
/// <summary>Togliere da sole dall'elenco le aste concluse. Acceso di serie.</summary>
public bool AutoRemoveFinished { get; set; } = true;
/// <summary>Trattenere le aste su cui hai puntato, per controllarle a mano.</summary>
public bool AutoRemoveKeepWithMyBids { get; set; } = true;
/// <summary>Trattenere le aste vinte: c'è da confermare l'acquisto su Bidoo.</summary>
public bool AutoRemoveKeepWon { get; set; } = true;
/// <summary>Trattenere le aste di cui non si è vista la fine: esito da controllare.</summary>
public bool AutoRemoveKeepUnclear { get; set; } = true;
/// <summary>
/// Versione dello schema delle impostazioni, per le migrazioni una tantum.
///
/// <para>Un file scritto da una versione precedente non ha questo campo e vale
/// quindi 0. Serve perché un predefinito nuovo <b>non arriva</b> a chi ha già un
/// file salvato: il valore su disco vince sempre, ed è giusto che sia così, tranne
/// quando il vecchio valore veniva da una misura che si è poi rivelata sbagliata.
/// Vedi <see cref="Migra"/>.</para>
/// </summary>
public int SettingsSchemaVersion { get; set; }
/// <summary>
/// Sospendere le puntate nella fascia indicata. Acceso di serie.
/// L'asta resta comunque Attiva e riprende da sola: vedi <see cref="BiddingHours"/>.
/// </summary>
public bool QuietHoursEnabled { get; set; } = true;
/// <summary>Prima ora della fascia sospesa, inclusa.</summary>
public int QuietHoursStart { get; set; } = BiddingHours.DefaultStart;
/// <summary>Prima ora in cui si torna a puntare.</summary>
public int QuietHoursEnd { get; set; } = BiddingHours.DefaultEnd;
/// <summary>
/// Ritirarsi quando dall'altra parte c'è un'autopuntata armata. Acceso di serie.
/// Vedi <see cref="AutoBidDuel"/> per il meccanismo e i numeri misurati.
/// </summary>
public bool AutoBidDuelWithdrawEnabled { get; set; } = true;
/// <summary>
/// Risposte automatiche di fila oltre le quali ci si ritira. Predefinito 5.
/// Più basso risparmia prima ma rischia di scambiare per macchina un avversario
/// umano regolare; più alto è prudente e costa puntate.
/// </summary>
public int AutoBidDuelResponses { get; set; } = AutoBidDuel.RispostePerConcludere;
// 🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥
// RILEVAMENTO COMPETIZIONE E HEAT METRIC
@@ -779,6 +860,36 @@ namespace AutoBidder.Utilities
private const int CACHE_TTL_MS = 2000;
/// <summary>Schema corrente. Alzarlo fa girare <see cref="Migra"/> una volta sola.</summary>
public const int SchemaCorrente = 1;
/// <summary>
/// Porta avanti un file scritto da una versione precedente. Restituisce true se
/// qualcosa è cambiato, così chi chiama sa che va risalvato.
///
/// <para><b>Perché serve.</b> Il valore su disco vince sui predefiniti, ed è
/// giusto: sono scelte dell'utente. Ma due di quei valori non erano scelte, erano
/// conseguenze di misure sbagliate — il tetto di 40 richieste al secondo e
/// l'anticipo di 1000 ms erano stati tarati quando il tetto stesso accodava le
/// chiamate e gonfiava il ping a 444 ms. Senza migrazione, chi aggiorna si ritrova
/// le correzioni di oggi già annullate dal proprio file, e conclude che non
/// funzionano.</para>
///
/// <para>Si tocca solo chi ha ancora esattamente il vecchio valore. Chi lo aveva
/// già cambiato di suo se lo tiene: quella è una scelta, e non spetta a un
/// aggiornamento ribaltarla.</para>
/// </summary>
public static bool Migra(AppSettings s)
{
if (s == null || s.SettingsSchemaVersion >= SchemaCorrente) return false;
if (Math.Abs(s.MaxRequestsPerSecond - 40) < 0.001) s.MaxRequestsPerSecond = 0;
if (s.DefaultBidBeforeDeadlineMs == 1000) s.DefaultBidBeforeDeadlineMs = 500;
s.SettingsSchemaVersion = SchemaCorrente;
return true;
}
public static AppSettings Load()
{
// Percorso comune: nessun lucchetto, nessuna allocazione.
@@ -794,11 +905,20 @@ namespace AutoBidder.Utilities
return cached;
AppSettings loaded;
var daRisalvare = false;
try
{
loaded = File.Exists(_file)
var esiste = File.Exists(_file);
loaded = esiste
? JsonSerializer.Deserialize<AppSettings>(File.ReadAllText(_file)) ?? new AppSettings()
: new AppSettings();
// Un file nuovo nasce già allo schema corrente: la migrazione riguarda
// solo chi arriva da una versione precedente.
if (!esiste) loaded.SettingsSchemaVersion = SchemaCorrente;
else daRisalvare = Migra(loaded);
}
catch
{
@@ -806,6 +926,15 @@ namespace AutoBidder.Utilities
loaded = _cached ?? new AppSettings();
}
// Scritto qui e non dentro Save: Load è sul percorso caldo e non deve
// toccare il disco, ma una migrazione capita una volta sola nella vita
// del file, e lasciarla non salvata la farebbe ripetere a ogni avvio.
if (daRisalvare)
{
try { File.WriteAllText(_file, JsonSerializer.Serialize(loaded, new JsonSerializerOptions { WriteIndented = true })); }
catch { /* si riproverà al prossimo avvio */ }
}
_cached = loaded;
Interlocked.Exchange(ref _cacheExpiryTicks, DateTime.UtcNow.AddMilliseconds(CACHE_TTL_MS).Ticks);
return loaded;
+20
View File
@@ -100,6 +100,17 @@ namespace AutoBidder.Utilities
/// <summary>Doppioni con lo stesso id: si tiene il più recente.</summary>
public bool RemoveDuplicates { get; set; } = true;
/// <summary>
/// Copertura minima richiesta, da 0 a 1: sotto questa soglia l'asta è stata
/// agganciata a metà e va tolta. Null = non filtrare sulla copertura.
///
/// <para>È il criterio per «non seguita dall'inizio alla fine». Non serve
/// nessun campo nuovo nei record: la copertura si ricava dal prezzo finale,
/// vedi <see cref="AuctionIntegrity.ObservedCoverage(double,int)"/>. Vale
/// quindi anche per lo storico raccolto prima che questa regola esistesse.</para>
/// </summary>
public double? MinObservedCoverage { get; set; }
/// <summary>Aste chiuse prima di questa data. Null = nessun limite.</summary>
public DateTime? OlderThan { get; set; }
}
@@ -161,6 +172,15 @@ namespace AutoBidder.Utilities
if (options.RemoveWithoutValue && r.BuyNowPrice is not > 0)
Mark(r, "senza valore del prodotto");
if (options.MinObservedCoverage is { } soglia)
{
// Copertura non calcolabile = asta senza prezzo finale: qui vale come
// "non seguita", perché non c'è modo di dire il contrario.
var copertura = AuctionIntegrity.ObservedCoverage(r);
if (copertura is null || copertura.Value < soglia)
Mark(r, "seguita solo in parte");
}
if (options.OlderThan is { } limite && r.EndedAt < limite)
Mark(r, "più vecchia del limite");
}
+22 -1
View File
@@ -500,8 +500,29 @@ namespace AutoBidder.ViewModels
OnPropertyChanged(nameof(LatencyDisplay)); // ? AGGIUNTO - latenza cambia quando si attiva/disattiva
}
/// <summary>
/// Rinfresca il solo tempo mancante, senza che sia arrivata una risposta nuova.
///
/// <para>Il valore era già giusto — <see cref="TimerDisplay"/> lo ricava dalla
/// scadenza ancorata all'orologio del server, non dall'ultimo poll — ma nessuno lo
/// richiedeva finché non arrivava un'interrogazione. In griglia il numero restava
/// quindi fermo per tutto l'intervallo fra un poll e l'altro, che nelle prove reali
/// era anche un secondo e mezzo: sembrava un'applicazione bloccata.</para>
///
/// <para>Si notificano solo le tre proprietà che dipendono dal tempo. Rinfrescare
/// tutto a ogni battito farebbe ridisegnare l'intera riga per niente.</para>
/// </summary>
public void RefreshTimeDisplay()
{
if (_lastState == null || IsFinished) return;
OnPropertyChanged(nameof(TimerDisplay));
OnPropertyChanged(nameof(TimerUrgency));
OnPropertyChanged(nameof(SortRemainingMs));
}
public event PropertyChangedEventHandler? PropertyChanged;
protected void OnPropertyChanged(string propertyName)
{
PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName));
+1 -1
View File
@@ -50,7 +50,7 @@ DisableProgramGroupPage=yes
DisableDirPage=auto
OutputDir={#OutputDir}
OutputBaseFilename=AutoBidder-{#AppVersion}-setup
OutputBaseFilename=AutoBidder_{#AppVersion}
SetupIconFile=..\Icon\favicon.ico
UninstallDisplayIcon={app}\{#AppExeName}
UninstallDisplayName={#AppName} {#AppVersion}
+30 -5
View File
@@ -67,8 +67,20 @@
creare. Vuoto = la minor successiva all'ultimo tag. Vedi VersioneDaTag. -->
<Versione Condition="'$(Versione)' == ''"></Versione>
<!-- Note di rilascio; vuote = ricavate dai commit dall'ultimo tag. -->
<Note Condition="'$(Note)' == ''"></Note>
<!--
Note di rilascio. Vuote = solo il numero di versione.
Arrivano da una variabile d'ambiente, non da -p:. MSBuild spezza il valore
di una proprietà sulle virgole: `-p:Note=uno, due` diventa la proprietà
Note=uno più l'opzione " due", e si finisce su MSB1006 "proprietà non
valida". Una nota di rilascio in italiano contiene quasi sempre una
virgola, quindi il passaggio per riga di comando è inutilizzabile.
L'ambiente non ha questo problema, e MSBuild legge le variabili
d'ambiente come proprietà.
-p:Note= resta accettato per chi lo passa a mano senza virgole.
-->
<Note Condition="'$(Note)' == ''">$(AUTOBIDDER_NOTE)</Note>
<!-- ── Perché serve azzerare queste variabili ──────────────────────────
`dotnet test` e `dotnet publish` lanciati da dentro MSBuild ereditano
@@ -421,9 +433,22 @@
<Target Name="Pubblica" DependsOnTargets="DeterminaVersione">
<Message Importance="High" Text="== Pubblicazione dell'eseguibile ($(V)) ==" />
<!-- ── Perché gli intermedi vanno altrove ──────────────────────────────
La sola opzione -o sposta il risultato ma lascia gli INTERMEDI nella obj/
del progetto, che è la stessa che usa la compilazione di debug lanciata da
VS Code. La compilazione WPF genera un progetto temporaneo `_wpftmp.csproj`
a ogni giro, e quando ci trova dentro lo stato di un'altra configurazione
smette di produrre le classi parziali dello XAML: si ottiene una raffica di
CS2001 «file di origine MainWindow.g.cs non trovato», a giri alterni, senza
che il codice sia cambiato.
Era già il rimedio adottato per i test; qui era rimasto indietro, e bastava
pubblicare mentre si compilava in VS Code per rompere la compilazione di
debug. L'opzione artifacts-path sposta gli intermedi, la -o continua a
decidere dove finisce l'eseguibile: le due convivono. -->
<Exec WorkingDirectory="$(Radice)"
EnvironmentVariables="$(AmbientePulito)"
Command="dotnet publish &quot;$(Csproj)&quot; -c Release -r win-x64 --nologo -v q --self-contained true -p:PublishSingleFile=true -p:IncludeNativeLibrariesForSelfExtract=true -p:EnableCompressionInSingleFile=true -p:PublishReadyToRun=true -p:PublishTrimmed=false -p:Version=$(V) -p:AssemblyVersion=$(V).0 -p:FileVersion=$(V).0 -p:InformationalVersion=$(V) -o &quot;$(CartellaPubblicazione)&quot;" />
Command="dotnet publish &quot;$(Csproj)&quot; -c Release -r win-x64 --nologo -v q --artifacts-path &quot;$(CartellaProve)&quot; --self-contained true -p:PublishSingleFile=true -p:IncludeNativeLibrariesForSelfExtract=true -p:EnableCompressionInSingleFile=true -p:PublishReadyToRun=true -p:PublishTrimmed=false -p:Version=$(V) -p:AssemblyVersion=$(V).0 -p:FileVersion=$(V).0 -p:InformationalVersion=$(V) -o &quot;$(CartellaPubblicazione)&quot;" />
<Error Condition="!Exists('$(CartellaPubblicazione)\AutoBidder.exe')"
Text="Pubblicazione fallita: AutoBidder.exe non trovato." />
@@ -453,7 +478,7 @@
Command="&quot;$(Iscc)&quot; /Qp &quot;/DAppVersion=$(V)&quot; &quot;/DSourceExe=$(CartellaPubblicazione)\AutoBidder.exe&quot; &quot;/DOutputDir=$(CartellaPacchetti)&quot; &quot;$(Iss)&quot;" />
<PropertyGroup>
<Setup>$(CartellaPacchetti)\AutoBidder-$(V)-setup.exe</Setup>
<Setup>$(CartellaPacchetti)\AutoBidder_$(V).exe</Setup>
</PropertyGroup>
<Error Condition="!Exists('$(Setup)')" Text="Installatore non trovato: $(Setup)" />
@@ -528,7 +553,7 @@
<CurlCfg>$(Tmp)\curl.cfg</CurlCfg>
<CorpoJson>$(Tmp)\release.json</CorpoJson>
<RispostaJson>$(Tmp)\risposta.json</RispostaJson>
<Setup>$(CartellaPacchetti)\AutoBidder-$(V)-setup.exe</Setup>
<Setup>$(CartellaPacchetti)\AutoBidder_$(V).exe</Setup>
</PropertyGroup>
<MakeDir Directories="$(Tmp)" />