Показать сообщение отдельно
  #14 (permalink)  
Старый 15.12.2020, 09:33
Профессор
Отправить личное сообщение для laimas Посмотреть профиль Найти все сообщения от laimas
 
Регистрация: 14.01.2015
Сообщений: 12,990

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

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

$show_weather = $weather->main->temp;

затем

$w_weather = ceil($show_weather);

затем

$sw_weather = $w_weather. "°C";

чтобы вывести

$sw_weather."<br />

это что за баталии такие? Если нужна температура для вывода в строку и для графики, то почему не

$weather = round($weather->main->temp) . '°C';

и вывод с строке

. $weather . "<br />

и где-то для графики $weather. Прочтите еще раз и внимательно о ceil(), это ли нужно использовать?

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

PS. Кстати, температура отдается за определенную дату, которая указана в объекте, то есть текущая дата не обязательно может совпадать с датой за которую получены данные. А значит не

dete(format)

а

date(format, $weather->dt)

Последний раз редактировалось laimas, 15.12.2020 в 09:38.
Ответить с цитированием