Diskuze: Sečtení sloupce v listView

Matyáš Černohous

Tvůrce

Matyáš Černohous:20.8.2013 15:47

Zdravím, chtěl bych se zeptat jak by se dali sečíst hodnoty ve sloupci v listView. Mám listView se 3 sloupci a potřebuji sečíst ten poslední a vypsat do textboxu. Zkoušel jsem tohle:

int sum = 0;
foreach (ListViewItem li in listView1.Items)
  sum += int.Parse(li.Subitems[2].Text);

Ale nešlape to.
Díky za rady

Odpovědět

20.8.2013 15:47

Luboš Běhounek Satik

Tvůrce

Luboš Běhounek Satik:20.8.2013 16:06

Specifikuj blíže výraz "nešlape to" - nic to nedělá, padá to, nejde to zkompilovat?

Nahoru Odpovědět

20.8.2013 16:06

https://www.facebook.com/peasantsandcastles/

Matyáš Černohous

Tvůrce

Matyáš Černohous:20.8.2013 17:41

Zkompilovat to jde, ale když zmáčknu na button abych dostal ten součet, tak to hodí hlášku: Vstupní řetězec nemá správný formát. Mám ListView propojený s Db a datovýtyp toho sloupce je integer, tak nevím co tam je špatný formát...

Nahoru Odpovědět

20.8.2013 17:41

Luboš Běhounek Satik

Tvůrce

Luboš Běhounek Satik:20.8.2013 18:38

v

li.Subitems[2].Text

Máš něco, co asi není číslo.
Používáš celá nebo desetinná čísla?

Nahoru Odpovědět

20.8.2013 18:38

https://www.facebook.com/peasantsandcastles/

Matyáš Černohous

Tvůrce

Matyáš Černohous:20.8.2013 19:17

Jen celá...

Nahoru Odpovědět

20.8.2013 19:17

Luboš Běhounek Satik

Tvůrce

Luboš Běhounek Satik:20.8.2013 19:28

Tak tam nemáš číslo.
Zkus si na ten řádek dát breakpoint (ve VS defaultně klávesou F9) a až se ti to tam zastaví na tom řádku, tak se koukni na obsah toho Textu.

Nahoru Odpovědět

20.8.2013 19:28

https://www.facebook.com/peasantsandcastles/

Jakub Lásko[Saarix]

Tvůrce

Jakub Lásko[Saarix]:21.8.2013 11:10

Možná by nebylo špatné zkusit u toho sloupce 2 použít metodu GetType, aby jsis potvrdil správnost typu.

Nahoru Odpovědět

21.8.2013 11:10

Časem je vše možné.

Matyáš Černohous

Tvůrce

Matyáš Černohous:21.8.2013 12:59

A mohl bys mi prosím poradit jak to použít ? Ještě nikdy jsem to nepoužíval.

Nahoru Odpovědět

21.8.2013 12:59

Jan Vargovský

Tvůrce

Jan Vargovský:21.8.2013 13:01

Debugoval jsi to jak ti řekl Satik ? Když si ten program na tom místě zastavíš, zjistíš kompletně co to je za typ, jestli tam nejsou nějaké špatné znaky atd...

Nahoru Odpovědět

21.8.2013 13:01

matesax

Tvůrce

matesax:21.8.2013 16:14

Možná pro něj bude jednodušší MessageBox, když se jedná o string...

Nahoru Odpovědět

21.8.2013 16:14

Neaktivní uživatel

Člen

Neaktivní uživatel:21.8.2013 19:00

Zkus tohle:

                int sum = 0;
                int x = 0;
                foreach (ListViewItem li in listView1.Items)
                {
                    if (int.TryParse(li.SubItems[2].Text.Trim(), out x))
                        sum += x;
                    else
                        MessageBox.Show(li.SubItems[2].Text.Trim());
                }
MessageBox.Show(sum);

Pro všechny neparsovatelný stringy v danym sloupci ti to hodí msgbox, kde se ti ukáže, čim vlastně ten sum chceš plnit (a na konci vyhodí sumu)

Editováno 21.8.2013 19:00

Nahoru Odpovědět

21.8.2013 19:00

Neaktivní uživatelský účet

matesax

Tvůrce

matesax:21.8.2013 19:50

TryParse, out? Proč ne try catch?

for(int index = 0; index < listView1.Items.Count; index++)
    try
    {
        sum += int.Parse(listView1.Items[index].Text);
    }
    catch(FormatException e)
    {
        MessageBox.Show(e.Message + ", in " + listView1.Items[index].Text);
    }

Jinak geniální názvy...

Nahoru Odpovědět

21.8.2013 19:50

Neaktivní uživatel

Člen

Neaktivní uživatel:21.8.2013 21:23

u názvů vycházim z toho, co tam má (u testování označení x nepovažuju za problém - přednější je objevit, kde je chyba). To co si psal, nepude (respektive vyhodí exception vždy) - v tomhle případě sou důležitý subitemy (jedno okýnko v tabulce), ne itemy (jeden řádek).

Jinak proč jo/ne TryParse/try catch? Máš skoro jedno + takle víš jestli ti to sečetlo aspoň něco (+ nováček bude spíš znát tryparse než try catch - i když bude házet msgbox při každym průběhu cyklem, tak kompilátor de stejně stopnout...).

Nahoru Odpovědět

21.8.2013 21:23

Neaktivní uživatelský účet

matesax

Tvůrce

matesax:21.8.2013 21:31

Nechápeš OOP. Musíš dodržovat pojmenovávání OOP v C#... (TextBox.Text, ListViewItems.Items[index].Text, ddd.ToString, ddd.ddd.ToString, ...) Názvy by se měly opakovat - tedy ve všech bodech Frameworku, které řeší obdobný problém najdeš vždy stejné pojmenování - liší se jen objekt, ve kterém se nachází...

Větvení je neřád. V tom TryParse je try blok, ale udělá největší prasárnu - out a výsledek převede do bool - pak se musí větvit. Když dáš try{parse}, nevětvíš, neoutuješ a navíc si můžeš odchytit libovolnou zprávu... catch(výjimka)

Editováno 21.8.2013 21:35

Nahoru Odpovědět

21.8.2013 21:31

matesax

Tvůrce

matesax:21.8.2013 21:36

A chyba:

ListView.Items[index].Text

Nahoru Odpovědět

21.8.2013 21:36

Luboš Běhounek Satik

Tvůrce

Luboš Běhounek Satik:21.8.2013 21:55

V TryParse uvnitř interně žádný try-catch blok není a v případě nečíselného vstupu chyby je rychlejší o několik řádů.

Nahoru Odpovědět

21.8.2013 21:55

https://www.facebook.com/peasantsandcastles/

Neaktivní uživatel

Člen

Neaktivní uživatel:21.8.2013 22:05

možná nerozumim OOP (psal sem to akorát proto, že není jasné, zda si matesak poradil s debugováním), ale ty asi nerozumíš listViewu (listView.Items[index].SubItems[2].Text)

Editováno 21.8.2013 22:06

Nahoru Odpovědět

21.8.2013 22:05

Neaktivní uživatelský účet

matesax

Tvůrce

matesax:21.8.2013 22:15

Což mi může být šumafuk - když to v životě nebudu potřebovat... (Moc nepoužívám Formy - natož jeho Controly...)

Editováno 21.8.2013 22:16

Nahoru Odpovědět

21.8.2013 22:15

Jan Vargovský

Tvůrce

Jan Vargovský:21.8.2013 22:58

Raději použiju tryparse než zbytečný try-catch block, který zpomaluje tisickrát více program. Jak bys to vyřešil, kdyby to tam nebylo ? Každý si myslí, že vyřeší vyhození exception jen pomocí try-catch... Ale jaký dopad to má na výkon už nikdo neřeší...

Nahoru Odpovědět

21.8.2013 22:58

Jan Vargovský

Tvůrce

Jan Vargovský:21.8.2013 23:04

Ještě doplňím, jednu moudrou hlášku, kterou mi dal kolega. Vždycky se dívej na ten program tak, jako bys ho měl denně používat. Věř mi, že kdybys měl vadné data a skákal by ti tam denně 500x textbox, že by ten program brzy letěl do koše

Nahoru Odpovědět

21.8.2013 23:04

Luboš Běhounek Satik

Tvůrce

Luboš Běhounek Satik:21.8.2013 23:06

Já hlavně pořád nechápu, co mají všichni proti ref/out, přehlednost kódu to nijak nesnižuje, protože je to vidět v hlavičce metody i v kódu při jejím volání (té funkce).

I když to také skoro nepoužívám, tak si dovedu představit situace, kdy se to dá využít a hlavně při práci s nativníma knihovnama se bez toho člověk často neobejde, protože tam je tenhle přístup běžný.

Nahoru Odpovědět

21.8.2013 23:06

https://www.facebook.com/peasantsandcastles/

Matyáš Černohous

Tvůrce

Matyáš Černohous:22.8.2013 1:20

Vidím že se tu to tady pořádně rozjelo.... zkusil jsem ten kód a jsem zmatenější víc než na začátku. V messageBoxu se objevila správná hodnota, ale už se to neobjevilo v textboxu. Viz: http://www.2i.cz/7a7785af7c
Jinak s tím debugováním jsem si moc neporozuměl

Editováno 22.8.2013 1:21

Nahoru Odpovědět

22.8.2013 1:20

matesax

Tvůrce

matesax:22.8.2013 6:42

OOP také zpomaluje. Tady nejde o rychlost... V C# jde o správnost OOP návrhu, přehledný nezaujatý kód atd. Try blok je rozhodně lepší, než větvení. Tedy v místě, jako je toto - kde k pádu dojde výjimečně...

Nahoru Odpovědět

22.8.2013 6:42

Neaktivní uživatel

Člen

Neaktivní uživatel:22.8.2013 7:47

Dáváš to do správnýho textboxu (zkontroluj/zkopíruj název - vstup by ten textbox měl mít ok)?

nejakyTextBoX.Text = sum.ToString();

Nahoru Odpovědět

22.8.2013 7:47

Neaktivní uživatelský účet

Matyáš Černohous

Tvůrce

Matyáš Černohous:22.8.2013 13:25

Tak už to funguje, měl jsem tam jen špatnej textBox.
Díky

Nahoru Odpovědět

22.8.2013 13:25