Vérifier que ma variable contient bien une valeur de ma structure

Version imprimable

Bonjour,

j'utilise la structure pour la première fois parce que j'ai une variable qui ne peux prendre que 6 valeurs (en base je fais un check pour voir si elle est égale à une de ces valeurs).

donc j'ai ma structure (incomplète encore):
Code:

1 2 3 4 5 6 public struct TypeCategorie { public static const string Secteur = "Secteur"; public static const string Complexe = "Complexe"; }
et ma variable qui prend une de ces valeurs:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 public string Categorie { get; set { if (value == TypeCategorie.Secteur) { Categorie = value; } else { throw new Exception("La catégorie d'exploitation est incorrecte: elle doit correspondre a secteur, complexe..."); } } }
je voulais déjà savoir si ce code était correcte et si c'était possible de vérifier si ma variable était bien égale à un des champs de ma structure sans faire la vérification champs par champs.

30/06/2011, 11h20
Er3van

Si TypeCategorie était une classe, tu pourrais lui affecter une méthode statique du type

Code:

public static boolean IsTypeCategorie(String value) { ... }

Ca serait sans doute plus propre que de tout tester dans ta classe consommatrice.

ça revient à faire une vérification champs par champs?
je voulais savoir s'il existait un moyen de faire ça

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
public string Categorie
        {
            get;
            set
            {
                if (value == ValueOf(TypeCategorie))
                { Categorie = value; }
                else
                { throw new Exception("La catégorie d'exploitation est incorrecte: elle doit correspondre a secteur, complexe..."); }
            }
        }

Une fonction déjà toute faite qui permettrait de faire ça?

30/06/2011, 11h48
NicoL__

Et pourquoi pas une enum ça simplifierait la chose non ?
30/06/2011, 11h53
Er3van

Tu pourrais utiliser le mot clé IS si tu construisais tes avec la classe TypeCategorie, mais à mon avis tu les construits avec String...

(Et ça ferait une utilisation farfelue...)
30/06/2011, 11h54
youtpout978
le problème de l’énumération c'est qu'il ne peut pas renvoyer de string ce qui reviendrait à faire un switch derrière pour récupérer la string correspondant a mon enum alors qu'avec la structure je peut directement affecté une string en faisant:
Code:

1 2 3 Exploitation test=new Exploitation(); test.Categorie = Exploitation.TypeCategorie.Secteur;

bon bein on va faire une vérification champs par champs

Code:

1
2
3
4
5
6
7
8
9
 
private bool IsTypeCategorie(string value)
        {
            bool ok = false;
            if (value == TypeCategorie.Complexe || value == TypeCategorie.Secteur)
            { ok = true; }
 
            return ok;
        }

30/06/2011, 12h44
NicoL__

un enum peut renvoyer : toString()
http://msdn.microsoft.com/en-us/library/16c1xs4z.aspx

Autre option :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
public class TypeCategorie
        {
           private static const string Secteur = "Secteur";
           private static const string Complexe = "Complexe";
           private static const List<String> categories = new List<string> { Secteur, Complexe };
 
           private bool IsTypeCategorie(string value)
           {
               if (categories.contains(value)) {
                      return true;
               }
               return false;
           }
        }

30/06/2011, 13h53
youtpout978
pour l'enum toString on est obligé de faire:
Code:

1 2 3 Enum typeCategorie=TypeCategorie.Complexe; String categorie=typeCategorie.ToString();
et pour la classe ça m'oblige a créer une liste contenant toutes les valeur en plus je voudrais pourvoir faire simplement:
Code:

1 2 String categorie=TypeCategorie.Complexe;
on dirait qu'il n'y a pas vraiment de méthode permettant le listing de tout ce qu'une structure contient ce qui m'aurai permis de faire une simple recherche dessus.

tu veux peut être créer un List<TypeCategorie>, franchement je crois pas que tu es compris ce qu'est une struct par rapport à une class... ça se ressemble beaucoup.
Enfin moi ce que j'ai compris de ton problème ça donne ça comme solution :
Code:

1 2 enum TypeCategorie {Secteur, Complexe}
Variable :
Code:

1 2 3 4 5 6 7 8 9 public TypeCategorie Categorie { get; set; } public String CategorieString { get{return Categorie.toString()} }

Si, si tu veux y accéder aussi en lecture :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
public class TypeCategorie
        {
           private static const string _secteur = "Secteur";
           public static string Secteur {
                 get { return _secteur ; }
           }
 
           private static const string _complexe = "Complexe";
           public static string Complexe {
                 get { return _complexe ; }
           }
 
           private static const List<String> categories = new List<string> { Secteur, Complexe };
 
           private bool IsTypeCategorie(string value)
           {
               if (categories.contains(value)) {
                      return true;
               }
               return false;
           }
        }

Là tu peux bien faire TypeCategorie.Complexe, et tu peux ne pas parcourir l'ensemble. Je vois difficilement comment faire mieux dans ton cas...

Voilà ce que j'ai fais:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
 
 
    public class Exploitation : ObjetDomaine
    {
        public string Identifiant { get; set; }
        public string Parent_id { get; set; }
        public string Libelle { get; set; }
        public string Categorie
        {
            get { return Categorie; }
            set
            {
                if (TypeCategorie.categories.Contains(value))
                { Categorie = value; }
                else
                { throw new Exception("La catégorie d'exploitation est incorrecte: elle doit correspondre à secteur, complexe..."); }
            }
        }
 
 
        public struct TypeCategorie
        {
            public const string Secteur = "Secteur";
            public const string Complexe = "Complexe";
            public const List<string> categories = new List<string> { Secteur, Complexe };
 
        }
 
        public Exploitation() { }
 
    }

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
 
 public class Exploitation : ObjetDomaine
    {
        public enum TypeCategorie {Secteur, Complexe}
        private TypeCategorie  cat;
        public string Identifiant { get; set; }
        public string Parent_id { get; set; }
        public string Libelle { get; set; }
 
        public string Categorie
        {
            get { return cat.ToString(); }
            set
            {
               cat = (TypeCategorie ) Enum.Parse(typeof(TypeCategorie ), value, true);
            }
        }
 
        public Exploitation() { }
 
    }

ça enverra une exception (ArgumentException) si la valeur n'est pas possible.

30/06/2011, 15h32
youtpout978

merci pour cette seconde solution, je comprends pas pourquoi ils n'ont pas rajouté la possibilité d'énumérer des string et autre ...
30/06/2011, 15h53
Nathanael Marchand

On parle de ca?

Code:

var list = Enum.GetNames(typeof(MonEnum));
30/06/2011, 15h53
NicoL__

Oui c'est vrai, de base l'enum se converti implicitement en int donc c'est pratique si on manipule des entiers. Et en général en base de données le pk et autre id sont numérique donc ça marche bien.
Avec les string les enum sont moyennement pratique, c'est un peu tordu.
30/06/2011, 15h59
Nathanael Marchand

Je rajoute égalment que les structures NE SONT PAS FAITES POUR CA
La seule différence entre une classe et une structure c'est que la première est un type valeur, l'autre un type référence.

Citation:

Envoyé par Nathanael Marchand

Je rajoute égalment que les structures NE SONT PAS FAITES POUR CA

+1000

Dans le code de l'appli sur laquelle je bosse au bureau il y avait un truc comme ça, j'ai sauté au plafond quand je l'ai vu :
Code:

1 2 3 4 5 6 7 8 9 public struct ItemState { public const int ToBeSaved = 0; public const int Saved = 1; public const int NotModified = 2; public const int Deleted = 3; public const int UnSelected = 4; public const int Incomplete = 5; }
Ici c'est très clairement un enum qu'il faut utiliser. On peut tester si une valeur appartient à l'enum avec la méthode Enum.IsDefined (ça peut prendre un paramètre une valeur de l'enum, ou le nom d'une valeur de l'enum)

Citation:
Envoyé par tomlev
+1000

Dans le code de l'appli sur laquelle je bosse au bureau il y avait un truc comme ça, j'ai sauté au plafond quand je l'ai vu :
Code:

1 2 3 4 5 6 7 8 9 public struct ItemState { public const int ToBeSaved = 0; public const int Saved = 1; public const int NotModified = 2; public const int Deleted = 3; public const int UnSelected = 4; public const int Incomplete = 5; }
Ici c'est très clairement un enum qu'il faut utiliser. On peut tester si une valeur appartient à l'enum avec la méthode Enum.IsDefined (ça peut prendre un paramètre une valeur de l'enum, ou le nom d'une valeur de l'enum)
Surtout qu'en fait, que ca soit une structure ou une classe, ca change strictement rien car aucune instance n'est jamais créée, c'est des membres statiques ou des constantes remplacées au moment de la compilation!

30/06/2011, 16h14
youtpout978

dans ce cas là l'enum est faite pour ça mais qu'elle est le meilleur moyen pour énuméré des strings ?
30/06/2011, 16h26
tomlev

Citation:

Envoyé par youtpout978

dans ce cas là l'enum est faite pour ça mais qu'elle est le meilleur moyen pour énuméré des strings ?

Tu veux dire pour récupérer toutes les valeurs possibles ? Nathanael te l'a indiqué plus haut : Enum.GetNames(typeof(MonEnum))
30/06/2011, 16h28
NicoL__

Ben l'Enum c fait pour ça, c'est juste qu'il faut utiliser les méthodes static d'Enum pour manipuler sa face "String".
30/06/2011, 17h16
youtpout978
Citation:

Envoyé par tomlev

Tu veux dire pour récupérer toutes les valeurs possibles ? Nathanael te l'a indiqué plus haut : Enum.GetNames(typeof(MonEnum))

non je parlais pas de cette méthode mais plutôt de ça
Code:

1 2 public enum TypeCategorie { Secteur="Secteur", Complexe="Complexe" }
30/06/2011, 17h23
Nathanael Marchand

Oui mais on a dit que ca, c'était pas possible! Un enum c'est un entier point final. Qu'est ce que ca change pour toi que derriere ca soit un string, un int ou un guid?

je sais très bien que c'est impossible c'est pour ça que je suis partie sur le struct pour pouvoir faire directement

Code:

1
2
3
 
Exploitation test=new Exploitation();
test.Categorie = Exploitation.TypeCategorie.Complexe;

finalement je suis partie sur ça (qu'est ce qu'on peut s'emmerder sur des trucs aussi simple):

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
 
 public class Exploitation : ObjetDomaine
    {
        public enum TypeCategorie { Secteur, Complexe }
        private string _categorie;
        public string Identifiant { get; set; }
        public string Parent_id { get; set; }
        public string Libelle { get; set; }
 
        public string Categorie
        {
            get
            {
                if (IsTypeCategorie(_categorie))
                    return _categorie;
                else
                    throw new Exception("La catégorie d'exploitation est incorrecte ou ne contient aucun élément");
            }
            set
            {
                if (IsTypeCategorie(value))
                    _categorie = value;
                else
                    throw new Exception("La catégorie d'exploitation est incorrecte: elle doit correspondre a secteur, complexe...");
            }
        }
 
        private bool IsTypeCategorie(string value)
        {
            if (Enum.GetNames(typeof(TypeCategorie)).Contains(value))
                return true;
            else
                return false;
        }
 
        public Exploitation() { }
 
    }

Ah ben la du coup, t'as loupé le message de tomlev:

Code:

IsTypeCategorie(_categorie)

peut se remplacer par:

Code:

Enum.IsDefined(typeof(TypeCategorie), _categorie));

Mais cela dit, je vois toujours pas pourquoi tu te trimballes des strings:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 public class Exploitation : ObjetDomaine { public enum TypeCategorie { Secteur, Complexe } public string Identifiant { get; set; } public string Parent_id { get; set; } public string Libelle { get; set; } public TypeCategorie Categorie {get;set;} public Exploitation() { } }
Ca marche très bien! Si tu veux l'afficher dans un GUI WPF ou Silverlight tu utilises un converter pour representer la valeur par un autre string, une image, etc.

30/06/2011, 18h06
Er3van

Si tu voulais des strings, la solution à base de classe que je t'ai proposé fonctionne bien. Tu peux même la coupler avec un fichier de ressource pour être plus propre.

disons que mon objet exploitation correspond à une table de ma base qui à un champs id, parent_id,libelle, categorie ...

j'ai fait ça

Code:

1
2
3
4
5
6
7
8
 
private bool IsTypeCategorie(string value)
        {
            if (Enum.IsDefined(typeof(TypeCategorie), value))
                return true;
            else
                return false;
        }

j'essaye de trouver s'il existe une librairie permettant de faire ce genre de chose

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
 
public enumType Categorie:String
{
Secteur="secteur"
}
 
public enumType Modele:Voiture
{
ferrari=voitureFerrari()
}
 
private static voiture voitureFerrari()
{
....
}

30/06/2011, 19h33
Nathanael Marchand

J'ai rien compris a ton code, mais en tout cas je pense qu'il y'a un problème de modélisation dans la base! Tu ne devrais pas avoir les valeurs sous la forme de chaine de caractères pour les valeurs de tes enumerations! Il faut les stocker sous une forme d'entier et éventuellement créer une jointure avec une table qui contient les valeurs. Tout ca dans un soucis de normalisation.
30/06/2011, 20h14
youtpout978

vu la tête de la base c'est pas cette table qui va la pourrir plus qu'elle ne l'est déjà
01/07/2011, 09h18
Er3van

Citation:

Envoyé par youtpout978

vu la tête de la base c'est pas cette table qui va la pourrir plus qu'elle ne l'est déjà

C'est comme ça qu'une application se dégrade.

Partant de ce principe là, tu peux aussi créer un index sur un texte, ne plus mettre de clé primaire et renommer toutes tes tables en camel case...
01/07/2011, 10h33
NicoL__

Er3van je crois qu'aujourd'hui la règles pour le nommage des table c'est bien la camelCase...:roll:
01/07/2011, 11h25
Er3van

Cela me surprend...

http://sqlpro.developpez.com/cours/standards/#L2.2
01/07/2011, 11h44
tomlev

Citation:

Envoyé par Er3van

Cela me surprend...

http://sqlpro.developpez.com/cours/standards/#L2.2

Ouais mais l'article est pas tout neuf, et les conventions, ça peut évoluer...

C'est vrai que jusqu'ici, en pratique j'ai surtout vu des noms sous la forme décrite dans l'article, mais on voit de plus en plus de noms de tables en CamelCase, notamment dans les articles de Microsoft concernant Entity Framework et les technos qui tournent autour

De toutes façons une convention, ce n'est pas unique, tout le monde n'est pas obligé d'adopter la même
01/07/2011, 14h06
Er3van

Oui, et jusqu'à ce que Microsoft affirme clairement une volonté d'uniformiser ça, ça restera à la discrétion des (développeurs et) dba.

Je suis partit sur les enums finalement

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
 
   public class Exploitation : ObjetDomaine
    {
        public string Identifiant { get; set; }
        public string Parent_id { get; set; }
        public string Libelle { get; set; }
        private string _categorie;
 
        public enum TypeCategorie { Secteur, Complexe, LS, Systeme, Gare, Voie }
 
        public string Categorie
        {
            get
            {
                if (IsTypeCategorie(_categorie))
                    return _categorie;
                else
                    throw new Exception("La catégorie d'exploitation est incorrecte ou ne contient aucun élément");
            }
            set
            {
                if (IsTypeCategorie(value))
                    _categorie = value;
                else
                    throw new Exception("La catégorie d'exploitation est incorrecte: elle doit correspondre à un secteur, complexe...");
            }
        }
 
        private bool IsTypeCategorie(string value)
        {
            if (Enum.IsDefined(typeof(TypeCategorie), value))
                return true;
            else
                return false;
        }
 
        public Exploitation() { }
 
    }

Voilà la base pour info:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
 
/****** Objet*:  Table [dbo].[TRD_EXPLOITATION]    Date de génération du script*: 07/04/2011 09:20:01 ******/
SET ANSI_NULLS ON
GO
SET QUOTED_IDENTIFIER ON
GO
SET ANSI_PADDING ON
GO
CREATE TABLE [dbo].[TRD_EXPLOITATION](
	[EXP_ID] [numeric](18, 0) IDENTITY(1,1) NOT NULL,
	[EXP_EXP_ID] [numeric](18, 0) NULL,
	[EXP_LIBELLE] [varchar](50) COLLATE French_CI_AS NOT NULL,
	[EXP_CATEGORIE] [varchar](50) COLLATE French_CI_AS NOT NULL,
	[EXP_USR_CRE] [varchar](10) COLLATE French_CI_AS NULL,
	[EXP_DAT_CRE] [datetime] NULL,
	[EXP_USR_MAJ] [varchar](10) COLLATE French_CI_AS NULL,
	[EXP_DAT_MAJ] [datetime] NULL,
PRIMARY KEY CLUSTERED 
(
	[EXP_ID] ASC
)WITH (PAD_INDEX  = OFF, IGNORE_DUP_KEY = OFF) ON [PRIMARY]
) ON [PRIMARY]
 
GO
SET ANSI_PADDING OFF
GO
ALTER TABLE [dbo].[TRD_EXPLOITATION]  WITH CHECK ADD FOREIGN KEY([EXP_EXP_ID])
REFERENCES [dbo].[TRD_EXPLOITATION] ([EXP_ID])
GO
ALTER TABLE [dbo].[TRD_EXPLOITATION]  WITH CHECK ADD CHECK  (([EXP_CATEGORIE]='Voie' OR [EXP_CATEGORIE]='Gare' OR [EXP_CATEGORIE]='Systeme' OR [EXP_CATEGORIE]='LS' OR [EXP_CATEGORIE]='Complexe' OR [EXP_CATEGORIE]='Secteur'))

je passe à résolu merci pour votre aide :ccool: