[C#] tussen 2 uren in.

[Verwijderd]

Ik wil een melding generen als het huidige uur tussen 2 voorgedefinieerde uren ligt.
Bijvoorbeeld tussen 10:00 en 11:00 of tussen 23:00 en 4:00

Ik heb diverse dingen geprobeerd, maar krijg het niet generiek genoeg om het in alle gevallen voor elkaar te krijgen.

Het volgende heb ik geprobeerd:

code:

 System.Threading.Thread.CurrentThread.CurrentCulture = new System.Globalization.CultureInfo("nl-NL");
        DateTime Current = DateTime.Now;
        DateTime MinBound = new DateTime(DateTime.Now.Year, DateTime.Now.Month, DateTime.Now.Day, 23, 0, 0, DateTimeKind.Local);
        DateTime MaxBound = new DateTime(DateTime.Now.Year, DateTime.Now.Month, DateTime.Now.Day, 7, 0, 0, DateTimeKind.Local);
        
        if (Current.Hour == 23)
        {
            Current = Current.AddHours(1);
        }
        if ((MinBound.Hour >= Current.Hour) && (Current.Hour <= MaxBound.Hour))
        {
            Label1.Text += "<br> lig te slapen";
        }

Daarnaast heb ik ook de timespan geprobeerd i.p.v. DateTime en daarbij loop ik tegen dezelfde problemen aan.

[Hydra]

Err, gewoon:

DateTime lower = bla
DateTime higher = bla2

if(lower < now && higher > now)
{
TADAAA!
}

.Net kent operator overloading dus je kunt gewoon datetimes met mekaar vergelijken.

[Mawlana]

C#:

if(Current >= MinBound && Current <= MaxBound)
{
    Label1.Text += "<br> lig te slapen";
}

Of denk ik nu te simpel?

[Verwijderd]

C#:

if(MinBound < MaxBound)
{
    if(Current >= MinBound && Current <= MaxBound)
    {
        Label1.Text += "<br> lig te slapen";
    }
}
else
{
    if(Current >= MinBound || Current <= MaxBound)
    {
        Label1.Text += "<br> lig te slapen";
    }
}

[Verwijderd]

Hydra schreef op vrijdag 28 november 2008 @ 13:53:
Err, gewoon:

DateTime lower = bla
DateTime higher = bla2

if(lower < now && higher > now)
{
TADAAA!
}

.Net kent operator overloading dus je kunt gewoon datetimes met mekaar vergelijken.

Zo'n constructie heb ik ook geprobeerd, maar ik wil het dag onafhankelijk hebben.
In jou voorbeeld gaat het mijn inziens fout:
DateTime lower = 28-11-2008 23:00
DateTime higher = 28-11-2008 4:00
Hierdoor zal hij niet in de if komen.

[Niemand_Anders]

Ik zou het als volgt doen:

C#:

TimeSpan window = MaxBound.Substract(MinBound); 
TimeSpan offset = MaxBound.Substract(Current);

if (offset <= window)
 //valt in time window
else
 //valt er buiten

De code bepaald of het verschil met Current minder of gelijk is dan het verschil met de minimale datum. Deze code werkt dus ook over meerdere dagen.

Overigens neem ik aan dat MaxBound day + 1 moet zijn omdat je anders nooit er tussen min en max kunt zitten omdat MaxBound (7 uur 's ochtends) eerder valt dan MinBound (11 uur 's avonds, maar op dezelfde dag)

[yade]

Verwijderd schreef op vrijdag 28 november 2008 @ 14:03:

C#:

if(MinBound < MaxBound)
{
    if(Current >= MinBound && Current <= MaxBound)
    {
        Label1.Text += "<br> lig te slapen";
    }
}
else
{
    if(Current >= MinBound || Current <= MaxBound)
    {
        Label1.Text += "<br> lig te slapen";
    }
}

Dit is niet mooi hè?

[Verwijderd]

Verwijderd schreef op vrijdag 28 november 2008 @ 14:03:

C#:

if(MinBound < MaxBound)
{
    if(Current >= MinBound && Current <= MaxBound)
    {
        Label1.Text += "<br> lig te slapen";
    }
}
else
{
    if(Current >= MinBound || Current <= MaxBound)
    {
        Label1.Text += "<br> lig te slapen";
    }
}

Ik denk met een aanvulling van .Hour het moet gaan lukken maar mooi is het niet...

code:

if (MinBound.Hour < MaxBound.Hour)
        {
            if (Current.Hour >= MinBound.Hour && Current.Hour <= MaxBound.Hour)
            {
                Label1.Text += "<br> lig te slapen";
            }
        }
        else
        {
            if (Current.Hour >= MinBound.Hour || Current.Hour <= MaxBound.Hour)
            {
                Label1.Text += "<br> lig te slapen";
            }
        }

[Verwijderd]

Niemand_Anders schreef op vrijdag 28 november 2008 @ 14:14:
Ik zou het als volgt doen:

C#:

TimeSpan window = MaxBound.Substract(MinBound); 
TimeSpan offset = MaxBound.Substract(Current);

if (offset <= window)
 //valt in time window
else
 //valt er buiten

De code bepaald of het verschil met Current minder of gelijk is dan het verschil met de minimale datum. Deze code werkt dus ook over meerdere dagen.

Overigens neem ik aan dat MaxBound day + 1 moet zijn omdat je anders nooit er tussen min en max kunt zitten omdat MaxBound (7 uur 's ochtends) eerder valt dan MinBound (11 uur 's avonds, maar op dezelfde dag)

In dit geval zou je MaxBound +1 doen, maar als je de tijd nu wilt wijzigen naar minbound = 10:00 en maxbound = 15:00 dan werkt het weer niet goed

[Verwijderd]

C#:

if((MinBound < MaxBound && Current >= MinBound && Current <= MaxBound) ||
((MinBound > MaxBound) && (Current >= MinBound || Current <= MaxBound))
    {
        Label1.Text += "<br> lig te slapen";
    }

Zo beter? Ik vind de condities dan niet zo erg leesbaar.

[Verwijderd]

yade schreef op vrijdag 28 november 2008 @ 14:17:
[...]

Dit is niet mooi hè?

Nope, niet de meest mooie oplossing, maar het is wel de enige die tot nu toe werkt.

[Verwijderd]

En met onderstaande vang ik natuurlijk af dat ik die dag iets langer op bed lig

code:

 if (Current.DayOfWeek == DayOfWeek.Saturday)
{
}

[Rainmaker1987]

Mij lijkt de volgende code behoorlijk generiek:

C#:

DateTime Current = DateTime.Now;
DateTime MinBound = new DateTime(DateTime.Now.Year, DateTime.Now.Month, DateTime.Now.Day, 20, 0, 0, DateTimeKind.Local);
DateTime MaxBound = new DateTime(DateTime.Now.Year, DateTime.Now.Month, DateTime.Now.Day, 7, 0, 0, DateTimeKind.Local);
if (MinBound.Hour > MaxBound.Hour)
     MaxBound = MaxBound.AddDays(1);
if ((MinBound <= Current) && (Current <= MaxBound))
     Console.WriteLine("Lekker slapen");

Het checkt eerst even of MinBound en MaxBound op dezelfde dag liggen (dat is immers zo wanneer MaxBound.hour > MinBound.hour wanneer je er vanuit gaat dat de totale periode nooit meer dan 24 uur mag zijn) Als dit niet zo is gewoon MaxBound een extra dagje geven en de vergelijk daarna is vrij eenvoudig