Как довести код JS до нормального состояния?
getArt(){} вместо getart(){}
var random = Math.random(); //потом используем его в обеих случаях, ибо нех дважды получать случайное число )
ну и все Оо, не знаю , что вы подразумеваете под "довести код до конца|до нормального состояния"
Ну и в глобальную область видимости (window.truePainterName, window.link) по возможности как можно меньше залезать.