Скоро я стану профессионалом?

Question

[JaPoznajuMir]

Я тут задавал вопрос, можно ли присылать код сюда, так как большинство сказало да, то присылаю. Моя задача стать лучше и получить комментарии от опытных разработчиков. Я буду стараться присылать только небольшие поделки(не больше 150 строк).

Вот моя первая поделка https://jsfiddle.net/drwsy648/ Если все таки лучше присылать ссылку на гитхаб, то я его заведу))

Сделаем мир чище! Избавим его от гавнокодера. Убивать меня не надо, просто помочь))

Comments

[Евгений Вольф]

Убивать меня не надо, просто помочь))

Это уж давайте мы сами решим! :))))

[Евгений Вольф]

А чего собственно сделать нужно? Написать комментарии по коду? Стилистическое оформление кода комментировать нужно? :)

[JaPoznajuMir]

Евгений Вольф, Правильный ли я подход выбрал, может нужно на классы переписать, может код не поддерживаемый или не понятный для коллег)) Неужели я хорошо написал его?))

[Алексей]

PhpStorm сказал замечательный код, всего 4 помарки нашел (ну типа в функции keyControl не хватает var перед переменной diraction_scroll), а так всё отлично.

[JaPoznajuMir]

Алексей, хм, просто я кое-кому это показал... И мне сказали, что тот кто увидит этот код застрелиться)) И что такое не должно существовать)) А это сказали опытные коллеги)) Но к сожалению у них нет времени дать полный комментарий, поэтому я пришел сюда.

[Алексей]

JaPoznajuMir, PhpStorm оценивает код с точки зрения синтаксиса, и не более того. Я просто скопировал ему и посмотрел, что он выделил.
А если как человек оценивать, то по мне так он вообще не работает. Попробовал изменить 2 слайда на 50, попробовал изменить ширину на 20 и высоту на 10, и никакого эффекта нет.

[Алексей]

в css тоже есть помарки

.container-carousel {
...
    width: 100%;
    width: 600px;
...
}

[Евгений Вольф]

JaPoznajuMir, я сразу уточню, я не эксперт по JS'у, по этому могу давать только общие рекомендации по коду. Из того, что бросилось мне в глаза сразу же:
1. input_number_slide - имена переменных в формате snake_case, не то, что бы так делать "нельзя", но лучше это у более опытных JS'ников, какие сейчас в тренде стандарты.
2. У Вас очень много вот таких сочетаний: ){ , пробела между ними нет, хотя как минимум в одном варианте, подобный пробел есть. От этого стилистика кода разная в разных строках, это не правильно.
3. Список переменных объявленный в самом верху - переменные, как я понял объявлены в хаотичном порядке и не сортируются ни по какому признаку... - такого быть не должно.

[JaPoznajuMir]

Алексей, Хм, странно, а у меня все работает по ссылке, когда меняю на 50, становится 50 слайдов их можно полистать либо по стрелкам на клавиатуре, либо нажав на стрелки на слайдере.

[tema_sun]

150 строк это дофигища.
Решай узкую задачу и показывай код, который относится ТОЛЬКО к этой задаче.

Пример выше - это простыня, которую хочется сразу закрыть.

[JaPoznajuMir]

tema_sun, А сколько максимум строк должно быть?

[Максим Корольский]

JaPoznajuMir
Идеально - одна
В худшем случае - две
Три уже совсем плохо

[fdroid]

Коллеги, что скажете по поводу моего кода? Можно ли как-то оптимизировать или так сойдёт?

10 INPUT a
20 INPUT b
30 LET c=a+b
40 PRINT c
50 GOTO 10

[fdroid]

Василий, как тогда лучше сделать? Хочу научиться программировать.

[fdroid]

Василий, этот код не работает, в нём минимум две ошибки.

[fdroid]

Василий, мой работает, задачу сложения двух чисел решает. Ваш - нет.
Проверял тут torinak.com/qaop