Kā jūs izietu no situācijām, kad nepiekrītat koda pārskatīšanas novērtējumam?

Kad rodas nesaskaņas par koda pārskatīšanas novērtējumu, es izmantoju šo pieeju:

1. Pievēršu uzmanību komentāram: Cenšos saprast galveno iemeslu viņu iebildumiem un ieteikumus uzlabojumiem. Iespējams, es kaut ko esmu palaidis garām vai viņu skatījums balstās uz citu pieredzi.

2. Analizēju savu pozīciju: Pārskatu savu sākotnējo lēmumu un argumentēju, kāpēc tieši šo pieeju esmu izvēlējies. Svarīgi ir skaidri formulēt savus argumentus.

3. Tieši apspriežu: Tā vietā, lai būtu pasīvi neapmierināts, es uzsāku diskusiju ar pārskatītāju. Tas var būt sinhrons komunikācija ziņojumā, tikšanās vai komentārs pašā kodu pārskatīšanas sistēmā. Mans mērķis ir — ne uzvarēt strīdā, bet nonākt pie optimālā risinājuma.

4. Iesniedzu alternatīvus risinājumus: Ja mans risinājums neapmierina, es esmu gatavs apsvērt citus variantus, ko ierosina pārskatītājs, vai kopīgi izstrādāt jaunu pieeju, ņemot vērā abu viedokļus.

5. Nepieciešamības gadījumā, iesaistu trešo personu: Ja pēc diskusijas vienošanās netiek panākta, es varu vērsties pie cita komandas locekļa vai vadītāja, lai saņemtu neatkarīgu viedokli.

Piemērs par diskusiju kodu pārskatīšanas sistēmā:

// Mans komentārs pie piezīmes
// Sveiki! Paldies par pārskatīšanu. Es saprotu jūsu viedokli par this.findViewById izmantošanu.
// Es izvēlējos šo pieeju veiktspējas apsvērumu dēļ, jo šeit View dziļums ir mazs.
// Vai varētu paskaidrot, kāpēc tieši šajā gadījumā ieteicat izmantot ViewBinding?

Ja pārskatītājs iesaka optimizāciju, kas šķiet pārāk apgrūtinoša:

Es attēloju datus tabulas formātā, ilustrējot ierosinātās optimizācijas ietekmi uz veiktspēju šajā konkrētajā gadījumā, salīdzinot ar tās īstenošanas laiku:

Optimizācija	Potenciālais veiktspējas pieaugums	Realizācijas laiks
Sākotnējais risinājums	Bāzes	0
Ierosinātā optimizācija	Minimāla (apmēram 1-2 ms)	1 stunda

Galvenais mans mērķis ir — uzlabot koda kvalitāti un padarīt to labāku, nevis tikai aizstāvēt savu viedokli. Es esmu atvērts konstruktīvai kritikai un gatavs mainīt savu lēmumu, ja tiks sniegti pārliecinoši argumenti.