Kod incelemelerinizi nasıl eğlenceli hale getirebilirsiniz (ve korkutucu değil)

Kod incelemelerinde adil payımı yaptım. Adil paylaşım derken çok şey kastediyorum. Destansı bir özelliğin kod incelemesi yaptınız mı? Sahibim. Harika bir deneyim değildi, çünkü 100. dosya gözden geçirilinceye kadar koda bakmaktan yorulmuştum. "Evet, bana iyi görünüyor" diyip onayımı vermeye çok yaklaştım.

Ancak kod incelemeleri bu şekilde çalışmaz. Başladıktan sonra, ona bağlı kalmalı ve sonuna kadar bitirmelisin. Elbette ara verebilirsiniz, ancak sonra bağlamı kaybetmeye başlarsınız ve şunun gibi her şeye yeniden başlamanız gerekir:

Yine de dalıyorum. Ağrı, benim konuşmam gereken şey değil.

Öyleyse nasıl daha az korkunç hale getirebiliriz? Birincisi, çekme isteğini yapmadan önce ve hatta kodlamaya başlamadan önce bile özellikleri güzel küçük parçalara ayırmayı bir noktaya getirmeliyiz. Bir mini özelliği devreye almak destansı bir özellikten daha kolay olduğu için geliştirme ve dağıtımlar daha hızlı hale gelir. Bu, değişiklikler oldukça küçük olduğu için kod incelemelerini daha kolay ve daha hızlı hale getirir.

Harika! Umarım makalemi beğenmişsindir. Hadi eve gidelim ...

Bekle, dur !

Bu, nasıl daha az korkunç hale getirileceğiyle ilgiliydi, ama bunu eğlenceli hale getirmeye ne dersiniz?

Kod incelemelerinde memleri kullanma

Sorumluluk reddi: Aşağıdakiler işverenimin değil kendi düşüncelerimdir.

Bunu yoldan çıkarmak istedim, çünkü bu potansiyel olarak tartışmalı bir konu olabilir. Her neyse, memler ekleyerek kod incelemelerimizi daha hafif hale getiriyoruz.

Buradan göz dönmesini duyabiliyorum! Yine de beni dinle. Bunu zaten memnuniyetle açıklayan başka biriyle bağlantı kurmaktan daha iyi bir açıklama olabilir mi?

Hem gözden geçiren hem de kodu gözden geçirilen kişi için eğlenceli hale getirmeye yardımcı olan bir üsluptur. Bir örnek, benim içindeki ilk "zımba" olabilir image wallet. Rubocop'un dosyalarımızda bazı çift boşlukların eksik olduğunu fark ettim, bu yüzden birisi fazladan beyaz boşluk içeren bir çekme isteği gönderdiğinde, onlara bir doge vermek hiç akıllıca değildi.

Basit bir resimdi ama yine de mesaj çok etkiliydi. İnsanların gördüklerinde güldüklerini hatırlıyorum. İnsanlar "korkutulmak" istemedi, bu yüzden herkes fazladan boşlukta daha dikkatli oldu.

Please remove the extra whitespaceÇekme isteğinde (PR) sadece eski olanı yazmak yerine, bulucuyu açmak, doge'yi bir yoruma sürüklemek ve yayınlamak daha kolaydı , özellikle birden fazla köpek varsa.

Geri tepemez mi?

Tamamen olabilir. Herkesin ekibimizi takip etmesi gerektiğini söylemiyorum. Gerçekten takımın kişiliğine bağlı. Bizimki yaş ortalamalarımın altı yaşında, bu da çoğunun memlerle ilgili olabileceği anlamına geliyor. Takımınızın ortalama yaşı kırk olsa farklı bir hikaye olurdu (tabii ki 4chan sakinleri veya en son memlerle güncel değillerse!).

Takım arkadaşlarınızı en iyi siz bilirsiniz, gerçekten. Birkaç ay birbirimizle birlikte olduktan sonra (özellikle zamanın neredeyse% 100'ünde eşleştiğimiz için), tabiri caizse, birbirimizin mizahı hakkında bir fikir edindik. Bu, PR'larımızda memleri görmemizi ve sadece gülmemizi sağladı (tabii ki sorunu düzeltirken).

Memlerden ne gibi faydalar elde edersiniz?

Bir faydası, incelemeyi daha hızlı hale getirmesidir (en azından benim için). PR'yi de gönderen kişi aslında neyi daha hızlı yapması gerektiğini anlıyor. Birkaç kelime okumak yerine bir resim görüyorsunuz ve hepimiz resimlerin kelimelerden daha yüksek sesle konuştuğunu biliyoruz. İşte benden birkaç örnek personal image wallet™:

Bir resmi ilk kez ortaya çıkardığımda, ne yapmaları / düzeltmeleri gerektiğine dair bir açıklama / açıklama ekliyorum. Yukarıdakiler temel olarak yukarıdaki "eski" kodun kaldırılmasını söylüyor. Silin. Onu yok et!

Bu Ruby'ye özgüdür - pretzel dotoperatör temelde bir try-catch. Ben sadece simit gerektiren pasajı vurgulayıp bu resmi yapıştırıyorum. Çabuk ve kolay!

Bu ikisini sadece birbirinin yerine kullanıyorum ve ÇOĞUNLUKLA Ruby'de # frozen_string_literal: truesihirli yorumu her zaman dosyanın en üstüne ekliyoruz . Dümendeki Rubocop ile Elsa ve Bay Freeze'i gitgide daha az gördük. Ayrıca donması gereken herhangi bir sabit için de görünürler.

Bu oldukça basittir (umarım). Meslektaşlar bunu gördüklerinde, bu bir yazım hatası yaptıklarının bir işaretidir. Genellikle yazım hatasını da vurgularım, böylece hızlı ve kolay görülmesi (ve düzeltilmesi).

Bunlar sadece birkaçı. Unutmayın, çok fazla şey genellikleiyi bir şey değil. Bu yüzden memlerinizi eklerken dikkatli olun. Bazen dikkat dağıtıcı olabileceğinden çok fazla mem eklememek en iyisidir. Doğru ton dengesini bulmak ve mesajı olabildiğince çabuk göndermek, bir çekme talebini tamamlamanın en iyi yoludur.

Hepsi bu millet

Sonuç olarak, eğlenmek veya sadece işte ciddi ve profesyonel kalmak istiyorsanız, bu gerçekten size kalmış. Bazıları memlerin profesyonel olmadığını düşünebilir ve tabii ki resmi işlerde de olabilirler. Bir çekme talebinin işle ilgili resmi bir şey olabileceği veya olduğu iddia edilebilir, ancak bence bu yalnızca dokümantasyon veya incelemeler için kullanırsanız geçerlidir. Aksi takdirde, bence "ifade özgürlüğü" (evet, o kartı çektim!) Ve olduğu gibi alınmalı.

Benim için söylemek istediğim şeyi aktarıyor: fazladan boşlukları kaldırın, yazımı düzeltin, bir sabiti dondurun veya bir satır veya kod satırını kaldırın. Fazla yazmam gerekmiyor, meslektaşımın başka bir kelime grubuna karşı bir sürü kelimeyi okuması gerekmiyor ve bu ruh halini biraz hafifletiyor. Herkes kazanır! Bununla bunu resimlerimden biriyle bitireceğim