🏆 Рейтинг ІТ-работодателей 2019: уже собрано более 5000 анкет. Оцените свою компанию!
×Закрыть

Code review для початківців

Початківцям буде корисно, щоб їх код переглянули і спеціалісти і початківці, вказали на потенційні помилки, слабкі місця, а також початківцям і самим буде корисно побути у ролі code reviewer-а

Вже є окрема тема з такою пропозицією

Ця тема тільки спроба організувати товариство

Поки бачу цей формат так: у верхніх коментарях автор публікує посилання на код і короткий опис, а у відповідь отримує «переглянув і відписав в репозиторії»

LinkedIn
Допустимые теги: blockquote, a, pre, code, ul, ol, li, b, i, del.
Ctrl + Enter
Допустимые теги: blockquote, a, pre, code, ul, ol, li, b, i, del.
Ctrl + Enter

Доброго дня, написав типу плагін на 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 має добре розвивати вміння програмувати

Краще замість методу

searchElement

, використовувати document.querySelector там де це необхідно?

Там де використовував searchElement можеш сміло використовувати document.querySelector

Доброго дня. Шукаю допомоги.
github.com/nicko-woo/hmiDbWebApp
Суть проблеми: намагаюся реалізувати додаток на .NET Core, SignalR, який на веб-сторінці показує певні дані з бд, і при зміни даних у базі вони мають миттєво оновлюватися на сторінці. Проект компілюється, запускається, але жодних даних з бд на сторінці не відображує. Буду дуже вдячний, так як у .NET я початківець і знайомих, на жаль, немає)

Глянув поверхнево, спробуй написати тест на додавання в БД і читання

Дякую, вже розібрався, AJAX-запит був некоректним

codereview.stackexchange.com

Кто им виноват что гуглить не умеют

гуглить — это только для сеньоров

Что это будет полезно начинающим понятно, а вот специалисты после 2 ревью забьют. Без мотивации для ревьювящего обречено на неудачу. Особенно после комментариев от проверяемого, как в одной из тем, неточная цитата — «да тут форматирования на 2 минуты, могли бы себе отформатировать..»

Так, тому будуть правила після яких можна відправляти на перегляд

такой формат не взлетит, надо по минимум разрешить 5 обсыраний кода на 1го новичка в его ветке, иначе народ будет ленится ревьювить, так сказать не будет внутренней мотивации

За ці пару тижні вже 3+ теми з перегляду коду:

а лінуватись будуть за будь-який умов, навіть якщо додати фізичне покарання за помилки

так они иявляются примерами подхода о котором я писал, там еще и как проехались по тс

Вот и мотивация))

Подписаться на комментарии