[prev in list] [next in list] [prev in thread] [next in thread] 

List:       php-doc-bugs
Subject:    [DOC-BUGS] [PHP-BUG] Doc #75518 [NEW]: Better code examples for article "Choosing an API [to access 
From:       "codedokode at gmail dot com"<php-bugs () lists ! php ! net>
Date:       2017-11-13 17:59:42
Message-ID: bug-75518 () bugs ! php ! net
[Download RAW message or body]

From:             codedokode at gmail dot com
Operating system: Irrelevant
PHP version:      Irrelevant
Package:          Documentation problem
Bug Type:         Documentation Problem
Bug description:Better code examples for article "Choosing an API [to access MySQL server]"

Description:
------------
The article http://php.net/manual/en/mysqlinfo.api.choosing.php
"Choosing an API [to access MySQL server]" has incorrect code examples
that can mislead users. 

First, there is no error handling. In real world examples, of course,
error handling is required. For PDO, this can be solved with one line,
enabling exceptions (although my personal opinion is that this mode
should be teh default). But for mysqli, the error handling should be
made explicitly by adding if operator after every method call. 

Also, the code examples should have at least one argument inserted via
placeholders. In real world code almost all queries will have parameters
and it is important to show that parameters are easier to insert using
PDO.

I can correct the code examples using edit.php.net if this change is
approved. 

I would also like to add a line "supports exceptions" to API comparison
table. 



-- 
Edit bug report at https://bugs.php.net/bug.php?id=75518&edit=1
-- 
Try a snapshot (PHP 5.4):   https://bugs.php.net/fix.php?id=75518&r=trysnapshot54
Try a snapshot (PHP 5.5):   https://bugs.php.net/fix.php?id=75518&r=trysnapshot55
Try a snapshot (trunk):     https://bugs.php.net/fix.php?id=75518&r=trysnapshottrunk
Fixed in SVN:               https://bugs.php.net/fix.php?id=75518&r=fixed
Fixed in release:           https://bugs.php.net/fix.php?id=75518&r=alreadyfixed
Need backtrace:             https://bugs.php.net/fix.php?id=75518&r=needtrace
Need Reproduce Script:      https://bugs.php.net/fix.php?id=75518&r=needscript
Try newer version:          https://bugs.php.net/fix.php?id=75518&r=oldversion
Not developer issue:        https://bugs.php.net/fix.php?id=75518&r=support
Expected behavior:          https://bugs.php.net/fix.php?id=75518&r=notwrong
Not enough info:            https://bugs.php.net/fix.php?id=75518&r=notenoughinfo
Submitted twice:            https://bugs.php.net/fix.php?id=75518&r=submittedtwice
register_globals:           https://bugs.php.net/fix.php?id=75518&r=globals
PHP 4 support discontinued: https://bugs.php.net/fix.php?id=75518&r=php4
Daylight Savings:           https://bugs.php.net/fix.php?id=75518&r=dst
IIS Stability:              https://bugs.php.net/fix.php?id=75518&r=isapi
Install GNU Sed:            https://bugs.php.net/fix.php?id=75518&r=gnused
Floating point limitations: https://bugs.php.net/fix.php?id=75518&r=float
No Zend Extensions:         https://bugs.php.net/fix.php?id=75518&r=nozend
MySQL Configuration Error:  https://bugs.php.net/fix.php?id=75518&r=mysqlcfg


-- 
PHP Documentation Bugs Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

[prev in list] [next in list] [prev in thread] [next in thread]