Мой ООП практикум

Решил попрактиковаться в написании сайта на ООП.

Столкнулся с проблемой что в функцию не передаеться результат, полученный из базы.

PHP:

1.  
2. <?php
3.  
4. class News{
5.  
6. public $table = "news";
7. public $on_page = 5;
8.  
9. function get_values()
10. {
11. $this->result = mysql_query ("SELECT * FROM $this->table ORDER BY id desc");
12. return $this->result;
13. }
14.  
15. function get_values_limit()
16. {
17. $this->result = mysql_query ("SELECT * FROM $this->table ORDER BY id desc LIMIT $this->begin, $this->on_page");
18. return $this->result;
19. }
20.  
21. function get_value()
22. {
23. $this->result = mysql_query ("SELECT * FROM $this->table WHERE id = '$this->id' LIMIT 1");
24. return $this->result;
25. }
26.  
27.  
28.  
29.  
30. function generate_values()
31. {
32. while ($info = mysql_fetch_array($this->result))
33. {
34. if (strlen($info[2]) > 300)
35. {
36. $info[2] = substr($info[2], 0, 300);
37. $info[2] .= "... <a href='?task=news&id=$info[0]' class='news_links'>подробнее</a>";
38. }
39. $content = "<table align='center' class='news'>
40.  <tr>
41.    <td class='top_news_corners'>&nbsp;</td>
42.  </tr>
43.  <tr>
44.    <td class='news_text'><b>$info[1]</b><br />$info[2]</td>
45.  </tr>
46.  <tr>
47.    <td class='bottom_news_corners'>&nbsp;</td>
48.  </tr>
49. </table><br />";
50. }
51.  
52. return $content;
53. }
54.  
55.  
56. }
57. ?>

Далее обработчик:

PHP:

1.  
2. <?php
3.  
4. $N = new News();
5.  
6. if(isset($_GET['id']))
7. {
8. $N->id = Validate::protection($_GET['id'], 10, TRUE, TRUE);
9. }
10. else
11. {
12. $N->id = 1;
13. }
14. if(isset($_GET['page']))
15. {
16. $page = Validate::protection($_GET['id'], 10, TRUE, TRUE);
17. }
18. else
19. {
20. $page = 1;
21. }
22.  
23. $N->rows = mysql_num_rows($N->get_values());
24.  
25. if(!isset($_GET['id']))
26. {
27. if ($N->rows != 0)
28. {
29. $N->url = "?task=news";
30.    
31. $N->begin = ($page - 1) * $N->on_page;
32. $N->num_pages = ceil($N->rows/$N->on_page);
33.  
34. $N->get_values_limit();
35.    
36. echo $N->generate_values();
37. echo Validate::paginator($page, $N->num_pages, $N->url);
38. }
39. }
40. else
41. {
42. if (mysql_num_rows($N->get_value()) != 0)
43. {
44. echo $N->generate_value();
45. }
46.  
47. }
48.  
49. ?>

Проблема кроется где то тут:
$N->get_values_limit();
echo $N->generate_values();

Подскажите, что не так.

Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource in Z:\home\ranetky\www\engine\class\news.php on line 32. Если это что то даст)

Обсуждать сам стиль написания кода не желательно))

 

[/quote]

Да нормальный код) Я не стремлюсь же к ооп идеалу, а просто хоть чуть чуть приблизиться) я думаю мне удалось)

 

О. А сможете выразить код который в первом посте в вашем коде?)

 

А как же он должен выглядеть, что бы был ооп?

На даном примере например.

 

Только давайте сначала разберемся с проблемой. Вот переписал немного обработчик:

PHP:

1.  
2. <?php
3. if (!defined("security")) die (/*"Доступ закрыт!"*/);
4.  
5. $N = new News();
6.  
7. if(isset($_GET['id']))
8. {
9. $N->id = Validate::protection($_GET['id'], 10, TRUE, TRUE);
10. }
11. else
12. {
13. $N->id = 1;
14. }
15. if(isset($_GET['page']))
16. {
17. $page = Validate::protection($_GET['id'], 10, TRUE, TRUE);
18. }
19. else
20. {
21. $page = 1;
22. }
23.  
24. if(!isset($_GET['id']))
25. {
26. $news = $N->get_values();
27.  
28. if (mysql_num_rows($news) != 0)
29. {
30. $news = $N->get_values_limit();
31.  
32. $N->url = "?task=news";
33.    
34. $N->begin = ($page - 1) * $N->on_page;
35. $N->num_pages = ceil($N->rows/$N->on_page);
36.  
37. $N->result = $news;
38.  
39. //$N->get_values_limit();
40.    
41. echo $N->generate_values();
42. echo Validate::paginator($page, $N->num_pages, $N->url);
43. }
44. }
45. else
46. {
47. if (mysql_num_rows($N->get_value()) != 0)
48. {
49. echo $N->generate_value();
50. }
51.  
52. }
53.  
54. ?>

Класс:

PHP:

1.  
2. <?php
3. if (!defined("security")) die (/*"Доступ закрыт!"*/);
4.  
5. class News{
6.  
7. public $table = "news";
8. public $on_page = 5;
9.  
10. function get_values()
11. {
12. $result = mysql_query ("SELECT * FROM $this->table ORDER BY id desc");
13. return $result;
14. }
15.  
16. function get_values_limit()
17. {
18. $result = mysql_query ("SELECT * FROM $this->table ORDER BY id desc LIMIT $this->begin, $this->on_page");
19. return $result;
20. }
21.  
22. function get_value()
23. {
24. $tresult = mysql_query ("SELECT * FROM $this->table WHERE id = '$this->id' LIMIT 1");
25. return $result;
26. }
27.  
28.  
29.  
30.  
31. function generate_values()
32. {
33. while ($info = mysql_fetch_array($this->result))
34. {
35. if (strlen($info[2]) > 300)
36. {
37. $info[2] = substr($info[2], 0, 300);
38. $info[2] .= "... <a href='?task=news&id=$info[0]' class='news_links'>подробнее</a>";
39. }
40. $content = "<table align='center' class='news'>
41.  <tr>
42.    <td class='top_news_corners'>&nbsp;</td>
43.  </tr>
44.  <tr>
45.    <td class='news_text'><b>$info[1]</b><br />$info[2]</td>
46.  </tr>
47.  <tr>
48.    <td class='bottom_news_corners'>&nbsp;</td>
49.  </tr>
50. </table><br />";
51. }
52.  
53. return $content;
54. }
55.  
56. function generate_value()
57. {
58. while ($info = mysql_fetch_array($this->result))
59. {
60. $content = "<table align='center' class='news'>
61.  <tr>
62.    <td class='top_news_corners'>&nbsp;</td>
63.  </tr>
64.  <tr>
65.    <td class='news_text'><b>$info[1]</b><br />$info[2]</td>
66.  </tr>
67.  <tr>
68.    <td class='bottom_news_corners'>&nbsp;</td>
69.  </tr>
70. </table>";
71. }
72. return $content;
73. }
74.  
75. }
76. ?>

Проблема осталась..

 

Ну тогда мне посоветуйте)

И с проблемкой помогите разобраться)