non credo sia utile utilizzare un ciclo... se per esempio i tag img che ti interessano hanno tutti lo stessa proprietà class puoi applicare l'evento onClick al nome del class invece di impostarlo all'id, e puoi fare a meno del ciclo...