Question

I assume this should be fine

bool prefMatch = false;

// Is the frequency the same?
prefMatch = string.Compare(user.Frequency, pref.Action.ToString()) == 0;

so if user.Frequency is “3” and pref.Action.ToString() is “3” then it should set the prefMatch to true right? I’m getting false and I’ve definitely checked the 2 values in the watch tab in VS 2008 just to be sure they’re the same

Answer 1

You can just use ==

prefMath = (user.Frequency == pref.Action.ToString());

Though string.Compare will also work. I suggest there is a problem elsewhere.

— Edit

Also, just for completeness, there is no point assigning a variable to something, and then assigning it again directly after. It’s slightly confusing to do so, so better to leave it unassigned, or assign it all in one spot. This way the compiler can help you if you have a case where it doesn’t get assigned like you think. It is, obviously, acceptable to assign first if you wrap the second assignment in a try/catch though.