It seems like you are calling the size function count() as part of the test condition. You might want to compute the size beforehand, and not on each iteration.
If the size of the collection does not change during the iteration, it is
generally a good practice to compute it beforehand, and not on each iteration:
for($i=0;$i<count($array);$i++){// calls count() on each iteration}// Betterfor($i=0,$c=count($array);$i<$c;$i++){// calls count() just once}
Loading history...
86
$line = $lines[$j];
87
$linlen = strlen($line);
88
$newline = '';
89
for ($i = 0; $i < $linlen; ++$i) {
90
$c = substr($line, $i, 1);
91
$dec = ord($c);
92
if (($dec == 32) && ($i == ($linlen - 1))) { // convert space at eol only
93
$c = '=20';
94
} elseif (($dec == 61) || ($dec < 32)
95
|| ($dec > 126)) { // always vcard_encode "\t", which is *not* required
96
$h2 = floor($dec / 16);
97
$h1 = floor($dec % 16);
98
$c = $escape . $hex["$h2"] . $hex["$h1"];
99
}
100
if ((strlen($newline) + strlen($c)) >= $line_max) { // CRLF is not counted
101
$output .= $newline . $escape . $eol; // soft line break; " =\r\n" is okay
102
$newline = ' ';
103
}
104
$newline .= $c;
105
} // end of for
106
$output .= $newline;
107
if ($j < count($lines) - 1) {
108
$output .= $linebreak;
109
}
110
}
111
112
return trim($output);
113
}
114
115
/**
116
* Class vCard
117
*/
118
class vCard
119
{
120
public $properties;
121
public $filename;
122
123
/**
124
* @param $number
125
* @param string $type
126
*/
127
public function setPhoneNumber($number, $type = '')
128
{
129
// type may be PREF | WORK | HOME | VOICE | FAX | MSG | CELL | PAGER | BBS | CAR | MODEM | ISDN | VIDEO or any senseful combination, e.g. "PREF;WORK;VOICE"
mambax7 /
wflinks
mambax7
If the size of the collection does not change during the iteration, it is generally a good practice to compute it beforehand, and not on each iteration: