Go yourself on or the rules of communication in the team

 3r3176. 3r3-31.
Post-response to article https://habr.com/post/432700/3r315. 3r3164. 3r3161.  3r3176.
If I followed the advice from this article, it would be enough for me to show emotion and tell the author "Go yourself on you don't understand anything!". :)
3r3161.  3r3176.
However, this would not help to convey my point. So let's take a closer look. 3r3164. 3r314. 3r3161.  3r3176. 3r318. Quotation 1:
3r3161.  3r3176. 3r3105. If a person is incompetent, it is necessary to give him a clear understanding of this, and not to take care of his tender feelings to the detriment of everything else.
I do not agree with the basis of this statement. I believe that a person can not be competent or incompetent. Such a generalized black and white approach does not work in practice. Even the most pumped senior may not know any things. Conversely, juniors sometimes have great ideas. 3r3164. 3r3161.  3r3176.
Getting personal ("you are not competent!") To the review code instead of specific arguments is too easy. If you are so smart senior, work hard, explain why it is in this place of the code that everything should be different. You can not explain - better do not write anything, because it is possible and you do not fully understand. 3r3164. 3r3161.  3r3176.
At the same time, it is certainly necessary to talk about specific problems in the code. 3r3164. 3r3161.  3r3176.
A normal person is happy to discuss a reasoned position. And take hostile negative emotions. Who would ever want to work with a toxic team member? 3r3164. 3r3161.  3r3176. 3r340. Quote 2: 3r33112. 3r3161.  3r3176. 3r3105. A person can send you a review code with the same mistakes again and again and you need to respond to this with politeness and a smile?
If a person makes mistakes time after time and does not try to grow somehow, then he should be fired. Talk with team lead on this. But hysteria is not necessary anyway. Well, simply because it does not help. 3r3164. 3r3161.  3r3176.
Negative emotions can generate only negative emotions. And errors in the code will not fix it. 3r3164. 3r3161.  3r3176.
3: 3r3112. 3r3161.  3r3176. 3r3105. The greater the responsibility in the profession, the greater should be the resistance to stress.
I worked with the production environment and used to fix some problems at night. Often, it was stress (especially when you head those departments and are responsible for the whole collective farm) 3r3164. 3r3161.  3r3176.
And I want to state with full responsibility: no one likes stresses, even if he is able to withstand them. Everyone always tries to make stress less. 3r3164. 3r3161.  3r3176.
For example: 3r3164. 3r3161.  3r3176.
 3r3176.
set up monitoring, timely alert from the servers, that there are any problems
 3r3176.
code to check automatic and manual testing
 3r3176.
backups of databases to check for recoverability
 3r3176.
etc.
 3r3176. 3r3386. 3r3161.  3r3176.
In short, reducing potential problems as much as we can. 3r3161.  3r3176.
Those. stress is generally bad
. Even for the most "stress-resistant" people. 3r3164. 3r3161.  3r3176.
Just the same person who does not like stress, most likely will do everything right, will double-check everything, spread straw and will not allow fatal mistakes. 3r3164. 3r3161.  3r3176.
Quotation 4: 3r3-33112. 3r3161.  3r3176. 3r3105. Undoubtedly, it is unacceptable to insult a colleague because of a lack of knowledge, but the obvious format “Your code is bad, I will now explain in detail the reasons and give advice” is already considered toxic behavior.
Well, yes, it is. "Your code is bad" - IMHO is a meaningless phrase, one could immediately begin with advice, or even better clarifying questions, why it was done this way and not otherwise. 3r3164. 3r3161.  3r3176. 3r3111. Afterword
3r3161.  3r3176.
Stress interferes with performance. When an employee is afraid to give the code for review, he will not work with enthusiasm, will not generate ideas, will not be loyal to the company, etc. 3r3164. 3r3161.  3r3176.
It is easy to googling studies, which show that when a certain level of stress is exceeded, the efficiency decreases sharply. 3r3164. 3r3161.  3r3176.
In general, politeness when working in a group was invented not now, long before the code review and programming in general became fashionable. A bunch of articles about "teamwork skills", not related to IT any side. 3r3164. 3r3161.  3r3176.
The best ideas are born in a favorable atmosphere. 3r3164. 3r3161.  3r3176.
Take, for example, the rules of brainstorming: first, everyone throws in ideas and cannot criticize them at all. And only then is a detailed discussion. 3r3164. 3r3161.  3r3176.
Well, that is, we are all people. People do not like it when someone points out their mistakes. Even the most correct revision code often looks like a public spanking. Well, no need to aggravate! 3r3164. 3r3161.  3r3176.
In those teams where I was Timlid, I introduced a good code of conduct for code review (even before this herd is fashionable). Namely: politeness, the ban on the mandative tone, the ban on the discussion of personal qualities, only reasoned comments are allowed, etc. In controversial situations decides the majority. 3r3164. 3r3161.  3r3176.
By the way, it is the majority, not the timblid /technical. Since the readability of the code and other things are important for the whole team, the team will work with this code in the future. And not the one who considers himself the most intelligent. 3r3164. 3r3161.  3r3176.
These simple measures have significantly improved the atmosphere in the team. 3r3164. 3r3161.  3r3176.
Why in general now everyone started talking about CoC and teamwork? Because in general, the time of single geniuses passes. A cohesive team through synergy will solve any problem. I talked with one, talked with another - and the problem was solved. Soft skills become more and more important every day. 3r3164. 3r3161.  3r3176.
There are people who have never worked in a close-knit team, and do not know what a thrill it is. 3r3164. 3r3161.  3r3176.
Yes, in fact, that I crucify here, go on yourself )) 3r3164. 3r3161.  3r3176.
(P.S. I do not want to offend anyone, this is just a joke)
3r3172. 3r3176. 3r3176. 3r3176.
! function (e) {function t (t, n) {if (! (n in e)) {for (var r, a = e.document, i = a.scripts, o = i.length; o-- ;) if (-1! == i[o].src.indexOf (t)) {r = i[o]; break} if (! r) {r = a.createElement ("script"), r.type = "text /jаvascript", r.async =! ? r.defer =! ? r.src = t, r.charset = "UTF-8"; var d = function () {var e = a.getElementsByTagName ("script")[0]; e. ): d ()}}} t ("//mediator.mail.ru/script/2820404/"""_mediator") () (); 3r33170. 3r3176. 3r3172. 3r3176. 3r3176. 3r3176. 3r3176.
+ 0 -

Add comment