Логичные места для ООП/OOP #25

yunasc · 2014-05-10T14:57:41Z

ИМХО самое первое место которое стоит переделать на классы это captcha.

$captcha = new Captcha;
$captcha->generate(); // а можно и без этого
echo $captcha->getHtmlForm(); // бежим в View или просто шаблонизатор и получает шаблон для вывода, куда вставляем нужные переменные

if (!$captcha->validate())
    stderr('Error', 'Captcha fail');

Осталось подумать как-же передавать нужный hash от строки, можно как hidden поле в форме.

Дальше идет вывод формы комментариев и сами комментарии:

$comments = new Comments($torrentid, array('type' => 'torrent')); // передать можно ведь и другой вариант, т.е искать будет по другой колонке просто.
$comments->fetch(array('page' => 3, 'pp' => 25)); // или любой другой вариант
echo $comments->getHtml(); // тут уже запрос побежал во вьюуху, забрал шаблон и вывел кусок html.

P.S: Я совсем не желаю использовать такие конструкторы:

$query = SphinxQL::create($ql)->select('id')
    ->from('torrents')
    ->option('ranker', $matcher[$search_type])
    ->limit(1000);
$query->match('descr', SphinxQL::expr(custom_escape_fn($query, $descrsearchstr, $search_type)));

$results = $query->execute();

Буду рад выслушать критику.

The text was updated successfully, but these errors were encountered:

yunasc · 2014-05-22T05:20:30Z

Как лучше выложить сделанный класс для Captcha?

Думаю про отдельный репозиторий.
Вопрос 2: можно-ли вести 2-й репозиторий, как под-папка первого, и что-б коммиты из под суб-репы попадали в общий?

neonko · 2014-05-26T11:40:42Z

можно просто сделать отдельную ветку (Branch)

yunasc · 2014-05-26T14:34:29Z

Тоже вариант, но неудобство в том что разные ветки - и потом черри между ними - брр.

yunasc · 2014-05-26T14:35:10Z

А так код вышел такой:

<?php

require('core/core.php');

include 'classes/CaptchaImage.php';

$imagestring = $_SESSION['imagestring_' . $_GET['imagehash']];

if (empty($imagestring))
    $imagestring = 'WTF?';

$image = new CaptchaImage($imagestring);

$image->output_image();

?>

<?php

require('core/core.php');

define('COOKIE_SALT', 'some salt to protect cookies and do other stuff');

include 'classes/Captcha.php';
include 'classes/CommentsCaptcha.php';

$cap = new GenericCaptcha();

if ($_SERVER['REQUEST_METHOD'] == 'POST') {
    $is_valid_cap = $cap->validateCaptcha();
}

echo <<<VIEW
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
    <head><title>Captcha Test</title>
    <link rel="stylesheet" href="styles/style.css" type="text/css">
    </head>
<body>
<form action="index.php" method="post">
VIEW;

echo $cap->getHtmlForm();

$validation_result = include 'templates/validation.php';

echo <<<VIEW
<br /><input type="submit" value="Проверить" />
</form>
Результат проверки кода: $validation_result
</body>
</html>
VIEW;

?>

https://github.com/yunasc/captcha

neonko · 2014-05-26T17:41:22Z

что насчет реализации неких "Entity" типа User или Torrent ?

yunasc · 2014-05-27T08:17:32Z

Думал о таком, но с капчей легче всего.

Потом есть смысл колупать Torrent и только после User.
Почему User в последнюю очередь? Да потому-что много где get_user_class() надо будет заменять...

Apmyp · 2014-09-17T14:56:00Z

Давайте попробуем накрутить что-то для Comments, например. Что внутри Comments::fetch будет? sql_query? Или обертку напишем для конструктора запросов и от него наследовать будем?

yunasc · 2014-09-21T06:06:32Z

Я думаю чуть-чуть по другому:

::fetch нельзя, только ->fetch ;) У нас-же не статический метод.

Что там внутри - я думаю, что неплохо хотя-бы использование sql_query, а сам запрос (его шаблон) брать из свойств класса.
Как его вызывать - хороший вопрос.
Prepared statements? У нас не mysqli.
PDO? не уверен, но вроде большой геморрой по части текстов
Значит, остается вариант использовать свой/хорошо известный DBAL.

Тут надо смотреть. Я, с одной стороны, хочу сделать нормально, с другой, не хочу слишком привязыватьтся к каким-то вещам намертво - переделывать будет мучительно. А с первого раза сделать правильный выбор - та еще задачка :)

Apmyp · 2014-09-21T06:09:10Z

Имел ввиду не статический метод, конечно.
Использовать PDO, обертку для него (свою или посмотреть) и уже от неё Comments.

yunasc · 2014-09-21T06:12:07Z

А зачем использовать PDO?
В всем движке, сейчас, используется mysql_.

Лучше воткнуть DBAL, который, пока, будет использовать mysql_.
Далее переводить на него модули, какие-то через конструктор запросов (не люблю я их), и потом уже разом сменить на mysqli_.

Apmyp · 2014-09-21T06:13:02Z

Например потому что mysql_ больше не поддерживается, не рекомендуется и вообще нет нет нет?

yunasc · 2014-09-21T06:15:18Z

Ты не прав: устарело расширение mysql, но не mysqlnd.
Тем более, что никто, пока, не заявлял совместимость с PHP 5.6 где, по моей памяти, удалили расширение mysql, но оставили mysqli и mysqlnd.

Функции mysql_ никогда не устаревали.

yunasc added the enhancement label May 10, 2014

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Логичные места для ООП/OOP #25

Логичные места для ООП/OOP #25

yunasc commented May 10, 2014

yunasc commented May 22, 2014

neonko commented May 26, 2014

yunasc commented May 26, 2014

yunasc commented May 26, 2014

neonko commented May 26, 2014

yunasc commented May 27, 2014

Apmyp commented Sep 17, 2014

yunasc commented Sep 21, 2014

Apmyp commented Sep 21, 2014

yunasc commented Sep 21, 2014

Apmyp commented Sep 21, 2014

yunasc commented Sep 21, 2014