Ignore:
Timestamp:
12/18/12 16:15:29 (11 years ago)
Author:
douglasz
Message:

Ticket #3236 - Correcoes para Performance: Function Within Loop Declaration.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/phpgwapi/inc/class.http.inc.php

    r5912 r7673  
    265265                                                return('5 it was not specified a valid POST method values array'); 
    266266                                        } 
    267                                         for($request_body = '',Reset($values),$value=0;$value<count($values);Next($values),$value++) 
     267                    $values_count = count($values); 
     268                                        for($request_body = '',Reset($values),$value=0;$value<$values_count;Next($values),$value++) 
    268269                                        { 
    269270                                                if($value>0) 
     
    296297                                        if(@is_array($header_value)) 
    297298                                        { 
    298                                                 for(Reset($header_value),$value=0;$value<count($header_value);Next($header_value),$value++) 
     299                        $header_value_count = count($header_value); 
     300                                                for(Reset($header_value),$value=0;$value<$header_value_count;Next($header_value),$value++) 
    299301                                                { 
    300302                                                        if(!$success = $this->PutLine("$header_name: ".$header_value[Key($header_value)])) 
     
    327329                                        { 
    328330                                                $now = gmdate('Y-m-d H-i-s'); 
    329                                                 for($cookies = array(),$domain=0,Reset($this->cookies[0]);$domain<count($this->cookies[0]);Next($this->cookies[0]),$domain++) 
     331                        $cookies_count = count($this->cookies[0]); 
     332                                                for($cookies = array(),$domain=0,Reset($this->cookies[0]);$domain<$cookies_count;Next($this->cookies[0]),$domain++) 
    330333                                                { 
    331334                                                        $domain_pattern = Key($this->cookies[0]); 
     
    335338                                                                ($match == 0 || $domain_pattern[0] == '.' || $this->request_host[$match-1] == '.')) 
    336339                                                        { 
    337                                                                 for(Reset($this->cookies[0][$domain_pattern]),$path_part=0;$path_part<count($this->cookies[0][$domain_pattern]);Next($this->cookies[0][$domain_pattern]),$path_part++) 
     340                                $cookies_count = count($this->cookies[0][$domain_pattern]); 
     341                                                                for(Reset($this->cookies[0][$domain_pattern]),$path_part=0;$path_part<$cookies_count;Next($this->cookies[0][$domain_pattern]),$path_part++) 
    338342                                                                { 
    339343                                                                        $path = Key($this->cookies[0][$domain_pattern]); 
    340344                                                                        if(strlen($this->request_uri) >= strlen($path) && substr($this->request_uri,0,strlen($path)) == $path) 
    341345                                                                        { 
    342                                                                                 for(Reset($this->cookies[0][$domain_pattern][$path]),$cookie = 0;$cookie<count($this->cookies[0][$domain_pattern][$path]);Next($this->cookies[0][$domain_pattern][$path]),$cookie++) 
     346                                        $cookies_count = count($this->cookies[0][$domain_pattern][$path]); 
     347                                                                                for(Reset($this->cookies[0][$domain_pattern][$path]),$cookie = 0;$cookie<$cookies_count;Next($this->cookies[0][$domain_pattern][$path]),$cookie++) 
    343348                                                                                { 
    344349                                                                                        $cookie_name = Key($this->cookies[0][$domain_pattern][$path]); 
     
    353358                                                        } 
    354359                                                } 
    355                                                 for(Reset($cookies),$cookie=0;$cookie<count($cookies);Next($cookies),$cookie++) 
     360                        $cookies_count = count($cookies); 
     361                                                for(Reset($cookies),$cookie=0;$cookie<$cookies_count;Next($cookies),$cookie++) 
    356362                                                { 
    357363                                                        $cookie_name = Key($cookies); 
     
    525531                        { 
    526532                                $secure = Key($this->cookies); 
    527                                 for($domain = 0,Reset($this->cookies[$secure]);$domain<count($this->cookies[$secure]);Next($this->cookies[$secure]),$domain++) 
     533                $cookies_count = count($this->cookies[$secure]); 
     534                                for($domain = 0,Reset($this->cookies[$secure]);$domain<$cookies_count;Next($this->cookies[$secure]),$domain++) 
    528535                                { 
    529536                                        $domain_pattern = Key($this->cookies[$secure]); 
    530                                         for(Reset($this->cookies[$secure][$domain_pattern]),$path_part=0;$path_part<count($this->cookies[$secure][$domain_pattern]);Next($this->cookies[$secure][$domain_pattern]),$path_part++) 
     537                    $cookies_count = count($this->cookies[$secure][$domain_pattern]); 
     538                                        for(Reset($this->cookies[$secure][$domain_pattern]),$path_part=0;$path_part<$cookies_count;Next($this->cookies[$secure][$domain_pattern]),$path_part++) 
    531539                                        { 
    532540                                                $path=Key($this->cookies[$secure][$domain_pattern]); 
    533                                                 for(Reset($this->cookies[$secure][$domain_pattern][$path]),$cookie=0;$cookie<count($this->cookies[$secure][$domain_pattern][$path]);Next($this->cookies[$secure][$domain_pattern][$path]),$cookie++) 
     541                        $cookies_count = count($this->cookies[$secure][$domain_pattern][$path]); 
     542                                                for(Reset($this->cookies[$secure][$domain_pattern][$path]),$cookie=0;$cookie<$cookies_count;Next($this->cookies[$secure][$domain_pattern][$path]),$cookie++) 
    534543                                                { 
    535544                                                        $cookie_name = Key($this->cookies[$secure][$domain_pattern][$path]); 
Note: See TracChangeset for help on using the changeset viewer.