💫 About Me:

🎓 I have graduated from Karabük University - Computer Programming

💻 I'm open to work as Frontend Developer

🌱 I'm currently learning React Native

💬 Ask me about Javascript

💻 Tech Stack:

📊 GitHub Stats:

weather-app's People

Contributors

Watchers

weather-app's Issues

Code Review notlarım

WeatherContext.tsx

let'le tanımlanan değişkenleri const'la tanımlaman mümkün ve öyle yapman daha doğru.
Örneğin selectedCity için map yerine find methodunu kullanarak tek seferde değişken atamasını yaparak let'ten kurtulabilirsin. why const over let diye aratabilirsin.
arr değişkeni için: map içinde arr.push yerine doğrudan filter kullanarak kodu istediğin veriyi dönecek şekilde düzenleyebilirsin.
initialCity'i const'a çevirmek mümkün, ve öyle olması daha doğru. localStorageCity nin varlığına göre atanması yapılıyorsa const initialCity = localStorageCity ?? defaultCity ile yapılabilir. https://www.scaler.com/topics/javascript-nullish-coalescing-operator/

let initialCity: string
const localStorageCity =
  typeof window != 'undefined'
    ? window.localStorage.getItem('weatherapp.city')
    : null
localStorageCity
  ? (initialCity = localStorageCity)
  : (initialCity = defaultCity)

DailyForecast Dosyası

props parametresini kaldırıp data'yı destructuring-assignment ile alabilirsin.
Bu gibi değerleri üstte birer değişkene atayıp bileşenin içinde öyle kullanman daha temiz olur moment.unix(props.data.dt).format('dddd')
imaj src'sini de değişkene taşıyabilirsin.

Bu üstteki iki maddenin aslında temel amacı, logic'i olabildiğince JSX kodlarından arındırmak, tepeye bakıp hangi değişkenler var görüp, JSX'de de evet burası tamamen HTML dönen şablon kısmı diye yorumlamak.

Genel
Comment ekleme Cohortta önerildiği için sanırım fazla yapılmış. Zaten bileşen, değişken, fonksiyon adları kendini açıklayan isimler olduğundan çoğuna ihtiyaç yok. Elzem olan yerlerde comment eklemekte fayda var.

Kodlama

API çağrılarını ayrı bir dosyada tutmakta fayda var, projede tek yerde çağırılıyor olsa da. getWeather'o Context Provider içinden çıkarabilirisin.

README

Son seçilen şehir bilgisini lokalde tuttuğunu ona göre kullanıcı her girdiğinde kendi şehrini görebilir notuny da ekleyebilirsin.
Getting Started'a npm install'a da eklemende fayda var.

Klasör ve Dosyalar

Kurulumda deneysel app routing özelliğine yes dediğin için _document.tsx dosyası da gelmiş, projende kullanmayacaksa kurulumda bunu seçmesen daha iyi olur.
cities.json'da kullanılmayan bölge bilgisi var, kullanmayı planlamıyorsan çıkarabilirsin. Eğitsel, demo amaçlı da olsa projelerinizde yüksek kaliteyi hedeflemeye çalışın bu örnek de sizi buna bir adım yakınlaştırır.

Görsel Öneri

Ana div'i sayfaya dikey ortayabilirsin.
Düşük ve yüksek sıcaklıklar aynı divdeki p'ler olduğundan şehir değişince bir kayma oluyor gibi. Benim çözüm önerim bu iki div'i %50 genişlik verip, metni de sağa yaslamak olur.

Sorularım