Asp mvc 5 condition dans une view

Bonjour,

J'ai un tableau qui liste des messages, je voudrais faire en sorte de mettre en avant les lignes dont le message n'est pas encore lu.
J'ai essayé cela :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

if (!item.Vu) { ViewBag.vu = "active"; } else { ViewBag.vu = "";   } <tr class=@ViewBag.vu>

---

Mais il me met la class="active" partout.... bien sur ce morceau de code et mis dans une boucle foreach et item correspond a un message à la fois.

Quelqu'un aurait une idée?

Merci

Bonjour,

Le code que tu as fourni n'est pas suffisant pour te donner une piste de solution. Pour être en mesure de t'aider, il faudra que tu fournisses plus d'information.

On aura besoin probablement tout le code de l'action qui exécuté, tout comme la logique qui d'initialiser ton tableau liste des messages.

Si tu ne souhaites pas fournir plus d'information, il faudra mettre débogueur pour vérifier le contenu de ton tableau de messages à la suite de l'initialisation ou tout autre traitement avant l'appel de méthode qui permet de parcourir ce dernier, pour vérifier si les données sont déjà cohérentes avec qui sont attendues.

Bon courage!

Bonjour,
Déjà je ne comprends pas pourquoi tu utilises le viewBag, normalement il est fait pour transférer les données entre le controller et la vue, il a une portée sur toute la vue.
Personnellement ce que je ferai c'est que dans la classe qui correspond à l'objet item je rajouterai une propriété en lecture myClass que j'alimenterai tel que tu l'as fais dans ta vue.

Du coup, dans ta vue tu n'aurais que

Code:

---

<tr class="@item.myClass">

---

Pour que ce soit propre il faut bien évidemment que la classe qui correspond à l'objet item soit un viewModel, car on ne peut pas rajouter des informations d'affichage dans un modèle métier.

Une autre option serait de passer par une classe static Converter qui transformerai un booléen en nom de classe css.

Cordialement
Soazig

Citation:

---

Envoyé par soazig

Bonjour,
Déjà je ne comprends pas pourquoi tu utilises le viewBag, normalement il est fait pour transférer les données entre le controller et la vue, il a une portée sur toute la vue.
Personnellement ce que je ferai c'est que dans la classe qui correspond à l'objet item je rajouterai une propriété en lecture myClass que j'alimenterai tel que tu l'as fais dans ta vue.

Du coup, dans ta vue tu n'aurais que

Code:

---

<tr class="@item.myClass">

---

Pour que ce soit propre il faut bien évidemment que la classe qui correspond à l'objet item soit un viewModel, car on ne peut pas rajouter des informations d'affichage dans un modèle métier.

Une autre option serait de passer par une classe static Converter qui transformerai un booléen en nom de classe css.

Cordialement
Soazig

---

ou plus simplement dans ta boucle. (le @{} ne sera p-e pas nécessaire en fonction de ta programmation)

Code:

---

1 2 3 4 5 6 7 8 9

@{ string myClass = string.Empty; if(!item.VU){ myClass = "active"; } }   <tr class="@myClass ">

---

Je ne suis pas fan de mettre dans un modèle de données des propriétés de représentation. Je préfère que ce soit la responsabilité de la vue :)

Bonjour,
Pour Dokho1000,

Citation:

---

Envoyé par Dokho1000

Je ne suis pas fan de mettre dans un modèle de données des propriétés de représentation. Je préfère que ce soit la responsabilité de la vue :)

---

Dans le modèle, je suis d'accord, dans un view modèle cela ne me choque pas (j'avais bien précisé si c'est un view modele).
sinon la solution de la variable définie dans la vue, peut marcher aussi.

Cordialement
Soazig