2014-02-13 2 views
0

Должен быть более эффективный способ написания этого кода (см. Ниже) может ли кто-нибудь предложить решение, которое будет показывать несколько Ifs более эффективным образом?Вложенные условные требования PHP

if ( isset($_POST["submit"]) 
    || isset($_POST["submit_x"]) 
    ) { 

    // strip_tags removes tags and then we compare to the original contents 
    if (strip_tags($_POST['fld_comments']) !== $_POST['fld_comments']) { 

     // Drop post as it had html in it (!== means Not Identical) 
     echo '<h2>No html tags allowed in comments</h2>'; 
     $blindError = true; 

    } else { 

     // run secondary code such to process form 

    } 
} 
+4

Определение "эффективной". – Dai

+1

Почему бы вам просто не отменить теги из комментария и сохранить его? Что заставляет вас хотеть отказаться от этого в любом случае? – jtheman

+0

Отступы (которые я добавил для вас) делают его более читаемым и гораздо легче устранять неполадки. –

ответ

2

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

Я имею в виду:

if (isset($_POST["submit"]) || isset($_POST["submit_x"])) { 
+0

Согласен. Я сам предпочитаю альтернативный синтаксис if(): и endif; – jtheman

+0

@jtheman Обычно проверка размещенных значений и печать HTML в любом случае не будут одинаковыми. Я также предпочитаю альтернативный синтаксис для HTML, но его код по-прежнему подходит для того, что ему нужно. – kapa

+1

@ AD7six Проклятье, спасибо: D. – kapa

Смежные вопросы