[bug] Pricewatch sortering opslag GB/dag foutief

Pagina: 1
Acties:

Acties:
  • 0 Henk 'm!

  • Bubbles
  • Registratie: Juni 2001
  • Laatst online: 11:36

Bubbles

DPC-Crew

2 the resQ

Topicstarter
In de pricewatch kan je bij (ssd) opslag sorteren op GB/dag. Echter gaat die sortering de mist in. Zie als voorbeeld dit plaatje:
Afbeeldingslocatie: https://tweakers.net/ext/f/IMSBbMRIk76T8JxakzwRwAoq/full.png

Ik heb hier gekozen voor een aflopende sortering. We zien eerst 87.7GB staan, gevolgd door 637,4GB, aangezien een 8 groter is dan een 6, compleet er voorbijgaand aan het feit dat een hondertal echt wel groter is dan een tiental. Ietsje lager zien we dit verschijnsel nog een paar keer terugkomen. Zie ook deze link voor de daadwerkelijke pagina.

Misschien de values splitsen in getal en unit-grootte, in plaats van op een simpele string te sorteren?

Vouw mee bij Fatal Error Group! - F@H: #10


Acties:
  • +1 Henk 'm!

  • winkbrace
  • Registratie: Augustus 2008
  • Laatst online: 24-08 15:17
Oh, daar is er eentje tussendoor geslipt. Bedankt, we pakken het op!

Acties:
  • 0 Henk 'm!

  • ACM
  • Registratie: Januari 2000
  • Niet online

ACM

Software Architect

Werkt hier

@winkbrace dit is domweg een fout bij de definitie van deze speciale "virtuele specificatie", geen bug van onze code. Het zou ook in onze code opgelost kunnen worden (door tekstuele uitkomsten met "natural ordering" te sorteren), maar in dit geval kan het ook eenvoudiger door de "virtuele specificatie"-definitie wat te veranderen :)

@Glaanie Die heb ik bij deze aangepast, dus hopelijk werkt ie nog zoals je bedoelde :P

Acties:
  • 0 Henk 'm!

  • Glaanie
  • Registratie: Juni 2002
  • Nu online

Glaanie

Medewerker Product Content

All your spec are belong to us

ACM schreef op zaterdag 9 februari 2019 @ 09:43:
@winkbrace dit is domweg een fout bij de definitie van deze speciale "virtuele specificatie", geen bug van onze code. Het zou ook in onze code opgelost kunnen worden (door tekstuele uitkomsten met "natural ordering" te sorteren), maar in dit geval kan het ook eenvoudiger door de "virtuele specificatie"-definitie wat te veranderen :)

@Glaanie Die heb ik bij deze aangepast, dus hopelijk werkt ie nog zoals je bedoelde :P
Het is mij onduidelijk wat je nou precies hebt veranderd :p Ik heb op iteration gekeken in de hoop daar de oude code te treffen, maar of je hebt daar dezelfde fix lopen testen, of ik zie het verschil niet ... Mag je mij maandag laten zien :)