Как довести код JS до нормального состояния?

Question

[romanu4]

Помогите довести код до конца, ошибки, замечания, буду очень благодарен

function getart() { 
   
  var platform = "http://first/painters/" ;
  var currentSet = [1,2,3,4,5,6,7,8,9];
  var art = document.getElementById("art");
  var randomdir = currentSet[Math.floor((Math.random()*currentSet.length))];
  $.getJSON("painters/" + randomdir + "/data.json", function(json) {
  var paintings = json.paintings;
  var image = Math.floor((Math.random()*paintings)+1);
      art.src = platform + randomdir + "/" + image + ".jpg";
      window.truePainterName = i18n.t("painters." + randomdir, { lng: window.lang });
      window.link = json.link.local;
  });
};

Comments

[vasIvas]

А может стоит конвенции погуглить?

Answer 1

[IceJOKER]

getArt(){} вместо getart(){}

var random = Math.random(); //потом используем его в обеих случаях, ибо нех дважды получать случайное число )

ну и все Оо, не знаю , что вы подразумеваете под "довести код до конца|до нормального состояния"

Comments

[Ranwise]

var random = Math.floor(Math.random()); вроде если вынести, то random всегда будет равен 0

[IceJOKER]

Ranwise: согласен, исправил.

Answer 2

[Дмитрий]

Помимо выше сказанного, я бы не плодил так много слов var, а перечислял бы объявления переменных через запятую.

Comments

[Sergej]

Это не всегда удобно =)
Но совет верный.

[Константин Китманов]

Есть разные code style. Я раньше ратовал за перечисление через запятую, сейчас перешел на отдельный var для каждой переменной -- так их проще добавлять и убирать.

[Владимир Родкин]

Константин Китманов: Да, так проще поддерживать старый код. Раньше тоже через запятую перечислял. Но при правках это добавляет кучу проблем =)

[Константин Китманов]

Владимир Родкин: ага. И диффы аккуратнее выглядят.

[Дмитрий]

Константин Китманов: учитывая то, что по возможности все переменные нужно объявлять в начале (ф-ции, файла, метода), их проще перечислять через запятую.

[Константин Китманов]

Дмитрий Пищалка: не обязательно. Когда в начале функции полтора десятка переменных, из которых половина -- однобуквенные счетчики из for-циклов, то создается очень неприятное впечатление.

[Дмитрий]

Константин Китманов: это просто рекомендация, правило хорошего тона (не нашел доклад к сожалению), Дело в том, что компилятор пробежавшись по коду, собирает все переменные и объявляет их в начале области видимости (ф-ции или файла вовсе), и чтобы изначально у скрипта не было переменных undefined, рекомендуется их объявлять в самом начале кода, естественно счетчиков это может не касаться (во всяком случае я не понимаю смысла объявлять i в самом начале). Вот еще хорошая статья по поводу оформления вашего js.

Answer 3

[shqn]

Ну и в глобальную область видимости (window.truePainterName, window.link) по возможности как можно меньше залезать.