Invalid token 'else' in class, struct, or interface member declaration

Question

i try to highlight record ...like when any one wants to upload documet then in repeater i try to highlight this new record and when user click on this document then this becomes as normal position means not highlight

<tr style='<%#if(DataBinder.Eval(Container.DataItem, "ViewedType")== 1) 
 { %> background-color: yellow;  <% }
   else { <% background-color: white;
    <%} %>'>

but it shows me error

CS1519: Invalid token 'else' in class, struct, or interface member declaration

Source Error:

Line 128:  style='<%#if(DataBinder.Eval(Container.DataItem, "ViewedType")== 1) 
Line 129:  { %> background-color: yellow;  <% }
Line 130:  else { <% background-color: white;
Line 131:  <%} %>'>
Line 132:  <%--<td>

how to solve?

You should really be doing this logic in the code behind, before binding the values, rather than in the template. If the bound column is simply the color you can just bind the background color to that. — Servy, Feb 05 '14 at 19:22

John Gibb · Answer 1 · 2014-02-13T20:44:47.040

1

You can't use control structures (like if statements) inside of a databinding expression tag (which is <%# %>), but you also can't use DataBinder inside of a regular tag (<% %>).

I'd recommend just using the conditional operater inline like this:

<tr style='background-color: <%# (bool) DataBinder.Eval(Container.DataItem, "ViewedType") ? "yellow" : "white" %>'>

edited Feb 13 '14 at 20:44

answered Feb 05 '14 at 19:23

John Gibb

10,603
2
37
48

it shoW me error "if(DataBinder.Eval(Container.DataItem, "ViewedType")== 1) " in this line ERROR IS CS0103: The name 'Container' does not exist in the current context – user3265377 Feb 05 '14 at 19:31
Yea, that's correct, you can only use DataBinder inside of `<%# %>` tags. I've got anohter idea, will edit – John Gibb Feb 05 '14 at 19:41
@user3265377 Try now, that should work for you I think. – John Gibb Feb 05 '14 at 19:44
it shows me error Operator '==' cannot be applied to operands of type 'object' and 'int' – user3265377 Feb 05 '14 at 20:26
@user3265377 Try now. You needed a cast. – John Gibb Feb 05 '14 at 21:00
this shows me error ...Specified cast is not valid. – user3265377 Feb 05 '14 at 22:02
You gotta figure this out on your own. Figure out what `ViewedTypes` is, apparently it's not an `int` even though that's what you're comparing it to. If it's a string, then use `(string) DataBinder.Eval(Container.DataItem, "ViewedTypes") == "1"` instead. – John Gibb Feb 05 '14 at 22:11
and if it is bit... then what would i do – user3265377 Feb 06 '14 at 05:09
Cast it to a bool, use `(bool) DataBinder.Eval(Container.DataItem, "ViewedTypes")` – John Gibb Feb 06 '14 at 17:08
hey i try this .. and it shows me error .... Operator '==' cannot be applied to operands of type 'bool' and 'string' – user3265377 Feb 07 '14 at 00:02
`` – John Gibb Feb 07 '14 at 16:08
what does what about 1 mean? – John Gibb Feb 10 '14 at 23:15
Yes. a `bit` in sql is a `bool` in c#. That means that you can just use it directly and not have to compare it to 1. – John Gibb Feb 11 '14 at 15:33
No. Literally just put this and try it. `` – John Gibb Feb 13 '14 at 20:44
Most of this thread isn't questions about data binding - it's moved onto questions about types in C#.HTML/JavaScript aren't strongly typed, but once you go into a code block, you're in strongly-typed land. http://en.wikipedia.org/wiki/Type_safety – DrShaffopolis Feb 13 '14 at 21:01

score 1 · Answer 2 · answered Feb 05 '14 at 19:49

1

Try

<tr style='background-color: <%# ChooseColor((int)DataBinder.Eval(Container.DataItem, "ViewedType")) #>;'>

where

protected string ChooseColor(int viewedType){
    if (viewedType == 1) return "yellow"; else return "white";
}

answered Feb 05 '14 at 19:49

John Saunders

160,644
26
247
397

Nice option too, I think you'll need to add a cast in the markup since DataBinder.Eval returns an object. Something like `<%# ChooseColor((int) DataBinder....` – John Gibb Feb 05 '14 at 19:50
@JohnSaunders the code which is protected string is in .aspx or in .aspx.cs – user3265377 Feb 05 '14 at 20:27
.aspx.cs. There should be only a bare minimum of C# code in the .aspx file. Nothing more than simple data binding expressions. – John Saunders Feb 05 '14 at 20:49

Invalid token 'else' in class, struct, or interface member declaration

2 Answers2