New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Feature]: Ranking Cheapest Station #96
Comments
Muss man sich mal anschauen mit "Erste günstige Tankstelle". |
Die Intention war, den Shortpreis für die Darstellung in VIS zu nutzen. So wie Schmakus es erklärt, wird der Preis normalerweise ohne den Tausendstel Euro (3rd) kommuniziert. So kann jeder selbst entscheiden, ob er in VIS 1,88€ oder 1,88hoch9 darstellen lässt. Einige Tankstellen setzen die Tausendstel-Stelle auch auf 4. Da geht es nur um die Darstellung der Zahl. Das schließt übrigens auch die Sprachausgabe ein. Zu "cheapest":
|
"Bei 1.889 € sagt niemand, dass der Preis bei 1.89 € liegt, sondern immer 1.88 €" "generell entscheidet, dass nur Stationen, die günstiger sind, als die bisher Günstigste, selbst Günstigste werden können" |
Eine Rundung des Preises im
Ich habe mir heute noch mal den Response von der API angeschaut und gesehen, dass die Stationen dort nicht in der Reihenfolge, wie sie in der UI konfiguriert sind zurückkommen. Reihenfolge UI Darauf wird dann die Sortierung angewandt (e5 Sortierung). Und angesichts dessen wird, auch wenn die 1. Tankstelle, die in der UI konfiguriert ist und gerade einer der Tankstellen mit dem billigsten Preisen ist, nicht als Erstes angezeigt wird. Ich werde was einbauen, dass der Response nach der Reihenfolge der Stationen in der UI sortiert wird und dann erst nach den Preisen. Jetzt sollten die billigsten Tankstellen nach der UI Reihenfolge angezeigt werden. @Klaus-Lucas Ist das so OK für dich? |
@xXBJXx auf jeden Fall. Danke für die Mühe. |
@Klaus-Lucas Ich habe auf GitHub eine Testversion mit der neuen sortier Funktion hochgeladen kannst du sie bitte testen und mir dann Bescheid geben, ob sie bei dir so funktioniert wie du sie haben wolltest? |
@xXBJXx Sieht korrekt aus. Wie geschrieben sind die Tankstellen nach Prio 0-8 eingetragen. |
Also so wie es soll? |
@xXBJXx Danke für die Anpassung. |
Is your feature request related to a problem? Please describe.
Laut Issue 50
Der Adapter geht alle eingetragenen Stationen nacheinander durch und holt die Preise von Tankerkoenig ab und speichert sie in die Datenpunkte. Dabei vergleicht er bei jeder Station, ob sie verglichen mit den anderen Stationen nicht günstiger ist. Wenn ja, wird sie auch in Cheapest geschrieben.
Ich habe meine Tankstellen sortiert nach Entfernung eingegeben. Bei gleichen Preisen gibt der Adapter unter Cheapest (s.o.) die letzte Tankstelle aus, die den gleichen Preis hat, also bei mir die weit entfernteste. Meiner Meinung nach sollte der Adapter beim Vergleichen feststellen, dass es schon eine Tankstelle mit dem Preis gibt und die folgenden mit gleichem Preis ignorieren.
Ich persönlich fände ich es noch gut, wenn der Short Preis gerundet ausgegeben würde, also bei 1,879€ 1,88€. Ist aber Ansichtssache.
Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: