Правилност на клас.

[vik96]

Понеже не можах да намера стара тема направих нова.
Послушах съветите на другите и си редактирах класа :

<?
class userLogs
{
	public $logfile = 'log.html';

	
	public function setInfo()
	{
	$ip = $_SERVER['REMOTE_ADDR'];
	$page = $_SERVER['PHP_SELF'];
	$referer = $_SERVER['HTTP_REFERER'];

	$getinfo = $this->getArray($_GET); 	

	$tek = "$this->old <br /> =========================================== <br /> <b>IP</b>: $ip <br /><b>Page</b>: $page <br /> <b>Referer</b>: $referer  <br /><b>GET Query</b>:   $getinfo ";
	
	file_put_contents($this->logfile,$tek); 
	}
	


	protected function getArray($array)
	{
	foreach ( $array as $flag=>$part )
		{
		$start .= " <b>$flag</b>:$part , ";
		}
	return $start;
	}



	public function setFile($log)
	{
	$this->logfile = $log;
	}

}
?>

Сега правилно ли е?

 

[emagi]

Какво целиш да направиш с този клас?
Не вярвам, че може да ми обясниш защо ползваш ООП!
Карай си с процедурно програмиране!ООП е за големи проекти.
Иначе може да ползваш и конструктур за инициализиране на първоначалните променливи.....
С две думи:опитал си се(което е хубаво), но нищо не е станало, защото си напъхал обикновен(процедурен) код в един клас.Дефактно ти нямаш никаква ползва от това и разликата с функционалното програмиране е никаква, просто кода е "обвит" в един текст, с име "class"

 

[vik96]

Какво целиш да направиш с този клас?
Не вярвам, че може да ми обясниш защо ползваш ООП!
Карай си с процедурно програмиране!ООП е за големи проекти.
Иначе може да ползваш и конструктур за инициализиране на първоначалните променливи.....
С две думи:опитал си се(което е хубаво), но нищо не е станало, защото си напъхал обикновен(процедурен) код в един клас.Дефактно ти нямаш никаква ползва от това и разликата с функционалното програмиране е никаква, просто кода е "обвит" в един текст, с име "class"

Не става въпрос за функционалноста ,а просто искам да се науча да пиша правилен код за по-големи проекти.Този клас съм го направил само за тест Може ли да ми кажеш къде греша ? Иначе класа работи..

 

[devent]

Не грешиш. Наистина можеш да ползваш конструктор, но можеш и без него да минеш. Като цяло няма никакъв смисъл в момента от кода ти. Реално това, можеш да го постигнеш и без клас. Т.е подкрепям думите на @emagi, за да не се повтаряме

 

[StormBreaker]

Е то ако става на въпрос за това - всяко едно нещо можеш да го направиш и с OOP и в процедурен вариант. Въпроса е ти самия да оцениш за отделен проект как ще ти е най-удобно да действаш. А едно от положителните страни на ООП е, че като напишеш веднъж един клас можеш много лесно да си го ползваш и в други проекти без да се налага много интегриране.