Как в моем случае облагородить код, сделать красивей, не руша логику?

Question

[frees2]

Тестирую историю, вроде все работает, демо для тестирования на
www.dulsky.eu/v/6sPiPA4ogwA
История обновляется внизу, смотришь и переключаешь историю, хоть браузером.

Но будучи в js быдлокодером, хотелось спросить, как облагородить код, сделать красивей, не руша логику? (JSON.stringif позволяет загружать с любого места и все что угодно, но тут всего лишь строка.

history.pushState(\''.$videoId.'\', null, \''.$videoId.'\') ;

где videoId может быть лентой JSON )

var i = 0;
(function(history)
{ var pushState = history.pushState;
history.pushState = function(state)
 { if 
(typeof history.onpushstate == "function")

{history.onpushstate({state: state});} 

return pushState.apply(history, arguments); }})
(window.history);
window.onpopstate = history.onpushstate = function(e)
{ 
i++; test=JSON.stringify(e.state);
var reg_pusto = test;
if (reg_pusto.length>10)
{ test =test.replace(/"/g,"");
loc =window.location.pathname;
loc = loc.substr(3,52);
loc =loc.replace(/=list=/g,"&list=");
locris = loc.substr(0,11);
 if (locris.length<11) {locris2=''} 
else
{ locris2= '<span onclick="ytplayer.loadVideoById(\''+loc+ '\', 0);return false"><img src="https://i1.ytimg.com/vi/'+ locris + '/default.jpg" align="left" width="120" height="70"></span>' };
if (test==loc) { test2= ' ' ;}
 else
{ test2='<span onclick="ytplayer.loadVideoById(\''+ test + '\', 0);return false"><img src="https://i1.ytimg.com/vi/'+ test + '/default.jpg" align="right" width="120" height="70"></span>' ; };
$('#ms').html(''+ test2 + ''+i+''+ locris2 + ''); }
else{  };};

Answer 1

[Павел Осадчук]

Я не знаю js но общие принципы рефакторинга подскажу.
1) покройте изменяемый код тестами
2) устраняйте дублирование, по одному за раз
3) устранили, запускайте тесты
4) продолжать до полного удовлетворения кодом

Comments

[Arris]

1) покройте изменяемый код тестами

а вот с этого места можно попродробнее? и без отсылок к гуглу?

Answer 2

[Назар Мокринский]

Отформатируйте для начала, а то кастрюля с макаронами получается)

Answer 3

[aplic]

Вот я смотрю на это
''+ test2 + ''+i+''+ locris2 + ''
Этим можно детей пугать.

Comments

[frees2]

Есть замечательная глобальная компания на букву Г, там требования - оставлять поля 'павапыапар' для заполнения дизайнерами, им удобно, они видят места, js писать в строчку без пробелов слева, выделять логические элементы пробелами, код обязательно писать вместе с css на одной странице. Клянусь, это не треш.

Answer 4

[Andrew Dabich]

Нужно научиться писать код по стилю, по его структуре для поддержки и понимания самого кода. А не мучить глаза в каше символов.

ru.wikipedia.org/wiki/%D0%A1%D1%82%D0%B0%D0%BD%D0%...

Comments

[frees2]

для меня восприятие кода не зависит от стиля, , разметка реально мешает, логически она не соответствует восприятию.

[Andrew Dabich]

Благородный код этот код который будет понятен всем. И будет каждому говорить о том что он делает. И восприятие его облегчается. Мне например часто приходиться дорабатывать чей-то код. И меня радует то что я его по ходу понимаю, а если попадается код - "мусор", скажем так, это усложняет работу + время убивается много. Нужно тренировать в себе стиль, без него код не будет благородным. Особенно это нужно в командной работе. И вообще хорошие манеры программиста, это красиво оформленный код.