Год назад я перешел с классических ОО-языков, таких как Java, на JavaScript. Следующий код определенно не рекомендуется (или даже не корректен) в Java:

if(dayNumber = getClickedDayNumber(dayInfo))
{
    alert("day number found : " + dayNumber);
}
function getClickedDayNumber(dayInfo)
{
    dayNumber = dayInfo.indexOf("fc-day");
    if(dayNumber != -1) //substring found
    {
        //normally any calendar month consists of "40" days, so this will definitely pick up its day number.
        return parseInt(dayInfo.substring(dayNumber+6, dayNumber+8));
    }
    else return false;
}

В основном я только что узнал, что я могу присвоить переменную значению в операторе условия if, и сразу же проверить присвоенное значение, как будто оно логическое.

Для более безопасной ставки я обычно делю это на две строки кода, сначала назначаю, а затем проверяю переменную, но теперь, когда я нашел это, мне просто интересно, является ли это хорошей практикой или нет в глазах опытных разработчиков JavaScript?