Burakpncr
commented
3 years ago Cihat değerlendirmelerin için çok teşekkürler. Node modules konusunda aslında biraz dalgınlığım yüzünden bu şekilde oldu react dosyaları direk .gitignore içinde olduğu için diğer konuda da çok haklısın reactteki projelerimdeki istekleri o şekilde yapıyordum normalde bir dahakine daha dikkatli olurum yeniden değerlendirmelerin ve tavsiyelerin için çok teşekkür ederim . :)
Oncelikle odevi yaptigin ve basarili bir sekilde output'u bastigin icin tebrik ederim 🚀 Surasi belki daha iyi olabilirdi dedigim noktalari buradan not olarak iletiyorum. Yanlis yorumladigim noktalar olabilir, bunlari buradan konusabiliriz ☺️
Bence yorum satirlarini yazmana gerek yok. Yorum satirlarini daha karmasik logicler icin kullanmanin daha dogru oldugunu dusunuyorum.
node_modules icin .gitignore dosyasi olusturabilirsin diyecektim ama sen zaten bu issue'yu olusturmussun 😄 Issue acarak ilerlemen guzel bence : )
service.js istek attigimiz URL'i bir constant olarak tanimlayabiliriz. Bunu sebebi de istek atacagimiz her yerde bu urli elle yazarsak okunmasi zor olabilir. bunun yerine bir degiskene atayabiliriz veya axios uzerinde baseURL olusturabiliriz. Degisken olarak tanimlamak istersek de su sekilde yazabiliriz.
const API_URL = https://jsonplaceholder.typicode.comdaha sonra bu degiskeni asagi da kullanabiliriz.Eger katilmadigin noktalar var ise yorumlar da konusabiliriz : )