Как разместить несколько условий NOT внутри инструкции cfif?

Question

Это мой код:

<cfif (CFFILE.FileSize GT (4194304))> 
Delete File 
<cfelseif CFFILE.serverFileExt neq "png" > 
Delete File 
<cfelse> 
It's ok 
</cfif> 

Я хочу добавить «JPG» и «JPEG» в список разрешенных типов файлов. Я пробовал, но без успеха.

<cfif (CFFILE.FileSize GT (4194304))> 
Delete File 
<cfelseif ((CFFILE.serverFileExt neq "png") OR (CFFILE.serverFileExt neq "jpg") OR (CFFILE.serverFileExt neq "jpeg")) > 
Delete File 
<cfelse> 
It's ok 
</cfif> 

Любые идеи?

РЕШЕНИЕ:

Мои ИЛИ заявление должно быть И, спасибо Мигель-F, и с Busches Improvment, это конечный результат:

<cfif (CFFILE.FileSize GT (4194304))> 
    Delete File 
<cfelseif NOT ListFindNoCase("jpg,jpeg,png", CFFILE.serverFileExt)> 
    Delete File 
<cfelse> 
    It's ok 
</cfif> 

Спасибо, ребята!

Answer 1

Ваш код синтаксически корректный, но логика неверна. Это условие всегда будет истинным, потому что вы используете OR. Как только одно из условий верно, условие проходит. Попробуйте использовать AND вместо этого, как это:

<cfif (CFFILE.FileSize GT (4194304))> 
    Delete File 
<cfelseif (CFFILE.serverFileExt NEQ "png") AND (CFFILE.serverFileExt NEQ "jpg") AND (CFFILE.serverFileExt NEQ "jpeg")> 
    Delete File 
<cfelse> 
    It's ok 
</cfif> 

См Busches ответ способ вы можете конденсироваться ваш код немного, если вы используете функцию ColdFusion ListFindNoCase().

Answer 2

Я думаю, что проблема в вашем коде - это просто использование «ИЛИ» в вашем заявлении. Если я правильно прочитаю ваш образец, он будет удалять файл каждый раз.

Я считаю, что это будет иметь эффект, который вы ищете.

<cfelseif ((CFFILE.serverFileExt neq "png") 
     AND (CFFILE.serverFileExt neq "jpg") 
     AND (CFFILE.serverFileExt neq "jpeg")) > 

Вы не ищете ситуации, когда потенциально удаленный файл удовлетворяет любой один условий (что и «ИЛИ» делает), потому что любое расширение файла будет неравным, по крайней мере, 2 из 3 вариантов.

Скорее, вы ищете ситуации, в которой потенциально удаленном файловом удовлетворяет все условий (что «И» делает.

Answer 3

Как заявил Мигель-F, ваш вопрос вам нужно использовать AND вместо OR. Я также рекомендую, в таких случаях, чтобы переключиться на использование ListFindNoCase() вместо использования нескольких OR/AND заявлений.

<cfif CFFILE.FileSize GT 4194304> <!--- You also don't need all those() here ---> 
    Delete File 
<cfelseif NOT ListFindNoCase("png,jpg,jpeg", cffile.ServerFileExt)> 
    Delete File 
<cfelse> 
    It's ok 
</cfif> 

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