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/calendar/inc/class.boicalendar.inc.php

    r7655 r7673  
    14341434                        //$this->debug('parse_parameters array return_value: '._debug_array($return_value,False)); 
    14351435 
    1436                 for ($i = 0; $i < count($return_value); ++$i) { 
     1436        $return_value_count = count($return_value); 
     1437                for ($i = 0; $i < $return_value_count; ++$i) { 
    14371438                                $name = strtolower($return_value[$i]['param']); 
    14381439                                $value = $this->strip_quotes($return_value[$i]['value']); 
    … …  
    16611662                                        if (!empty ($event[$value])) { 
    16621663                                                if ($multiples && $value != 'exdate') { 
    1663                                                         for ($i = 0; $i < count($event[$value]); ++$i) { 
     1664                            $event_value_count = count($event[$value]); 
     1665                                                        for ($i = 0; $i < $event_value_count; ++$i) { 
    16641666                                                                $str .= $this->fold(strtoupper(str_replace('_', '-', $value)) . $this->build_parameters($event[$value][$i], $value)); 
    16651667                                                                } 
    … …  
    16741676                                case 'uri' : 
    16751677                                        if (!empty ($event[$value])) { 
    1676                                                 for ($i = 0; $i < count($event[$value]); ++$i) { 
     1678                        $event_value_count = count($event[$value]); 
     1679                                                for ($i = 0; $i < $event_value_count; ++$i) { 
    16771680                                                        $str .= $this->fold(strtoupper(str_replace('_', '-', $value)) . $this->build_parameters($event[$value][$i], $to_text)); 
    16781681                                                        } 
    … …  
    16821685                                        if (!empty ($event[$value])) { 
    16831686                                                if ($multiples) { 
    1684                                                         for ($i = 0; $i < count($event[$value]); ++$i) { 
     1687                            $event_value_count = count($event[$value]); 
     1688                                                        for ($i = 0; $i < $event_value_count; ++$i) { 
    16851689                                                                $str .= $this->fold(strtoupper(str_replace('_', '-', $value)) . $this->build_parameters($event[$value][$i], $value)); 
    16861690                                                        } 
    … …  
    17121716                                                if (@ $this->parameter[$key]['type'] != 'function') { 
    17131717                                                        if ($multiples && count($event[$value]) > 1) { 
    1714                                                                 for ($i = 0; $i < count($event[$value]); ++$i) { 
     1718                                $event_value_count = count($event[$value]); 
     1719                                                                for ($i = 0; $i < $event_value_count; ++$i) { 
    17151720                                                                        $str .= $this->fold(strtoupper(str_replace('_', '-', $value)) . $this->build_parameters($event[$value][$i], $value)); 
    17161721                                                                } 
    … …  
    17211726                                                                $function = $this->parameter[$value]['function']; 
    17221727                                                        if ($multiples) { 
    1723                                                                 for ($i = 0; $i < count($event[$value]); ++$i) { 
     1728                                $event_value_count = count($event[$value]); 
     1729                                                                for ($i = 0; $i < $event_value_count; ++$i) { 
    17241730                                                                        $str .= $this->fold(strtoupper(str_replace('_', '-', $value)) . ':' . $this-> $function ($event[$value][$i])); 
    17251731                                                                } 
    … …  
    17321738                                case 'cal-address' : 
    17331739                                        if (is_array($event[$value][0])) { 
    1734                                                 for ($j = 0; $j < count($event[$value]); ++$j) { 
     1740                        $event_value_count = count($event[$value]); 
     1741                                                for ($j = 0; $j < $event_value_count; ++$j) { 
    17351742                                                        $temp_output = $this->build_parameters($event[$value][$j], $value); 
    17361743                                                        if ($temp_output) { 
    … …  
    17481755                        } 
    17491756                if (!empty ($event['x_type'])) { 
    1750                         for ($i = 0; $i < count($event['x_type']); ++$i) { 
     1757            $event_x_type_count = count($event['x_type']); 
     1758                        for ($i = 0; $i < $event_x_type_count; ++$i) { 
    17511759                                $str .= $this->build_xtype($event['x_type'][$i], ':'); 
    17521760                                } 
    … …  
    17551763                if ($ical_item == 'vtimezone') { 
    17561764                        if ($event['tzdata']) { 
    1757                                 for ($k = 0; $k < count($event['tzdata']); ++$k) { 
     1765                $event_tzdata_count = count($event['tzdata']); 
     1766                                for ($k = 0; $k < $event_tzdata_count; ++$k) { 
    17581767                                        $str .= 'BEGIN:' . strtoupper($event['tzdata'][$k]['type']) . "\r\n"; 
    17591768                                        $str .= $this->build_card_internals(strtolower($event['tzdata'][$k]['type']), $event['tzdata'][$k]); 
    … …  
    17631772                        } 
    17641773                elseif ($event['alarm']) { 
    1765                         for ($k = 0; $k < count($event['alarm']); ++$k) { 
     1774            $event_alarm_count = count($event['alarm']); 
     1775                        for ($k = 0; $k < $event_alarm_count; ++$k) { 
    17661776                                $str .= 'BEGIN:VALARM' . "\r\n"; 
    17671777                                $str .= $this->build_card_internals('valarm', $event['alarm'][$k]); 
Note: See TracChangeset for help on using the changeset viewer.