Nazwy id/klas zaczynających się od cyfr nie jest dobrą praktyką.
W twoim przypadku po odwołaniu się do do pól nie sprawdzasz value, a dodatkowo nigdy one nie zwrócą wartości null (no chyba, że element nie istnieje).
document.getElementById('1').value==""
Tak możesz sprawdzić zawartość czy nie jest pusta. Poza tym druga część w warunku if nie ma najmniejszego sensu.