Avatar billede Hansx Nybegynder
22. december 2009 - 19:25 Der er 5 kommentarer og
1 løsning

Er denne database læsning korrekt ?

Hej venner,

Jubii så er det lykkes for mig at læse og skrive til databasen. Meeen for en sikkerhedskyld ville jeg vide om den code jeg har skrevet er korekkt ?

Forresten når jeg havne i catch blokken , hvordan kan jeg sende bruger til en anden side derfra noget med Response.Redirekt("siden")
NB: Response.Redirekt("siden") Den skal ikek skrives efter som functiobeb findes i en class..

............................................................

public static OpretAltprop BrugerLogin(OpretAltprop Login)
        {
            string connectionString = (string)ConfigurationSettings.AppSettings["OpretConnectionString"];
            string commandText = "KUNDE_LOGIND_TJEK";
            try
            {
                using (SqlConnection conn = new SqlConnection(connectionString))
                {
                    using (SqlCommand comm = new SqlCommand(commandText, conn))
                    {
                        comm.CommandType = CommandType.StoredProcedure;
                        comm.Parameters.AddWithValue("@EMAIL", Login.EMAIL);
                        comm.Parameters.AddWithValue("@KODEORD", Login.KODEORD);
                        comm.Connection.Open();

                        using (IDataReader r = comm.ExecuteReader())
                        {
                            OpretAltprop brugerinfo = new OpretAltprop();
                            while (r.Read())
                            {
                                brugerinfo.FORNAVN = r["FORNAVN"].ToString();
                                brugerinfo.KUNDEID = Convert.ToInt32(r["KUNDE_ID"]);
                            }
                            return brugerinfo;
                        }
                    }
                }
              }
              catch (SqlException ex)
              {
                  throw ex;
              }
              catch (Exception ex)
              {
                  throw ex;
              }
      }
Avatar billede arne_v Ekspert
22. december 2009 - 19:35 #1
Det ser rimeligt godt ud.

Forslag:
- brug mixed case ikke all upper case for property navne
- kald OpretAltprop constructor inden i loekken
- test paa om der er mere end en raekke
Avatar billede bvli Praktikant
23. december 2009 - 09:50 #2
- og lad være med at kaste din exception igen - det ødelægger din stacktrace. (brug throw; i stedet for throw ex; - eller lad helt være med at catch'e hvis du ikke kan håndtere det)
Avatar billede arne_v Ekspert
24. december 2009 - 00:50 #3
Det var en rigtig god pointe.
Avatar billede arne_v Ekspert
10. januar 2010 - 04:30 #4
hans ?
Avatar billede Hansx Nybegynder
24. juli 2010 - 15:58 #5
tak
Avatar billede arne_v Ekspert
24. juli 2010 - 19:56 #6
Og du tog selv point fordi??
Avatar billede Ny bruger Nybegynder

Din løsning...

Tilladte BB-code-tags: [b]fed[/b] [i]kursiv[/i] [u]understreget[/u] Web- og emailadresser omdannes automatisk til links. Der sættes "nofollow" på alle links.

Loading billede Opret Preview
Kategori
IT-kurser om Microsoft 365, sikkerhed, personlig vækst, udvikling, digital markedsføring, grafisk design, SAP og forretningsanalyse.

Log ind eller opret profil

Hov!

For at kunne deltage på Computerworld Eksperten skal du være logget ind.

Det er heldigvis nemt at oprette en bruger: Det tager to minutter og du kan vælge at bruge enten e-mail, Facebook eller Google som login.

Du kan også logge ind via nedenstående tjenester