Sepertinya kode Anda belum ditinjau :-)
Tujuan dari tinjauan kode adalah untuk mendapatkan kode dalam kualitas yang layak, dan untuk mengetahui bahwa Anda telah mendapatkan kode dengan kualitas yang layak. Ketika kode pengembang yang tidak berpengalaman ditinjau, maka dapat digunakan untuk mengajarkan cara menulis kode yang lebih baik, sambil menghindari membuat frustrasi pengembang itu.
Peninjau seharusnya tidak pernah mengubah kode Anda. Mereka dapat membuat saran yang kurang lebih kuat bagaimana mereka ingin kode Anda diubah, dan mereka dapat memutuskan apakah akan menerima kode Anda atau tidak.
Jika ulasan berjalan dengan benar / jika saya meninjau kode Anda, apa yang mungkin akan Anda dapatkan adalah beberapa komentar bagaimana saya akan menulis kode yang dapat Anda pelajari, atau abaikan - ini adalah hal-hal di mana saya memiliki pendapat dan Anda bebas untuk memiliki pendapat berbeda. Di daerah saya, penamaan fungsi yang baik, variabel, dan sebagainya dianggap penting, sehingga Anda dapat memperoleh beberapa saran untuk meningkatkan penamaan. Biasanya Anda harus membuat perubahan dalam kasus itu (kadang-kadang dengan mencari nama yang lebih baik untuk sesuatu). Terkadang saya akan menemukan bug. Anda memperbaikinya. Terkadang saya menemukan hal-hal yang saya anggap bug, dan saya salah. Jika sulit untuk melihat bahwa kode itu benar, Anda membuatnya lebih jelas. Jika saya salah, Anda memberi tahu saya.
Jika saya berpikir bahwa desainnya secara umum tidak benar, maka ini seharusnya sudah dibahas sebelumnya. Kami kemudian harus berpikir tentang apakah desain Anda cukup baik, dengan mempertimbangkan berapa banyak pekerjaan yang terlibat dalam suatu perubahan, atau mungkin saya hanya salah dan desain Anda lebih baik. Kita harus berakhir dengan kesepakatan.
Jika pengulas dan penerima ulasan tidak setuju, maka kami memiliki masalah. Karena itu berarti bahwa salah satu dari kita tidak mampu melakukan kerja tim, atau salah satu dari kita tidak dapat membedakan antara desain yang baik atau buruk, atau keduanya. Ini belum tentu salahmu. Sayangnya ada pengembang yang senior dan tidak mengerti, dan itu akan menjadi masalah bagi perusahaan dan untuk Anda.
Jika itu terjadi, pikirkan sangat, sangat keras: Apakah Anda memiliki masalah dalam menerima kritik yang beralasan? Jika itu masalahnya, Anda perlu mengubah sikap Anda. Apakah Anda terlalu berpengalaman untuk melihat mengapa peninjau itu benar? Jika itu masalahnya, itu tidak masalah. Percayai pengulas dan pelajari. Apakah Anda yakin tahu lebih baik daripada pengulas? Terima ulasan, tetapi tanyakan pengembang ketiga yang tepercaya tentang pendapatnya. Ingat Anda bisa benar-benar yakin pada diri sendiri dan benar, tetapi Anda juga bisa benar-benar yakin pada diri sendiri dan salah.