Новая тема   Ответить

 Lexikon
Yacc писал:
Нет
это я как пример поинтересовался
сейчас уже потихоньку начал втыкать в эти все тонкости.
При мер был как момент понять как и куда.

 Yacc
В конкретно данном примере класс не нужен вообще, он тут тебе вообще ничего не решает.

Надо мыслить "П" в ООП как "проектирование". В этом смысле ООП начинается задолго до написания первой строчки кода.

 Lexikon
grozny писал:

Это для локалки код пишется.

Вот тут всплыла еще такая проблема при работе цыкла

Fatal error: Allowed memory size of 134217728 bytes exhausted (tried to allocate 500001 bytes)

установлен WAMP

хотелось бы разобраться, это можно как то решить не прибегая к увеличении памяти? Может какие то переменные должны очищаться, после каждого круга, это вообще возможно?

 idk2045
Lexikon писал:
что за цикл-то? 130 метров это дофига.

 Lexikon
Yacc писал:

Возможно.
У меня массив на 25 значений, цыклом нужно пройти 20к строк, по этим значениям нужно сформировать текст он зависит от определенных параметров которые указаны в значениях массива.

 Lexikon
grozny писал:
состоит 25к повторов, каждый цыкл возвращает массив на 25 значений, каждое значение, это строка в среднем на 50 символов

 idk2045
Lexikon писал:
Lexikon писал:
когда проходишь большой файл, или делаешь большую выборку из базы, то обычно все не хранишь в памяти, а по одной строке отрабатываешь и сразу записываешь куда надо. ну если есть такая возможность по алгоритму.

 Lexikon
grozny писал:
сразу не глянул, тут похоже проблема в классе Snoopy.class
Это старонний, не мой, просто подключаю его, чтоб он имитировал обращение браузера. Без него ни как. По крайней мере я незнаю как это сделать проще. Если без него то выдает что у меня нет возможности сохранять кукисы и все. Нужно как то в заголовках это всё передавать, чтоб обойти это. Я пробовал это сделать, но нет.

 Lexikon
всё, всплыла проблема
было так
919704
1120432
1341712

стало так
695768
665536
654416

Это кол-во памяти после каждого цикла
забыл закоментить массив в цикле в который доавлялись новые массивы, чтоб получить один многомерный.

 Lexikon
начал изучать ООП.
Поставил для себя задачу сделать дерево каталогов и вложить в них файлы, по идеи должно было получиться так

./dirData/_general/images/
./dirData/_general/pages/
./dirData/_general/fileNameList.dat

./dirData/09.03.2014/images/
./dirData/09.03.2014/pages/
./dirData/09.03.2014/fileNameListNew.dat

написал для этого такой класс class.DirStructure.php


<?php

/*
Класс для создания заданной структуры директорий
*/

class dirStructure
{
   // Переменные с именами директорий
   public $dirData          = "dirData";
   public $dirImages          = "images";
   public $dirPages          = "pages";
   public $dirGeneral         = "_general";

   // Переменные с именами файлов
   public $fileNameList       = "fileNameList.dat";
   public $fileNameListNew      = "fileNameListNew.dat";


   function createDir(){
        // Получаем текущую дату   
        $dateToday = date("m.d.Y");

      // Пути к основным директориям и файлам
      $toDirGeneral = $this -> dirData."/".$this -> dirGeneral."/";
      $toMainDirImages = $this -> toDirGeneral."/".$this -> dirImages."/";
      $toMainDirPages = $this -> toDirGeneral."/".$this -> dirPages."/";
      $toFileNameList = $this -> toDirGeneral."/".$this -> FileNameList;

      // Пути к новым директориям и файлам
      $toDirDate = $this -> dirData."/".$this -> dateToday."/";
      $toNewDirImages = $this -> toDirDate."/".$this -> dirImages."/";
      $toNewDirPages = $this -> toDirDate."/".$this -> dirPages."/";
      $toFileNameListNew = $this -> toDirDate."/".$this -> FileNameListNew;

      if (!is_dir($this -> dirData)){
         mkdir($this -> dirData);
               echo "Директория <b><i>".$this -> dirData."</i></b> СОЗДАНА! Путь к директории : (<b>".$this -> dirData."</b>). <br />";
        }


        if (!is_dir($this -> toDirGeneral)){
         mkdir($this -> toDirGeneral);
               echo "Директория <b><i>".$this -> dirGeneral."</i></b> СОЗДАНА! Путь к директории : (<b>".$this -> toDirGeneral."</b>). <br />";
        }
        if (!is_dir($this -> toMainDirImages)){
         mkdir($this -> toMainDirImages);
               echo "Основная директория <b><i>".$this -> dirImages."</i></b> СОЗДАНА! Путь к директории : (<b>".$this -> toMainDirImages."</b>). <br />";           
        }
        if (!is_dir($this -> toMainDirPages)){
         mkdir($this -> toMainDirPages);
               echo "Основная директория <b><i>".$this -> dirPages."</i></b> СОЗДАНА! Путь к директории : (<b>".$this -> toMainDirPages."</b>). <br />"; 
      }

          
   }
}




?>

Но вот ничего, кроме как создалась только первая директория /dirData/ и вылезли ошибки "Notice: Undefined property: dirStructure::$toDirGeneral in"

это файл script.php


<?php

header("Content-Type: text/html; charset=UTF-8");

include ("./class.DirStructure.php");

$object = new dirStructure;

$object -> createDir();

?>


Получется что те пути которые я формирую из переменных, вообще не так должны создаваться.
Подскажите люди добрые, что не так.
Спасибо!

 Stek
Для начала классы не должны ничего выдавать типа echo .

Далее $this->toDirGeneral у тебя не определено, отсюда и ошибка.

Ты определил $toDirGeneral как внутреннюю переменную в функции. А обращаешься к переменной класса $this->toDirGeneral.

 Lexikon
вон оно что.
Сейчас стал менять и пошло.
Понял ошибку, я почемуто думал что нужно всегда обращаться через $this ->, даже внутри функции.
Спасибо!
Но вот с echo, а как тогда быть если сообщить, что возникла та или иная проблема, к примеру в классе функция проверки наличия файлов. Как тогда выдать инфу что файлы не создалиь?

 Sterx
исключения или писать в какой нибудь
$this->error

 Stek
Если ошибки отлавливаешь сам, то верни false при неудаче или true при успехе . По началу так легче. Позже можно обрастать экзепшенами и прочими вещами. Но на начальном этапе это лишнее.

 Yacc
Lexikon писал:
Зачем тебе для решения этой задачи класс, экземпляр которого еще и создавать надо? Во-вторых с таким решением смысла писать какой-то там класс вообще нет. Это равносильно тому, что ты прямо в индексе писал бы mkdir, mkdir, ... Завтра поменяется структура директории и ты что будешь делать? Новый класс писать?

Правильное решение выглядит так примерно:
Код:

 Stek
Из "недочетов":
1. имя класса с маленькой буквы, должно быть с большой
2. функция в классе без указания public
3. в создании директории нет проверки на результат
4. в создании файла тоже как и в пункте 4

Совершенно не понял смысл с array_pop($root);

Ну и на довесок, вместо DIRECTORY_SEPARATOR можно везде смело использовать "/" , на виндах работает так же без проблем, как и на линксе.

А вообще молодец, что не поленился

 Yacc
Я-то писал чтобы Лексикон лес за деревьями увидел. Ты вон даже смысла array_pop($root) не понял, поди теперь вот в этом разберись:

Код:

Имена файлов имеет смысл отфильтровать еще, но это если неизвестно кто и как будет их формировать. А если для себя, то это уже неваляшка.

 Stek
Вот это треш с эзкепшенами ты наворотил. Я бы по началу упростил все, выкинув экзепшены, переопределение ошибок и разделив логику создания файлов и директорий, с их проверками.

<?php
error_reporting(E_ALL);

class Foo {
    /***
     * Элементарная рекурсия в 7 строк.
     * @param $path
     * @param $config
     * @return bool
     */
    public static function make_tree($path, $config) {
        /*
         * просто читаем конфиг.
         * Если элемент массив - значит это директория. Создаем и передаем ее конфиг на себя
         * Если элемент ключ - значит просто файл
         * */
        foreach ($config as $node_key => $node_value) {
            if (is_array($node_value)) {
                self::create_dir($path.'/'.$node_key);
                self::make_tree($path.'/'.$node_key, $node_value);
            } else {
                self::create_file($path.'/'.$node_value);
            }
        }
        return true;
    }

    /***
     * Создаем директорию, если доступна запись и такой директории нет.
     * По желанию можно удалять существующую, переименовывать и прочее.
     * Дополнительные проверки и т.п. Суть в том, создание отделено от логики.
     * @param $path string
     * @return bool
     */
    public static function create_dir($path) {
        if (!file_exists($path) && is_writable(dirname($path))) {
            mkdir($path, 0777, true);
            return true;
        } else {
            return false;
        }
    }

    /***
     * Та же фигня что и выше
     * @param $path
     * @param string $content
     * @return bool
     */
    public static function create_file($path, $content = '') {
        if (!file_exists($path) && is_writable(dirname($path))) {
            file_put_contents($path, $content);
            return true;
        } else {
            return false;
        }
    }
}

$files   = [
    'dir1' => [
        'file1',
        'file2',
        'dir2' => [
            'dir3' => [
                'file18',
                'file4'
            ],
            'dir4' => []
        ],
        'file5'
    ],
    'file6',
    'dir5' => [
        'file7'
    ]
];

$root = realpath(dirname(__FILE__)).'/dir0';

// создаем нужную директорию если необходимо
Foo::create_dir($root);

// а тут создаем по конфигу
Foo::make_tree($root, $files);


Вообще по хорошему, стоило бы выделить файловые операции в отдельный класс. Тогда их потом можно из других классов использовать , зная что там уже есть все необходимые проверки и прочее.

 Lexikon
Сразу хочется сказать Вам обоим спасибо за представленные коды.

Yacc писал:

Создание директорий через класс, это как вариант начала познания ООП. То что пишется в книгах и показывается в видеоуроках, это все наглядно, нужно начать самому что то делать чтоб понять до конца как оно работает.

Вот еще интересны такие моменты как (: - это как я знаю "оператор разрешения области видимости". И далее если цитировать строки из книги - "Статические функции полезны для совершения действий, относящихся к самому классу, но не к конкретным экземплярам этого класса."
Я просто не до конца понимаю сути создания статического метода? Как в моем понимании так, статический метод нужен для использования его внутри только самого класса, при этом обратиться к нему можно только через :: и это создано только для того чтоб его не вызывали в объекте?

И касаемо:
if (count($errors = Foo::baz ($root, $files)) > 0) {
    printf('<pre>%s</pre>', print_r($errors, 1));
}
выше говорилось что в классах ненужно использовать всяческие операторы выводов, но тут все таки вывод через print_r() или в данном случае это есть исключание?

 Stek
Lexikon писал:
Просто тупо требуется опыт и перелапачивание сотни вариантов, что бы научиться определять где и как удобнее.
Yacc сделал через статик, я повторил, что бы не сильно отличаться. Тут дело вкуса больше, ну или личного удобства.

Код:

Вот в примере выше класс у тебя уже полностью создал свой объект, и объектов может быть много и все они со своими значениями $id.

А со static уже не получится так.

Lexikon писал:
Это же вне класса сделано, просто как дебаг вывод окончательного результата. Т.е. к классу отношения не имеет.

 Yacc
Stek писал:
То что ты не в состоянии понять как это работает не значит что это трэш. Порядочные люди добавляют к таким сентенция imho.

Посмотрим на твой код. Неискушенному читателю вроде ТС-а может показаться, что он и вправду лучше чем мой: чистенький такой, прокомментировал все. Но взглянем правде в глаза.

Во-первых комменты. Блять как же меня бесят эти комменты, благо нетбинс их сворачивает сразу, а то бы меня кондрат бил по три раза на день. Вот скажи мне зачем методу в котором кроме mkdir нихуя нет комментарий длиннее чем сам метод? Я считаю, что если код нуждается в комментариях, то это плохой код. Особенно понравился коммент "Элементарная рекурсия на 7 строк". Это 5.

Во-вторых методы create_dir и create_file возвращают false в случае неудачи, но в методе create_tree этот результат не проверяется.

В третьих любое исключение или warning и даже notice в твоем коде приведет к остановке.

В четвертых клиент твоего класса в случае ошибки при создании файла или папки никогда не узнает что именно произошло, в лучшем случае он получит false.

Короче: поделка достойна абзаца в брошюре "Эта книга без затей, про ООП для детей".


Lexikon писал:
Статический метод это метод класса, то есть для вызова этого метода не нужно создавать объект этого класса.

Lexikon писал:
Выше много чего говорилось, не нужно принимать все за чистую монету. Заявление о том, что в классах нельзя использовать операторы вывода - бред сивой кобылы.

Вообще в кодинге нет такого слово "нельзя", все что позволяет сделать язык, можно и нужно использовать. Конечно при условии, что ты понимаешь что и зачем ты делаешь.

 Stek
Yacc писал:
Вообще то они для Lexikon написаны и их смысл показать, почему именно так я сделал.


Yacc писал:
Вот если бы ты почитал комменты, то эти бы вопросы у тебя не возникли.


Yacc писал:
Хреново, когда не читают комменты.

Yacc писал:
Ахуенно хуево, когда не читают комменты

Yacc писал:
Только вот для того, что бы очистить уже существующею директорию или файл, мне достаточно изменить функцию их создания. А тебе придется лезть в свой цикл и там накручивать дополнительный код. Плюс код создания директории у тебя в двух точках дублируется.

У каждого свой стиль написания. Я к примеру стараюсь упрощать код, разделять его на логические части. Потом самому же будет легче расширять. И х.з. чего ты кипятишься

 Lexikon
Чем больше работаю с ПХП тем больше всего нового появляется
Это я еще без БД работаю, а там еще доп инфы появится.
А если вникать во всё, что связано то я х.з. сколько времени нужно на изучение.
Вот когда создают какие то серьезные движки, я думаю там не только нужно знать сам язык программирования, но и взаимодействия с сервером. Да и для серьезных проектов где нужна скорость там уже как я понял могут идти связки PHP с С++, для динамичности и интерактивности Ajsx, jQuery, JS хотя по сути эти первые два это по сути тот же JS. Ну и не останется в стороне БД .
Вот тоже интересно, на чем пишут всяческие видеоплееры, те которые используют тьюбы? Java, ActionScript или что то другое.
Вот чититал, про гугл, так как я понял у них основное на чем пишут так это Python.
Всё это очень интересно, жалко времени мало!

 Lexikon
Stek писал:
Я два раза писал одно и то же, но с разными знаниями. Всё работает одинаково, но вот коды разные Учитывая прошлые косяки в следующий раз уже начинаешь писать по другому.
Это как почерк

 Stek
Lexikon писал:
Поэтому нет смысла, пытаться узнать сразу все. Технологии даже в пхп меняются каждый год. Ухватить все не возможно, разве что только не целыми днями этим заниматься. Но тогда кто то должен тебе твое проживание оплачивать.

Lexikon писал:
Гугл вроде свой GO разрабатывает, который ближе к процедурному, чем к OOP.

Lexikon писал:
Это аксиома, что программер одну и ту же задачу каждый раз будет решать по разному