Whats up with this?

[JJ]

Hi all,

whats wrong with the following code:

DateTime calcDate = new DateTime();

DateTime dDate = DateTime.Now;

if (calcDate.DayOfWeek(dDate) != "Saturday") || (calcDate.DayOfWeek(dDate)
!= "Sunday")

{

}

Thanks,

Jim

[aaron]

umm..not sure what you are trying to do...but I'll suggest anyway.

DateTime dateTime = DateTime.Now;

if (dateTime.DayOfWeek == DayOfWeek.Tuesday)
{
Console.WriteLine("Yes it is tuesday");
}

"JJ" <jm***@bellatlantic.net> wrote in message
news:ur*************@TK2MSFTNGP09.phx.gbl...

Hi all,

whats wrong with the following code:

DateTime calcDate = new DateTime();

DateTime dDate = DateTime.Now;

if (calcDate.DayOfWeek(dDate) != "Saturday") || (calcDate.DayOfWeek(dDate)
!= "Sunday")

{

}

Thanks,

Jim

[Mattias Sjögren]

Jim,

whats wrong with the following code:

DateTime calcDate = new DateTime();

DateTime dDate = DateTime.Now;

if (calcDate.DayOfWeek(dDate) != "Saturday") || (calcDate.DayOfWeek(dDate)
!= "Sunday")

{

}

DayOfWeek is a property, you're using it like a method. It returns a
value of the enum type DayOfWeek, not a string. Try it like this

if (dDate.DayOfWeek != DayOfWeek.Saturday ||
dDate.DayOfWeek != DayOfWeek.Sunday)

But then, that code is pretty much useless, since it will always
evaluate to true.

What did you actually want to accomplish?

Mattias

--
Mattias Sjögren [MVP] mattias @ mvps.org
http://www.msjogren.net/dotnet/
Please reply only to the newsgroup.

[JJ]

Hi Matt,

Trying to detect in if statement that its not Sat or Sun do the
following code.
So I should use the logical and " && " instead, correct?

Thanks,

JJ
"Mattias Sjögren" <ma********************@mvps.org> wrote in message
news:%2***************@TK2MSFTNGP10.phx.gbl...

Jim,

whats wrong with the following code:

DateTime calcDate = new DateTime();

DateTime dDate = DateTime.Now;

if (calcDate.DayOfWeek(dDate) != "Saturday") || (calcDate.DayOfWeek(dDate)!= "Sunday")

{

}

DayOfWeek is a property, you're using it like a method. It returns a
value of the enum type DayOfWeek, not a string. Try it like this

if (dDate.DayOfWeek != DayOfWeek.Saturday ||
dDate.DayOfWeek != DayOfWeek.Sunday)

But then, that code is pretty much useless, since it will always
evaluate to true.

What did you actually want to accomplish?

Mattias

--
Mattias Sjögren [MVP] mattias @ mvps.org
http://www.msjogren.net/dotnet/
Please reply only to the newsgroup.

[JJ]

Ah it was the brackets around the expressions that was causing the Invalid
expression.

But at the same time Mattias you were saying that my logic with the if was
going to fail anyway because of using Or correct?

I should be using the and operator && instead in order for the logic to
work, correct?

Thanks,
JJ

"Mattias Sjögren" <ma********************@mvps.org> wrote in message
news:%2***************@TK2MSFTNGP10.phx.gbl...

Jim,

whats wrong with the following code:

DateTime calcDate = new DateTime();

DateTime dDate = DateTime.Now;

if (calcDate.DayOfWeek(dDate) != "Saturday") || (calcDate.DayOfWeek(dDate)!= "Sunday")

{

}

DayOfWeek is a property, you're using it like a method. It returns a
value of the enum type DayOfWeek, not a string. Try it like this

if (dDate.DayOfWeek != DayOfWeek.Saturday ||
dDate.DayOfWeek != DayOfWeek.Sunday)

But then, that code is pretty much useless, since it will always
evaluate to true.

What did you actually want to accomplish?

Mattias

--
Mattias Sjögren [MVP] mattias @ mvps.org
http://www.msjogren.net/dotnet/
Please reply only to the newsgroup.

[JJ]

Gotcha thanks John !

JJ
"Jon Skeet" <sk***@pobox.com> wrote in message
news:MP************************@news.microsoft.com ...

JJ <jm***@bellatlantic.net> wrote:

Ah it was the brackets around the expressions that was causing the Invalid expression.

But at the same time Mattias you were saying that my logic with the if was going to fail anyway because of using Or correct?

Yes. Look at your code carefully, and think about what it means: you're
specifying that you want something done either if it's not Sunday, or
it's not Saturday. It'll *always* be either not Sunday or not Saturday,
because it can't be both Saturday and Sunday at the same time.

I should be using the and operator && instead in order for the logic to
work, correct?

Either that or rephrase it as (in pseudo-code)

if (! (day==Saturday || day==Sunday) )
{
}

--
Jon Skeet - <sk***@pobox.com>
http://www.pobox.com/~skeet/
If replying to the group, please do not mail me too