Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> Klasa odpowiadająca za dynamiczne linki
marcinek37
post
Post #1





Grupa: Zarejestrowani
Postów: 239
Pomógł: 0
Dołączył: 2.06.2011

Ostrzeżenie: (0%)
-----

Przyjazne linki można tworzyć za każdym razem ręcznie, np.
[PHP] pobierz, plaintext 
  1. <?php
  2. ...
  3. echo'<a href="strony/'.$title.'-'.$id.'/">zobacz więcej</a>';
  4. ...
  5. ?>
[PHP] pobierz, plaintext


Jednak w przypadku, gdy sytuacja wymusza na nas zmianę struktury linka na np. materialy/tytul-10/, trzeba będzie dokonywać zmian w wielu miejscach. Im większa strona, tym więcej roboty.

Dlatego wziąłem się za pisanie klasy, która może pomóc rozwiązać ten problem. Uruchamia się ją w ten sposób:
[PHP] pobierz, plaintext 
  1. <?php
  2. ...
  3. echo'<a href="'.Seo::link('index.php?m=info&id=10&title=tytul').'">zobacz więcej</a>';
  4. ...
  5. ?>
[PHP] pobierz, plaintext


m - nazwa kontolera
id - informuje, jaki materiał załadować
title - zmienna wyłącznie do pozycjonowania

Klasa wygląda tak:
[PHP] pobierz, plaintext 
  1. <?php
  2. class Seo{
  3.  
  4. public static function link($link_to_convert){
  5. $link = str_replace('index.php?', '', $link_to_convert);
  6. foreach(explode('&', $link) AS $var){
  7. $parts = explode('=', $var);
  8. if($parts[0] == 'm'){ $method = $parts[1]; }
  9. else{ $vars[$parts[0]] = $parts[1]; }
  10. }
  11. if(method_exists('Seo', $method)){ return self::info($vars); }
  12. else{ return $link_to_convert; }
  13. }
  14.  
  15.  
  16. public static function info($vars){
  17. if($vars['id'] && $vars['title']){ $link = '/strona/'.$vars['title'].'-'.$vars['id'].'/'; }
  18. else{ $link = '/'; }
  19. return $link;
  20. }
  21.  
  22.  
  23. }
  24. ?>
[PHP] pobierz, plaintext


Każdy link w tworzonym przeze mnie Frameworku będzie mieć strukturę index.php?m=MODUŁ + [cała reszta zmiennych związana z modułem].
Dlatego na początku w metodzie "link" ze zmiennej $link kasowany jest ciąg "index.php?". Następnie cała reszta dzielona jest na zmienne, z czego zmienna $m jest szczególna, bo informuje, jaką metodę ma uruchomić - o ile istnieje.

Następnie metoda ta jest uruchamiana i generuje ładny link - o ile to metody trafią porządane zmienne.
Na razie jest tylko jedna metoda "info", ale w przyszłości dojdą kolejne.

Co sądzicie o takim rozwiązaniu?
Go to the top of the page
+Quote Post
 
 Start new topic
Odpowiedzi
ziolo
post
Post #2





Grupa: Zarejestrowani
Postów: 82
Pomógł: 20
Dołączył: 17.01.2009
Skąd: Kraków

Ostrzeżenie: (0%)
----- 

[PHP] pobierz, plaintext 
  1.  
  2. <?php
  3.     ...
  4.     echo'<a href="'.Seo::link('index.php?m=info&id=10&title=tytul').'">zobacz więcej</a>';
  5.     ...
  6.     ?>
  7.  
[PHP] pobierz, plaintext


Pierwsza uwaga nie związana z samym tematem. Ale jak widzę coś takiego to przypominają mi się moję początki z php i jak ja mogłem taki kod php pisać ?
Nie wstawiaj elementów html w echo w pliku php. Tylko wstawiaj elementy php do pliku html.

[PHP] pobierz, plaintext 
  1.  
  2.   <html>
  3.   ...
  4.   <a href="<?php echo Seo::link('index.php?m=info&id=10&title=tytul')?>">zobacz więcej</a>
  5.   ...
  6.   </html>
[PHP] pobierz, plaintext


Nie prościej to wygląda ?

Druga sprawa to Ty tu jesteś architektem, Ty piszesz metodę wieć czemu narzucasz sobie taki fatalny format wejściowy tej metody

[PHP] pobierz, plaintext 
  1.  
  2.   <html>
  3.   ...
  4.   <a href="<?php echo Seo::link('info', array('id' => 10,  'title' => 'tytul'))?>">zobacz więcej</a>
  5.   ...
  6.   </html>
[PHP] pobierz, plaintext

Nie lepiej tak ? Tu już nic nie musisz parsować. Bo bez sensu wprowadzać taki dziwny format wejściowy, żeby go potem parsować.
Co do samego pomysłu użycia metody statycznej Seo::link w widoku jak najbardziej prawidłowe podejście.

Ten post edytował ziolo 22.05.2014, 23:23:19
Go to the top of the page
+Quote Post

Posty w temacie
- marcinek37   Klasa odpowiadająca za dynamiczne linki   22.05.2014, 22:12:32
- - skowron-line   No niestety nie ma to nic wspolnego z OOP. Poszuka...   22.05.2014, 22:36:28
- - Crozin   W porównaniu do pierwszego lepszego routera z dowo...   22.05.2014, 22:37:09
- - marcinek37   Router daje informacje, jaki kontroler ma zostać z...   22.05.2014, 22:52:49
- - ziolo   [PHP] pobierz, plaintext  <?php ... ...   22.05.2014, 23:11:28
- - slog   Klasa ma przede wszystkim złą nazwę i może wprowad...   24.05.2014, 10:20:20
- - marcinek37   jaki macie zatem pomysł na stworzenie funkcji, któ...   28.05.2014, 16:46:29
- - !*!   Nie ma na to lekarstwa. Klasy generujące linki z m...   28.05.2014, 17:16:24
- - Crozin   Cytat[...] chcę zrobić prostą malą stronę, której ...   28.05.2014, 17:24:26
- - Pyton_000   Prosze, pierwszy lepszy link z google https://gith...   28.05.2014, 17:51:26
- - marcinek37   @!*! Twoja rozwiązanie też wydaje mi się m...   28.05.2014, 17:51:50
- - !*!   Nikt Ci nie broni tworzenia linku w metodzie a do ...   28.05.2014, 18:04:10
- - marcinek37   @Pyton_000 Będę miał kilka pytań co do Twojego prz...   10.07.2014, 20:30:32
- - Pyton_000   Ta klasa którą Ci podałem będzie dobra. Masz tam p...   10.07.2014, 21:07:30
- - marcinek37   gdy go uruchamiam, pokazuje się jakiś dziwny komun...   11.07.2014, 11:24:16
- - Pyton_000   Ten dziwny komunikat mówi Ci o tym że nie umiesz c...   11.07.2014, 11:29:52

« Następny starszy · Object-oriented programming · Następny nowszy »
 

Reply to this topicStart new topic
2 Użytkowników czyta ten temat (2 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 

Tryb wyświetlania: Przełącz na: Standardowy · Przełącz na: Linearny+ · Drzewo

Śledź ten temat · Wyślij temat na e-mail · Wydrukuj ten temat · Subskrybuj to forum

RSS Aktualny czas: 13.10.2025 - 07:02
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn