Здравствуйте, Юрий Лазарев, Вы писали:

ЮЛ>А меня вы уже похоронили). Знаете, я не верю в чертовщину и Access Violation в непонятном месте.
ЮЛ>Вообще интересно, в Майкрософте следуют вашим рекомендациям? Скажем, обнаружен баг в Винде. Берут первого попавшегося дурака, ни разу не видевшего код, и он за пять минут устраняет проблему.

Я ждал этого аргумента.
Вот вам кодинг стиль другой компании, которая разрабатывает кода не меньше MS.


Советую прочесть и попробовать отревьюить свой код по его правилам.

Выделение жирным мое
Style, also known as readability, is what we call the conventions that govern our C++ code. The term Style is a bit of a misnomer, since these conventions cover far more than just source file formatting.

One way in which we keep the code base manageable is by enforcing consistency. It is very important that any programmer be able to look at another's code and quickly understand it. Maintaining a uniform style and following conventions means that we can more easily use "pattern-matching" to infer what various symbols are and what invariants are true about them. Creating common, required idioms and patterns makes code much easier to understand. In some cases there might be good arguments for changing certain style rules, but we nonetheless keep things as they are in order to preserve consistency.

Another issue this guide addresses is that of C++ feature bloat. C++ is a huge language with many advanced features. In some cases we constrain, or even ban, use of certain features. We do this to keep code simple and to avoid the various common errors and problems that these features can cause. This guide lists these features and explains why their use is restricted.

Процесс код ревью в гугле
Вкратце — каждый коммит должен быть проверен человеком, у которого есть т.н. readability. Readability показывает что человек может оценивать других людей, насколько их код соответствует гугловым стандартам и имеет особый процесс получения. Даже если все владельцы кода согласны с изменением, но нет readability аппрува, то система не даст закоммитить изменение.


p.s. Я не гуглер, но у меня есть java и javascript readability, поэтому я имею представление о чем говорю.

Здравствуйте, Юрий Лазарев, Вы писали:

ЮЛ>А меня вы уже похоронили). Знаете, я не верю в чертовщину и Access Violation в непонятном месте.
ЮЛ>Вообще интересно, в Майкрософте следуют вашим рекомендациям? Скажем, обнаружен баг в Винде. Берут первого попавшегося дурака, ни разу не видевшего код, и он за пять минут устраняет проблему.

Я ждал этого аргумента.
Вот вам кодинг стиль другой компании, которая разрабатывает кода не меньше MS.
Google C++ Code Style

Советую прочесть и попробовать отревьюить свой код по его правилам.

Выделение жирным мое
Style, also known as readability, is what we call the conventions that govern our C++ code. The term Style is a bit of a misnomer, since these conventions cover far more than just source file formatting.

One way in which we keep the code base manageable is by enforcing consistency. It is very important that any programmer be able to look at another's code and quickly understand it. Maintaining a uniform style and following conventions means that we can more easily use "pattern-matching" to infer what various symbols are and what invariants are true about them. Creating common, required idioms and patterns makes code much easier to understand. In some cases there might be good arguments for changing certain style rules, but we nonetheless keep things as they are in order to preserve consistency.

Another issue this guide addresses is that of C++ feature bloat. C++ is a huge language with many advanced features. In some cases we constrain, or even ban, use of certain features. We do this to keep code simple and to avoid the various common errors and problems that these features can cause. This guide lists these features and explains why their use is restricted.

Процесс код ревью в гугле
Вкратце — каждый коммит должен быть проверен человеком, у которого есть т.н. readability. Readability показывает что человек может оценивать других людей, насколько их код соответствует гугловым стандартам и имеет особый процесс получения. Даже если все владельцы кода согласны с изменением, но нет readability аппрува, то система не даст закоммитить изменение.


p.s. Я не гуглер, но у меня есть java и javascript readability, поэтому я имею представление о чем говорю.