29

在下面的代码中，IDE 在最​​后一个括号中提醒我“缺少返回语句”。这导致我在这里询问内部的回报try{}是否可以或应该在外部。

非常感谢。

public function getFileNamesFromKeywords( array $ids, $format ) {
    try {
      if(self::$dbLink) {
        $ids = implode(',',$ids);
        $query = 'SELECT d.id, d.wfid, d.docid , k.keyword, k.value'.
          'FROM keywords k'.
          'INNER JOIN documents d '.
          'ON k.document_id = d.id'.
          'WHERE k.document_id IN ('.$ids.')';
        $results = self::$dbLink->query($query);

        if( $results === false ) {
          throw new Exception('Ocurrió un error al consultar a la DB.', 500);
        }
        $results = $results->fetchAll(PDO::FETCH_ASSOC);
        $filenames = $this->buildFileNames( $results, $ids, $format );
      }
      else {
        throw new Exception('No hay una conexión establecida con la DB.', 500);
      }
      return $filenames;
    }
    catch(Exception $e) {
      $this->error = 'Error al intentar conectar con la BD: ' . $e->getMessage();
    }
  } //<----- Missing return statement
4

4 回答 4

36

如果抛出并捕获异常，函数将返回什么？

您应该在 catch 块中或在 try-catch 块之后有一个 return 语句。仅在 try 块中有一个 return 语句是不够的。

于 2013-09-23T15:25:24.363 回答
16

如果您在任何位置的函数内放置 return 语句，则预计该函数必须返回某些内容，并且由于您已将 return 语句放置在 try-catch 块中，当 IDE 评估 thw 代码时，它会注意到您没有当您的尝试失败时，有一个返回语句。

我建议在函数顶部创建一个初始化为 false 的 $response 变量，然后将 $filenames 分配给它，然后在 try-catch 块返回 $response 之后。

function getFilenames(){
    $response = false;

    try{
        //your code
        $response = $filenames;
    }catch{

    }

    return $response;
}

通过这样做，您可以确保该函数始终返回您需要的结果或错误的结果。

于 2013-09-23T16:04:08.723 回答
1

您收到的消息只是一个警告，因为您的代码可能不会返回任何内容。如果您想停止警告，最好的选择是为您的捕获添加一个返回。

只需在右大括号之前添加 return 即可。

catch(Exception $e) {
    $this->error = 'Error al intentar conectar con la BD: ' . $e->getMessage();
    return null;
}
于 2013-09-23T15:29:10.557 回答
1

我相信一般来说，return 语句的正确位置在 finally 部分内

function doSomenthing() {
    $result = SomeDefaultValue;
    try {
        // code
        $result = ResultValueIfEverythingIsOK;
    }
    catch {
        // error handler code
    }
    finally {
        return ($result);
    }
}

您可以在错误处理程序代码中获得未处理的异常，然后您不能保证 return 语句将被执行，除非它在 ​​finally 部分中。

于 2021-01-28T17:39:34.113 回答