Warning: Cannot use a scalar value as an array in /home/admin/public_html/forum/include/fm.class.php on line 757

Warning: Invalid argument supplied for foreach() in /home/admin/public_html/forum/include/fm.class.php on line 770

Warning: Invalid argument supplied for foreach() in /home/admin/public_html/forum/topic.php on line 737
Форумы портала PHP.SU :: Корректен ли код?

 PHP.SU

 Программирование на PHP, MySQL и другие веб-технологии
PHP.SU Портал     На главную страницу форума Главная     Помощь Помощь     Поиск Поиск     Поиск Яндекс Поиск Яндекс     Вакансии  Пользователи Пользователи


 Страниц (1): [1]   

> Без описания
Поиск в теме | Версия для печати
Gover
Отправлено: 20 Июля, 2012 - 20:21:42
Post Id



Новичок


Покинул форум
Сообщений всего: 25
Дата рег-ции: Февр. 2007  
Откуда: Беларусь


Помог: 0 раз(а)
Я в php не слишком силен, поэтому прошу помощи проверить мой код на ошибки. Все ли здесь корректно?

PHP:
скопировать код в буфер обмена
  1.  
  2. <?PHP
  3. $cache_time = 24*60*60; // 24 hours
  4.  
  5. $url = 'http://www.calend.ru/img/export/informer_7.png';
  6. $path = 'images/informer_7.png';
  7.  
  8. if(file_exists($path) and strtotime('now') - filemtime($path) < $cache_time) {
  9.     $im = file_get_contents($url);
  10. } else {
  11.     $im = file_put_contents($path, file_get_contents($url));
  12. }
  13.  
  14. ?>
 
 Top
DlTA
Отправлено: 20 Июля, 2012 - 20:39:40
Post Id



Постоянный участник


Покинул форум
Сообщений всего: 2952
Дата рег-ции: Окт. 2010  


Помог: 53 раз(а)
а запустить и проверить не судьба?
 
 Top
PATCH
Отправлено: 20 Июля, 2012 - 20:57:31
Post Id



Частый посетитель


Покинул форум
Сообщений всего: 924
Дата рег-ции: Апр. 2011  


Помог: 22 раз(а)




 в начало кода!
 
 Top
Gover
Отправлено: 20 Июля, 2012 - 21:07:59
Post Id



Новичок


Покинул форум
Сообщений всего: 25
Дата рег-ции: Февр. 2007  
Откуда: Беларусь


Помог: 0 раз(а)
DlTA пишет:
а запустить и проверить не судьба?

запустил проверил, вроде работает

я имел ввиду правильно ли прописаны условия в if
 
 Top
LIME
Отправлено: 20 Июля, 2012 - 21:40:08
Post Id


Активный участник


Покинул форум
Сообщений всего: 10732
Дата рег-ции: Нояб. 2010  


Помог: 322 раз(а)
Gover пишет:
strtotime('now')
PHP:
скопировать код в буфер обмена
  1. time();
а вообще кэш лучше удалять после изменений в админке например
но это лирика
вам виднее
 
 Top
IllusionMH
Отправлено: 20 Июля, 2012 - 21:46:01
Post Id



Активный участник


Покинул форум
Сообщений всего: 4254
Дата рег-ции: Февр. 2011  
Откуда: .kh.ua


Помог: 242 раз(а)
Gover,
Цитата:
strtotime -- Преобразует текстовое представление даты на английском языке в метку времени Unix

а это формат в миллисекундах, т.е. еще нужно на 1000 умножить
 
 Top
snikers987
Отправлено: 20 Июля, 2012 - 22:27:29
Post Id



Участник


Покинул форум
Сообщений всего: 1239
Дата рег-ции: Сент. 2011  
Откуда: Крым


Помог: 25 раз(а)
IllusionMH нет, метка времени в секундах


-----
Когда всматриваешься в тёмную бездну, учти, что кто-то может смотреть на тебя из неё...
 
My status
 Top
LIME
Отправлено: 20 Июля, 2012 - 22:29:44
Post Id


Активный участник


Покинул форум
Сообщений всего: 10732
Дата рег-ции: Нояб. 2010  


Помог: 322 раз(а)
я зря постил?
(Добавление)
IllusionMH ошибочка
 
 Top
Gover
Отправлено: 21 Июля, 2012 - 00:22:47
Post Id



Новичок


Покинул форум
Сообщений всего: 25
Дата рег-ции: Февр. 2007  
Откуда: Беларусь


Помог: 0 раз(а)
В целом, как я понял, замечаний нет.
Ну и ладненько Насвистываю
 
 Top
LIME
Отправлено: 21 Июля, 2012 - 00:32:02
Post Id


Активный участник


Покинул форум
Сообщений всего: 10732
Дата рег-ции: Нояб. 2010  


Помог: 322 раз(а)
time()!!!!!!
 
 Top
Gover
Отправлено: 21 Июля, 2012 - 00:36:39
Post Id



Новичок


Покинул форум
Сообщений всего: 25
Дата рег-ции: Февр. 2007  
Откуда: Беларусь


Помог: 0 раз(а)
Поменял на time(), спасибо.
 
 Top
IllusionMH
Отправлено: 21 Июля, 2012 - 18:47:19
Post Id



Активный участник


Покинул форум
Сообщений всего: 4254
Дата рег-ции: Февр. 2011  
Откуда: .kh.ua


Помог: 242 раз(а)

snikers987, LIME, спс.
Затупил. Опозорился и нуб А?!
 
 Top
Поиск в теме | Версия для печати
Страниц (1): [1]
Сейчас эту тему просматривают: 0 (гостей: 0, зарегистрированных: 0)
« Вопросы новичков »


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



Powered by PHP  Powered By MySQL  Powered by Nginx  Valid CSS  RSS

 
Powered by ExBB FM 1.0 RC1. InvisionExBB