Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
Reply to this topicStart new topic
> [www] http://frogss.com
zaba12
post
Post #1





Grupa: Zarejestrowani
Postów: 223
Pomógł: 0
Dołączył: 29.08.2003
Skąd: Gliwice

Ostrzeżenie: (10%)
X----


Prosze o ocenę http://frogss.com
Go to the top of the page
+Quote Post
Cysiaczek
post
Post #2





Grupa: Moderatorzy
Postów: 4 465
Pomógł: 137
Dołączył: 26.03.2004
Skąd: Gorzów Wlkp.




1. Pomieszany kod logiki z prezentacyjnym.
2. http://img110.imageshack.us/my.php?image=cmsik1kd1.png
3. Zaraz po instalacji: http://img136.imageshack.us/my.php?image=cmsik2gq6.png

Dalej nawet nie sprawdzam. Dlaczego? Dlatego, że przeciętny użytkownik nie chce grzebać w kodzie i ściągnie joomle.

Ogólnie kod oceniłbym na jakieś 2/10
Proponuję, żebyć popracowął nad tym wszystkim i przemyślał raz jeszcze całość.
Mimo, ze projekt mi sie nię podoba, to życzę jak najlepiej i pozdrawiam.
Go to the top of the page
+Quote Post
zaba12
post
Post #3





Grupa: Zarejestrowani
Postów: 223
Pomógł: 0
Dołączył: 29.08.2003
Skąd: Gliwice

Ostrzeżenie: (10%)
X----


po pierwsze sciagles stara wersje aktualna to 0.6 wersja a ty scaigles 0.5 chyba na to wyglada..
Go to the top of the page
+Quote Post
NetJaro
post
Post #4





Grupa: Zarejestrowani
Postów: 475
Pomógł: 0
Dołączył: 1.04.2005
Skąd: Warszawa

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


index.php
  1. <?php 
  2. //Engine strony
  3. include ("config.php");
  4. include ("module/news.php");
  5. //Koniec Engine strony
  6.  
  7. function pokaz() {
  8. global $cmd,$id,$news_limit,$mode,$act,$lan;
  9. if(isset($id)) {
  10.  
  11. czytaj('');
  12. }
  13. elseif ($mode == ''){
  14. news($news_limit);
  15. } elseif ($mode == 'add_news'){
  16. news_t_open();
  17. echo '<table style="font-size: 10px; color: black;" cellspacing="0" cellpadding="0" width="100%">
  18. <tr>
  19. <td align="left" style="padding-left: 3px;">
  20. <b>Administracja Newsami</b> :: <font color="silver"><<a href="#">'.$lan['news_add_news'].'</a>></font>
  21. </td>
  22. </tr>
  23. </table>';
  24. news_t_close();
  25. news_tresc_open();
  26.  dodaj_newsa();
  27. news_tresc_close();
  28. }
  29. }
  30.  
  31.  
  32. //Ładowanie Skóry
  33. include ("theme/$theme/index.php");
  34. ?>


1. Gdzie jest system szablonów?!
2. Optymalizacji więcej (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) " używa się tylko w wyjątkowych okolicznościach (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

module/artykuly.php
  1. <?php
  2. function dzialy() {
  3. global $PHP_SELF,$top,$lan;
  4. $query = "SELECT * FROM artykuly_dzialy ORDER BY tytul1";
  5. $result = mysql_query ($query) or die(mysql_error());
  6. while($rowek = mysql_fetch_array($result)) {
  7.  
  8. $id = $rowek['id'];
  9. $tytul1 = $rowek['tytul1'];
  10.  
  11. $tytul1 = stripslashes($tytul1);
  12. echo '<table width="98%" align="center">
  13. <tr>
  14. <td><table width="100%" border="0" cellspacing="0" cellpadding="0">
  15. <tr><td height="15"><table border="0" cellspacing="0" cellpadding="0"><tr align="left"><td width="20"></td><td>';
  16. $ile = mysql_num_rows(mysql_query("SELECT * FROM artykuly WHERE dzial='".$tytul1."' AND status=1"));
  17. if($ile<>0) {
  18. lista($dzial);
  19. }
  20. echo '<a href="artykuly.php?dzial='.$tytul1.'"><b>&rsaquo;</b> '.$tytul1.'</a> '.$lan['article_art'].':
  21. <b>('.$ile.')</b> </td></tr></table></td></tr>
  22. </table></td>
  23. </table>';
  24. }
  25. echo '<br /><center><a href="artykuly.php?mode=add_article">'.$lan['article_add'].'</a></center>';
  26. }
  27. //...
  28. ?>

3. Gdzie jest cache zapytań do SQL'a.. przydałaby się również jakaś dobra klasa do obsługi bazy danych..
4. global'ów juz się chyba nie używa..

Dużo do zmiany.. Ale może być dobrze (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
Go to the top of the page
+Quote Post
acztery
post
Post #5





Grupa: Zarejestrowani
Postów: 945
Pomógł: 7
Dołączył: 15.03.2005
Skąd: katowice

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


jak poprzednicy globali juz sie nie stosuje. co do klasy do bazy danych polecam DB_DataObject super sprawa.

kod wyglada jak śmietnik. Nie ma smartów czy innego systemu szablonów. Każda zmiana w kodzie może byc męczoca i czasochlonna.

-3/10

Ten post edytował acztery 3.09.2006, 22:47:17
Go to the top of the page
+Quote Post
ano
post
Post #6





Grupa: Zarejestrowani
Postów: 435
Pomógł: 40
Dołączył: 16.02.2003
Skąd: Wrocław

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


Po przeczytaniu:
Cytat
Pare osób uważa ze ten system to kopia znanego wam systemu Jportal lecz tak nie jest to nie jest zadna kopia jak ktos tak uwaza to niech sie powiesi nie mamy zamiaru słuchać jak ktoś będzie nam wmawiał ze to jest kopia (IMG:http://forum.php.pl/style_emoticons/default/exclamation.gif)
[...]
Gotowa do sciągniecia z downloadu zapraszam
Sciagnij - Download System ma 2 jezyki Ang I Pol mozna nim zrobic all z poziomu administracji!.

Odechciało mi się ściągać tego "Najlepszego systemu CMS".
Go to the top of the page
+Quote Post
bim2
post
Post #7





Grupa: Zarejestrowani
Postów: 1 873
Pomógł: 152
Dołączył: 9.04.2006
Skąd: Berlin

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


Szablony z jp. Normalnie wychował się na JP :|
Go to the top of the page
+Quote Post

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

 



RSS Aktualny czas: 25.08.2025 - 11:22