AGorkemAksoz
commented
4 months ago Düzeltildi yani herhalde :)
Open ozgurelmasli opened 4 months ago
AGorkemAksoz
commented
4 months ago Düzeltildi yani herhalde :)
https://github.com/AGorkemAksoz/CryptoApp-SwiftUI/blob/eefd15794bcf85b71635d8af79658142f95f2c85/CryptoApp-SwiftUI/Utilities/NetworkingManager.swift#L27
Bu function'ların static olmasının bir sebebi var mı ?
Bu şekilde yazmak test yazmayı zorlaştıracaktır.
Protocol-based bir hale dönüştürebiliriz. Kullandığımız yerlere dependecy injection kullanarak erişebilir hale getiririz. Tabii asıl getirisi unit test yazarken olacaktır.
Ekstra olarak NetworkManager'ın Util altında olmaması daha doğru olacağını düşünüyorum.
Final keyword koymakta bize o class'ın başka bi classtan inherit alınmayacağını garanti edere complier'ın build süresinde yarar sağlacaktır.
Araştırırken Final Keyword in Swift diye aratırsan daha detaylı anlatım bulacaksındır :)