[Phoryn]

#csharp #programowanie

Panowie mam problem ponieważ mam tablicę jedno wymiarową string gdzie są same wartości numeryczne (również wartości przecinkowe) a muszę z tego wyznaczyć minimalną wartość... I tu jest problem bo nie mogę tego przekonwertować.. Próbowałem tak :

static double fMin(string[] tab)
{
double[] tab2 = new double[tab.Length];
for (int i = 0; i < tab.Length; i++) {
tab2[i] = Convert.ToDouble(tab[i]);
}

double min = /tab.First()

for (int i = 1; i < tab.Length; i++)
{
if (tab2[i] < min)
min = tab2[i];
}

return min;
}

Lecz niestety wyrzuca mi błąd, że "nieprawidłowy format ciągu wejściowego"... Ktoś mi coś poradzi ;) ?

[RRybak]

@Phoryn: nie wiem jak w c# ale dużo języków się wykłada na tym, że gdzieś w stringu jest przecinek, a powinna być kropka (albo odwrotnie)

[m.....y]

@Phoryn: typowane języki och ach

[w.....j]

@mojkrajtakipiekny: no bo lepiej liczyć na szczęście ( ͡° ͜ʖ ͡°)

[ZapomnialWieprzJakProsiakiemByl]

@Phoryn:
Bo przecinek w jednym kraju służy do oddzielenia tysięcy a winnym części ułamkowej. Gdzie indziej używa się kropki zamiast przecinka.
Masz IFormatProvider w wersji przeciążonej Convert.ToDouble

[ZapomnialWieprzJakProsiakiemByl]

@Phoryn:
using System.Globalization;
Convert.ToDouble(tab[i], new CultureInfo("pl-pl"));
Convert.ToDouble(tab[i], new CultureInfo("en-us"));
itp...

[K.....o]

@Phoryn:

gejowskie (bo po stringu), ale najszybsze:

Array.Sort(tab, (x,y) => String.Compare(x.Name, y.Name));

mądrzejsze:

var tab2 = new double[tab.Lengh]

int i = 0;

tab.forEach(x =>

{

tab2[i] = Convert.ToDouble(tab[i])

});

Array.Sort(tab2);

return tab2

da się to pewnie jeszcze prościej zapisać, jakoś tak:

Array.Sort(Array.ConvertAll(tab.Split(','), Double.Parse));

return tab

[K.....o]

Lecz niestety wyrzuca mi błąd, że "nieprawidłowy format ciągu wejściowego"... Ktoś mi coś poradzi ;) ?

w pętli np. tab2[i].Replace(',','.'); //cuś takiego może odwrotnie, przecinek z kropką

@Phoryn:

[krzysio19]

@Phoryn: dla jakiej wartości stringa wyrzuca wyjątek?

char[] charArray = tab[i].ToCharArray();

[ZapomnialWieprzJakProsiakiemByl]

@Kamasetto: Ale namieszałeś... ( ͡° ͜ʖ ͡°) I pewnie nic z tego nie zadziała poprawnie.

Przejrzałbym plik i zrobił to wykorzystując odpowiednią kulturę albo dostosowując ją
var cultureInfo = CultureInfo.InvariantCulture.Clone() as CultureInfo;
cultureInfo.NumberFormat.NumberDecimalSeparator = ","; // tu okreslamy separator dla ułamków
cultureInfo.NumberFormat.NumberGroupSeparator = "."; // tu określamy separator dla tysięcy akurat kropa ale może to być dowolny string
for (int i = 0; i < tab.Length; i++)

[K.....o]

@ZapomnialWieprzJakProsiakiemByl co Ci mieszaleme? T cala metoda jest do zapisania jednym returnem. W 3 min.

[ZapomnialWieprzJakProsiakiemByl]

@Kamasetto: A która z nich ma zadziałać? Może zwyczajnie nie kumam bo programistą nie jestem...

[K.....o]

http://pastebin.com/htpx7jUR

Jedna linijka wystarczy.

[K.....o]

@wczoraj cześć

[ZapomnialWieprzJakProsiakiemByl]

@Kamasetto:
new string[] { "1.22", "1.23", "7.68", "9.45","1,122.1" }
i już nie zadziała( ͡° ͜ʖ ͡°)
przecież chodzi o błąd formatu

[K.....o]

@ZapomnialWieprzJakProsiakiemByl: a skąd wiesz jak wygląda tablica wejściowa?

[K.....o]

co nadal nie zmienia faktu, że w tej jednej linijce to się wykna

[ZapomnialWieprzJakProsiakiemByl]

@Kamasetto: No nie wiem, Ty też nie wiesz i tu jest problem.( ͡º ͜ʖ͡º)

[Phoryn]

@ZapomnialWieprzJakProsiakiemByl: @Kamasetto: @krzysio19:

Akurat sprawdziłem i wszędzie są kropki... nie ma przecinków, więc to nei może być problem tego...
Może po prostu napisałbym nową metodę, która przepisze mi tablicę jednowymiarową stringów na tablicę jednowymiarową intów...

[ZapomnialWieprzJakProsiakiemByl]

@Phoryn: To wystarczy, że zrobisz tak Convert.ToDouble(i, new CultureInfo("en-us"));
i powinno działać chyba, że masz spacje między tysiącami:)
using System.Globalization; -to też musisz dodać.
Amerykanie używają kropki do części ułamkowej a my przecinka stąd masz błąd formatu bo wzięło z do konwersji ustawienia domyślne komputera

[Phoryn]

@ZapomnialWieprzJakProsiakiemByl: Rozwiązałem to w ten sposób, że napisałem nową metodę która zamienia mi tylko tablicę. I działa ;)

static double[] StringDoTablicy(string[] tab) {

string a;
double[] tablica = new double[tab.Length];
for (int i = 0; i< tab.Length; i++)
{
a = tab[i];
tablica[i] = StringToDouble(a);
}
return tablica;
}

Szkoda, że się tak długo męczyłem nad taką głupotą