1. PHP / Говнокод #22017

    −17

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    51. 51
    52. 52
    53. 53
    54. 54
    55. 55
    56. 56
    57. 57
    58. 58
    59. 59
    60. 60
    61. 61
    62. 62
    63. 63
    64. 64
    65. 65
    66. 66
    67. 67
    68. 68
    69. 69
    70. 70
    71. 71
    72. 72
    73. 73
    74. 74
    75. 75
    76. 76
    77. 77
    78. 78
    79. 79
    <?php
namespace app\controllers;


use app\models\auth\OrdersModel;
use yii\helpers\Url;
use yii\data\Pagination;
use yii\base\Configurable;
use yii\web\Linkable;
use yii\base\Object;
use Yii;
require(__DIR__ . '/../models/OrdersModel.php');

class OrdersController extends \app\common\controllers\CabinetBaseController
{
	
	
    public function actionIndex()
    {
        $obj = new OrdersModel();
        $orders = $obj->get_orders(\Yii::$app->user->identity->id);

        if(isset($_GET) && !empty($_GET))
            $this->ProductsToCart($_GET['products_data']);

      for($i=0;$i<count($orders);$i++){
          if($orders[$i]["status"]==0 )
              $orders[$i]["status"]="<p style='color: #197b30'>Новый</p>";
          else if($orders[$i]["status"]==1)
              $orders[$i]["status"]="<p style='color: #000'>Принят</p>";
          else if($orders[$i]["status"]==2)
              $orders[$i]["status"]="<p style='color: #197b30'>Выполнен</p>";
          else if($orders[$i]["status"]==3)
              $orders[$i]["status"]="<p style='color: #fe0000'>Удален</p>";
          else if($orders[$i]["status"]==4)
              $orders[$i]["status"]="<p style='color: #000'>Скопмлектован</p>";
          else if($orders[$i]["status"]==5)
              $orders[$i]["status"]="<p style='color: #000'>На доставке</p>";
          else if($orders[$i]["status"]==6)
              $orders[$i]["status"]="<p style='color: #fe0000'>Отменен</p>";
          else if($orders[$i]["status"]==7)
              $orders[$i]["status"]= "<p style='color: #cec818'>Ожидание оплаты</p>";

          $orders[$i]['date']=substr($orders[$i]['date'], 0, strrpos($orders[$i]['date'], ' '));
          $orders[$i]['products_info']=$obj->get_orders_products($orders[$i]['id']);

      }
        for($j=0;$j<count($orders);$j++)
            for($k=0;$k<count($orders[$j]['products_info'])-1;$k++)
                $orders[$j]['products_info'][$k]['product_name'].=", ";


        $active_order=$obj->GetActiveOrder(Yii::$app->user->identity->id);



        return $this->render('index', [
            "orders" => $orders,
            "active" =>$active_order
        ]);


    }

    public  function ProductsToCart($products){

        $time=getdate();
        $orders= new OrdersModel();
        $data['products']=$products;

        $data['user_name']=Yii::$app->user->identity->name;
        $data['user_email']=Yii::$app->user->identity->email;
        $data['time']=$time['year'].'-'.$time['mon'].'-'.$time['mday'].' '.$time['hours'].':'.$time['minutes'].':'.$time['seconds'];
        $orders->products_to_cart($data);
        header("Location:". Yii::$app->homeUrl."/cart");

    }

}

    На реальном проекте

    Запостил: C3-PO, 21 Января 2017

    Комментарии (10) RSS

    • ava eskrano 22.01.2017 03:47 # +7

      typical Yii code
      Ответить

    • ava barop 22.01.2017 03:50 # 0

      да и типикал пхп так-то

      В пых приходит пафос: фреймворки, мвц, ооп, неймспейсы.
      Но от идеологии языка-то никуда не уйдешь. Все равно header посреди логики, все равно хеши, все равно блевотная, нелокализуемая, нескинизируемая верстка внутри, все равно сказочная реализация документа с заказом через ", "
      Ответить

      • ava C3-PO 22.01.2017 14:50 # 0

        В моделях там вообще жесть - SQL уязвимости, обращения к http запросам, куски верстки ...
        Ответить

    • ava Fike 22.01.2017 13:38 # 0

      yii2, require
      Ответить

      • ava C3-PO 22.01.2017 14:44 # 0

        Знаешь какой был аргумент? "Главное что работает". Я просто ах...ел
        Ответить

        • ava Fike 22.01.2017 15:06 # 0

          Мне тоже надо один проект перевести с cron-over-wget на что-то нормальное, при этом я для этого проекта вообще ни разу не программист. Мне тоже.
          Ответить

        • ava guest 13.02.2017 23:26 # 0

          Главное что работает - это аргумент для 99% заказчиков, которые не платят за изыски организации кода и уж тем более не смогут этого когда либо оценить... поэтому всё верно!
          (Мне лично фиолетового, работает! остальное отрефактурим как время будет... убиваться из-за вёрстки в контроллере - удел студентов/идеалистов)
          Ответить

    • ava Int 02.02.2017 13:36 # 0

      Есть похожее без всяких yii, живёт года с 2001 наверное. Лишь бы не падало.
      Ответить

    Добавить комментарий