Dotaz: Decimal separator

[pdp7]

Příloha:

• SwitchHardware..cs (57256 bytů)

Ahoj, mám tu jeden ovladač rozhraní ASCOM, který čte hodnoty z čidel kolem arduina a zobrazuje v nějakých intervalech na obrazovku PC. Je to psáno v C# a potíž je v tom, že to funguje jen v případě, když je v systému nastavena tečka jako desetinný oddělovač. Jak je tam čárka, tak to hubuje - nesprávný formát vráceného řetězce! Je nějaký elegantní způsob jak tomu říct aby bral jako oddělovač oboje? Snad mě s tím nepošlete kamsi když je to na widle.

Myslím, že je to na řádcích 121 - 127 ale nejsem si jistý a nevím jestli bude fungovat tohle:

internal static string SwitchStateVoltage = "0.00".Replace(",",".");

Odpovědi

[X]

tak to hubuje - nesprávný formát vráceného řetězce!

Mues napsat presne co to vraci a zaroven log z toho software?

[pdp7]

Jé Ahoj, abych to nemusel znova opisovat, odkážu na astro-forum.cz kde jsem tam jako Astrolama, příspěvek č. 18 a další. Celý výpis hlášení ve v tom err_ascom.zip a jsou tam i nějaké screeny. Spínač má svůj ovládací program "control center", ale ke stejným funkcím se dá dostat i v Nighttime Imaging 'N' Astronomy (to je ten tmavě červeno-černý screenshot), právě díky tomu ASCOM (standard pro ovládání astronomických zařízení).

Skutečně po změně oddělovače na tečku v ovládacích panelech všechno funguje normálně, ale v české mutaci widlí je nastavená výchozí čárka.

[ehmmm]

Takovy provni tip je, ze to bude tenhle radek:

return Convert.ToDouble(SerialCommands[id]);

Asi bych si v c# nastavil DecimalSeparator, ve stringu udelal replace dle potreby a pak by to melo fungovat vzdycky.

[X]

Staci radek 889 v SwitchHardware.cs :

return Convert.ToDouble(SerialCommands[id]);

upravit na:

return Convert.ToDouble(SerialCommands[id], CultureInfo.InvariantCulture);

a pridat uplne nahoru:

using System.Globalization;

Mimochodem ta konverze je tam jeste minimalne jednou tak to muzes opravit i s tim..

[pdp7]

Už to sviští :) Díky

[X]

Mohl by jsi pls potvrdit na Gitu, ze to chodi?

[pdp7]

Mrkev v zimě, nějak to chlapci nefunguje ten jeho build. Můj jde tak já nevím, napsal jsem mu to tam.

[ml]

Já bych použil trochu jinou podobu. Vyčetl bych si, jaký decimal separator používá systém a tím bych tečku nahradil.

Už jsem to tam někdy i udělal (zkusím dohledat, ale nic na tom neni) a při nějakém prohlížení kódu mě docela pobavil komentář "nehradí tečku desetinným oddělovačem, ať je jím cokoliv"

Funguje mi to ve windows i linuxu a bez ohledu na to, jaký je nastavený jazyk a co je desetinný oddělovač (třeba nahrazuju tečku tečkou, ale to mi může být ukradené). Totéž je možná potřeba udělat s thousend separator.

Pro váš případ teda třeba takto.

internal static string SwitchStateVoltage = "0.00".Replace(aktuální_decimal_separátor,".");

[ml]

Jen doplním moje řešení, ze kterého jsem udělal standard (mám to ve více programech), je to funkce, která jako vstup dostává text a jako výstup vrací double/float, přičemž tohle je zrovna verze, která vrací nulu pokud ten textový řetězec neni platný (tady se to tak hodilo, jinak třeba může vyvolat výjimku nebo použít jinou hodnotu). Jak jsem zjistil, zrovna v tom programu, ze kterého to kopíruju jsem na mnoha místech použil i TimeSeparator a Dateseparator a možná i něco dalšího, ale spíš při převodu hodnot na text.

 

function gp_val_float (const vstup : string):double;
begin
Result := (StrToFloatDef(StringReplace(vstup,'.',DecimalSeparator,(.rfReplaceAll.)),0));
end;

Tohle je teda v Lazarusu (syntaxe Pascalu) ale v jakémkoliv jiném jazyce to musí být úplně stejné, pokud mám k dispozici funkci, která očekává řetězec v národním tvaru (já jako programátor ovšem nemůžu tušit, jaké jazykové nastavení bude mít uživatel) a já mám řetězec v nějakém pevném (např. anglickém) tvaru, tak se s tím musím takto popasovat.

To, že mi kompilátor proměnnou DecimalSeparator označuje jako deprecated je celkem jedno, to jsem už v nějakém programu taky řešil (místo ní je tam nějaká struktura, kde je všechno národní nastavení soustředěné a ta se má přednostně používat), to jsou už jen technické detaily, kde si tohle nastavení zjistit. Jde o ten princip, že namísto natvrdo zadaného řetězce platného pro daný stroj, je lepší udělat to univerzálně.