Czy ten kod wygląda na obiektowy

Prosze o opinie

[PHP] pobierz, plaintext 
<?php
class gallery{
 
private $baza;//Laczenie z baza 
public  $biblioteka = '1';
 
 
function __construct(sqlcon $baza,$type,functions $fun,$biblioteka) 
{
$this->db = $baza;
$this->id_type = $type;
$this->func = $fun;
$this->libary = $biblioteka;
}
 
public function show_folders($image){
 
$galeria = $this->db->query('SELECT * FROM '.DB_PREFIX.'gallery_folders WHERE `folder_id_type`= '.intval($this->id_type).'');
WHILE($g = $this->db->fetch_assoc($galeria)){
 
$g_t = array('folder_id' => $g['id'],
'folder_nazwa' => $g['folder_nazwa'],
'folder_naz_t' => $this->func->utnij($g['folder_nazwa'],'12'));
 
echo' <div class="zdjecie">
<a href="gallery.php?act=open_folder&folder_id='.$g['id'].'"><img src="'.$image.'"
                border="0" alt="'.$g['folder_nazwa'].'"></a><br />
               <a href="gallery.php?act=open_folder&folder_id='.$g['id'].'">'.$this->func->utnij($g['folder_nazwa'],'12').'</a>
               </div>';
 
 
}
}
public function show_photos($id){
 
$this->id = $id;
 
$query = $this->db->query('SELECT * FROM '.DB_PREFIX.'gallery WHERE `folder_id` = '.intval($this->id).'');
 
$q = $this->db->query('SELECT * FROM '.DB_PREFIX.'gallery_folders WHERE `id` = '.intval($this->id).'');
while($qq=$this->db->fetch_assoc($q)){
echo '<B>ť</B> <b>'.$qq['folder_nazwa'].'<b>'; 
}
echo '<br><br>';
 
while($g=$this->db->fetch_assoc($query)){
 
if($this->libary == '1')
{
$bib = 'lightbox[roadtrip]';
}
elseif($this->libary == '2')
{
$bib = 'lytebox[vacation]';
}
elseif($this->libary == '3')
{
$bib = 'lightbox[atomium]';
}
 
elseif($this->libary == '4')
{
$bib = 'prettyPhoto[gallery]';
}
 
echo '<a href="'.$g['big_image'].'" border="0" rel="'.$bib.'">
<img border="0" src="'.$g['small_image'].'"  height="80" width="80"></a>';
 
 
}
}
public function load_libary_photos(){
 
if($this->libary == '1')
{ 
echo'<link rel="stylesheet" href="./scripts/thickbox/css/lightbox.css" type="text/css" media="screen" />
<script src="./scripts/thickbox/js/prototype.js" type="text/javascript"></script>
<script src="./scripts/thickbox/js/scriptaculous.js?load=effects,builder" type="text/javascript"></script>
<script src="./scripts/thickbox/js/lightbox.js" type="text/javascript"></script>';
}
elseif($this->libary == '2')
{ 
echo'<script type="text/javascript" language="javascript" src="./scripts/lytebox/lytebox.js"></script>
<link rel="stylesheet" href="./scripts/lytebox/lytebox.css" type="text/css" media="screen" />';
}
elseif($this->libary == '3')
{
echo'<script type="text/javascript" src="./scripts/slimbox/js/mootools.js"></script>
<script type="text/javascript" src="./scripts/slimbox/js/slimbox.js"></script>
<link rel="stylesheet" href="./scripts/slimbox/css/slimbox.css" type="text/css" media="screen" />
';
}
 
elseif($this->libary == '4')
{
echo'<script src="./scripts/bib4/js/jquery.js" type="text/javascript" charset="utf-8"></script>
<link rel="stylesheet" href="./scripts/bib4/css/prettyPhoto.css" type="text/css" 
media="screen" title="prettyPhoto main stylesheet" charset="utf-8" />
<script src="./scripts/bib4/js/prettyPhoto.js" type="text/javascript" charset="utf-8"></script>
';
 
}
echo'<style>
#wrapper {
    widt: 100%;
    clear: both;
}
.zdjecie {
    floa: left;
    text-alig: center;
    paddin: 10px 0;
}
.zdjecie img {
    displa: block;
    margi: 0 10px;
}
</style>';
}
 
 
//Koniec klasy
}
?>
[PHP] pobierz, plaintext 

Napewno wygląda "niebiesko"

Sprobuj

[PHP] pobierz, plaintext 
<?php
class gallery{
 
private $baza;
var $image;
...
?>
[PHP] pobierz, plaintext 

jakos niewiem jak to zrobic ;p

Dodaj tag [PHP] w swoim pierwszym poście. O to też chodzi w poście guilty82.

Poza tym, tak wygląda na obiektowy ale standardów się nie trzyma. Poza tym wymieszałeś PHP4 i PHP5. Po co te var?

tutaj jest odpowiedz

dziekuje za sugestie


hmmm

Var mam używać tylko jak zmienna znajduje się w konstruktorze ?

W PHP5 w ogóle nie używasz var. Od tego masz odpowiednie public, private oraz protected.

acha dziękuje bardzo za pomoc

Kod poprawiłem

czy teraz ma jakieś lepsze standardy

To oczywiście nie jest żaden wymóg, ale dobrym zwyczajem jest by metody klasy wykonywały jedynie pewne operacje, nie wyświetlały zaś treści. Twoja klasa może więc np zwrócić tablicę zawierającą wszelkie potrzebne elementy, a to osobny plik odpowiedzialny za widok niech te treści wyświetli korzystając z podanej tablicy

Czy mogę użyć zwykłego include();

bo innego pomysłu nie mam

Względnie, jak już ktoś się uprze, żeby cokolwiek wyświetlić, to niech to będzie jakaś jedna metoda show.

Class gallery, a nazwy właściwości, metod i zmiennych wszelakiego typu po polsku. Proponuję przenieść się jednak na ten angielski, w najgorszym przypadku konsekwentnie pisać po polsku. Wszystko.

Baza jak rozumiem jest to obiekt bazy danych?
Jeśli tak, to spokojnie możesz użyć konstrukcji: construct(nazwaKlasyBazyDanych $baza, ...) będziesz miał pewność, że obiekt zawsze i wszędzie bez względu na poziom zmęczenia będzie obiektem bazy danych.

[PHP] pobierz, plaintext 
<?php
if($this->biblioteka == '1')
{
$bib = 'lightbox[roadtrip]';
}
if($this->biblioteka == '2')
{
$bib = 'lytebox[vacation]';
}
if($this->biblioteka == '3')
{
$bib = 'lightbox[atomium]';
}
 
if($this->biblioteka == '4')
{
$bib = 'prettyPhoto[gallery]';
}
?>
[PHP] pobierz, plaintext 

Jeśli właściwość $this->biblioteka może być albo 1, albo 2, albo 3, albo 4, to nie stosuj czterech ifów po sobie. Lepiej jest zrobić elseif. W tym przypadku jeśli $this->biblioteka będzie 1 to i tak sprawdzi wszystkie inne warunki, które spokojnie mógłby sobie pominąć.

Poza tym w jakimś tam stopniu wygląda na obiektowy, ale jednocześnie wygląda na straszliwie zakręcony, trudny do modyfikacji i rozbudowy. A to wszystko przez ten spagettowaty kod html w środku.

Jakiego includa? o_O

np masz jakiś plik sterujący galerią: gallery.php
W nim masz grafikę galerii (tzn html) i wywołujesz tam metody klasy galerii odpowiedzialne za zwrócenie tablicy zdjęć/kategorii etc
I w tym pliku (gallery.php) pętlą po ww tablicy wyświetlasz te foty/kategorie i co tam jeszcze

Ma to na celu uporządkowanie kodu i oddzielenie (choćby tak podstawowe, skoro nie mvc) wartstw prezentacji i danych (w powyższym przykładzie na warstwie prezentacji jest kod html w pliku gallery.php a warstwa danych - to klas sterująca galerią. Jej zadaniem jest jedynie zwrócenie odpowiednich danych na podstawie zadanych parametrów [np id kategorii])

Rzeczywiście ten if to nie był wspaniały pomysł ,Poprawione dzięki


Z tym widokiem mogę zrobić tablice

[PHP] pobierz, plaintext 
<?php
$g_t = array('folder_id' => $g['id'],
 'folder_nazwa' => $g['folder_nazwa'],
 'folder_naz_t' => $this->func->utnij($g['folder_nazwa'],'12'));
?>
[PHP] pobierz, plaintext 

no ale jak ja wykorzystac

Np tak (tylko idea, kodu pisać nie będę ;P):
W klasie galerii:
1. Metoda getFolders zwraca listę folderów do wyświetlenia juz posortowaną odpowiednio z ew dodatkowymi informacjami (podpis, ilość zdjęć w środku etc). W pliku gallery.php widzisz ją jako, np:


Iterując po niej wyświetlasz foldery, w analogiczny sposób zdjęcia (inna metoda je zwraca oczywiście

[PHP] pobierz, plaintext 
<?php
foreach($aFoldersList AS $sFolder) {
    echo '<a href="gallery.php?folder=' . $sFolder['id'] . '" title="' . $sFolder['name'] . '">' . $sFolder['name'] . ' (Zdjec: ' . $sFolder['photos'] . ')</a>';
}
?>
[PHP] pobierz, plaintext 

etc