Re: Help With Warning Message

Path	csiph.com!fu-berlin.de!uni-berlin.de!individual.net!not-for-mail
From	"J.O. Aho" <user@example.net>
Newsgroups	comp.lang.php
Subject	Re: Help With Warning Message
Date	Tue, 24 Aug 2021 00:33:43 +0200
Lines	44
Message-ID	<ioim27F5698U1@mid.individual.net> (permalink)
References	<7no5igtt055jqdalqa2u2vlkfeu6bvhnag@4ax.com> <sfuut5$lb8$1@dont-email.me> <8ia6ig94heh1gjujblafvn17or10ufdlik@4ax.com> <ioh040Fpg0kU1@mid.individual.net> <sg0oqs$g6i$1@dont-email.me>
Mime-Version	1.0
Content-Type	text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding	7bit
X-Trace	individual.net Cl7FaIe7w8Xykn70mnqIeA2ZCLLgsm5H9pT2tptqyeSh7sY353
Cancel-Lock	sha1:5wjEFYcaM3RuUcYEo0eLQ3ri6vk=
User-Agent	Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.13.0
In-Reply-To	<sg0oqs$g6i$1@dont-email.me>
Content-Language	en-US-large
Xref	csiph.com comp.lang.php:18778

Show key headers only | View raw

On 23/08/2021 20.19, Lew Pitcher wrote:

> And, to be complete, the OP should check the value in $result4
> before invoking the fetch() method. So the logic fragment should
> look more like
>    $result4 = $dbh->query($query4);
>    if ($result4 === FALSE)
>    {
>      /* query() failed for some reason - abort this activity */
>    }
>    else
>    {
>      $da = $result4->FETCH(PDO::FETCH_ASSOC);
>      if ($da === FALSE)
>      {
>        /* fetch() failed for some reason - abort this activity */
>      }
>      else $da_name = $da['airport_name'];
>    }
> 
Make the code even better (and more readable), we should skip the else-part

    $result4 = $dbh->query($query4);
    if ($result4 === FALSE)
    {
      throw new Exception("Query failed: " + $dbh->errorInfo()[2]);
    }

    $da = $result4->FETCH(PDO::FETCH_ASSOC);
    if ($da === FALSE)
    {
        throw new Exception("Fetch failed.");
    }

    $da_name = $da['airport_name'];


Of course this requires you take care of the exception higher up in the 
code and give a nice general error message, log the exception.

-- 

  //Aho

Thread

Help With Warning Message Call Me Tom <noemail@noemail.com> - 2021-08-22 20:00 -0400
  Re: Help With Warning Message Lew Pitcher <lew.pitcher@digitalfreehold.ca> - 2021-08-23 01:50 +0000
    Re: Help With Warning Message Call Me Tom <noemail@noemail.com> - 2021-08-23 01:03 -0400
      Re: Help With Warning Message "J.O. Aho" <user@example.net> - 2021-08-23 09:13 +0200
        Re: Help With Warning Message Lew Pitcher <lew.pitcher@digitalfreehold.ca> - 2021-08-23 18:19 +0000
          Re: Help With Warning Message Call Me Tom <noemail@noemail.com> - 2021-08-23 15:26 -0400
          Re: Help With Warning Message "J.O. Aho" <user@example.net> - 2021-08-24 00:33 +0200
      Re: Help With Warning Message Lew Pitcher <lew.pitcher@digitalfreehold.ca> - 2021-08-23 11:47 +0000
  Re: Help With Warning Message Arno Welzel <usenet@arnowelzel.de> - 2021-08-30 10:21 +0200

csiph-web