form e` una parola usata dal DOM, e non e` bene usarla come nome di un oggetto interno.

Poi secondo logica (umana) il test dovrebbe essere:
if(form1.denominazione.value.length > 50)

Altrimenti il confronto controlla se il campo inizia per numero e fa il confronto tra tale numero (che al 99% sara` zero) e il 50.