С точки зрения безопасности, ваш код это пример того, как делать точно не стоит, я понимаю что ваша задача была показать защиту от спама, и показать как работает ajax, но видео по объяснению ajax у других не занимает более 5 минут. В вашем случае 25, а вся ваша защита сводится к подмене неймов в инпутах. Так же хотел отметить, что идея с подменой стара как мир, и реализуется более красиво, вам на заметку: Генерировать неймы не рандомно хеш функциями, а Шифрованием от от самого нейма, что позволит пройтись циклом по всем индексам массива полей (полей ведь может быть и более 10, а в разных формах на разных страницах ещё и разные неймы), так же как минимум вам стоит обработку ошибок обернуть отдельно в функцию, так как у вас сплошной копипаст кода. Не подумайте, что моя цель все обосрать, вовсе нет, опять же повторюсь, я понимаю что видео ориентировано на новичков, но в этом и есть проблемла, просто ваш код должен быть эталонным как сточки зрения безопасности, так и с точки зрения культуры написания, как минимум исключить повторяющийся код. объясню почему это важно: скорее всего новички просто посмотрят и скопипастят ваш код и возьмут его на вооружение, и очень вероятно отправят такой код в ПРОД какому нибудь клиенту. Далее думаю последствия объяснять не нужно... В целом видео у вас очень хорошие и приятный голос, но очень прошу, если учите людей и помогаете им войти в профессию, то ведите их по правильным путям, прививайте им культуру написания кода. В конце концов не прививайти им плохие привычки в кодинге. P.S. Другие варианты по защите от спама : Токинезация форм, позволяет эффективно бороться с спамом На стороне сервера. Google recaptcha v3, невидимая капча без необходимости ввода чего либо и поиска светофоров на картинках
Здравствуйте, видео класс! Спасибо. Хорошо бы с видео прикреплять исходники уроков.
Спасибо. Буду знать)
@@feodoraxis можно получить css и lib,.js?
@@ZolotarevPavel я уже всё удалил) Но libs.min.js точно не нужен. Там только jquery и jquery.maskedinput.
С точки зрения безопасности, ваш код это пример того, как делать точно не стоит, я понимаю что ваша задача была показать защиту от спама, и показать как работает ajax, но видео по объяснению ajax у других не занимает более 5 минут. В вашем случае 25, а вся ваша защита сводится к подмене неймов в инпутах.
Так же хотел отметить, что идея с подменой стара как мир, и реализуется более красиво, вам на заметку:
Генерировать неймы не рандомно хеш функциями, а Шифрованием от от самого нейма, что позволит пройтись циклом по всем индексам массива полей (полей ведь может быть и более 10, а в разных формах на разных страницах ещё и разные неймы), так же как минимум вам стоит обработку ошибок обернуть отдельно в функцию, так как у вас сплошной копипаст кода.
Не подумайте, что моя цель все обосрать, вовсе нет, опять же повторюсь, я понимаю что видео ориентировано на новичков, но в этом и есть проблемла, просто ваш код должен быть эталонным как сточки зрения безопасности, так и с точки зрения культуры написания, как минимум исключить повторяющийся код.
объясню почему это важно: скорее всего новички просто посмотрят и скопипастят ваш код и возьмут его на вооружение, и очень вероятно отправят такой код в ПРОД какому нибудь клиенту. Далее думаю последствия объяснять не нужно...
В целом видео у вас очень хорошие и приятный голос, но очень прошу, если учите людей и помогаете им войти в профессию, то ведите их по правильным путям, прививайте им культуру написания кода. В конце концов не прививайти им плохие привычки в кодинге.
P.S.
Другие варианты по защите от спама :
Токинезация форм, позволяет эффективно бороться с спамом
На стороне сервера.
Google recaptcha v3, невидимая капча без необходимости ввода чего либо и поиска светофоров на картинках
Спасибо большое за развёрнутый комментарий и критику. Буду иметь ввиду. В будущем, возможно, перезапишу видео по-новой и учту ваши советы)
А ты молодец. Раскритиковал по-уму. Теперь ты должен куда-то выложить свое решение для чайников, как я.
Привет ты клас👍🤗