Das ist nach veralteten Standards geschrieben,
von denen abzuraten ist...
Das all-Objekt ist zwar noch unterstützt, gilt aber als antiquiert.
Statt "document.all.2.src" ist "document.getElementById('2').src"
zu verwenden, dann solltest du zumindest erstmal einen großen Fehler weniger haben...