Code review для початківців
Початківцям буде корисно, щоб їх код переглянули і спеціалісти і початківці, вказали на потенційні помилки, слабкі місця, а також початківцям і самим буде корисно побути у ролі code reviewer-а
Вже є окрема тема з такою пропозицією
Ця тема тільки спроба організувати товариство
Поки бачу цей формат так: у верхніх коментарях автор публікує посилання на код і короткий опис, а у відповідь отримує «переглянув і відписав в репозиторії»
15 коментарів
Додати коментар Підписатись на коментаріВідписатись від коментарівДоброго дня, написав типу плагін на JavaScript, хотів би почути думку більш досвітчений розробників на рахунок самого коду.
github.com/Sasha-Zayets/modals-js
Дуже всім вдячний)
Подивився demo (там треба виправити помилку, 1 хвилина)
У методі searchElement використовується document.querySelector, який і так повертає HTMLElement або null, тому обгортку searchElement можна видалити
searchElement (selector) { const element = document.querySelector(selector); if (!!element) { return element; } else { return null; } }Код гарно форматований і зрозумілі назви змінних
А ось гра screeps.com має добре розвивати вміння програмувати
Краще замість методу
, використовувати document.querySelector там де це необхідно?
Там де використовував searchElement можеш сміло використовувати document.querySelector
Дуже дякую
Доброго дня. Шукаю допомоги.
github.com/nicko-woo/hmiDbWebApp
Суть проблеми: намагаюся реалізувати додаток на .NET Core, SignalR, який на веб-сторінці показує певні дані з бд, і при зміни даних у базі вони мають миттєво оновлюватися на сторінці. Проект компілюється, запускається, але жодних даних з бд на сторінці не відображує. Буду дуже вдячний, так як у .NET я початківець і знайомих, на жаль, немає)
Глянув поверхнево, спробуй написати тест на додавання в БД і читання
Коментар порушує правила спільноти і видалений модераторами.
Дякую, вже розібрався, AJAX-запит був некоректним
codereview.stackexchange.com
Кто им виноват что гуглить не умеют
гуглить — это только для сеньоров
Что это будет полезно начинающим понятно, а вот специалисты после 2 ревью забьют. Без мотивации для ревьювящего обречено на неудачу. Особенно после комментариев от проверяемого, как в одной из тем, неточная цитата — «да тут форматирования на 2 минуты, могли бы себе отформатировать..»
Так, тому будуть правила після яких можна відправляти на перегляд
За ці пару тижні вже 3+ теми з перегляду коду:
а лінуватись будуть за будь-який умов, навіть якщо додати фізичне покарання за помилки
-