Муки творчества

кроме php

docx

Тоже не в счет, такая есть задачка)

Кстати, да, с удалением временно каталога я, видимо, поторопился. Хз, сколько времени будет длиться конвертация,

пока поднимется XSLT 2.0 процессор на Java, пока отработает TCPDF...

tcpdf

Всем привет!

Наваял/напастил,наговнякал вот такой примерный SHELL-скрипт конвертора jpeg —> png.

#!/bin/sh
convert $1 $2

Пока правда его не отлаживал, сразу, конечно, ничего не заработает :-)

А сюда пришел за «гнилыми помидорами» от сообщества: поругайте, подскажите, что можно сделать лучше.

Какие есть явные косяки, кроме SHELL?

Пока правда его не отлаживал
подскажите, что можно сделать лучше.

Отладить?

А сюда пришел за «гнилыми помидорами» от сообщества

[trollmode] ImageMagick же, юзай только православный gd и только php. [/trollmode] А чем так плох, tcpdf?

Code style? Отступы? Одинаковая инфа для каждого дока, одинаковые шрифты? Неотлаженое? Бессмысленные дебаги? Полно ненужных отступов? Очевидные комменты?

Это понятно, у мну больше вопрос к опытным кодерам, есть ли явные огрехи, шо прямо глазки потекли :-)

Code style? Отступы?

Согласен с отступами беда, еще не приводил в порядок.

Добавь $page_name в список параметров функции Convertion. Не понятно откуда она там вылазит.
$proc,$xmlFile,$xslTable,$saxonProc со всеми кишками можно засунуть в Convertion
Extract можно тоже вызывать из Convertion, в который передавать только $page_name, $doc_name и $tmp_dir
$upload_path - это скорее $download_path
$page_name - это скорее $output_page

Ну и всякие вещи типа $doc_name = «commands.docx»; конечно же никуда не годятся.

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

Наваял/напастил,наговнякал вот такой примерный php-скрипт конвертора docx —> pdf.

Ты не наваял, и не напастил и не наговнякал _скрипт конвертора_. Ты просто взял либу, дал на вход один говно-формат, и получил (с помощью либы) другой. Впрочем, EXL тебе уже все верно выше сказал. Вопрос в том зачем ты это принес сюда?

поругайте, подскажите, что можно сделать лучше.

Так нечего ругать. Там нет ничего, кроме небольшой демки работы TCPDF.

Пока правда его не отлаживал, сразу, конечно, ничего не заработает :-)

Отлаживал? Отлаживал? :)

Согласен с отступами беда, еще не приводил в порядок.

В твоем случае, конечно, отступы очень важны :))

Вот спасибо, комментарий по делу.

Ну и всякие вещи типа $doc_name = «commands.docx»; конечно же никуда не годятся.

Да, но это на первое время, чтобы оно признаки жизни подало, потом я хочу что оно получало имя файла из $POST массива, хотя тоже не фонтан, но это только начало пути.

Для естетики... мне главное, что этот Франкенштейн чё-нить сказал)

Вопрос в том зачем ты это принес сюда?

Чтобы получить дельный совет.

И один толковый коммент имеет место быть, надо заметить :-)

Ааа. Ну не забудь тогда как-нить красиво прокомментировать потом

/******* ITS MA9I PERVA9I PROGA! SOSYNKI ******
******** MOU KOIIURAIT AZAZAZAZAZAZAZAZA ******
********   KYJLCCC XAKEP BAC9I PYPOK     ******

Ты пользуешься ООП-либой, а сам городишь процедурщину.

Что поделать не умею я в ООП нормально, не джавист ;-)

Если моя работа заказчику понравится, тогда буду думать о дальнейшем развитии

в ООП-парадигме.

И один толковый коммент имеет место быть, надо заметить :-)

Не знаю где ты что там увидел. Все кто пытаются тебе что-либо советовать вообще что-либо _кроме remove_ твоего кода, очевидно, сами любители говнокодить. Мой тебе совет – отнеси это на говнокод.ру, там парни оценят.

Не, она первая такого плана и на пыхе)

Троллинг засчитан ;-)

Если моя работа заказчику понравится

Заказчик будет оценивать результат. _Результат работы_ (не твоей либы). Т.е. ты к оценке результат не имеешь отношения от слова _совсем_. Ты же просишь здесь людей оценить _твой код_. Я дал тебе уже дельный совет – с таким извращением сходи на говнокод.ру – там тебе подскажут что делать.

Злой ты какой-то сегодня :-)

Видно JVM тормозит больше обычного.

P.S. За помидоры спасибо :-)

Хорошо бы еще что-нибудь придумать, чтобы не было шуршания диска в плане распаковки, входных и выходных файлов. Можно конечно и так, но потом всё это надо будет чистить и что-то придумывать для создания временных ссылок. Проще просто забрать файл прямо из потока, обработать и вернуть также ничего не куда не записывая, кроме как в память.

Злой ты какой-то сегодня :-)

Нет, это просто последствия разочарования от корреляции названия темы с разделом web-dev.

P.S. За помидоры спасибо :-)

Ну все как ты просил.

Ага, тоже об этом думал)

Но мне пока это чудо-юдо надо запустить IRL.

Нет, это просто последствия разочарования от корреляции названия темы с разделом web-dev.

Если что пусть модеры потрут или в толксы перенесут, чтоб ты нервы не тратил ;-)

Слишком поздно :) Вот если б ты сразу назвал тему «Оцените мой первый говнокод», (как это обычно и делают), я бы и не зашел сюда. Но я уже ухожу, так что продолжайте, не стесняйтесь :)

Да ладно тебе. Он админ, у него дети кушать хотят, вот и приходиться говнокодить для веб на фрилансе.

А что, питомник уже денег не приносит?

Мопед питомник не мой, кстати, хорошо, что напомнил, надо его оттуда убрать.

у него дети кушать хотят

Бездомные дети вас не забудут (с) О.Бендер

:-D

О, теперь точно буду знать кого кастовать, если задумаю подобную хрень реализовать на Джава через REST API, используя docx4j.

Попробуй ещё от глобальных переменных избавиться.

И это, отступы должны приводиться в порядок в процессе написания кода, а не после.

OK, спасибо, постараюсь.

Добавь $page_name в список параметров функции Convertion.

Верно подмечено, уже вижу, что с ней у меня явный косяк есть.

                $handle = fopen($page_name,"w");
                fclose($handle);

Это что за хрень?
Сначала код приведи в порядок, а потом на ревью выкладывай

собаку убери, она там не нужна

Не понял, где ты там ее увидел?

Вот вы всего говорите, Царь, Царь...

а меня например интересует, почему не работает код ZipArchive(); обернутый в функцию Extract?

sudo cast deep-purple

Короче, пошел я на http://www.php.su/functions/custom/?1 для начала...

if (@file_exists(dirname(__FILE__).'/lang/eng.php')) {

не ходи туда
начинай отсюда http://www.phptherightway.com/

Extract можно тоже вызывать из Convertion.

Для начала,в силу вышеизложенного, просто перенесу туда её код.

И так по зернышку, по зернышку приду к рабочему варианту.

Мопед не мой, это паста из примеров автора TCPDF :-)

Спасибо)

Ну как тебе сказать. Где соблюдение PSR? Не используй «@» в PHP, никогда. Какой смысл использовать unset перед завершением функции? Какой вообще смысл его использовать, это вопрос для другой дискуссии конечно же.

$handle = fopen($page_name,"w");
fclose($handle);

// FYI
touch($pageName);

Какой смысл использовать unset перед завершением функции?

Так предлагают делать авторы Saxonica, за touch спасибо, сам думал его использовать.

Потом прочитал руководства на php.net, чего-то дернуло меня на fopen.

Благодарю за отклик.

Но ведь переменные, которые ты удаляешь, уже и так почти теряют свою область видимости и соотв. они будут уничтожены автоматически, unset никоим образом не освобождает какую либо память, он просто уничтожает ссылку на данные, а уничтожать будет когда то потом gc.

Я в курсе.

Просто пока «склеил» разные куски кода из примеров используемых проектов и получил

монстрика по ссылке.

Теперь постараюсь навести порядок.

Надо писать вот так:

$zip->open('file.zip', ZipArchive::CREATE);

тогда работает.

Кстати, а что ты предпочитаешь вместо tcpdf, wkhtmltopdf ?

И чем таки плох tcpdf ?

Я не пишу Web-приложения, прости меня, дружище.

Поэтому не знаю, плох tcpdf или хорош.