这是用PHP编写长条件的好方法吗？

我必须在PHP中评估一个很长的条件，因此，为了避免错误并尝试编写更可读的代码，我执行了以下操作：

 //this returns 1 when true, and nothing when false, although expected TRUE or FALSE

  $isNameValid=strlen($dataDecoded['nombre'])>=3;

  $isDescriptionValid=(strlen($dataDecoded['descripcion'])>=10) &&  strlen($dataDecoded['descripcion'])<=300;

  $isPriceValid=$dataDecoded['precio'] >0;

  $isImageValid=(($dataDecoded['imagen'] != "") && ($dataDecoded['imagen'] != NULL) );

 if($isNameValid==1 && $isDescriptionValid==1 && $isPriceValid==1 && $isImageValid==1)
  {
      echo "ok";
  }
  else{
      echo "no";
  }

这似乎很好，但可能是一种奇怪的做事方式。我想避免以下情况，因为我发现这些情况更令人困惑，更容易出错

if(strlen($dataDecoded['nombre'])>=3 && ... && ...)

---

有更好的方法吗？我做的不对吗？谢谢

是的，可以接受。但是，这里的变量都是布尔型的，所以您甚至不需要==1

if($isNameValid && $isDescriptionValid && $isPriceValid && $isImageValid)

---

这真的取决于你想如何处理它。 这是一种选择还是可行的选择？ 是漂亮还是方便

从我所看到的，我猜你有一个验证目的和一个操作输入，这取决于验证。为什么不创建一个处理输入和验证的函数或类呢？在那里，你可以得到你想要的所有脏代码。在您的逻辑代码中，您只需执行一个类

$someClass = new SomeClass();
$someClass->validate($fields);
if ($someClass->isValidated()) ...

这样的话，你实际上会遵循一些标准，而它的目的是作为一个验证器为所有的？取决于您的需要和您的数据

例如，三元ifs

$isNameValid = count($dataDecoded['nombre'])>=3 ? true : false;
$isDescriptionValid = count($dataDecoded['descripcion']) >= 10 && count($dataDecoded['descripcion']) <= 300 ? true : false;
$isPriceValid = count($dataDecoded['precio']) > 0 ? true : false;
$isImageValid = empty($dataDecoded['imagen']) === false ? true : false;


if ($isNameValid && $isDescriptionValid && $isPriceValid && $isImageValid) ...

---

我不喜欢在这里创建额外的变量；这使得代码难以维护和不可用。我建议将验证逻辑分解为易于阅读、可维护、可重用的功能：

function valid($data) {
    return validName($data['nombre']) && 
           validDescription($data['descripcion']) &&
           validPrice($data['precio']) &&
           validImage($data['imagen']);
}

function validName($name) {
    return strlen($name) >= 3;
}

function validDescription($desc) {
    return strlen($desc) >= 10 && strlen($desc) <= 300;
}

function validPrice($price) {
    return $price > 0;
}

function validImage($image) {
    return $image !== "" && $image != NULL;
}

$dataDecoded = [
    "nombre" => "foo",
    "descripcion" => "foo bar foo bar",
    "precio" => 15,
    "imagen" => "foo.png"
];

// now your main code is beautiful:
echo (valid($dataDecoded) ? "ok" : "no") . "\n";

---

但条件如何返回1或无，而不是真或假？非常优雅的想法，非常感谢，非常漂亮的代码。