Вообще чем меньше html в классе тем лучше)
Инкапсуляцию соблюдать нужно, свойства тоже желательно скрывать если они из вне не вызываются .
Зачем ты наследуешь connect_Db(как понимаю соединение с БД)

И выяснить действительно ли юзер грузит картинку, а не скрипт с расширением jpg|png может только ф-я getimagesize (я так думаю).

Создай какой нибудь текстовый файл и дай ему расширение jpg
И эта проверка

public function CorFormatOfImage(){
        $arr = array("image/png", "image/jpeg", "image/gif");
        $type = $_FILES["filename"]["type"];
        if(in_array($type, $arr)) {
            return true;
        }

Его спокойно пропустит, а вот getimagesize - хрен :)

PHPprogrammer

m4a1fox
А зачем наследовать класс, если можно обращаться к независимому статичному классу?

:)

class connectDb() {
//тут пишется про соеденение сБД
}

class SomeClass() {
connectDb::method
}

m4a1fox
Оно

m4a1fox
Когда имя класса короткое, то у меня выглядит так: DB::_()->method

Все! Теперь точно запомнил! а то и правда, neadekvat готов просто прибить!

Меня если честно смущает вот это.....

if(isset($_POST['download'])) {
    $t = $upload->troo();
    if($upload->CheckIsUploadedFile()){
        if($upload->CorFormatOfImage()) {
            echo $upload->MoveImage($t);
            echo $upload->ImageShow($t);
            echo $upload->NameOfImageShow($t);
            echo $upload->PathToFileShow($t);
            echo $upload->TypeOfImageShow();
        }else {
            echo $upload->MistakeTypeImage();
        }
    } else {
        echo $upload->MistakeEmptyFielsForm();
    }

неужели в слассе это нельзя сделать????? просто если это сделать в классе, то тогда с именем файла магия какая-то твориться!

http://phpforum.ru/index.php?showtopic=0&v...dpost&p=1284610
*открыл шампанское*

В остальном - блин, насоветуют же люди. И зачем столько методов? На каждый чих буквально.
На самом деле выделение метода определяется не тем, сколько в нем строк, а своей задачей.
Например, у меня в классе парсера был метод логирования всего в пару строк - но это потому что было экономнее и понятнее было вызывать $this->log(), чем писать постоянно эти две строки.
А у тебя код наоборот, стал запутаннее. Большую часть методов можно упразднить.

И, да, избавляйся нафиг от html в классах.

m4a1fox
Если интересно, можешь покурить

neadekvat

m4a1fox

Я б оптимизацией еще занялся.

А вообще, весь этот дикий ужас можно свернуть в пару методов.
|
\/

if(isset($_POST['download'])) {
    $t = $upload->troo();
    if($upload->CheckIsUploadedFile()){
        if($upload->CorFormatOfImage()) {
            echo $upload->MoveImage($t);
            echo $upload->ImageShow($t);
            echo $upload->NameOfImageShow($t);
            echo $upload->PathToFileShow($t);
            echo $upload->TypeOfImageShow();
        }else {
            echo $upload->MistakeTypeImage();
        }
    } else {
        echo $upload->MistakeEmptyFielsForm();
    }

Так вроде пути относительны. А насчет кода, убираю его... Так что это еше не конец.
alex12060

PHPprogrammer

Итак, продолжаем! Вот собственно поправил.... Как теперь Вам?

class DownloadImage {
    
    public $abc = 'abcdefghijklmnopqrstuvwxyzABSCDEFGHIJKLMNOPQRSTUVWXYZ';
    public $path = 'file/';
    public $ok = '<b>Image is download correct.</b>';
    
    public function CheckIsUploadedFile() {
        if(is_uploaded_file($_FILES["filename"]["tmp_name"])) {
            return true;
        }
    }
       
    public function MoveImage($t) {
        move_uploaded_file($_FILES["filename"]["tmp_name"], "".$this->path."".$t);
        $true  = $this->ok;
    return $true;
    }
    
    public function ImageShow($t) {
        return M4A1_HOST.$this->path.$t;
    }
    
    public function TypeOfImage(){
        return pathinfo($_FILES['filename']['name'], PATHINFO_EXTENSION);
        }

    public function NameFile() {
        return $this->ChangeNameOfImage(strtolower($_FILES["filename"]["name"]));
        }
    
    
    public function CorFormatOfImage(){
        $arr = array("image/png", "image/jpeg", "image/gif");
        $type = $_FILES["filename"]["type"];
        if(in_array($type, $arr)) {
            return true;
        }
    }
 
    public function ScanDirForImage() {
        $scan = scandir($this->path);
            for ($i=0; $i<count($scan); $i++) {
                if($scan[$i] != '.' && $scan[$i] != '..') {
                $true[] = $scan[$i];
                }
            }
        return $true;
    }
    
     function RandomNameToImage() {
           $str = substr(str_shuffle($this->abc), 0, 12).'.'.$this->TypeOfImage();
           //$str = $this->NameFile();
     return $str;
     }
     
     public function DeleteFile($file) {
        unlink($this->path.$file);
        return true;
     }
}

$upload = new DownloadImage;
?>

Тебе же уже ответили по поводу getimagesize

inpost
Да, ответили! Вот только не знаю как его использовать..... Пробую!

---

Спустя 33 минуты, 11 секунд m4a1fox написал(а):

---

Есть один ступор! Функция getimagesize проверяет картинку кот. уже закачена. Не получается у меня задать ей параметр проверки файлаб еще не загруженного, может надо использовать так

getimagesize($_FILES["filename"]["tmp_name"])

???

---

Спустя 3 минуты, 10 секунд m4a1fox написал(а):

---

Нет! Так тоже не получается!

---

Спустя 50 минут, 22 секунды m4a1fox написал(а):

---

Прошу помощи, так как уже теряюсь

    $image_info = getimagesize($upload->path.$t);
            echo $image_info['mime'];

Говорит что файла нет, хотя он в папке присутствует!

Конкретнее, вот такая ошибка

Warning: getimagesize(file/MxkNjAyOwYdB.JPG) [function.getimagesize]: 
failed to open stream: No such file or directory in 
Y:\home\localhost\www\oop\modules\download\show.php on line 10

Хотя файл, стопитьсот процентов наместе! Что не так?
А не так то - что я проверяю еще не закаченный файл!

m4a1fox
На, короче посмотри мой код, проверки файла, я повыдирал из разных файлов наиболее важные моменты

---

Спустя 2 минуты, 54 секунды PHPprogrammer написал(а):

---

$image_info = getimagesize($this -> file['tmp_name'])

PHPprogrammer
Сейчас разбираюсь! Спасибо! Буду изучать!

---

Спустя 16 минут, 17 секунд m4a1fox написал(а):

---

Все! Вроде разобрался! Спасибо что помогли! Конечно грубовато получилось, но... получилось!

Может конечно и троллингом занимаюсь, но все же.... Вот класс
Добавил метод про уменьшение картинки. Честно - скопирован со стороннего сайта

class DownloadImage extends showLogin {
    
    public $abc = 'abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ';
    public $path = 'file/';
    public $ok = '<b>Image is download correct.</b>';
    public $allowType = array('image/jpeg', 'image/jpg', 'image/png', 'image/gif');
    public $onlyType = array('jpeg', 'jpg', 'png', 'gif');
    
    public function CheckIsUploadedFile() {
        if(is_uploaded_file($_FILES["filename"]["tmp_name"])) {
            return true;
        }
    }
       
    public function MoveImage($t) {
        move_uploaded_file($_FILES["filename"]["tmp_name"], "".$this->path."".$t);
        $true  = $this->ok;
    return $true;
    }
    
    public function ImageShow($t) {
        $true = M4A1_HOST.$this->path.$t;
        return $true;
    }
    
    public function TypeOfImage(){
        return pathinfo($_FILES['filename']['name'], PATHINFO_EXTENSION);
        }
    
    public function ChangeNameOfImage($str) {
        $str = str_replace(" ", "_", $str);
        $str = str_replace("-", "_", $str);
        return $str;
    }

    public function NameFile() {
            $name_s = $this->ChangeNameOfImage(strtolower($_FILES["filename"]["name"]));
        return $name_s;
        }
    
    
    public function CorFormatOfImage(){
        $arr = array("image/png", "image/jpeg", "image/gif");
        $type = $_FILES["filename"]["type"];
        if(in_array($type, $arr)) {
            return true;
        }
    }
    
    public function GetImageSize() {
        if(!empty($_FILES["filename"]["name"])){
        $type = getimagesize($_FILES['filename']['tmp_name']);
        if(in_array($type['mime'], $this->allowType)){
            return true;
            }
            }
            else
            return false;
    }
 
    public function ScanDirForImage() {
        $scan = scandir($this->path);
            for ($i=0; $i<count($scan); $i++) {
                if($scan[$i] != '.' && $scan[$i] != '..') {
                $true[] = $scan[$i];
                }
            }
        return $true;
    }
    
     function RandomNameToImage() {
           $str = substr(str_shuffle($this->abc), 0, 12).'.'.$this->TypeOfImage();
     return $str;
     }
     
     public function DeleteFile($file) {
        $true = $this->onlyType;
        for($i=0; $i<count($true); $i++){
            unlink($this->path.$file.'.'.$true[$i]);
        }
        
        return true;
     }
     
     ////////////////////////////////////////////////////////////////////////
     public function ResizeImage($filename, $w, $h) 
      { 
        // Имя файла с масштабируемым изображением 
        $filename = $filename;     
        // определим коэффициент сжатия изображения, которое будем генерить 
        $ratio = $w/$h; 
        // получим размеры исходного изображения 
        $size_img = getimagesize($filename); 
        // Если размеры меньше, то масштабирования не нужно 
        if (($size_img[0]<$w) && ($size_img[1]<$h)) return true; 
        // получим коэффициент сжатия исходного изображения 
        $src_ratio=$size_img[0]/$size_img[1]; 
    
        // Здесь вычисляем размеры уменьшенной копии, чтобы при масштабировании сохранились 
        // пропорции исходного изображения 
        if ($ratio<$src_ratio) 
        { 
          $h = $w/$src_ratio; 
        } 
        else 
        { 
          $w = $h*$src_ratio; 
        } 
        // создадим пустое изображение по заданным размерам 
        $dest_img = imagecreatetruecolor($w, $h);   
        $white = imagecolorallocate($dest_img, 255, 255, 255);        
        if ($size_img[2]==2)  $src_img = imagecreatefromjpeg($filename);                       
        else if ($size_img[2]==1) $src_img = imagecreatefromgif($filename);                       
        else if ($size_img[2]==3) $src_img = imagecreatefrompng($filename); 
    
        // масштабируем изображение     функцией imagecopyresampled() 
        // $dest_img - уменьшенная копия 
        // $src_img - исходной изображение 
        // $w - ширина уменьшенной копии 
        // $h - высота уменьшенной копии         
        // $size_img[0] - ширина исходного изображения 
        // $size_img[1] - высота исходного изображения 
        imagecopyresampled($dest_img, $src_img, 0, 0, 0, 0, $w, $h, $size_img[0], $size_img[1]);                 
        // сохраняем уменьшенную копию в файл 
        if ($size_img[2]==2)  imagejpeg($dest_img, $filename);                       
        else if ($size_img[2]==1) imagegif($dest_img, $filename);                       
        else if ($size_img[2]==3) imagepng($dest_img, $filename); 
        // чистим память от созданных изображений 
        imagedestroy($dest_img); 
        imagedestroy($src_img); 
        return true;          
      }
}
$upload = new DownloadImage;

PHPprogrammer
Пишу его для интереса! Для соц сети - это топорно будет.