?

Log in

No account? Create an account

Previous Entry Share Next Entry
Конструктив и деструктив
gelena_s
Итак, если все-таки речь зашла о конструктивной критике, то вначале говорим о том, что понравилось, что зацепило в хорошем смысле. Затем говорим о том, что можно было бы добавить, что еще можно улучшить.

И вопрос: каково должно быть соотношение того, что понравилось и того, что, подчеркну, именно на ваш взгляд, можно улучшить или добавить?

Как и какими словами нужно говорить об улучшениях, чтобы человек не уходил в защиту? Как подчеркнуть то, что это именно ваше мнение?


Подписаться

Recent Posts from This Journal

  • Про шаги

    Если кто не заметил, так в том самом комментарии я посоветовала использовать шаги из АА, как способ выхода из ситуации. Лично я считаю, что это…

  • Немного конкретики

    Расширим предыдущий вопрос, добавим немного деталей. Имеем: женщине за пятьдесят, в разводе лет десять, муж побежал за новой молодостью и вторым…

  • Про сервис

    Я тогда с клиенткой посмеялась больше над маминым контролем в виде снятой «вредной» шкурки. Есть женщины и мужчины, которые умудряются…



  • 1
Напомнило. У нас принята практика код-ревью (то есть, одни программисты пишут код, другие его потом просматривают на предмет несовершенств). И вот есть одна довольно юная девушка, про которую, когда она делает ревью, говорят:
- Ну да, она потом приходит: ребята, мур-мур, вы тут все хорошо написали, все конечно работает... НО Я БЫ ПЕРЕПИСАЛА

(ну и дальше понимают, что да, действительно придется переписывать).

Мне кажется, это необидное выражение.

Конечно нет.
Но главное тут, что это, в данном случае, обязанность - критиковать чужой код и найти в нем как можно больше косяков.

Задача код-ревью - не критиковать чужой код и искать в нем как можно больше косяков. Докопаться можно и до столба, тем более, что качество кода - вещь субъективная. Любой программист переписал бы любой чужой код - просто потому, что он сам код пишет иначе, вот и все. Потому что все люди - разные.

Задача ревью - не пропустить очевидный чужим глазам косяк, и только лишь. Причем косяк - это не когда "плохо", это когда "а вот в такой ситуации, про которую ты не подумал, твой код отработает некорректно". Там, где этого не обьясняют, а ревью делают - просто генерируют сотрудникам понижение мотивации на пустом месте и ни для чего.

Edited at 2017-07-08 02:43 pm (UTC)

Я упростила смысл для не-программистов.

  • 1