Evet kesinlikle. Bunlar, bu kodu yazan kişinin koddan memnun olmadığının ve işe yarayana kadar büyük olasılıkla düştüğünün açık göstergeleridir. Gerçek meselelerin ne olduğunu anlamadılar ya da daha kötüsü, onları anladılar ve düzeltmek için çok tembellerlerdi.
Ancak, bunları düzeltmek için çok çaba sarfedilmesi gerektiği ve bu düzeltmelerin kendisiyle ilişkili riskleri olacağı konusunda bir uyarı verilir.
İdeal olarak, sorunun ne olduğunu çözebilecek ve düzgün bir şekilde düzeltebileceksiniz. Örneğin:
Modül A'ya bazı şeyler yapması için zaman aşımına uğradı. Böyle zamanlanmış değilse, kopacak.
Bu, Modül A'nın ne zaman kullanıma hazır olduğunu veya işlemeyi ne zaman bitirdiğini uygun şekilde göstermediğini göstermektedir. Belki de bunu yazan kişi, Modül A'yı tamir etmeyi rahatsız etmek istemedi veya bir sebepten ötürü yapamadı. Bu, gerçekleşmeyi bekleyen bir felakete benziyor, çünkü uygun sıralamadan ziyade şansla uğraşılan bir zamanlama bağımlılığı öneriyor. Bunu görseydim düzeltmek isterdim.
Bunu değiştirmeyin. İnan bana, işleri kıracaksın.
Bu size pek söylemez. Kodun ne yaptığına bağlı. Bu, bir nedenden ötürü, gerçekten de kodu kıracağı açık optimizasyonlar olduğu anlamına gelebilir. Örneğin, bir döngü başka bir kod parçasının dayandığı belirli bir değerde bir değişken bırakıyor olabilir. Veya bir değişken başka bir dizide test edilebilir ve değişken güncellemelerinin sırasını değiştirmek diğer kodu kırabilir.
SetTimeout'u kullanmanın iyi bir uygulama olmadığını biliyorum, ancak bu durumda kullanmak zorunda kaldım.
Bu kolay birine benziyor. Ne setTimeout
yaptığını görebilmeli ve belki de daha iyi bir yol bulmalısın.
Bununla birlikte, bu tür düzeltmeler refactorünüzün kapsamı dışındaysa, bu kodun içinde yeniden düzenlemeye çalışmanın çabalarınızın kapsamını önemli ölçüde artırabileceğinin belirtileridir.
En azından, etkilenen koda yakından bakın ve yorumu en azından sorunun ne olduğunu daha net bir şekilde açıkladığı noktaya getirip geliştiremediğinizi görün. Bu, bir sonraki kişiyi, karşılaştığınız gizemle yüzleşmekten kurtarabilir.