Οι αξιολογήσεις PR (Pull Request) είναι μια από τις πιο σημαντικές αδιδασκόμενες δεξιότητες στην ανάπτυξη λογισμικού. Εδώ θα σας δείξω τι έμαθα σε μια δεκαετία στον κλάδο.
Hex Values
Το πιο σημαντικό πράγμα για μια κριτική δημοσίων σχέσεων είναι ότι δεν αφήνετε ποτέ μια ακατέργαστη δεκαεξαδική τιμή να περάσει στη βάση κωδίκων σας. Βλέπετε, οι δεκαεξαδικοί αριθμοί χρησιμοποιούνται ευρέως από σχεδιαστές συστημάτων υπολογιστών και προγραμματιστές επειδή παρέχουν μια ακριβή τιμή για ένα χρώμα που εμφανίζεται στην οθόνη. Αν τα χρησιμοποιήσουμε χωρίς να τα τοποθετήσουμε σε μεταβλητές, ο επόμενος προγραμματιστής θα ξέρει ακριβώς ποιο χρώμα εννοούμε, χωρίς να χρειάζεται να ψάξει τη βάση του κώδικα για να βρει ποια δεκαεξαδική τιμή αντιπροσωπεύει η μεταβλητή μας.
Φανταστείτε ότι εργάζεστε για μια μεγάλη εταιρεία, ας πούμε τη McDonalds, μια εταιρεία με ένα σημαντικό παγκοσμίως αναγνωρισμένο σύνολο επώνυμων χρωμάτων.
Φανταστείτε το αφεντικό σας Ρόναλντ να σας λέει: «Γεια, αλλάζουμε το #FF0000 σε #C8A2C8 επειδή το Grimace είναι πολύ δημοφιλές αυτή τη στιγμή. Αλλάξτε κάθε κόκκινο σε μωβ! ΤΩΡΑ ΑΜΕΣΩΣ!"
Εάν ήσασταν έξυπνοι και χρησιμοποιούσατε την ίδια μεταβλητή για το #FF0000, η αλλαγή του χρώματος θα σας πάρει περίπου 30 δευτερόλεπτα για να βρείτε το αρχείο μεταβλητής και να αλλάξετε την τιμή. Εάν είστε μια τεμπέλης Lucy, που δεν αφιέρωσε χρόνο για να προετοιμαστεί, θα αφιερώσετε 45 δευτερόλεπτα για το Control + Shift + F στο VsCode για να βρείτε και να αντικαταστήσετε όλα. Μέχρι εκείνη τη στιγμή, ο Ρόναλντ θα σε είχε ρίξει στο κέικ σου και θα σε είχε αντικαταστήσει με έναν ασκούμενο.
Linters
Πολλοί smart alex θα πουν, "Ναι, αλλά δεν θα μπορούσατε απλώς να βάλετε ένα σωρό από αυτά τα ζητήματα επιπέδου επιφάνειας σε έναν κανόνα χνούδι και να το διορθώσετε αυτόματα το "eslint", ελευθερώνοντας τους προγραμματιστές να επικεντρωθούν σε σημαντικά μέρη του κώδικα που έχουν κάποια σχέση με την αίτηση;»
ΟΧΙ. Μην είσαι ηλίθιος. Πληρώνεστε $$$ την ώρα, θα θέλετε πραγματικά αυτή την ώρα να κωδικοποιήσετε; Εάν έχουμε έναν υπολογιστή να αφαιρέσουμε τα περιττά ερωτηματικά από το JavaScript μας, στη συνέχεια θα θέλουν να οδηγήσουν τα αυτοκίνητά μας.
Προσωπικό στυλ
Μπορείτε να πείτε τι συμβαίνει με αυτό το μπλοκ κώδικα; Θα σας δώσω τα 5 λεπτά.
Έχουν περάσει μόνο 4 λεπτά, αλλά έχω πράγματα να κάνω… Το μπλοκ else δεν είναι απαραίτητο γιατί η συνάρτηση θα επέστρεφε σιωπηρά απροσδιόριστο εάν δεν πληρούται η συνθήκη. Δεν μου αρέσουν οι άσκοπες επιστροφές και παρόλο που αυτό θα έχει μηδενική επίπτωση στον κώδικα από την πλευρά του χρήστη και ένα εντελώς ασήμαντο κόστος στην εφαρμογή συνολικά, πρέπει να πω σε αυτό το άτομο να το αλλάξει (αλλιώς ο κωδικός του δεν θα συγχωνευθούν στη βάση κώδικα). Η βάση κωδικών της ομάδας είναι σαν να περνάτε φαγητό γύρω από ένα τραπέζι, θέλετε να βάλετε τα δακτυλικά σας αποτυπώματα σε όσο το δυνατόν περισσότερο.
«Σας είπα ποτέ ότι αυτό εδώ [κωδική βάση] αντιπροσωπεύει ένα σύμβολο της ατομικότητάς μου και της πίστης μου στην προσωπική ελευθερία; … Λέγοντας προσωπική ελευθερία, εννοώ την ελευθερία να απορρίψω τον κώδικά σας». — Nic Cage, Wild at Heart
Εκτέλεση του κώδικα
Ναι, μην το κάνεις αυτό. Η αλλαγή κλαδιών χάνει πολύ χρόνο. Αυτό που θέλετε να κάνετε με τις κριτικές δημοσίων σχέσεων είναι απλώς να τις γυαλίσετε με τα μάτια σας και να αφήσετε τη μηχανή JavaScript/C compiler/κ.λπ. του εγκεφάλου σας να τρέξει τον κώδικα. Αν δείτε κάτι που δεν μοιάζει με κάτι που έχετε ξαναδεί, αφήστε ένα σχόλιο λέγοντας "Αυτό δεν φαίνεται σωστό" ή "Θα έπρεπε να το κάνει αυτό;". Σίγουρα, μπορεί να μην γνωρίζετε τι κάνει ο νέος κώδικας αξιόπιστα, αλλά δεν χρειάζεται να μάθετε πώς λειτουργεί το git stash. Οι συνάδελφοί σας θα σας ευχαριστήσουν για αυτό.
Πύλη
Οι κριτικές δημοσίων σχέσεων είναι ένας πολύ καλός τρόπος για να ενημερώνετε επανειλημμένα περισσότερα κατώτερα μέλη της ομάδας σας ότι ήσασταν εκεί για περισσότερο καιρό. Ο μόνος τρόπος για να ενισχύσετε τον κώδικά τους είναι να τους βάζετε επανειλημμένα να δημοσιεύουν τα PR τους έως ότου αποκαλύψουν και απομνημονεύσουν όλες τις προσωπικές σας «γκέτες» του στυλ κώδικα και «μυηθούν» πλήρως στην ομάδα.
Ασάφεια
Οι κριτικές δημοσίων σχέσεων μπορεί να είναι πολύ διασκεδαστικές, για τους λόγους που ανέφερα παραπάνω. Ένας τρόπος για να διατηρήσετε τη διασκέδαση ακόμα περισσότερο είναι να μην εγκρίνετε ΟΥΤΕ να απορρίψετε ένα PR, αλλά απλώς να αφήσετε ένα σωρό σχόλια. Εάν απορρίψετε ένα PR και πείτε "διορθώστε αυτό το πράγμα, τότε θα εγκρίνω" ο συνάδελφός σας θα είχε μια σειρά από επαγγελματικές προσδοκίες να ακολουθήσει. Απλώς πετάξτε ένα αστείο σχόλιο στο PR, ένα gif ή ίσως κάποιες αόριστες προτάσεις που υπονοούν ότι αν ξαναγράψουν τα πάντα εντελώς διαφορετικά, ίσως ίσως να μην ξέρω…. κάτι ίσως.
Μην τα κάνετε
Τέλος, η νούμερο 1 συμβουλή μου. Μην κάνετε κριτικές δημοσίων σχέσεων αν μπορείτε να αποφύγετε. Αυτά είναι για άτομα που δεν είναι τόσο απασχολημένα όσο εσείς. Εάν θέλετε να εντυπωσιάσετε τη διοίκηση και την ομάδα σας, το να μην κάνετε αναθεωρήσεις δημοσίων σχέσεων σας δείχνει πόσο σοβαρά παίρνετε τη δουλειά σας και τη δέσμευσή σας σε αυτή τη βάση κώδικα.
