Не могу сделать вывод из JSON запроса на php?

Question

[Костя Тестер]

В общем всё просто но в тоже время не могу сделать вывод вод кусок json результата


Подключился соответственно к json
Для того что бы вывести в php делаю через foreach

<?php foreach($data as $a): ?>
<?= $a->number ?>
<?php endforeach; ?>

Но не получается вывести строку number
Помогите может кто подскажет как правильно, может я что-то не то делаю? Заранее благодарен!

Comments

[Александр]

Можете показать весь JSON?

P.s. Попробуйте без foreach

[Костя Тестер]

Александр, обновил скриншот

[Александр]

Костя Тестер, пробовали без foreach?

[Костя Тестер]

Александр, пробовал не работает.

[DevMan]

Костя Тестер, не надо скриншотов - вставьте джейсон строкой.

[Костя Тестер]

{"except_days":"","arrival_date":null,"from":null,"uid":"086B_0_2","title":"Минск — Киев","departure_date":null,"start_time":"22:40","number":"086Б","short_title":"Минск — Киев","days":"ежедневно, кроме 18.07, 27.09, 10.10","to":null,"carrier":{"code":114,"offices":[],"codes":{"icao":null,"sirena":null,"iata":null},"title":"Беларуская Чыгунка"},"transport_type":"train","stops":[{"arrival":null,"departure":"2018-07-28 22:40:00","terminal":null,"platform":"","station":{"code":"s9613989","station_type":"station","title":"Минск-Пасс.","popular_title":"","short_title":"","codes":{"yandex":"s9613989","esr":"140210"},"transport_type":"train","station_type_name":"станция","type":"station"},"stop_time":null,"duration":0.0},{"arrival":"2018-07-28 23:57:00","departure":"2018-07-29 00:01:00","terminal":null,"platform":"","station":{"code":"s9614258","station_type":"station","title":"Осиповичи-1","popular_title":"","short_title":"","codes":{"yandex":"s9614258","esr":"146407"},"transport_type":"train","station_type_name":"станция","type":"station"},"stop_time":240,"duration":4860.0},{"arrival":"2018-07-29 00:30:00","departure":"2018-07-29 00:33:00","terminal":null,"platform":"","station":{"code":"s9614101","station_type":"station","title":"Бобруйск","popular_title":"","short_title":"","codes":{"yandex":"s9614101","esr":"147008"},"transport_type":"train","station_type_name":"станция","type":"station"},"stop_time":180,"duration":6780.0},{"arrival":"2018-07-29 01:25:00","departure":"2018-07-29 01:28:00","terminal":null,"platform":"","station":{"code":"s9614068","station_type":"station","title":"Жлобин-Пасс.","popular_title":"","short_title":"","codes":{"yandex":"s9614068","esr":"155004"},"transport_type":"train","station_type_name":"станция","type":"station"},"stop_time":180,"duration":10080.0},{"arrival":"2018-07-29 02:45:00","departure":"2018-07-29 03:09:00","terminal":null,"platform":"","station":{"code":"s9614088","station_type":"station","title":"Гомель-Пасс.","popular_title":"","short_title":"","codes":{"yandex":"s9614088","esr":"150000"},"transport_type":"train","station_type_name":"станция","type":"station"},"stop_time":1440,"duration":16140.0},{"arrival":"2018-07-29 03:45:00","departure":"2018-07-29 04:24:00","terminal":null,"platform":"","station":{"code":"s9614894","station_type":"station","title":"Терюха","popular_title":null,"short_title":null,"codes":{"yandex":"s9614894","esr":"150301"},"transport_type":"train","station_type_name":"станция","type":"station"},"stop_time":2340,"duration":20640.0},{"arrival":"2018-07-29 08:56:00","departure":null,"terminal":null,"platform":"","station":{"code":"s9614928","station_type":"station","title":"Киев-Пасс.","popular_title":"","short_title":"","codes":{"yandex":"s9614928","esr":"320308"},"transport_type":"train","station_type_name":"станция","type":"station"},"stop_time":null,"duration":36960.0}],"vehicle":null,"start_date":"2018-07-28","transport_subtype":{"color":null,"code":null,"title":null},"express_type":null}

Answer 1

[DevMan]

так вам итерировать нечего - у вас всего один объект, а не массив объектов.
форич на одном объекте прост перебирает его свойства и $a->number приведет к фаталу (ибо $а не является объектом).

в вашем случае нужно весь код заменить на просто <?= $data->number ?>
для наглядности - https://ideone.com/VM6kdT

Comments

[Костя Тестер]

Спасибо большое!

[DevMan]

Костя Тестер, пожалуйста)

Answer 2

[xmoonlight]

<? var_dump($data->number); ?>Так работает.
Значит и так тоже до первого объекта:

<?php foreach($data as $a): ?>
<?= $a ?>
<?php endforeach; ?>

Поэтому упрощённо - такая версия (все объекты делаем json-ами):

<?php foreach($data as $k=>$a): ?>
<?=(is_string($a))?"$k=>$a<br>":"$k=>".json_encode($a)."<br>" ?>
<?php endforeach; ?>

По-скольку json - это структура, то для извлечения произвольного поля нужно написать рекурсивную функцию, которая будет перебирать всё "дерево" и находить нужное значение. Или явно указывать нужное через цепочку "->" корректно задавая путь по "дереву" к нужному узлу.

Comments

[DevMan]

шорттаг не действует на эхо (если пых не древний).

[xmoonlight]

DevMan, весело... не знал я... у меня они всегда включены просто...

[DevMan]

xmoonlight, у меня они всегда выключены. но

5.4.0 Тег <?= доступен всегда, вне зависимости от настройки short_open_tag

[xmoonlight]

DevMan, ок, запомню.

[Кирилл Несмеянов]

xmoonlight, на будущее, рекомендую ознакомиться со стандартом PSR-1 и PSR-2. Там чёрным по белому расписано что делать можно, а что нельзя, включая причины того, почему что-то делать нельзя, а что-то можно.

[xmoonlight]

Кирилл Несмеянов, Спасибо, почитаю на досуге.

[DevMan]

xmoonlight, и нафига такие сложности?

Кирилл Несмеянов, пср - не гранит. хотя выбранный код-стайл и напоминает пхп 3-4.

[xmoonlight]

DevMan, какие именно сложности? я просто пояснил варианты, чтобы достать значение...

[DevMan]

xmoonlight,

для извлечения произвольного поля нужно написать рекурсивную функцию, которая будет перебирать всё "дерево" и находить нужное значение.

вот это - перебор.

[xmoonlight]

DevMan, ну, да, согласен... за то - полное покрытие возможных требований)

[Кирилл Несмеянов]

DevMan, ну в рекомендации ведь есть упоминание про шорт теги? Ну вот. Так почему бы сразу не прочитать вообще обо всём?) Разве плохо?

[DevMan]

Кирилл Несмеянов, не плохо конечно же, я и сам короткие теги не люблю и не использую. но:
1. PSR - это все же рекомендации
2. PSR запрещает короткие теги, но разрешает <?=

[Кирилл Несмеянов]

DevMan,
1. Вы забыли про то, что это "общепринятые рекомендации", так что учитывая это можно сказать, что это уже стандарт. Код не по PSR в 2018ом году воспринимается как дилетантский, разве нет?
2. И это разве плохо? "<?=" работает на любой конфигурации сервера, на неё не влияет наличие или отсутствие каких-либо флагов.

[xmoonlight]

Кирилл Несмеянов, Код в первую очередь должен работать/исполняться быстро, безопасно и как требует бизнес-логика проекта/сервиса.
А как он будет восприниматься - зависит уже от конкретного специалиста.
Для тех кто только начал кодить и особо ещё не научился - важны понты в виде соблюдения PSR.

[DevMan]

Кирилл Несмеянов,
1. нет: во многих компаниях/командах принят свой устоявшийся стайл, появившийся задолго до пср.
да и средства типа cs-fixer делают не нужным принуждение людей менять привычный им стиль.

2. не знаю или вы помните сколько было воплей, когда в пыхе сделали short_open_tag по дефолту выключенными, что, естественно привело к неработоспособности <?=. после чего в 5.4 сделали этот тег независимым от настроек.
лично я не уверен, что такое не повториться в будущем. и у меня нет никакого желания рисковать этим из-за сомнительной экономии в наборе нескольких символов (который, к тому же, можно вообще автоматизировать).

[Кирилл Несмеянов]

DevMan, ну хз, я пока не встречал того места, где было что-то отличное от PSR (естественно, уже после того, как он появился).

не знаю или вы помните сколько было воплей, когда в пыхе сделали short_open_tag по дефолту выключенными

Хм, я не помню этой истории. В целом, ваше рассуждение вполне логичное, что лучше перебздеть, нежели... С другой стороны шорттеги попортили крови довольно много кому, я до сих пор с содроганием впоминаю как дебажил самопальный sitemap и пытался понять откуда ошибки. Короче, шорретг "<?" правильно запретили, это логично, т.к. ведёт к куче проблем. А "<?=" не запретят в ближайшем будущем или вообще никогда, т.к. проблем связанных с ним не было и никому оно не мешает.

Т.е. я к тому, что из пыха убивают только проблемные конструкции, регистер глобалс, шорртеги, декларация через script и asp-like и проч. Но беспокоиться за "<?=" не имеет смысла, т.к. она не взодит в группу "ненужного функционала".

[Кирилл Несмеянов]

xmoonlight,

Код в первую очередь должен работать/исполняться быстро, безопасно и как требует бизнес-логика проекта/сервиса.

Требование "просто работает" применимо только к прототипам и одноразовым скриптикам. Такие характеристики, как:
1) Читаемость
2) Поддерживаем
3) Правильно скомпанован (я хз как сформулировать тезисы об инкапсуляции побочных эффектов и SOLID в целом), что повышает тестируемость и расширяемость.

Имеют первостепенную важность, если код является частью единой системы. В этом случае баги и недоработки фиксятся за пару минут, а новая задача, поверх старой не потребует лишних телодвижений. Я вообще не припомню случаев, где требовалось бы сделать N по тех.заданию и забить. Каждый проект, если он живёт, то постепенно дорабатывается, улучшается, а бизнес-логика меняется. Задача программиста - это найти границу между минимальной стоимостью разработки и поддержки. А в некоторых случаях (например, в Open Source) задача стоит только одна - максимальное качество кода.

А как он будет восприниматься - зависит уже от конкретного специалиста.

Согласен. Но делать его в стиле барокко и говнокодить, оправдываясь этим - не стоит.

Для тех кто только начал кодить и особо ещё не научился - важны понты в виде соблюдения PSR.

Да что уж там, давайте тогда и на отступы забъём. Кому нужны переносы строк и пробельчики? Это же просто понты!

Не, я согласен, что резон в этих словах есть. Для начинающих не важно на какой строке стоит фигурная скобка, но минимальные правила (только UTF-8 без BOM, опускание закрывающего `?>` и проч.) должны выполняться всегда. Это не прихоть, это решение огромного вороха проблем. Да даже далеко ходить не стоит: Почему пишет, что хедер уже послан? Этого бы не было, если бы человек удосужился выполнять простые правила PSR.

[DevMan]

Кирилл Несмеянов,

ну хз, я пока не встречал того места, где было что-то отличное от PSR

я встречал и встречаю. PSR-1/2 были приняты в 2012 году, на пыхе писали задолго до этого и продолжают поддерживать/развивать написанное тогда.

Но беспокоиться за "<?=" не имеет смысла, т.к. она не взодит в группу "ненужного функционала"

в пыхе ворох проблем, не решаемых годами из-за стремления сохранить обратную совместимость. и я совсем не удивлюсь, даже обрадуюсь, если/когда кор-тим решит с этим покончить.

а пока мне не сложно настроить автозамену <?= -> <?php echo в иде или на уровне оси и спать спокойно.

[Кирилл Несмеянов]

в пыхе ворох проблем, не решаемых годами из-за стремления сохранить обратную совместимость.

Не только. Ещё трудозатраты. Некоторый функционал и исправления не был принят (т.е. был отложен) просто потому, что очень много чего переписывать надо. Именно по-этому так форсится работа Стогова. Он взял и пофиксил один из таких "проблематичных" моментов, который очень сложно было исправить и реализовал функционал для будущих плюшек. Все обновления, начиная с 7.0+ - это лишь рефакторинг ядра и ничего больше, который позволил бы в будущем реализовать огромную кучу функционала и исправить старый.

[xmoonlight]

Кирилл Несмеянов,

Короче, шорретг "<?" правильно запретили, это логично, т.к. ведёт к куче проблем

Хотелось бы про них узнать, т.к. сам я на практике ни разу не столкнулся...

Да что уж там, давайте тогда и на отступы забъём. Кому нужны переносы строк и пробельчики? Это же просто понты!

Я пишу код в notepad++ и пользуюсь несколькими плагинами. Надо выложить код в репу - я пройдусь плагином Indent By Fold. Мне пока хватает такого простого IDE с головой и не вызывает затруднений или неудобств при разработке. Пока не вижу для себя смысла переходить на более "тяжёлые" IDE.

[Кирилл Несмеянов]

Хотелось бы про них узнать, т.к. сам я на практике ни разу не столкнулся...

Ну попробуйте создать простой XML файл с расширением php и подставить туда переменные.

Я пишу код в notepad++ и пользуюсь несколькими плагинами. Надо выложить код в репу - я пройдусь плагином Indent By Fold. Мне пока хватает такого простого IDE с головой и не вызывает затруднений или неудобств при разработке. Пока не вижу для себя смысла переходить на более "тяжёлые" IDE.

Ну хватает, значит хватает. Значит вы настолько опытный, что сходу можете найти проблему в простом и даже рабочем примере:

function starts_with(string $haystack, string $needle): bool
{
    return $needle !== '' && \substr($haystack, 0, \strlen($needle)) === $needle;
}

А если использовать IDE, то она очевидна

Лично я не могу держать в памяти весь код, что он возвращает, какие исключения кидает и прочее.

[xmoonlight]

Кирилл Несмеянов,

Ну попробуйте создать простой XML файл с расширением php и подставить туда переменные.

Не понял... Зачем для XML делать расширение PHP?!

Так вот делаю для sitemap (sitemap.xml):

<?
header("Content-type: text/xml");
$sitename=$_SERVER['HTTP_HOST'];
echo '<?xml version="1.0" encoding="UTF-8" ?>';
ob_start();
?><urlset>
 <url>
  <loc>http://<?=$sitename?>/goods/</loc>
  <lastmod>2016-04-20T02:53:35Z</lastmod>
  <priority>1</priority>
 </url>
..............
</urlset>

и в .htaccess:

<Files sitemap.xml>
AddType application/x-httpd-php .xml
</Files>

или не то имелось ввиду?

[Кирилл Несмеянов]

Не понял... Зачем для XML делать расширение PHP?!

Это правда. Не нужно, если есть нормальный роутинг, то проще написать более адекватный вариант, вроде:
$router->get('/sitemap.xml', 'Controller@action');

Но бывают случаи же и другие.

В любом случае у вас тут не просто так echo вперемешку с нормальным выводом. Предлагаю чуть отрефакторить и разделить логику и представление.

Возьмём обычный twig (соли, перца по вкусу).

<?xml version="1.0" encoding="UTF-8" ?>
<urlset>
 <url>
  <loc>http://{{ site.name }}/goods/</loc>
  <lastmod>{{ site.modified | rfc3339}}</lastmod>
  <priority>1</priority>
 </url>
..............
</urlset>

// ...
public function render(): string
{
    return $this->twig->render(..., ['site' => $this->site]);
}


// Вывод
\header('Content-type: text/xml');

$site = new Site('Name', new \DateTime());

echo (new Sitemap($site, ...))->render();

Теперь код стал чище, он стал тестируемым за счёт композиции объектов. А если заменить Site на интерфейс, то получится реализация инверсии контроля, т.е. можно будет добавить реализацию чтения настроек, например из БД. Всё круто, но код, внезапно, просто не будет работать там, где нарушена рекомендация в отключении шорттегов.

В любом случае вам очевидно, где проблемы шорттегов, иначе бы не костылили заголовок с помощью echo.

[xmoonlight]

Кирилл Несмеянов, ну т.е. проблема именно вначале заголовка <?xml, если файл обрабатывается php-обработчиком. Других и не встречалось. Правильно я понял?

[Кирилл Несмеянов]

xmoonlight, там ещё что-то было, связанное с XHTML 1.1. До появления HTML5 - это был самый популярный вариант. Ну и SVG конечно же.

А так да, убрали именно потому, что он конфликтует с другими форматами. И не только его убрали, можно было писать ещё "<% ... %>" и "<script type="application/php"> ... </script>".

[xmoonlight]

Кирилл Несмеянов, я стараюсь избавляться от объектов (и уж тем более от шаблонизаторов) по-максимуму насколько это возможно и это ощутимо сказывается на производительности исполнения кода.
В подобных случаях, как этот, preg_replace() c регексами - наше всё)

[xmoonlight]

Кирилл Несмеянов,

А так да, убрали именно потому, что он конфликтует с другими форматами.

Могли бы просто оступ (пробельный символ) учесть и проблема бы решилась.

[Кирилл Несмеянов]

xmoonlight,
1) Шаблонизаторы никак не влияют на производительность, т.к. в конце концов они тоже собираются в обычный php
2) Объекты быстрее (но массивы чуть быстрее инициализируются) и легче всего прочего, они лучше (эффективнее) работают с GC. Помимо этого они позволяют реализовывать автолоадинг, что нивелирует очевидное повышенное потребление памяти при жадной загрузке.
3) А регулярки зачем? Эти проблемы обычный phpcfb решают, ну или какие-нибудь CI, вроде styleci.

[Кирилл Несмеянов]

Могли бы просто оступ (пробельный символ) учесть и проблема бы решилась.

xmoonlight, могли. Но в те времена вроде не использовался bison (или yacc, что там сейчас?) для грамматики, а значит синтаксис патчить было сложнее. Хотя я могу ошибаться.

[xmoonlight]

Кирилл Несмеянов,

1) Шаблонизаторы никак не влияют на производительность, т.к. в конце концов они тоже собираются в обычный php
2) Объекты быстрее и легче всего прочего, помимо этого они лучше работают с GС
3) А регулярки зачем? Эти проблемы обычный phpcfb решают, ну или какие-нибудь CI, вроде styleci.

Регулярка вместо шаблонизатора.
А по остальным пунктам - я замерял время и вынужден не согласиться... Просто замерьте время:
1. на создание объекта (и соотв. выделяется доп. память под это) с методом и просто метод в виде функции (и сравните).
2. на рендер: регуляркой и шаблонизатором (и сравните).

[DevMan]

xmoonlight,

Хотелось бы про них узнать, т.к. сам я на практике ни разу не столкнулся...

да запросто: приносит мне соискатель свой код, а он у меня не запускается.

Кирилл Несмеянов,

Не только. Ещё трудозатраты.

это как-то отменяет написанное мной?

[xmoonlight]

DevMan, не ну это уже офф-топ) это настройки хостинга)

[DevMan]

xmoonlight, ну и зачем зависеть от них, если этого элементарно можно избежать? еще раз: ЭЛЕМЕНТАРНО.

[Кирилл Несмеянов]

на создание объекта (и соотв. выделяется доп. память под это) с методом и просто метод в виде функции (и сравните).

Я там дополнил выше ответ. На проектиках в 10 строк кода - это не аргумент, т.к. микросекунды (даже нано, а не микро) никто считать не будет. А на средних проектах под 50 метров (исходников имеется ввиду) - автолоадинг решает на порядки больше, т.к. просто не загружает в память неиспользуемый функционал.

2. на рендер: регуляркой и шаблонизатором (и сравните).

Ну да, шаблонизатор будет быстрее, т.к. он кеширует результат в PHP и ничего потом парсить не надо будет. Более того - регулярки довольно медленные и реализация лексическогго анализа поверх, например, lexertl, вместо встроенного pcre будет в несколько раз быстрее:

spoiler

Где:
1) Stateful - предварительная компиляция PCRE + preg_replace_callback
2) Stateless - тот же самый алгоритм, но без промежуточного кеширования.
3) Hoa - реализация на основе чистых preg_replace/preg_match
4) Parle - реализация поверх lexertl

[xmoonlight]

DevMan, элементарно - только со стороны сервера, но не со стороны каждого соискателя) Вы же не скажете каждому всегда писать полные теги, если он привык к коротким? Ещё раз: СО СТОРОНЫ СЕРВЕРА будет проще всего.

[DevMan]

xmoonlight, скажу. и если он не поймет - пойдет искать дальше.
нет желания повторяться, Как во всём проекте заменить старые теги на новые?

[Кирилл Несмеянов]

элементарно - только со стороны сервера, но не со стороны каждого соискателя) Вы же не скажете каждому всегда писать полные теги, если он привык к коротким? Ещё раз: СО СТОРОНЫ СЕРВЕРА будет проще всего.

1) Любой не джун и так будет использовать полные теги, т.к. у него есть плачевный опыт и минимальные знания стандартов.
2) Как уже упоминал DevMan и я - это решается обычным phpcfb или ci, если уж привык и ничего не поделать.

[xmoonlight]

Кирилл Несмеянов,

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

Если у меня нет плачевного опыта и я использую short-tags, значит я - джун?! Такая что ли логика?))

[Кирилл Несмеянов]

xmoonlight, если это ваш профильный язык, то однозначно да. А разве нет? Вы считаете, что человек, который не знает основ может считаться тем, на кого действительно можно положиться в разработке ПО? Миддла от ждуна как раз отличает отсутствие необходимости стоять над душой и не поправлять всё за ним. Можно поручить задачу и быть увереным, что она будет выполнена и покрыта тестами, что ревьювить почти ничего не надо будет. Или нет?

[DevMan]

Кирилл Несмеянов,

человек, который не знает основ

и снова здравствуйте: это не основы, это один из множества стайлов.

[Кирилл Несмеянов]

DevMan, множества? Предлагается в качестве альтернативы рассматривать мёртвый PEAR? Или wordpress, который применяется, внезапно, только в WP. Или какой-нибудь битрикс рассматривать в качестве источника вдохновения? Ну это даже не смешно. Всё остальное, либо PSR, либо основывается на PSR, просто добавляются мелочи поверх второго. Что Zend, что Symfony, что Laravel, что Drupal. Любую более-менее актуальную систему бери - везде будет PSR. Стоит уже смириться, что это уже не рекомендации, а стандарт)))

Или есть аргументация против моих тезисов?)

[DevMan]

Кирилл Несмеянов, я привёл аргументацию выше, вы предпочли её проигнорировать.

[DevMan]

Кирилл Несмеянов, ваша аргументация на уровне "мерседес - вот это машина, остальное - говно".
я участвую в проектах, которые появились за 10 лет до пср, я работаю на конторах, которые на рынке появились задолго не то, что пср, а вообще нормального пыха. и никто в здравом уме на таких проектах не будет менять принятый и устоявшийся стайл, под который уже заточены разрабы и вылизаны форматеры, линтеры и прочие сниферы.

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

[Кирилл Несмеянов]

ваша аргументация на уровне "мерседес - вот это машина, остальное - говно".

Если только в мире бы существовало 3 вида автомобилей и на мерседесе катался 9 человек из 10-ти. Остальные 10% просто потому, что жалко выкидывать на свалку свой тазик.

Попробуйте воспользоваться замечательным правилом бойскаута. Перед тем как закрыть файл - просто сделайте его перед этим чуть лучше. Я работал с кодом, который был написан во времена PHP4/5.0, а сейчас 7.1+ c PSR-12. И повышение качества - не такая уж трудоёмкая задача. Да, не весь код, половина просто была удалена за ненадобностью, но всё же.

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

Такие конторы называются "веб-студии", которые клепают одноразовые лендинги. Ах, ну да, ещё фриланс. Там не важно кто и как потом будет это поддерживать, главное вывалить клиенту что-то более-менее рабочее. А что у вас за "проекты", собственно такие, где всем срать, простите за прямоту, на то как написан код?

P.S. Да и во времена, когда кодстайл всего проекта исправляется одной командой в консоли - довольно печально бахвалиться тем, что ваш код игнорирует все возможные рекомендации.

[DevMan]

Кирилл Несмеянов,

Если только в мире бы существовало 3 вида автомобилей и на мерседесе катался 9 человек из 10-ти. Остальные 10% просто потому, что жалко выкидывать на свалку свой тазик.

в мире больше, чем 3 марки авто, и на мерседесах катается не 90%. но вы дальше продолжаете воспроизводить мантру "мерседес - вот это машина, остальное - говно".

Такие конторы называются "веб-студии", которые клепают одноразовые лендинги. Ах, ну да, ещё фриланс. Там не важно кто и как потом будет это поддерживать, главное вывалить клиенту что-то более-менее рабочее.

сам шучу - сам смеюсь?

А что у вас за "проекты", собственно такие, где всем срать, простите за прямоту, на то как написан код?

сам шучу - сам смеюсь, номер 2?

когда кодстайл всего проекта исправляется одной командой в консоли - довольно печально бахвалиться тем, что ваш код игнорирует все возможные рекомендации

нет бахвальства, есть целесообразность или ее отсутствие.

и будьте добры упоминать меня, если отвечаете мне.

[Кирилл Несмеянов]

в мире больше, чем 3 марки авто, и на мерседесах катается не 90%. но вы дальше продолжаете воспроизводить мантру "мерседес - вот это машина, остальное - говно".

DevMan, я до сих пор не услышал ответа на вопрос. Что в мире есть актуального и хотя бы на 1/10 такого же популярного как PSR? Пока что вы не привели ни одного примера, лишь хвастаетесь древними поделками, аргументируя это тем, что все привыкли. Это уже демагогия какая-то. Я вот, например, открываю какой-нибудь: getjump.github.io/ru-php-the-right-way/#%D1%80%D1%... и что вижу? PSR, мёртвый PEAR под PHP 5.2 и Zend, который является тем же PSR. Где все эти ваши "огромное количество других"?

сам шучу - сам смеюсь, номер 2?

Отнюдь. Мне действительно интересно почему ваша аргументация всегда основывается на том, что если вы возитесь с легаси, то и все остальные тоже должны с ним возиться, мол, по-этому PSR не у всех.

нет бахвальства, есть целесообразность или ее отсутствие.

А в чём целесообразность игнорирования общепринятых рекомендаций и возни с легаси? В том, что вам этого не нужно, т.к. код работает и его трогать не надо? Ну тогда не надо приводить его в пример, если этот код никто не трогает. А если трогает, то зачем он такой? Почему вы до сих пор не поправили всё автоматом? Не целесообразно, т.к. у вас нет новых разработчиков, которые приходят, смотрят на код, матерятся и уходят? Ну в таком случае да, ничего исправлять не нужно, т.к. это никому не облегчит жизнь.

[DevMan]

Кирилл Несмеянов,

Что в мире есть актуального и хотя бы на 1/10 такого же популярного как PSR?

какая связь с написанным мною?
или у вас клин на пире/зенде/прочих, и нет мысли что сотрудники компании способны сами для себя выработать стайл (на который пср очень похож, только наш более строгий) когда не то что пср-ов, a php fig вообще еще даже в зародыше не было?

Почему вы до сих пор не поправили всё автоматом? Не целесообразно, т.к. у вас нет новых разработчиков, которые приходят, смотрят на код, матерятся и уходят?

опять влажные фантазии.
под этот стайл написано куча линтеров, которые автоматом не поправишь.
а править лишь потому, что кто-то решил что

if ($a > $b) {
    $foo->bar($arg1);
}

более правильно, чем

if( $a > $b ) {
    $foo->bar( $arg1 );
}

или что допустимо использовать <?= вместо <?php echo смысла не имеет.

Мне действительно интересно почему ваша аргументация всегда основывается на том, что если вы возитесь с легаси, то и все остальные тоже должны с ним возиться

мне действительно интересно почему вы решили, что мы возимся с легаси? только потому что наш стайл устраивает нас больше, чем пср?

[Code Keeper]

DevMan,

или у вас клин на пире/зенде/прочих, и нет мысли что сотрудники компании способны сами для себя выработать стайл

Угу, вот разработчики битрикса выработали свой уникальный "стайл" которым следуют не только они, но и все их комьюнити. Вот до сих пор следуют.

[DevMan]

Code Keeper, и?

[Code Keeper]

DevMan, да в принципе и все. Как бы битрикс не отличается хорошими решениями, а так же не задает направления развития. Достаточно уже того что на нем работают только в странах СНГ, а крупные игроки(по слухам тот же эльдорадо) поняв в какую кабалу они попали стараются уйти от битрикса подальше потому что это просто выкачка денег

[DevMan]

Code Keeper, то есть битрикс налажал не потому, что налажал, а потому что он не использует пср для оформления кода.
л - логика. и она у вас просто потрясающая. правда со знаком минус.

[Code Keeper]

DevMan, О, я смотрю вы любитель оскорблять людей. :)

Логика очень простая, битрикс вместо того что бы следовать общепринятым стандартам и вовремя отрефакторить код, решил ввести свои собственные стандарты чтобы не признавать свои косяки.

Отступление от общепринятых стандартов - первый шаг в пропасть.

[DevMan]

Code Keeper, где вы узрели оскорбления? o_O

если вы не заметили, то в треде речь идет конкретно о пср-1/2, которые касаются исключительно оформления кода.
а теперь риторический вопрос: применение пср-1/2 сделает код битрикса лучше?

[Code Keeper]

DevMan,

где вы узрели оскорбления? o_O

Цитата:

л - логика. и она у вас просто потрясающая. правда со знаком минус.

:)

если вы не заметили, то в треде речь идет конкретно о пср-1/2, которые касаются исключительно оформления кода.
а теперь риторический вопрос: применение пср-1/2 сделает код битрикса лучше?

Да сделает, потому что соблюдение стандартов и приведение кода к стандартам это первый шаг к рефакторингу и улучшению качества всего приложения.

[DevMan]

Code Keeper,

Цитата

и что в ней оскорбительного?

Да сделает

govnokod.ru/23695 вот вам функция оформленная по пср за исключением отступов. она хороша?
да и примеры по вашей ссылке показывают что определенного код-стайла в битриксе либо нет вообще, либо никто его не соблюдает и не следит за соблюдением - пример нещитовый, поищите другой.

[Code Keeper]

DevMan,

и что в ней оскорбительного?

Ну раз вы считаете, что в ней нет ничего оскорбительного, тогда у вас женская логика. :)

govnokod.ru/23695 вот вам функция оформленная по пср за исключением отступов. она хороша?
да и примеры по вашей ссылке показывают что определенного код-стайла в битриксе либо нет вообще, либо никто его не соблюдает и не следит за соблюдением - пример нещитовый, поищите другой.

Нет "она" не хороша, но соблюдение стандарта(хотя я вижу только часть кода) уже помогает понять что данная функция требует рефакторинга и декомпозиции.

[xmoonlight]

Code Keeper,

Да сделает, потому что соблюдение стандартов и приведение кода к стандартам это первый шаг к рефакторингу и улучшению качества всего приложения.

ОМГ! Поздно пить боржоми, когда почки отказали! Это надо делать заранее, пока сроки не сгорели!

Тот кто считает шаги к рефакторингу и есть говнокодер! (это обо всех, а не лично Вам!)

Признаки, что Вам нужен рефакторинг:

1. Ваш программный продукт работает, но внесение новой функциональности иногда затягивается на недели;
   
2. В определённых местах Ваш код работает совершенно не так, как Вы того ожидали;
   
3. Вы часто ошибаетесь в сроках реализации поставленной задачи;
   
4. Вам приходится вносить однотипные изменения в разных местах.
   

Я проектирую всю архитектуру вначале в схемах и сразу вижу сколько раз я БУДУ ИСПОЛЬЗОВАТЬ! один и тот же код. Т.е. заранее делаю рефакторинг при проектировании архитектуры на верхнем уровне абстракции, чтобы не прыгать вверх-вниз и не тратить двойное (если не больше!) время на разработку проекта.

[xmoonlight]

DevMan, там даже проблема не в кодестайле, а в архитектуре... патч-на-патче... о чём речь?!

[Code Keeper]

xmoonlight,

ОМГ! Поздно пить боржоми, когда почки отказали! Это надо делать заранее, пока сроки не сгорели!

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

Конечно в компаниях где есть строгий workflow это явление редскость, но человеческий фактор никто не отменял.

Я проектирую всю архитектуру вначале в схемах и сразу вижу сколько раз я БУДУ ИСПОЛЬЗОВАТЬ! один и тот же код. Т.е. заранее делаю рефакторинг при проектировании архитектуры на верхнем уровне абстракции, чтобы не прыгать вверх-вниз и не тратить двойное (если не больше!) время на разработку проекта.

Отлично! Но опять же это если вы сами от начала и до конца занимались разработкой проекта(модуля, компонента и т.д.). А что делать если к Вам уже пришел готовый "продукт с душком"?

[DevMan]

xmoonlight, я знаю. поэтому и написал что пример не в тему.

Code Keeper,

тогда у вас женская логика

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

соблюдение стандарта(хотя я вижу только часть кода) уже помогает понять что данная функция требует рефакторинга и декомпозиции

a без стандарта не понятно?
а с другим стандартом, отличным от пср, не понятно?

https://goo.gl/uFrNEp - код, написанный по пср. он хороший?

[xmoonlight]

Code Keeper,

А что делать если к Вам уже пришел готовый "продукт с душком"?

1. восстановить архитектуру
   
2. документировать
   
3. переформировать на схеме функциональные блоки без повторения функциональности и т.д.
   
4. проработать стратегию реструктуризации существующей архитектуры
   
5. реализовать на коде
   

И только после этого можно начать дорабатывать (продолжать писать код) дальше.

[Code Keeper]

DevMan,

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

Ну, вот я тоже скромно констатировал факт. :)

a без стандарта не понятно?

Без стандарта, код тяжело читаем.

а с другим стандартом, отличным от пср, не понятно?

Приведите пример стандарта отличного от PSR которое широко используется в сообществе.

https://goo.gl/uFrNEp - код, написанный по пср. он хороший?

C точки зрения PSR - возможно. С точки зрения SOLID - нет. :)

[Code Keeper]

xmoonlight,

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

Если вам в реалиях аутсорса удается на это все выбить время, то я вам завидую.

[xmoonlight]

Code Keeper, надо уметь логично аргументировать свои действия для снижения затрат заказчика и он согласится на все 100%:
"Продолжим писать ща код - +30-50% к сроку.
Сделаем валидацию и реструктуризацию архитектуры - +10%.
Что делаем?"

[DevMan]

Code Keeper,

C точки зрения PSR - возможно. С точки зрения SOLID - нет. :)

вы пришли в спор о оформлении кода и заявили что "код, оформленный не по стандарту - гавно". я вам показал, что код, оформленный по стандарту и из крупного сообщества - тоже гавно (при чем не из-за солид'а, а сам по себе; это говнокод чистой воды, даже если выдернуть из фреймворка и написать отдельно на страничке). так с чем именно вы спорите?

Приведите пример стандарта отличного от PSR которое широко используется в сообществе

вы снова делаете ту же ошибку, что в начале спора. и ту же, что и предыдущий оратор.
читайте тред полностью, на этот вопрос я уже отвечал.

[Code Keeper]

xmoonlight,

"Продолжим писать ща код - +30-50% к сроку.
Сделаем валидацию и реструктуризацию архитектуры - +10%.
Что делаем?"

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

Прошу простить, но мы отошли от темы.

[Code Keeper]

DevMan,

вы пришли в спор о оформлении кода и заявили что "код, оформленный не по стандарту - гавно".

Уважаемый, не нужно за меня ничего додумывать. Пожалуйста ссылочку или скриншот где я делаю это утверждение.

при чем не из-за солид'а, а сам по себе; это говнокод чистой воды, даже если выдернуть из фреймворка и написать отдельно на страничке

С нетерпением жду от вас разбора, что именно не так в приведенном вами коде.

вы снова делаете ту же ошибку, что в начале спора. и ту же, что и предыдущий оратор.
читайте тред полностью, на этот вопрос я уже отвечал.

Перед тем как это написать я специально перечитал весь тренд, но нигде не нашел ссылки которая бы показала, что именно я у вас попросил(пожалуйста читайте мои сообщения целиком). Так что если таковая имеется, то вам не составит труда ее привести.

[DevMan]

Code Keeper,

Пожалуйста ссылочку или скриншот где я делаю это утверждение

к чему тогда была ссылка на битрикс?

С нетерпением жду от вас разбора, что именно не так в приведенном вами коде

подавление ошибок вместо их проверок.

нигде не нашел ссылки которая бы показала, что именно я у вас попросил

а мысли что об этом мог попросить другой не возникло?

[xmoonlight]

Code Keeper, DevMan, как насчёт завязать гонять траф впустую?

[DevMan]

xmoonlight, сори, бро. мне иногда забавно погонять фанатиков.

[Code Keeper]

DevMan,

к чему тогда была ссылка на битрикс?

Я еще раз прошу, читайте мои сообщения внимательно и целиком, тогда вы поймете что я имел ввиду и не будете додумывать нелепости.

а мысли что об этом мог попросить другой не возникло?

Я еще раз повторяю, то что я вас просил привести я в треде не нашел, еще раз перечитайте что именно я вас просил привести.

мне иногда забавно погонять фанатиков.

Исходя из ваших сообщений можно сделать вывод, что самый главный "фанатик" тут вы :)

[DevMan]

Code Keeper,

Я еще раз прошу, читайте мои сообщения внимательно и целиком, тогда вы поймете что я имел ввиду и не будете додумывать нелепости.

окей, не буду додумывать: какая связь между оформлением кода по пср и качеством кода в целом и битриксом (в котором вообще нет код-стайла) в частности?

Я еще раз повторяю, то что я вас просил привести я в треде не нашел, еще раз перечитайте что именно я вас просил привести.

фигасе: не помним даже на что сами отвечали o_O бывает
внимательно прочтите пару коментов до начала вашего выступления.

"фанатик" тут вы

это же именно я надрываюсь здесь доказывая, что есть только один код-стайл под названием пср, а другие стайлы - это отстой, и использование их - грехоподобно.

[xmoonlight]

Здесь могла бы быть ваша реклама! :)

[Code Keeper]

DevMan,

окей, не буду додумывать: какая связь между оформлением кода по пср и качеством кода в целом и битриксом (в котором вообще нет код-стайла) в частности?

Хорошо я повторю
Вы написали следующее

или у вас клин на пире/зенде/прочих, и нет мысли что сотрудники компании способны сами для себя выработать стайл

Я написал это

Угу, вот разработчики битрикса выработали свой уникальный "стайл" которым следуют не только они, но и все их комьюнити. Вот до сих пор следуют.

Это было приведено как пример, что не всегда отклонение от общепринятых стандартов хорошо, вот и все. Вы же везде видите "фанатиков" и т.д.

фигасе: не помним даже на что сами отвечали o_O бывает
внимательно прочтите пару коментов до начала вашего выступления.

Перечитал, так и не нашел ссылки, за сим делаю вывод что ее нет.

это же именно я надрываюсь здесь доказывая, что есть только один код-стайл под названием пср, а другие стайлы - это отстой, и использование их - грехоподобно.

Еще раз, хватит додумывать за меня. Тем более что я так и не увидел ссылки на "другие стайлы".

[DevMan]

Code Keeper,

Перечитал, так и не нашел ссылки, за сим делаю вывод что ее нет.

ее и нет. ибо ни какой другой стандарт, кроме пср, не популяризиловался.
однако, это не делает все другие код-стайлы, как открытые так и закрытые, автоматически плохими (хотя некоторые из них действительно плохи), а код, использующий их - ужасным.

Это было приведено как пример, что не всегда отклонение от общепринятых стандартов хорошо, вот и все.

в битриксе вообще нет стайла, и проблема у него вовсе не в стайле. и приводить его как пример в споре о стайле - ни о чем.
я привел пример, что не всегда следование общепринятым стандартам - хорошо.
так о чем спор вообще?

давайте иначе: я утверждаю, что используемый код-стайл никак не связан с качеством самого кода, и что не столь принципиально какой именно код-стайл используется; главное, чтоб он вообще был и был консистентным по всему проекту.
вы согласны с этим? или только пср и ничего другого?
напоминаю: речь не о пср в целом, а только о той его части, которая касается код-стайла.

[Code Keeper]

DevMan,

ее и нет. ибо ни какой другой стандарт, кроме пср, не популяризиловался.
однако, это не делает все другие код-стайлы, как открытые так и закрытые, автоматически плохими (хотя некоторые из них действительно плохи), а код, использующий их - ужасным.

Так плохие стандарты и не популяризируют. От старых привычек трудно избавляться по этому большинство компаний и разработчиков внедряют общепринятые стандарты чтобы люди которые приходят в компанию(команду) могли быстро и эффективно начинать работать, а не изучать привыкать к другим нормам тратя на это уйму времени.

в битриксе вообще нет стайла, и проблема у него вовсе не в стайле. и приводить его как пример в споре о стайле - ни о чем.

Зайдите на сайт битрикса и почитайте о их нормах и стандартах. У них есть свой стиль которому они следуют.

давайте иначе: я утверждаю, что используемый код-стайл никак не связан с качеством самого кода, и что не столь принципиально какой именно код-стайл используется; главное, чтоб он вообще был и был консистентным по всему проекту.
вы согласны с этим? или только пср и ничего другого?
напоминаю: речь не о пср в целом, а только о той его части, которая касается код-стайла.

Код стайл это часть общепринятой системы которая вырабатывалась годами в php комьюнити. То что его популялизировали - не просто так. Вы сами написали

ее и нет. ибо ни какой другой стандарт, кроме пср, не популяризиловался.

по этому можно сколько угодно продолжать спор, но каждый останется при своем мнение. Я вам предлагаю не мелочится и сразу написать этим ребятам https://sensiolabs.com/, что есть и другие замечательные стандарты. А еще лучше выступить на ближайшей php конференции с докладом "почему ПСР не панацея" или что-то в этом духе. Возможно комьюнити примет во внимание или даже возьмет часть стандартов которые вы предложите. Пока все что вы утверждаете это не более чем "изобретение велосипеда" в отдельно взятой компании(вебстудии) потому что так "кто-то там привык и ему влом переучиваться". Как-то так.
В общем всего вам доброго, не болейте.

[xmoonlight]

Code Keeper,

Так плохие стандарты и не популяризируют.

+

Зайдите на сайт битрикса и почитайте о их нормах и стандартах. У них есть свой стиль которому они следуют.

А как насчёт других CMS? Почему Вы о них ничего не рассказали?!

[DevMan]

Code Keeper,

Зайдите на сайт битрикса и почитайте о их нормах и стандартах. У них есть свой стиль которому они следуют.

если вы сами посмотрите на код, на который сами же ссылаетесь, то без труда сможете увидеть, что стиля никакого там нет. он может быть на сайте битрикса и задекларирован, но в коде не наблюдается никаких стандартов. повторю в очередной раз - пример нещитовый.
если следовать вашей логике, то и пср говно - используя его можно говнокодить как в yii.

Я вам предлагаю не мелочится и сразу написать этим ребятам https://sensiolabs.com/, что есть и другие замечательные стандарты. А еще лучше выступить на ближайшей php конференции с докладом "почему ПСР не панацея" или что-то в этом духе.

зачем? у меня нет проблем с код-стайлами: если на проекте используется пср-стайл - я совершенно спокойно и без истерик использую пср-стайл; если другой - совершенно спокойно и так же без истерик использую другой стайл. благо современные инструменты позволяют этим крутить как угодно.
но у меня есть проблемы с разными петями и васями, заявляющими "ты должен использовать только пср-стайл и ничего другого" прикрываясь лозунгами "так решило комюнити". извините, ребята, но я вам ничего не должен: если я делаю работу для комюнити, я делаю ее по стандартам комюнити; если я делаю работу для компании - я делаю ее по стандартам компании; если я делаю для себя - я делаю по своим стандартам. that's it.

Так плохие стандарты и не популяризируют.

не все стандарты открыты. вы их автоматом и ничего о них не зная записали в плохие, но фанатик почему-то я.

я - за свободу и право выбора, говорю, что код-стайл может быть любым, если он устраивает причастных; вы говорите, что стайл должен быть только по пср. но фанатик здесь я, не перепутайте.

В общем всего вам доброго, не болейте.

и вам не хворать.