Meddelande

Minska
No announcement yet.

Kan inte bestämma mig vilken kod som är bäst

Minska
X
 
  • Filter
  • Klockan
  • Show
Clear All
new posts

  • Kan inte bestämma mig vilken kod som är bäst

    Jag har en frågesport på min hemsida som jag försöker att förändra/förbättra.
    Just nu så vet jag inte om jag gör den bättre eller sämmre.
    Gammal kod:
    [kod]
    if (RadioButtonList1.SelectedValue.ToString() == ViewState["svar1"].ToString())
    {
    ratt += 1;
    textFont1 = "blue";
    }
    else
    textFont1 = "red";

    if (RadioButtonList2.SelectedValue.ToString() == ViewState["svar2"].ToString())
    {
    ratt += 1;
    textFont2 = "blue";
    }
    else
    textFont2 = "red";
    //osv till fråga nr 10
    [/kod]
    Sen försökte jag "uppgradera" koden. Jag tänkte att jag inte skulle behöva
    så många rader kod...
    [kod]
    //först denna för att kolla värdet
    private static string checkValue(int ratt, string thisValue, string nextValue)
    {
    string textColor = "";
    string sRatt = "";
    if (thisValue == nextValue)
    {
    ratt += 1;
    textColor = "blue";
    sRatt = ratt.ToString();
    }
    else
    {
    ratt += 0;
    textColor = "red";
    sRatt = ratt.ToString();
    }

    string svar = sRatt + "," + textColor;
    return svar;
    }
    //sedan denna för att splita
    private static string deleString(int i, string result)
    {
    string newResult = "";
    string[] nytt;
    if (i == 1)
    {
    nytt = result.Split(',');
    newResult = nytt[0].ToString();
    }
    else
    {
    nytt = result.Split(',');
    newResult = nytt[1].ToString();
    }
    return newResult;
    }

    //och sedan för att köra det
    ratt = Convert.ToInt32(deleString(1, checkValue(ratt, RadioButtonList1.SelectedValue.ToString(), ViewState["svar1"].ToString()).ToString()));
    textFont1 = deleString(2, checkValue(ratt, RadioButtonList1.SelectedValue.ToString(), ViewState["svar1"].ToString()).ToString());
    //osv up till fråga 10
    [/kod]
    Är jag helt ute och reser eller har ni något bättre förslag?
    saw
    _______________________
    www.saw.se
    _______________________

  • #2
    Jag har nu lyckats med att lösa problemet.
    Jag har nu lyckats med att lösa problemet.
    [kod]
    //göra mina variabler public
    public string textFont1 = "red";
    public string textFont2 = "red";
    //osv
    public int ratt = 0;
    //en ny class
    private string makeValue(RadioButtonList rbl, string vieState, string textFont)
    {
    if (rbl.SelectedValue.ToString() == vieState)
    {
    ratt += 1;
    textFont = "blue";
    }
    return textFont;
    }
    //och sedan
    textFont1 = makeValue(RadioButtonList1, ViewState["svar1"].ToString(), textFont1);
    [/kod]Det är nog inte omöjligt att någon kan hitta ett bättre/effektivare sätt, men jag är nöjd med att jag löste det.
    saw
    _______________________
    www.saw.se
    _______________________

    Kommentera

    Working...
    X