On Mar 10, 2008, at 8:32 AM, Ray Hauge wrote:
Murat BEŞER wrote:
I can't under stood but PHP gaves me an error:
"UnExcepted $this" for " || $this->getFileExtension($file) == 'jpg' "
When I removed jpg extension check it's okay... PHP script runs well.
What is the problem :)
public function loadImages($folder) {
$result = $this->filemanager->fecthFiles($folder);
$images = array();
if (sizeof($result)>=1 && $result !== false) {
foreach ($result as $file) {
if ($this->getFileExtension($file) == 'gif' || $this-
>getFileExtension($file) == 'png' || $this->getFileExtension($file)
== 'jpg') {
$images[] = array('name'=>$file);
}
}
}
return $images;
}
Try storing the value in a variable first. It'll also have the side-
effect of being marginally faster too.
Good call.
$extension = $this->getFileExtension($file);
if ($extension == 'gif' || $extension || 'png' || $extension ==
'jpg') {
Correction:
... $extension == 'png' ...
Personally, I like the in_array() version better - it cleaner and you
can expand upon it more more easily.
$extensions = array('gif, 'png', 'jpg' [, 'm4a'[, 'etc']]);
if (in_array($extension, $extensions)) { ... }
// do something
}
That might help, but I would think that the way you had it would
also work. Let us know what happens when you use the variable like
I showed above.
Thanks,
--
Ray Hauge
HTH,
~Philip
"Personally, most of my web applications do not have to factor 13.7
billion years of space drift in to the calculations, so PHP's rand
function has been great for me..." ~S. Johnson
--
PHP General Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
