За последние 24 часа нас посетили 44469 программистов и 2997 роботов. Сейчас ищут 1317 программистов ...

Мой первый вывод новостей

Тема в разделе "PHP для новичков", создана пользователем Dimon2x, 9 апр 2017.

Метки:
  1. Dimon2x

    Dimon2x Старожил

    С нами с:
    26 фев 2012
    Сообщения:
    2.211
    Симпатии:
    186
    Есть такое задание:

    1. Создайте класс новостей для сайта.

    2. Реализуйте страницу, на которой вы будете эти новости выводить используя только методы класса (к свойствам обращаться нельзя).

    Я это задание сделал, но думаю, что мой код очень ужасный и не правильный, что я не так сделал?

    А может надо было поместить все статьи в один json файл и вывести разметку циклом?

    Не говорите мне пока про базу данных, сначала я хочу потренироваться на jsone.

    index.php

    PHP:
    1. <?php
    2. header('Content-Type: text/html; charset=utf-8');
    4. class News{
    5.     public function __construct($file_get_contents){
    6.         $this->file_get_contents = file_get_contents($file_get_contents);
    7.     }
    8.     public function newsStr(){
    9.         return $this->file_get_contents;
    10.     }
    11.     public function newsArray(){
    12.         return json_decode($this->newsStr(), true);
    13.     }
    14.     public function newsCells(){
    15.         return $this->newsArray()['news'];
    16.     }
    17.     public function newsTitle(){
    18.         return $this->newsCells()['title'];
    19.     }
    20.     public function newsDate(){
    21.         return $this->newsCells()['date'];
    22.     }
    23.     public function newsAuthor(){
    24.         return $this->newsCells()['author'];
    25.     }
    26.     public function newsDescription(){
    27.         return $this->newsCells()['description'];
    28.     }
    29. }
    30. ?>
    31. <!DOCTYPE html>
    32. <html>
    33.     <head>
    34.         <title>Новости сайта</title>
    35.         <link rel="stylesheet" href="./style.css">
    36.     </head>
    37. <body>
    38.     <div class="content">
    39.         <h2>Последние новости</h2>
    40.         <div class="news">
    41.             <?php $news1 = new News('news1.json');?>
    42.             <div class="title"><?php echo $news1->newsTitle()?></div>
    43.             <div class="description"><?php echo $news1->newsDescription()?></div>
    44.             <div class="date inline-block"><span>Дата:</span> <?php echo $news1->newsDate()?></div>
    45.             <span>|</span>
    46.             <div class="author inline-block"><span>Добавил</span> <?php echo $news1->newsAuthor()?></div>
    47.         </div>
    48.        
    49.         <div class="news">
    50.             <?php $news1 = new News('news2.json');?>
    51.             <div class="title"><?php echo $news1->newsTitle()?></div>
    52.             <div class="description"><?php echo $news1->newsDescription()?></div>
    53.             <div class="date inline-block"><span>Дата:</span> <?php echo $news1->newsDate()?></div>
    54.             <span>|</span>
    55.             <div class="author inline-block"><span>Добавил</span> <?php echo $news1->newsAuthor()?></div>
    56.         </div>
    57.        
    58.         <div class="news">
    59.             <?php $news1 = new News('news3.json');?>
    60.             <div class="title"><?php echo $news1->newsTitle()?></div>
    61.             <div class="description"><?php echo $news1->newsDescription()?></div>
    62.             <div class="date inline-block"><span>Дата:</span> <?php echo $news1->newsDate()?></div>
    63.             <span>|</span>
    64.             <div class="author inline-block"><span>Добавил</span> <?php echo $news1->newsAuthor()?></div>
    65.         </div>
    66.     </div>
    67. </body>
    68. </html>
    css

    Код (CSS):
    1. content{
    2.     border: 1px solid;
    3.     width: 600px;
    4.     margin: 0 auto;
    5. }
    6. h2{text-align: center;}
    7. .news{
    8.     border: 3px solid orange;
    9.     padding: 10px;
    10. }
    11. .title, .description{
    12.     padding-bottom: 10px;
    13. }
    14. .inline-block{
    15.     display: inline-block;
    16.     vertical-align: top;
    17.     font-weight: bold;
    18. }
    news1.json

    Код (Text):
    1. {
    2.     "news": {
    3.         "title": "Изобрён вечный двигатель",
    4.         "date": "05.04.2017",
    5.         "author": "Admin",
    6.         "description": "Совсем недавно, придумали вечный двигатель, который не требует обслуживания, но в эксплуатацию его запретили, так как хотят, что бы люди постоянно  мучились в рабстве"
    7.     }
    8. }
     
    #1 Dimon2x, 9 апр 2017
  2. Zuldek

    Zuldek Старожил

    С нами с:
    13 май 2014
    Сообщения:
    2.381
    Симпатии:
    344
    Адрес:
    Лондон, Тисовая улица, дом 4, чулан под лестницей
    Вердикт:
    Задание вы действительно сделали но забив гвоздь микроскопом.
    Вы обернули сущность в класс совершенно не поняв зачем это нужно делать и почему с сущностями при реализации ооп подхода так нужно работать.
    И далее, как следствие, не поняли как именно это можно реализовать.

    Что необходимо сделать чтобы понять:
    попробуйте вдумчиво изучить курс от Зандстры "Объекты, практики, методики программирования на PHP...". Как-то так зовется книга.
     
    #2 Zuldek, 9 апр 2017
    Последнее редактирование: 9 апр 2017
    alexblack нравится это.
  3. Dimon2x

    Dimon2x Старожил

    С нами с:
    26 фев 2012
    Сообщения:
    2.211
    Симпатии:
    186
    ооп в основном применяют из-за наследования
     
    #3 Dimon2x, 9 апр 2017
  4. alexblack

    alexblack Старожил

    С нами с:
    20 янв 2016
    Сообщения:
    640
    Симпатии:
    381
    Крайне неверное представление об ООП
     
    #4 alexblack, 9 апр 2017
  5. Zuldek

    Zuldek Старожил

    С нами с:
    13 май 2014
    Сообщения:
    2.381
    Симпатии:
    344
    Адрес:
    Лондон, Тисовая улица, дом 4, чулан под лестницей
    Дмитрий, изучите то, что рекомендовано, вам это поможет, можете верить.
     
    #5 Zuldek, 9 апр 2017
  6. Fell-x27

    Fell-x27 Суперстар
    Команда форума Модератор

    С нами с:
    25 июл 2013
    Сообщения:
    12.156
    Симпатии:
    1.771
    Адрес:
    :сердА
    ООП в основном применяют из-за ООП. Перенес тему из ветки "для профи" в ветку "для новичков". Нет, это не зазорно, это отличная ветка, где подскажут, научат и не будут особо придираться :) Главное - прислушиваться.
     
    #6 Fell-x27, 9 апр 2017
  7. igordata

    igordata Суперстар
    Команда форума Модератор

    С нами с:
    18 мар 2010
    Сообщения:
    32.408
    Симпатии:
    1.768
    меня на реддите с моим ооп обругали =) но я согласен =(
     
    #7 igordata, 9 апр 2017
  8. mkramer

    mkramer Суперстар
    Команда форума Модератор

    С нами с:
    20 июн 2012
    Сообщения:
    8.600
    Симпатии:
    1.764
    А это фетиш такой - по одной новости в файл записывать, и потом по одному файлу в исходнике выводить? В json-е можно и массивы хранить. А ещё в php есть циклы. Только не говорите, что вам классы раньше циклов на вашем курсе объясняют.
     
    #8 mkramer, 9 апр 2017
  9. MouseZver

    MouseZver Суперстар

    С нами с:
    1 апр 2013
    Сообщения:
    7.820
    Симпатии:
    1.333
    Адрес:
    Лень
    json_decode умеет переводить json массив в объект и дальше работать как используя только методы класса
    --- Добавлено ---
    PHP:
    1. $json = '{"a":{"name":"WW1"},"b":{"name":"WW2"},"c":{"name":"WW3"},"d":{"name":"WW4"},"e":{"name":"WW5"}}';
    2.  
    3. echo implode ( '<br>', array_map ( function ( $OBJ ) { return $OBJ -> name; }, json_decode ( $json ) ) );
    --- Добавлено ---
    или блин я даун, не знаю что такое методы в ООП
    --- Добавлено ---
    Код (Text):
    1. // методы:
    2. function Name() {
    3. echo "<h3>John</h3>";
    4. }
     
    #9 MouseZver, 9 апр 2017
  10. Dimon2x

    Dimon2x Старожил

    С нами с:
    26 фев 2012
    Сообщения:
    2.211
    Симпатии:
    186
    Я вот так сделал, нормально ли, как улучшить?

    PHP:
    1. <?php
    2. header('Content-Type: text/html; charset=utf-8');
    4.  
    5. class News{
    6.     private $title;
    7.     private $date;
    8.     private $author;
    9.     private $description;
    10.    
    11.     public function setTitle($title){
    12.         $this->title = $title;
    13.     }
    14.    
    15.     public function getTitle(){
    16.         return $this->title;
    17.     }
    18.    
    19.     public function setDate($date){
    20.         $this->date = $date;
    21.     }
    22.    
    23.     public function getDate(){
    24.         return $this->date;
    25.     }
    26.    
    27.     public function setAuthor($author){
    28.         $this->author = $author;
    29.     }
    30.    
    31.     public function getAuthor(){
    32.         return $this->author;
    33.     }
    34.    
    35.     public function setDescription($description){
    36.         $this->description = $description;
    37.     }
    38.    
    39.     public function getDescription(){
    40.         return $this->description;
    41.     }
    42.  
    43.     public function getJson($fileName){
    44.         $json = file_get_contents($fileName);
    45.         $data = json_decode($json, true);
    46.         $news = new News();
    47.         $news->setTitle($data['news']['title']);
    48.         $news->setDate($data['news']['date']);
    49.         $news->setAuthor($data['news']['author']);
    50.         $news->setDescription($data['news']['description']);
    51.         return $news;
    52.     }
    53. }
    54.  
    55. $catalog = scandir(__DIR__ . '/article');
    56. ?>
    57.  
    58. <!DOCTYPE html>
    59. <html>
    60.     <head>
    61.         <title>Новости сайта</title>
    62.         <link rel="stylesheet" href="./style.css">
    63.     </head>
    64. <body>
    65.     <div class="content">
    66.         <h2>Последние новости</h2>
    67.         <?php
    68.             $ob1 = new News();
    69.             foreach($catalog as $file){
    70.                 if(pathinfo($file, PATHINFO_EXTENSION) == 'json'){
    71.                     $json = $ob1->getJson('article/' .$file);
    72.                    
    73.                     echo '<div class="news">';
    74.                         echo '<div class="title">';
    75.                             echo $json->getTitle();
    76.                         echo '</div>';
    77.                        
    78.                         echo '<div class="description">';
    79.                             echo $json->getDescription();
    80.                         echo '</div>';
    81.                        
    82.                         echo '<div class="inline-block"><span class="date">Дата:</span>';
    83.                             echo '<span>' . $json->getDate() .'</span>';
    84.                         echo '</div>';
    85.                         echo '<span> | </span>';
    86.                         echo '<div class="inline-block"><span class="date">Добавил:</span>';
    87.                             echo $json->getAuthor();
    88.                         echo '</div>';
    89.                        
    90.                     echo '</div>';
    91.                    
    92.                 }
    93.             }  
    94.         ?>
    95.     </div>
    96. </body>
    97. </html>
     
    #10 Dimon2x, 9 апр 2017
(Вы должны войти или зарегистрироваться, чтобы разместить сообщение.)