---
title: "Kan inte bestämma mig vilken kod som är bäst"
type: "forum-thread"
url: "https://www.webforum.nu/amne/dotnet/189462-kan-inte-bestämma-mig-vilken-kod-som-är-bäst"
topic: ".NET"
topic_url: "https://www.webforum.nu/amne/dotnet"
author: "saw"
published: "2012-03-14T09:25:59.000Z"
updated: "2012-03-14T21:14:43.000Z"
replies: 1
views: 694
page: 1
pages: 1
language: "sv-SE"
site: "webForum — webforum.nu"
rights: "Upphovsrätten till varje inlägg tillhör dess författare."
attribution: "Citera som: webForum, https://www.webforum.nu/amne/dotnet/189462-kan-inte-bestämma-mig-vilken-kod-som-är-bäst"
---

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

## #1 — saw, 2012-03-14T09:25Z

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:

```
        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
```

Sen försökte jag "uppgradera" koden. Jag tänkte att jag inte skulle behöva
så många rader 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
```

Är jag helt ute och reser eller har ni något bättre förslag? :stud

Permalänk: https://www.webforum.nu/p/189462

## #2 — saw, 2012-03-14T21:14Z

Jag har nu lyckats med att lösa problemet. 
Jag har nu lyckats med att lösa problemet. 

```
//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);
```

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.  :bire

Permalänk: https://www.webforum.nu/p/2296781

---

Tråden på webben: https://www.webforum.nu/amne/dotnet/189462-kan-inte-bestämma-mig-vilken-kod-som-är-bäst