Homework№5.OR-Media.Pre-front #11
Conversation
| var Name = prompt('Введите ваше имя'); | ||
| var reg = /\d/g; | ||
| if (Name.match(reg) === null) { | ||
| function reverse(Name) { |
There was a problem hiding this comment.
нет смысла написание функции в if, она все равно поднимается вверх, и написание такого функций в if плохая практика
| @@ -0,0 +1,14 @@ | |||
| var Name = prompt('Введите ваше имя'); | |||
| var reg = /\d/g; | |||
| if (Name.match(reg) === null) { | |||
There was a problem hiding this comment.
проверять на null не нужно это и так false будет значение. можно переписать как !Name.match(reg)
| <button class="ButtonLink" type="submit">LINK</button> | ||
| <button class="ButtonStyle" type="submit">STYLE</button> | ||
| <button class="ButtonMaket" type="submit">MAKET</button> | ||
| <script src="./js/index.js"></script> |
There was a problem hiding this comment.
подключение скриптов делается в конце body
| @@ -0,0 +1,36 @@ | |||
| document.getElementsByClassName("ButtonLink")[0].onclick = function () { window.open("https://vk.com/white_power69", "_blank") } | |||
There was a problem hiding this comment.
лучше использовать addEventListener
| @@ -0,0 +1,36 @@ | |||
| document.getElementsByClassName("ButtonLink")[0].onclick = function () { window.open("https://vk.com/white_power69", "_blank") } | |||
There was a problem hiding this comment.
такого рода строки (https://vk.com/white_power69) лучше выносить в константные переменные, а еще лучше создать файл на подобии config.js или config.json и туда перенести такие переменные в объект, так как такая строка может много где использоваться в проекте, и что бы везде ее не менять при необходимости была возможность сделать это в одном месте
| Cont.remove(); | ||
| var el = document.querySelector("Link"); | ||
| el.href = "./css/ThirdStyle.css"; | ||
| var Body = document.body; |
There was a problem hiding this comment.
повторяющийся код можно вынести в функцию. Отличаться будут только параметры.
No description provided.