У тебя конструкция такая:
if(day <= 31) {
if(day >= 1 || day == 31) {
//Январь, март, май, июль, авгус, октябрь, декабрь
} else if(day >= 1 || day == 30) {
// Апрель, июнь, сентябрь, ноябрь
} else if(. . . ) {
// февраль
} else {
//Неверно
}
Если мы вводим правильный день месяца, то мы всегда заходим в первый же if и игнорируем все последующие if-else, поскольку в первый уже зашли. Например, не только 31 января со ствует условию if(day >= 1 || day == 31), но и 2 апреля, и даже 29 февраля. Только все, кто не попал в switch-case внутри первого if, будут выкидывать "неверный день"
Если не слишком переписывать код, то надо заменить конструкции else if на простые if, и убрать внутри default. Но как только мы нашли совпадение, т. е. зашли в одну из веток switch-case и вывели в консоль дату, надо из функции или программы делать return.
А вообще раазумнее и короче было бы сначала проверять введенный месяц, а затем уже проверять, со ствует ли день введенному месяцу. Извини за псевдокод, но так должно быть понятнее:
if год > 0 {
if месяц in (январь, март, май, июль, авгус, октябрь, декабрь) {
if день in 1. . 31 { // выводим дату } else { // неверный день }
}
else if месяц in (апрель, июнь, сентябрь, ноябрь) {
if день in 1. . 30 { // выводим дату } else { // неверный день }
}
else if месяц == февраль {
if проверка_на_обычный_год OR проверка_на_високосный год { // выводим дату } else { // неверный день }
}
else { // неверный месяц }
}
else { // неверный год }