Запитання з тегом «code-reviews»

Я ніколи не знаходив ідеального способу перевірки коду, але часто мої клієнти вимагають їх. Здається, кожен клієнт робить їх по-різному, і я ніколи не відчував себе задоволеним ні в одному з них. Що було найефективнішим для вас для перевірки коду? Наприклад: Чи одна людина вважається воротарем якості та переглядає код, …

71 code-quality  code-reviews 

Нещодавно я почав працювати в місці з деякими значно старшими розробниками (близько 50+ років). Вони працювали над критичними програмами, що стосуються авіації, де система не могла знизитися. Як результат, старший програміст прагне кодувати таким чином. Він схильний вводити бульові елементи в об’єкти, щоб вказати, чи слід викинути виняток чи ні. …

64 c#  design  object-oriented-design  exceptions  code-reviews 

Я переглядав якийсь старий код, який я написав. Це працює, але це не великий код. Зараз я знаю більше, ніж тоді, тому могла вдосконалити. Це не поточний проект, але це поточний, робочий, виробничий код. Чи несемо ми відповідальність за повернення та вдосконалення коду, про який ми писали раніше, чи правильне …

64 code-quality  code-reviews 

Один з моїх членів команди, молодший програміст, володіє вражаючими навичками програмування для свого рівня досвіду. І під час перегляду коду я вірю в наголос на навчанні, не вказуючи на помилки. Але чи повинні брати участь молодші програмісти в оглядах коду для старших програмістів? Або в оглядах коду повинні брати участь …

55 teamwork  code-reviews  pair-programming 

Я великий віруючий у правило скаутського хлопчика : Завжди перевіряйте модуль чистіше, ніж коли ви його перевіряли. "Незалежно від того, хто був оригінальним автором, що робити, якщо ми завжди докладали певних зусиль, не важливих, як малих, щоб удосконалити модуль. Який був би результат? Я думаю, якщо ми всі дотримувались цього …

55 refactoring  code-reviews 

Я складаю деякі вказівки щодо огляду коду. У нас поки немає одного формального процесу і намагаємось його формалізувати. А наша команда географічно розподілена. Ми використовуємо TFS для управління джерелами (ми також використовували його для завдань / відстеження помилок / управління проектами, але ми перенесли це на JIRA ) з Visual …

55 code-reviews 

В кінці двотижневого спринту та завдання має перегляд коду, в огляді ми виявляємо функцію, яка працює, читабельна, але вона досить довга і має кілька запахів коду. Легка робота рефактора. Інакше завдання відповідає визначенню виконаного. У нас є два варіанти. Не вдалося перевірити код, щоб квиток не закрився в цьому спринті, …

53 agile  code-reviews 

Це запитання в ТА говорить про те, щоб виправити те, що, на думку ОП, є кодом заздрості . Ще один приклад, коли я бачив, як цитується ця витончена фраза, - це нещодавно відповідь тут у програмістах. Хоча я крапля в коментарі до цієї відповіді з проханням про інформації , яку …

53 refactoring  code-reviews  clean-code  code-smell 

Я досить довго працював над досить великим проектом з відкритим кодом, і це вже майже момент, коли я хотів би його випустити. Однак я самоучка, і я не знаю нікого, хто міг би адекватно переглянути мій проект. Кілька років тому я випустив невеликий шматочок коду, який дуже сильно зірвався (в …

51 open-source  self-improvement  code-reviews  personal-projects 

Чи важливо вказати на хороші частини коду під час перегляду коду та причини, чому він хороший? Позитивні відгуки можуть бути настільки ж корисними для розробника, який перевіряється, та для інших, які беруть участь у огляді. Ми робимо огляди за допомогою інструменту в Інтернеті, тому розробники можуть відкривати огляди на їх …

48 code-reviews 

Я є частиною команди консультантів, яка впроваджує нове рішення для замовника. Я відповідальний за більшість оглядів коду на кодовій базі клієнтів (React та javascript). Я помітив, що деякі члени команди використовують унікальні шаблони кодування для того, щоб я міг обрати файл випадковим чином, щоб сказати, хто був автором лише зі …

47 javascript  code-quality  code-reviews  reactjs 

За допомогою регулярних git та github я можу зробити перевірку коду, просто створивши запит на виклик гілки функцій, над якою працюю до гілки master. Як я можу робити огляди коду за допомогою git-flow? З робочим процесом, як "закінчення функції git flow", я плутаюся в тому, де насправді відбувається перегляд коду …

43 code-quality  git  code-reviews  github  gitflow 

У мене немає іншого вибору, окрім як працювати самостійно, і не можу знайти адекватне рішення для того, щоб переглядати свою роботу, перевіряти обгрунтованість, мати когось з ідеями мозкового штурму, обговорювати кращі практики тощо. Я думав, що отримаю відповідь через статтю Джеффа Етвуда: «Програмування« Один є найолітнішим номером » , найкраще, …

39 code-reviews  solo-development 

Я працював у двох компаніях, у кожної з яких була інша методологія, коли справа стосувалася оглядів коду: У першій компанії перевірка коду була проведена керівниками команд і була потрібна після завершення кожного модуля. Однак у другій компанії керівники команд не вимагали перевірки коду, а лише перевіряли на предмет функціональності та …

39 code-reviews  junior-programmer 

Нещодавно я почав працювати молодшим розробником. Окрім того, що я одна з найменш досвідчених людей в команді, я також жінка, яка зі всіма своїми проблемами працює в середовищі, де переважають чоловіки. У мене останнім часом виникають проблеми, тому що я відчуваю, що отримую занадто необгрунтовану педантичну критику щодо своєї роботи. …

39 code-quality  self-improvement  teamwork  code-reviews