PHP.SU

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


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

> Описание: Вопрос с подвохом
mailitorps
Отправлено: 14 Июля, 2020 - 19:31:35
Post Id


Новичок


Покинул форум
Сообщений всего: 1
Дата рег-ции: Июль 2020  


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




Коллега скинул свой сниппет и говорит там скрыт прикол, лично я ничего не вижу..
В чем подвох? Серьёзно, в чем здесь подвох

PHP:
скопировать код в буфер обмена
  1. public
  2. function indexAction(Application $app, Request $request)
  3. {
  4.     if ($app->getConfig('maintenance') && !in_array($request->getClientIp(), $app->getConfig('excludeIpsFromMaintenance'))) {
  5.         return $app['twig']->render('@MainLayouts/maintenance.html.twig', ['endDate' => $app->getConfig('maintenanceEndDate')]);
  6.     }
  7.  
  8.     try {
  9.         $databaseUser = $app->getDatabaseUser();
  10.         $user = new User($databaseUser);
  11.         if (!$databaseUser->getProfile()->isInitialized()) {
  12.             $preferences = $this->getApp()->getEm()->getRepository('\CS\Entity\OcPreferences')->findOneBy(['userid' => $databaseUser->getUid(), 'configkey' => 'lang', 'appid' => 'core']);
  13.             if (null != $preferences) {
  14.                 $form = $user->getInitForm($databaseUser->getProfile()->getDisplayname(), $preferences->getConfigvalue());
  15.             }
  16.         }
  17.  
  18.         return $app['twig']->render('@MainLayouts/init.html.twig', ['form' => $form->createView()]);
  19.     } catch (\Exception $e) {
  20.         throw $e;
  21.     }
  22. }

(Добавление)
Подозреваю что на строке 20 надо:
 
 Top
LIME
Отправлено: 15 Июля, 2020 - 09:17:33
Post Id



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


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


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




mailitorps пишет:
Подозреваю что на строке 20 надо:
почти
просто тут вообще нет смысла ловить исключение только для того чтобы пробросить его дальше
проще вообще убрать try блок
возможно раньше там были еще каки-то действия в catch и со временем они упразднились, а блок остался
так себе прикол, обычное дело


-----
DDD
 
 Top
Страниц (1): [1]
Сейчас эту тему просматривают: 1 (гостей: 1, зарегистрированных: 0)
« Вопросы новичков »


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



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

 
Powered by ExBB FM 1.0 RC1. InvisionExBB