[c#] try/catch, v katerem gradniku je napaka @ Slo-Tech

Forum » Programiranje »
[c#] try/catch, v katerem gradniku je napaka

[c#] try/catch, v katerem gradniku je napaka

japol :: 15. okt 2012, 11:32

Imam try/catch stavek.

public void Izr()
        {
            try
            {
// koda...
                fckk = System.Double.Parse(textBox1.Text, System.Globalization.NumberStyles.Number, System.Globalization.NumberFormatInfo.CurrentInfo);
// koda...
      for (int štV = 0; štV < (dataGridView1.Rows.Count - 1); štV++)
                {
                    a = double.Parse(dataGridView1.Rows[štV].Cells["Column2"].Value.ToString(), System.Globalization.NumberStyles.Number, System.Globalization.NumberFormatInfo.CurrentInfo);
                    b = double.Parse(dataGridView1.Rows[štV].Cells["Column3"].Value.ToString(), System.Globalization.NumberStyles.Number, System.Globalization.NumberFormatInfo.CurrentInfo);
                    v = double.Parse(dataGridView1.Rows[štV].Cells["Column4"].Value.ToString(), System.Globalization.NumberStyles.Number, System.Globalization.NumberFormatInfo.CurrentInfo);
}
// koda...
}
catch(FormatException e) // če je vpis črka,...
{

}
  catch (NullReferenceException e) // če sploh ni bilo vpisa
{

}

}

Potrebujem nekakšen MessageBox v katerem piše ali je bila napaka narejena v textBox1 ali v dataGridView1, s tem bi uporabnik takoj vedel kje naj išče napako in jo popravi.
A bi lahko pisal nekako tako?:

catch(FormatException e) // če je vpis črka,...
{
if (textBox1... == e)
{
MessageBox.Show ("Napačen vpis fckk.");
}
if (dataGridView1... == e)
{
MessageBox.Show ("Napačen vpis v tabelo.");
}
}

darkkk :: 15. okt 2012, 13:10

1. način: namesto parse uporabi tryparse in z rezultatom tryparsa kontroliraš "flow"
2. način: več try catch blokov
3. kombinacija zgornjih dveh:

try{
int fkkk;
bool ok = Int32.TryParse(tvojniz, out fkk, rest of crap)
if(!ok) throw new Exception("Napaka pri vnosu fkkk");

...
}
catch(Exception ex){
if(ex.msg == "napaka pri vnosu fkkk") MessageBox.Show....
}

Načeloma je sicer fajn, da malo bolj popedeniš exceptione, ki jih mečeš...

Zgodovina sprememb…

spremenil: darkkk (15. okt 2012 ob 13:11)

Red_Mamba :: 15. okt 2012, 13:10

poglej kaj ti e.Source vrne.
spremenljivka e direktno ne vem ce bo sla

[st.slika https://img.shields.io/badge/Slo-Tech-green.svg test]
Linkedin >> http://goo.gl/839Aua
Mamba's Crypto & ICO's: https://t.me/joinchat/AAAAAExTkO4P4UDy0fIZdg

japol :: 15. okt 2012, 13:56

Saj sedaj razmišljam če pri vsaki spremnljivki vstavim svoj try/catch.

Če prikažem primer na zgornji kodi:

public void Izr()
        {
try
            {
fckk = System.Double.Parse(textBox1.Text, System.Globalization.NumberStyles.Number, System.Globalization.NumberFormatInfo.CurrentInfo);
}
catch(FormatException e) // če je vpis črka,...
{
 MessageBox.Show ("Napačen vpis fckk.");
}
  catch (NullReferenceException e) // če sploh ni bilo vpisa
{
 MessageBox.Show ("Napačen vpis fckk.");
}
 
}

in spet za drugo spremenljivko:

public void Izr()
        {
try
            {
for (int štV = 0; štV < (dataGridView1.Rows.Count - 1); štV++)
                {
                    a = double.Parse(dataGridView1.Rows[štV].Cells["Column2"].Value.ToString(), System.Globalization.NumberStyles.Number, System.Globalization.NumberFormatInfo.CurrentInfo);
                    b = double.Parse(dataGridView1.Rows[štV].Cells["Column3"].Value.ToString(), System.Globalization.NumberStyles.Number, System.Globalization.NumberFormatInfo.CurrentInfo);
                    v = double.Parse(dataGridView1.Rows[štV].Cells["Column4"].Value.ToString(), System.Globalization.NumberStyles.Number, System.Globalization.NumberFormatInfo.CurrentInfo);
}
}
catch(FormatException e) // če je vpis črka,...
{
 MessageBox.Show ("Napačen vpis v tabelo.");
}
  catch (NullReferenceException e) // če sploh ni bilo vpisa
{
 MessageBox.Show ("Napačen vpis v tabelo.");
}
 
}

Ampak temu sem se želel izogniti, ker je veliko več tega kot sem tu v primeru prikazal. Bo veliko try/catch statmentov.

p.s. Saj bo druga spremenljivka, primer: (FormatException Napaka)

Zgodovina sprememb…

spremenil: japol (15. okt 2012 ob 13:58)

KrEn1234 :: 15. okt 2012, 15:15

Kot je že darkkk napisal:

try-Catch block je mišljen za izjemne dogodke(npr. če pade mrežna povezava,...). Exception je iz stališča resursev zelo draga zadeva. V tvojem primeru bi jaz raje uporabil tryparse in nato gradil novi ApplicationException.

Red_Mamba :: 16. okt 2012, 12:27

Jaz vcasih naredim tudi tako da v zacetku definiram spremenljivko:

int vrstica = 0;

potem pa prev vsako vrstico nafilam vrstica++;

k se exception zgodi pogledam vrednost spremenljivke vrstica in vem kje se usuje.
Veliko lazje kot malo morje try/catch-ov

[st.slika https://img.shields.io/badge/Slo-Tech-green.svg test]
Linkedin >> http://goo.gl/839Aua
Mamba's Crypto & ICO's: https://t.me/joinchat/AAAAAExTkO4P4UDy0fIZdg

darkkk :: 16. okt 2012, 12:35

Za to maš catch(Exception ex){ex.StackTrace}

Looooooka :: 16. okt 2012, 12:53

kar se tice .parse in .tryparse ti zgoraj napisanih parametrov ni treba pisat, ker so ze privzeti (System.Globalization.NumberFormatInfo.CurrentInfo...in tist System.Globalization.NumberStyles.Number je verjetno tudi odvec).
in zakaj bi lovil nullreference exceptione?...to pomeni, da program dela nekaj kar nisi predvidel. preden uporabit tiste celle- jih daj v eno lokalno sprejemljivko in preveri, da niso null.
Ce seveda slucajno NE dovolis null vnosov

Pa btw...gridu lahko za vsak column nastavis kaksno obliko vnosa zahtevas (string,datetime, format stevilke...) in sploh ne bos rabil preverjat v funkciji, ker bo ze grid poskrbel za to, da ti uporabnik ne vnasa dreka.

Zgodovina sprememb…

spremenilo: Looooooka (16. okt 2012 ob 12:56)

japol :: 18. okt 2012, 13:25

Glede System.Globalization.NumberFormatInfo.CurrentInfo itd... :
Testiral sem program pri kolegih, profesorju,... in dobil napačen izračun. To pa zato, ker imajo inštalirane wolframalpha, mathematico,... in tako imajo decimalno ločilo piko ne pa vejico. S tem sem želel rešiti to da program deluje z nastavitvami kot so v windowsih. Če ima nastavljeno piko, potem da vnaša piko, če pa vejico, pa vejico.
Na to temo je že bila ustvarjena tema:
[c#] textbox, vpis samo cifer, event _TextChanged, pika/vejica

Imam metodo z okoli 800 vrsticami kode. to se nahaja znotraj try, v catch sem pa želel le to da če je prišlo do napake v nekem textboxu, da prikaže messagebox. Da točno veš kje se je pojavila napaka. Saj če ne bo šlo drugače bom pisal try/catch znotraj metode, jih bo pa kar nekaj in temu sem se želel izogniti.

darkkk :: 19. okt 2012, 09:16

Moje mnenje, znotraj "večjih" metod se ne piše try/catch-ov (razen, če imaš opravka z nekimi nestabilnimi stvarmi, kot npr. branje/pisanje datotek), ampak mečeš ven exceptione, katere pohendlaš zunaj.

try{
    mojaGolomaznaMetoda(...);
}
catch (){}
catch(){}
...

japol :: 19. okt 2012, 17:13

Ok, pa na tak način. Sedaj mi podaj še primer kode npr. enega MessageBoxa znotraj catch, če se je pojavila napaka v spremenljivki "a" (recimo da "a" pripada textBox1), ki je znotraj metode mojaGolomaznaMetoda.

Vredno ogleda ...

	Tema	Sporočila	Ogledi	Zadnje sporočilo
	Tema	Sporočila	Ogledi	Zadnje sporočilo
»	[VB] datagridview korenje3 Oddelek: Programiranje	35	1936 (1319)	korenje3 2. mar 2013 17:03:40
»	C# INSERT statment ne vpise podatkov b00mer Oddelek: Programiranje	11	1203 (1077)	darkolord 25. maj 2012 10:24:49
»	c# regularni izraz bombacina Oddelek: Programiranje	20	2015 (1713)	darkolord 18. avg 2009 13:09:07
»	[C#] Ne prkaže takoj podatkov iggy1 Oddelek: Programiranje	8	1132 (995)	darkolord 7. avg 2007 14:02:55
»	[C#] DATA GRID VIEW iggy1 Oddelek: Programiranje	13	1941 (1821)	Kekec 20. apr 2007 08:37:40

Več podobnih tem

Zadnje novice

Zadnji članki

Išči:

Forum » Programiranje »
[c#] try/catch, v katerem gradniku je napaka

[c#] try/catch, v katerem gradniku je napaka