hur kan jag förenkla denna kod på endast få rader?

nabbiii
Gradering

#19478

hejsan,

jag har lyckats få denna kod att funka.. men det känns som om jag gör en massa onödigt... :S
och behöver hjälp med att få koden så effektiv som möjligt...

sen har jag också problem med att om Request.QueryString["id"] är ett värde som inte finns i db:en så förblir sidan bara tom... :S det jag vill då göra är att skriva ut "inga inlägg hittades"..

string strConnectionString = ConfigurationManager.ConnectionStrings["MySQL_ODBC"].ConnectionString;
        OdbcConnection sqlConn = new OdbcConnection(strConnectionString);
        OdbcCommand cmd = new OdbcCommand("SELECT * from ads WHERE id = '" + Request.QueryString["id"] + "'", sqlConn);
        OdbcDataReader odr = null;

        try
        {
            sqlConn.Open();
            odr = cmd.ExecuteReader();

            if (odr != null && odr.Read())
            {
                linkEdit.Attributes.Add("href", "?page=adEdit&id=" + odr["id"].ToString());
                lblName.Text = odr["name"].ToString() + ". ";
                parseToDateTime = (DateTime)odr["date"];
                lblDateTime.Text = "Insänt den " + parseToDateTime.Date.ToString("dd MMMM") + " kl " + parseToDateTime.ToString("t") + ". ";
                imgAdImage.ImageUrl = "usrImg/" + odr["image"].ToString();
                imgAdImage.AlternateText = odr["brand"].ToString() + " " + odr["model"].ToString();
                lblCity.Text = odr["city"].ToString() + ": ";    
            }
        }
        finally
        {
            try
            {
                if (odr != null) odr.Close();
                sqlConn.Close();
            }
            catch (Exception ex)
            {
                throw ex;
            }
        }

ok, löste det med

 DBTool dbt = new DBTool();
        DataSet dsShowSpecificAd = dbt.showAd(Request.QueryString["id"]);

        foreach (DataRow r in dsShowSpecificAd.Tables[0].Rows)
        {
linkEdit.Attributes.Add("href", "?page=adEdit&id=" + odr["id"].ToString());                 lblName.Text = odr["name"].ToString() + ". ";
                parseToDateTime = (DateTime)odr["date"];
                lblDateTime.Text = "Insänt den " + parseToDateTime.Date.ToString("dd MMMM") + " kl " + parseToDateTime.ToString("t") + ". ";
                imgAdImage.ImageUrl = "usrImg/" + odr["image"].ToString();
                imgAdImage.AlternateText = odr["brand"].ToString() + " " + odr["model"].ToString();
                lblCity.Text = odr["city"].ToString() + ": ";   

}

men behöver fortfarande hjälp med att om Request.QueryString["id"] är fel eller tom

Redigerades av nabbiii den 18 februari 2010. Läs tidigare inlägg.
Redigerades av nabbiii den 18 februari 2010. Läs tidigare inlägg.

Skriv ut

ID: #19478 | Skapades: 2010-02-18 22:38:54 | Tråden har 1 svar och har lästs 270 gånger.

Timmie
Timmie
Gradering

#19485

Kolla om Rows.Count eller noll så ser du om det finns några poster eller inte.

ID: #19485 | Skapades: 2010-02-19 20:59:48