[php] Stałe w programowaniu obiektowym Opcje

[eerie]

Witam

Czy używanie stałych w programowaniu obiektowym pod php7 to dobra praktyka? Pytam, bo gdy programowaliśmy w języku Java na strudiach, to nie pamiętam, abyśmy używali stałych. Poniżej podaję kod przykładowej klasy konfiguracyjnej mojej aplikacji. Prosiłbym o sugestie, czy może mieć taki kształt? Czy zamiast zmiennych nie powinienem wprowadzić stałych? Dopiero uczę się php7 i nie mam pewności, czy stałe w klasie konfiguracyjnej to dobry pomysł...

https://github.com/eerie79/framework2_svn/b...Core/Config.php

Pozdrawiam
Robert

[krzywy5830]

To, co nazywasz stałymi, to w rzeczywistości chronione pola statyczne klasy, do definiowania stałych używamy słowa kluczowego CONST:

[PHP] pobierz, plaintext 
class Config
{
    CONST MYSQL_HOST = 'localhost';
[PHP] pobierz, plaintext 

Dodam, że jeśli chcesz żeby stała była protected to możesz to zrobić jeśli masz wersję PHP 7.1:

[PHP] pobierz, plaintext 
class Config
{
    protected CONST MYSQL_HOST = 'localhost';
[PHP] pobierz, plaintext 

No i nospor ma rację, taka konfiguracja powinna być w oddzielnym pliku. Ja zwykle tworzę plik o nazwie np. database.php z taką zawartością:

[PHP] pobierz, plaintext 
<?php
return
[
	'host' => 'localhost',
	'port' => '3306'
	'db_name' => 'nazwa bazy',
	'db_user' => 'użytkownik',
	'db_password' => 'hasło'
];
[PHP] pobierz, plaintext 

Potem w tej klasie Config możesz zrobić:

[PHP] pobierz, plaintext 
class Config
{
	public function __construct()
	{
		$database = require('database.php');
		echo($database['db_user']); //wyświetli "użytkownik", oczywiście nie dawaj tego echo, to tylko przykład żeby ci pokazać jak to działa
	}
}
[PHP] pobierz, plaintext 

No i jeszcze to:

[PHP] pobierz, plaintext 
    public static function getServerDomain(): string
    {
        return str_replace('www.', '', $_SERVER['SERVER_NAME']);
    }
 
    public static function getServerEmail(): string
    {
        return 'kontakt@' . str_replace('www.', '', $_SERVER['SERVER_NAME']);
    }
[PHP] pobierz, plaintext 

Po co stworzyłeś metodę getServerDomain() skoro potem jej nie używasz? Możesz to zamienić na:

[PHP] pobierz, plaintext 
    public static function getServerDomain(): string
    {
        return str_replace('www.', '', $_SERVER['SERVER_NAME']);
    }
 
    public static function getServerEmail(): string
    {
        return 'kontakt@' . self::getServerDomain();
    }
[PHP] pobierz, plaintext 

Poza tym $administratorEmail czy $moderatorEmail przecież mogą się zmieniać. Warto to umieścić np. w bazie danych i cachować.

Ten post edytował krzywy5830 12.01.2018, 13:23:56