Pointing out flaws should always be incouraged. Discussions, disagreements and suggestions aswell.
I don't agree about having code monkeys to do no-brain work. There are certainly companies that have that, but I'm against it.
I'd like mention that for more inexperienced people it would be a good practice to first speak with seniors and others, since they probably understand the context of the situation better, and basically anyone else that could shine new information.
I just wanted to point out that there is authority that is derived from expertize, since it wasn't that clear in your opening comment.
That's not controversial, but authority of "senior VS junior" often does not correspond to "expert VS non-expert". Seniors are often seniors because they just got stuck with the same company for a long time, or because, even worse, they wrote code that nobody except for them can maintain. That's why the development process should never be structured hierarchically, as it will kill fresh ideas and make bright developers suffer. There is, of course, a great learning opportunity if less experienced people choose to learn from more experienced, but don't forget, the best mentors are open-minded and are also willing to learn from fresh ideas. Learning should always be a 2-way process.
Pointing out flaws should always be incouraged. Discussions, disagreements and suggestions aswell.
I don't agree about having code monkeys to do no-brain work. There are certainly companies that have that, but I'm against it.
I'd like mention that for more inexperienced people it would be a good practice to first speak with seniors and others, since they probably understand the context of the situation better, and basically anyone else that could shine new information.
I just wanted to point out that there is authority that is derived from expertize, since it wasn't that clear in your opening comment.
That's not controversial, but authority of "senior VS junior" often does not correspond to "expert VS non-expert". Seniors are often seniors because they just got stuck with the same company for a long time, or because, even worse, they wrote code that nobody except for them can maintain. That's why the development process should never be structured hierarchically, as it will kill fresh ideas and make bright developers suffer. There is, of course, a great learning opportunity if less experienced people choose to learn from more experienced, but don't forget, the best mentors are open-minded and are also willing to learn from fresh ideas. Learning should always be a 2-way process.
For me the senior/junior & expert/non-expert correspondence was clear, so thank you for pointing out that it may not be clear to other people.
With the rest I guess I can only agree