Странные вещи в моем C# коде... Что происходит?

Question

[nkurakov]

Есть кусок кода, который ломает мой мозг:

int timeNow = DateTime.Now.Hour;
            //int timeNow = 6;
            string hi = "";
            if ((timeNow >= 6) & (timeNow <= 12))
                hi = "Доброе утро, ";
            else if ((timeNow >= 12) & (timeNow <= 18))
                hi = "Добрый день, ";
            else hi = "Добрый вечер, ";
            labelHello.Text = hi + getName() + "!";

Проблема в следующем - какое бы значение не передавалось от DateTime.Now.Hour в переменную timeNow, ни одно из условий не выполняется. Проверил все досконально через отладчик.

DateTime.Now.Hour = 6 (скрин ниже)


Дальше timeNow = 6


Казалось бы, тут проверка должна выполниться, если я все правильно понимаю, но


Нет. В итоге переменной hi присваивается значение "Добрый вечер". Я уже и вручную как видите ставил значение 6, и 7 пробовал, и 13, проверки так или иначе не выполняются.

В чем тут дело?

Comments

[GavriKos]

& замените на && и роверьте

[nkurakov]

GavriKos, первым делом об этом подумал) не работает

[Борис Животное]

nkurakov, у меня с 6 выводится "Доброе утро..."

[Борис Животное]

nkurakov, & имеет смысл использовать, когда независимо прошло условие слева или нет, но должен выполниться код справа. Когда-то я даже сходу мог привести пример. Сейчас с ходу не вспомню и не придумаю, когда эта действительно нужно. Используй по умолчанию всегда &&. Это просто совет, это не ответ на вопрос.

[Борис Животное]

nkurakov, с одним & такой код крашнется, но так пишут часто (да элвис оператор есть и т.д., это всё понятно).

int? number = null;
if (number != null && number == 3)
{
    Console.WriteLine("number is not null");
}
else
{
    Console.WriteLine("number is null");
}

[Борис Животное]

Возможно, ты запускаешь не пересобирая проект. Выполняется старый код, где всегда "Добрый вечер". Помнится что-то такое мне очень давно встречалось. Может в студии какая-то настройка... Не помню...

[GavriKos]

nkurakov,

int timeNow = 6;
            string hi = "";
            if ((timeNow >= 6) & (timeNow <= 12))
                hi = "1, ";
            else if ((timeNow >= 12) & (timeNow <= 18))
                hi = "2, ";
            else hi = "3, ";
        Console.WriteLine ( hi + "!");

Выдало 1. Код выглядит верным

Answer 1

[Роман]

&& надо, & это булева операция, а не сравнение

Comments

[Rsa97]

Только если оба операнда имеют целочисленный тип.
Иначе и то и другое - логические И. Разница в том, что & всегда вычисляет оба операнда выражения, а && сначала левый и только если он true, то вычисляется правый операнд.
https://docs.microsoft.com/en-us/dotnet/csharp/lan...

Answer 2

[Sergey В.]

Все работает как надо: https://dotnetfiddle.net/qyC3vO
При timeNow = 6 пишет "Доброе утро"
Обратите внимание, что при timeNow = 12 у Вас предусмотрена обработка и в первом и во втором условии.

Answer 3

[Василий Банников]

Вроде всё работает

static string Greet(int hour)
{
    int timeNow = hour;
    string hi = "";
    if ((timeNow >= 6) & (timeNow <= 12))
        hi = "Доброе утро, ";
    else if ((timeNow >= 12) & (timeNow <= 18))
        hi = "Добрый день, ";
    else hi = "Добрый вечер, ";
    return hi;
}

Console.WriteLine(Greet(5)); // Добрый вечер
Console.WriteLine(Greet(6)); // Доброе утро
Console.WriteLine(Greet(7)); // Доброе утро
Console.WriteLine(Greet(11)); // Доброе утро
Console.WriteLine(Greet(12)); // Доброе утро
Console.WriteLine(Greet(13)); // Добрый день
Console.WriteLine(Greet(17)); // Добрый день
Console.WriteLine(Greet(18)); // Добрый день
Console.WriteLine(Greet(19)); // Добрый вечер

PS: в твоём случае вместо & лучше использовать &&, но в целом на работу это не влияет.