Попинайте ногами кому не лень

[Frutik]

Попинайте ногами кому не лень

Я мало пока понимаю в методике работы с бд на пхп, поэтому плиз попинайте ногами кому не лень нижеследующую функция на предмет что следует делать не так
(функция вырвана из контекста класса так что может что и не понятно)

    function GetMovieDetailed($id) {
	if ( !$this->GetConnectionID() ) {
	    return 0; // Unable to connect
	}

	$query = "SELECT * " .
	         "FROM movies " .
		 "WHERE id=" . $id . ";";

	if ( !( $query_result = pg_exec($this->ConnectionID, $query)) ) {
	    if ( pg_numrows($query_result !== 0 ) {
		return pg_fetch_object($query_result, 0);
	    } else {
		return 1; // No movie with such id found
	    }
	} else {
	    return -1; // Query error or connection lost
	}
    }

 

[Frutik]

Re: Попинайте ногами кому не лень

Сначала пну себя сам и побольнее:
нужно писать без ошибок...
В исправленом виде выглядит так:

    function GetMovieDetailed($id) {
	if ( !$this->GetConnectionID() ) {
	    return 0; // Unable to connect
	}

	$query = "SELECT * " .
	         "FROM movies " .
		 "WHERE id=" . $id . ";";

	if ( $query_result = pg_exec($this->ConnectionID, $query) ) {
	    if ( !(pg_numrows($query_result == 0) ) {
		return pg_fetch_object($query_result, 0);
	    } else {
		return 1; // No movie with such id found
	    }
	} else {
	    return -1; // Query error or connection lost
	}
    }

[/QUOTE]