Можно ли улучшить код?

Question

[adrenalinruslan]

Сделал меню sidenav и чтобы это меню закрывалось если кликнули не по нему в jquery. Нормальный ли такой код ? Какие дыры в нем могут быть ? И можно ли как то уменьшить , оптимизировать ?)

При клике на кнопку ( чтобы показать меню ):

$('.sidenav-btn').click(function() {
		var menu = '#' + $(this).attr('data-target');
		$('.sidenav-box, ' + menu).removeClass('active').addClass('active');
		$(menu + ' > .sidenav-menu').animate({left: "0"}, 300);
	});

Проверка при клике на document:

$(document).click(function(event) {
		var attr = $(event.target).attr('class');
		if(attr == 'sidenav-menu') {
			return false;
		} else if(attr == 'sidenav-box active') {
			$('.sidenav-menu').animate({left: "-100%"}, 300, function() {
				$('.sidenav-box').removeClass('active');
			});
		}
	});

Answer 1

[tarthur]

Забудьте этот код и начните решать другие задачи, чем больше тем лучше,. Оптимизируйте как знаете на данный момент и не более.
Продолжайте искать идеальное решение и вы будете постепенно замедлятся в развитии, давать себе задачи которые вам не давали, а потом не делать их потому, что, все учесть и перепроверить не так то просто.
Перфекционизм порождает лень, лень будет уничтожать мотивацию.

Comments

[Jumandjilos]

а ведь всё по факту! я вот тоже еще учусь, но проблема с перфекционизмом очень часто меня преследует, и это правда сильно тормозит развитие, тупо топчешься на одном проекте, и всё думаешь, как бы его довести до идеала, не решая при этом другие задачи, а потом и вообще напрочь отбивается желание учить что-то новое, так что да, забей, работай над другими задачами

[Ирина]

Jumandjilos, +++
иногда надо, чтобы кто-то постоянно напоминал, что 20% усилий дают 80% результата.

а это "улучшение кода" может преследовать годами, постоянно крутиться в голове.
выхода 2:
- все таки принять, что работает без ошибок - уже хорошо
- выделять время отдельное на рефакторинг кода. И в это время только этим и заниматься. и только это установленное время