[PHP-QA] [PHP-BUG] Req #67210 [NEW]: This is a core php feature update request

[mrdaniellee2020 at gmail dot com]

From:             mrdaniellee2020 at gmail dot com
Operating system: All
PHP version:      Irrelevant
Package:          Testing related
Bug Type:         Feature/Change Request
Bug description:This is a core php feature update request

Description:
------------
Hi,

Earlier today i was reading about the newly release Facebook Hack
Languages and Tools online with this they have added a few new features
which are quite interesting.

However there was one feature they added which i had already thought of
in the past which i thought would be excellent if php had this installed
by default.

Its literally to check return values of methods and classes in order to
check returned values just as easily as you can check input values.

I wrote a quick class explaining exactly this.
I think it would be useful to have by default in php as i think it would
help to trim down php unit testing massively.

All this class simple does is allow you to check the response / return
value of a function to ensure the correct data type is returned,
otherwise it throws an exception.

Please review my test script below and tell me what you think?
Im sure if this was placed in the core the actual syntax of the call
could be shortend to something like return('string', '1234'); or
similar.


Test script:
---------------
<?php

class Check {

        /* description
         * This class is used to check function return types *

        /* usage */
        //return Check::type('string', $data);

        /* supported return types
        is_array
        is_​bool
        is_​callable
        is_​double
        is_​float
        is_​int
        is_​integer
        is_​long
        is_​null
        is_​numeric
        is_​object
        is_​real
        is_​resource
        is_​scalar
        is_​string 
        */

        //set allowed return types
        public static $types =
array('array','bool','callable','double','float','int','integer','long','null','numeric','object','real','resource','scalar','string');


        //check return type
        public static function type($type, $value, $allowNull = false) {
                
                //check an allowed type is selected
                if(!in_array($type, self::$types)) {
                        throw new Exception('return type not listed');
                }

                //check for $allowNull / empty values
                if($allowNull == true) {
                        if($value == null || $value == false || $value == '') {
                                return $value;
                        }
                }

                //prepare method name used to check
                $function = 'is_'.$type;

                //check return type
                if(!$function($value)) {self::error($type, $value);}

                //return value
                return $value;
        }

        //throw generic error for failed return type
        public static function error($type = '', $value = '') {
                $backtrace = debug_backtrace();
                $error  = 'failed method return type found in
'.$backtrace[0]['file'].' at line '.$backtrace[0]['line'].' ';
                $error .= 'return type expected: '.$type.' - ';
                $error .= 'actual type returned: '.gettype($value).' - ';
                $error .= 'containing: '.json_encode($value);
                throw new Exception($error);
        }

}


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


-- 
PHP Quality Assurance Mailing List <http://www.php.net/>
To unsubscribe, visit: http://www.php.net/unsub.php