Похоже, ваш код еще не проверен :-)
Цель обзора кода - получить код приличного качества и узнать, что у вас есть код приличного качества. Когда код неопытного разработчика проверяется, его можно использовать, чтобы научить писать лучший код, избегая при этом разочарования этого разработчика.
Рецензент никогда не должен менять ваш код. Они могут сделать более или менее убедительные предложения о том, как они хотели бы, чтобы ваш код был изменен, и они могут решить, принять ваш код или нет.
Если проверка прошла правильно / если я рассмотрю ваш код, вы, скорее всего, получите некоторые комментарии о том, как я написал бы код, из которого вы можете поучиться или проигнорировать - это то, что у меня есть мнение, и вы можете получить другое мнение. В моей области хорошее именование функций, переменных и т. Д. Считается важным, поэтому вы можете получить некоторые предложения по улучшению именования. Обычно вы должны внести изменения в этом случае (иногда, найдя даже лучшее название для чего-то). Иногда я найду ошибки. Вы исправляете их. Иногда я нахожу вещи, которые я считаю ошибками, и я ошибаюсь. Если трудно понять, что код правильный, вы сделаете его более очевидным правильным. Если я просто ошибся, скажи мне.
Если я думаю, что дизайн, как правило, не так, то это должно было обсуждаться ранее. Затем мы должны подумать о том, достаточно ли хорош ваш дизайн, принимая во внимание, сколько работы вовлечено в изменение, или, может быть, я был просто неправ, и ваш дизайн лучше. Мы должны закончить с соглашением.
Если рецензент и рецензент не могут договориться, у нас есть проблема. Потому что это означает, что один из нас неспособен на командную работу, или один из нас не способен отличить хороший или плохой дизайн, или оба. Это не обязательно ваша вина. К сожалению, есть разработчики, которые являются старшими и невежественными, и это будет проблемой для компании и для вас.
Если это произойдет, подумайте очень, очень усердно: есть ли у вас проблемы с принятием обоснованной критики? Если это так, вам нужно изменить свое отношение. Вы слишком неопытны, чтобы понять, почему рецензент прав? Если это так, это не проблема. Доверяйте рецензенту и учитесь. Вы уверены, что знаете лучше, чем рецензент? Примите обзор, но спросите третьего доверенного разработчика о его мнении. Помните, что вы можете быть действительно уверены в себе и быть правы, но вы также можете быть действительно уверены в себе и ошибаться.