Trendyol Mağza Takipci API PHP (Github)

Merhabalar
iş hayatımda iyi yerlere gelmek için github profilimde bir kaç proje paylaşmak istiyordum
zamanı olan değerlendire bilir mi ?
eğer hoşunuza gittiyse star (yıldızlama) yaparsanız çok sevinirim
Github Projelinki : https://github.com/suphiyasin/Trendyol-Takipci-API
Değerlendirmelerinizi Bekliyorum

Merhaba, öncelikle düşüncenizde size katılıyorum. Bu sektörde portfolyo buradan gösterilir buradan incelenir ve değerlendirilir. Naçizane bir kaç tavsiyem olacak lütfen olumsuz olarak düşünmeyin, kendinizi geliştirmeniz için yorum yazıyorum.

• Öncelikle yazdığınız kodu herkesin anlayabileceği düzeyde yazmalısınız. Koda sadece siz bakmayacaksınız. Ne ne işe yarar bilinmesi gerek ve bu yüzden yorum satırı kullanmanız gerekiyor. PSR standartlarını da incelemelisiniz.
• Basit bir işlem bile olsa 1-2 dosyada işi bitirmeye çalışmayın. Her projenizde bir mimari olsun. SOLID prensiplerini takip edebilirsiniz. Bu prensibi takip ederseniz şu 2 dosyayı klasörlere ve dosyalara bölebilirsiniz.
• Olabildiğince özenli şekilde projeler paylaşın, formatlama kullanımına özen gösterin.

Başarılar dilerim

hocam 1. madde için çok haklısınız önceki projelerimde hiç dikkat etmiyordum bu dikkat etmiş halim gün geçtikce geliştiricem o yönünü
2. madde için kısa olunca kullanıcıların indirip test etmesinde üşengeçlik yapmazlar diye düşündüm bundan sonra daha detaylı bir proje paylaşımı yaparım teşekkürler
3. madde de gerçekten githubda paylaşıcak proje çok az aklıma geliyor bu yüzden tüm konularım özgün olmasını isityorum hem özgün hemde kötü değil dediğiniz gibi özenli yazmak istiyorum o konuda da kendimi geliştirmye devam ediyorum .

zamanınızı ayırıp bana tavisye ve yazı yazdığınız için teşekkürler

yukarı çıkması için up

Başarılarınızın devamını dilerim 🙂

Buna takilmayin. Onemli olan unique proje yapmak degil, projeyi olustururken izledigini adimlar, kod kaliteniz vs.

tematem'in dedigi gibi SOLID ve PSR onemli. Mesala $headers[] = 'Content-Length: 0'; yapmak yerine bir tane Header class'iniz olsa ve bu class addHeader() ve getHeaders() methodlari icerse proje isverenin gozunde direkt 2-3 basamak daha yukarda olur.
Diger bir ornek; curl kullanilan satirlar. Curl alakali islmeler icin tekrar eden kod yazmisin. Buna gorek yok. Hemen cakiyorsun ayrica bir tane class. Hem kodun temiz oluyor, hem tekrara dusmuyorsun hem de single responsibility principle tarafinda sorun yasamiyorsun.

Ayrica composer paketi haline getirirsen daha iyi olur. Kullanmak isteyenler tek komut ile senin kodunu projeye dahil edebilirler.

DRY, KISS, SOLID, YAGNI. Bu terimlere bakmani oneririm

Evet hocam KISS , YAGNI bunlara dikkat etmem gerek aslında gereksiz bilgiler var , ve çoğu projemde genellikle tüm fonksiyonların header bilgileri aynı oluyor onları bir değişkene atıp ordan çekmek matıklı olucaktır .
teşekkürler

ups