en bedre måde at styre konstanter med både et id og en værdi

Måske med enums:
http://www.dotnetfunda.com/codes/code838-how-to-get-enum-value-in-csharp-.aspx

Der både kan have værdi og et tal som id.

Næh egentlig ikke :) Jeg vil nu ikke bruge enums til det, de er besværlige at arbejde med i den sammenhæng der.

Men jeg ville gøre sådan her istedet:

public class ModelType
    {
        public int  Id { get; set; }
        public string Value { get; set; }
    }
public enum ModelTypeId { IkkeOplyst = 0, Koeleskab = 1 }
public static readonly ModelType[] modelTyper = new ModelType[]
            {
                new ModelType() { Id = (int)ModelTypeId.IkkeOplyst, Name = "Ikke oplyst" },
                new ModelType() { Id = (int)ModelTypeId.Koeleskab, Name = "Køleskab" }
            };
        public static ModelType GetModeltype(ModelTypeId modelTypeId)
        {
            var type = modelTyper.Where(x => x.Id == (int)modelTypeId).SingleOrDefault();

            return type ?? new ModelType { Id = -1, Name = "Ukendt" };

           
        }

Kan et Dictionary ikke bruges?

enum ModelTypeId { IkkeOplyst = 0, Koeleskab = 1 }

Dictionary<ModelTypeId, ModelType> modelTyper = new Dictionary<ModelTypeId, ModelType>
{
  {ModelTypeId.IkkeOplyst, "Ikke oplyst"},
  {ModelTypeId.Koeleskab, "Køleskab"}
};

Designet er helt forkert.

Det duer ikke at have en type med 2 værdier som skal have en bestemt relation men denne relation er ikke sikret via typen.

Saa:
* drop ModelType
* brug enum ModelTypeId (enum er det helt rigtige til den slags) overalt i koden
* lav en static helper method eller en extension method når der i forbindelse med output skal vises en tekst

Ja designet er forkert i det lille stykke kode, men jeg tror det er de færreste der tænker at det lille stykke kode skal bruges til andet end spørgsmålet :)

Hvis koden afspejler virkeligheden og dit eneste behov er en keyvalue lign. type så brug en keyvalue :)

KeyValuePair<int, string>

Eller hvorfor ikke bare bruge enum'en direkte?

Ak ja.. der er mange muligheder, når alt kommer til alt er det nok fordi spørgsmålet ikke er særligt præcist stillet ;-)

Et array af konstanter er lidt besynderligt :)

tak for svarene.

til arne_v : kan du give et eksempel på en sådan extension metode?

Til janus: hvad mener du med at bruge enum'en direkte ? - hvordan kan jeg gemme og printe tekst med eksempelvis mellemrum i en enum?

Det er ligegyldigt om der er 100 linier eller 1 million linier kode. Konceptet med to fields der skal matche uden at det sikres af koden er daarligt.

Demo af extension metode:

using System;

namespace E
{
    public enum Classic { HelloWorld, FooBar }
    public static class EnumPrint
    {
        public static string ToReadableString(this Classic e)
        {
            switch (e) {
                case Classic.HelloWorld:
                    return "Hello world";
                case Classic.FooBar:
                    return "Foo bar";
                default:
                    throw new Exception("Unexpected enum value for Classic");
            }
        }
    }
    public class Program
    {
        public static void Main(string[] args)
        {
            Classic e = Classic.FooBar;
            Console.WriteLine(e.ToReadableString());
            Console.ReadKey();
        }
    }
}

ahh.. var ikke klar over at det skulle bruges ifb. med print, men okay. Tjaa jeg ville nok lave det på samme måde som Arnes... eller måske noget alá:

[DIV]
public enum ModelTypeId { IkkeOplyst = 0, Koeleskab = 1 }

    public class ModelHelper
    {
        public Dictionary<ModelTypeId, string> KVP{get; set;}

        public ModelHelper()
        {
            KVP.Add(ModelTypeId.IkkeOplyst, "Ikke oplyst");
            KVP.Add(ModelTypeId.Koeleskab, "Køleskab");
        }

    }

    public class Foo
    {
        public Foo()
        {
            var print = new ModelHelper().KVP[ModelTypeId.Koeleskab];
        }
    }
[/DIV]

Hvis man kun skal bruge en beskrivelse på sin enum, så er det en mulighed at bruge Description attributten fra ComponentModel, altså:
[DIV]
using System.ComponentModel;

namespace ConsoleApplication7
{
    public enum ModelTypeId
        {
            [Description("Ikke Oplyst")]
            IkkeOplyst = 0,
            [Description("Køleskab")]
            Koeleskab = 1 } 

    public class TestMe
    {
        public TestMe()
        {
            var print = ModelTypeId.Koeleskab.EnumDescription();

        }
    }

    public static class EnumExtensions
    {
        public static string EnumDescription(this ModelTypeId value)
        {
            System.Reflection.FieldInfo fi = value.GetType().GetField(value.ToString());

            var attributes = (DescriptionAttribute[])fi.GetCustomAttributes(
                typeof(DescriptionAttribute),
                false);

            return attributes != null &&
                  attributes.Length > 0
                      ? attributes[0].Description
                      : value.ToString();
        }
    }

}

[/DIV]

Dictionary loesningen har praecis samme problem som den originale - der er ikke noget i koden som sikrer konsistens mellem de to vaerdier.

Description er derimod en glimrende maade at goere det paa. Det sikrer en konsistens og samler det hele i en enkelt type.

tak for hjælpen. Jeg synes at janus og arne skal dele pointene. Lægger i et svar?

mht. Description løsningen - hvordan laver jeg et foreach loop igennem enum'et ?

foreach (string name in Enum.GetNames(typeof(ModelTypeId)))
{
}

kunne benyttes - men hvordan får jeg adgang til at kalde EnumDescription() ?

foreach (ModelTypeId modelTypeId in Enum.GetValues(typeof(ModelTypeId)))
        {
int val = (int)modelTypeId;
string description = modelTypeId.EnumDescription();
        }

GetValues var løsningen. bare se bort fra det sidste spørgsmål.

svar fra mig

:)