Mam formularz do ladowania zdjecia i chcialbym zmienic calkowicie nazwe ladowanego zdjecia (tzn. bez dodawania jakis wartosci na poczatku nazwy itp.) jednak tak, zeby nie zmienic rozszerzenia zdjecia. Robie tak jak nizej - czy dobrze robie - czy jest to bezpieczne ?

[PHP] pobierz, plaintext 
<?php
if($_FILES['zdjecie']['type']=='image/gif' || $_FILES['zdjecie']['type']=='image/jpeg' || $_FILES['zdjecie']['type']=='image/bmp') 
{
        //int strrpos(string stog, string igla) - zwraca pozycje ostatniego wystapienia 'igla' w 'stog'
        //string substr_replace(string ciag, string zamiana, int start, int [dlugosc]) - zamienia 'ciag' na 'zamiana'
        //od 'start' o dlugosci (parametr opcjonalny) 'dlugosc'
 
$pozycjaOstatniejKropki = strrpos($_FILES['zdjecie']['name'], '.');
$nazwapliku = substr_replace($_FILES['zdjecie']['name'], $id, 0, $pozycjaOstatniejKropki); //$id - to nowa nazwa zdjecia
}
?>
[PHP] pobierz, plaintext 

w zasadzie dobrze, brakuje sprawdzenia, czy plik został wysłany, można też troszkę prościej:

[PHP] pobierz, plaintext 
<?php
$file = isset($_FILES['zdjecie']) ? $_FILES['zdjecie'] ? array('type' => 'empty');
if($file['type']=='image/gif' || $file['type']=='image/jpeg' || $file['type']=='image/bmp') 
{
  $tmp = explode('.', $file['name']);
  $nazwapliku = $id.$tmp[count($tmp) - 1];
}
?>
[PHP] pobierz, plaintext 

EDIT: literówka w kodzie

Dziękuję