Если вы действительно хотели использовать &&, а не ||, ваша форма выбора должна быть:
<select name = "when">
<option value="" selected="selected"> </option>
<option value="breakfast">Breakfast</option>
<option value="lunch">Lunch</option>
<option value="dinner">Dinner</option>
<option value="snacks">Snacks</option>
</select>
Вы не передаете пустое значение выбранного по умолчанию элемента, и значение && if
не будет оцениваться как true.
Редактировать: если подумать, ваша форма должна быть такой, иначе пустое значение по умолчанию не будет перехвачено. Wheter вы решили использовать && или ||