search

tkaratug / titan-mvc

Titan - A Simple MVC Framework
MIT License
105 stars 16 forks source link

config/config.php #7

Closed irfanevrens closed 8 years ago

irfanevrens commented 8 years ago

config/config.php

Bu dosya App sınıfında içeri alınıyor. Ayrıca, system/plugins/Cookie.php ve system/plugins/Session.php içinde de alınıyor. Gördüğüm kadarıyla config parametrelerine ihtiyaç duyuyor bu sınıflar. Bu config içindeki ayarlara app üzerinden ulaşılırsa kendini tekrar etmeyen bir yapı kurulmuş olur.

Tabi bunun için Cookie ve Session nesneleri oluştururken $app objesi gönderiliyor olmalı.

tkaratug commented 8 years ago

Bu konuda ben de aynı fikirdeyim fakat config dosyalarına erişmesi gereken her nesneye app objesini göndermek, pek best practice gibi gelmedi. O yüzden şimdilik bu şekilde bırakıyorum.

irfanevrens commented 8 years ago

Aslında burada temel bir problem var. O da merkezi olarak bir service manager'ın olmaması. Sanırım bir service manager geliştirirsek bu problem çözülmüş olur.

Ayrıca, App bu işin http katmanını yöneten ve MVC pattern'i ile şekillenmiş context'i olabilir. Session ya da Cookie vs bu context'e ulaşmasına çok da gerek yok aslında. Service manager üzerinden dolaylı olarak iletişim sağlanabilir.