Как улучшить этот код?

@hunker · Регистрация: 03.02.2013

Студворк — интернет-сервис помощи студентам

Код хорошего программиста чёткий, понятный, логичный, последовательный. Это видно с первого взгляда. Хорошего качества кода добиваются годами практики.
Всем хочется писать красивый код.
Например:

Кликните здесь для просмотра всего текста

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
class Tester
{
    public function getProcessTime($iterations, $functionName)
    {
        if(!is_string($functionName) || !is_int($iterations)) {
            throw new Exception('Invalid parameter type.');
        }
 
        $totalTime = $this->getTotalTime($iterations, $functionName);
 
        $this->render($iterations, $totalTime);
    }
 
    protected function render($iterations, $totalTime)
    {
        require_once('tmp.php');
    }
 
    protected function getTotalTime($iterationsNumber, $userFunctionName)
    {
        $startTime = microtime(true);
 
        for($i=1; $i<=$iterationsNumber; $i++) {
            call_user_func($userFunctionName);
        }
 
        return microtime(true) - $startTime;
    }
 
}

Небольшой класс, предназначенный для выполнения клиентского кода заданное количество раз и отдачи времени, потраченного на его выполнение (иногда бывает интерессно, что где работает быстрее).
Хотелось бы узнать, как сделать лучше? Буду рад любым замечаниям, будь-то: синтаксис, логика, последовательность, наименование переменных/методов. Надеюсь на вашу адекватность и проффесионализм).

@casual_visitor · 31.03.2013, 22:56

Сообщение от hunker

PHP
1
2
3
4
    protected function render($iterations, $totalTime)
    {
        require_once('tmp.php');
    }

уже не понятно.. зачем 2 аргумента в данной функции? видимо они используются в tmp.php? вот уже признаки нечеткости,непонятности...

@hunker · 31.03.2013, 23:01 **[ТС]**

Да, tmp - это шаблон, в нём выводятся эти переменные. А как, по-вашему, будет лучше?

KOPOJI · 31.03.2013, 23:14

Сообщение от hunker

PHP
5
if(!is_string($functionName) || !is_int($iterations)) {

Однако это не мешает ввести '' и 0

Сообщение от hunker

PHP
1
require_once('tmp.php');

1. require_once, require, include_once, include - это все не функции, скобки не нужны
2. Имхо, лишнее require_once, т.к. заведомо известно, что будет только одно подключение
3. сказано выше, даже не сразу обращаешь внимание на переменные
4. шаблон с названием 'tmp' как-то не наводит на мысль, что же там такое

Сообщение от hunker

PHP
1
call_user_func($userFunctionName);

опять никакой проверки. А ведь аргумент должен быть callable. Получите свое false и даже и не узнаете об этом.. В результате все остальное пойдет насмарку.

@hunker · 31.03.2013, 23:21 **[ТС]**

Хух, даже не ожидал) Про callable забыл, каюсь.
И как, всё-таки, понятней передавать переменные в шаблон?

@casual_visitor · 01.04.2013, 12:14

Сообщение от hunker

И как, всё-таки, понятней передавать переменные в шаблон?

В Вашем классе есть некий функционал.. И тут же вы подключаете шаблон для вывода в браузер. Имхо нарушение mvc происходит... Инкапсуляция так же нарушена. Ведь класс, в идеале, это самодостаточная программная единица, черный ящик и работать с ним надо через его интерфейс из паблик методов и свойств. А Вы вставляете шаблон в коде класса и ваш класс ничего не знает о том что в этом шаблоне..
Либо вы должны отдельно описать интерфейс, который должен реализовать конкретный шаблон и в своем классе заполнять его данными(благодаря описанию интерфейса класс уже будет знать что делать с этим шаблоном .. да и приличная IDE тоже подскажет что в этом интерфейсе), либо вообще выкинуть из класса всю работу с шаблонами. Так наверно будет попроще.. Тогда потребуется уже в самом шаблоне получить объект вашего класса и осуществить вывод.. так многие фреймворки делают..

KOPOJI · 01.04.2013, 20:35

Не по теме:

casual_visitor, я Вас неправильно понял наверное..? render()

@casual_visitor · 01.04.2013, 22:31

Не по теме:

Сообщение от KOPOJI

render()

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

@hunker · 07.04.2013, 03:43 **[ТС]**

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

@PlumBoom · 07.04.2013, 05:02

Хорошо бы использовать паттерн MVC

@hunker · 07.04.2013, 12:18 **[ТС]**

Ух, спасибо, что-то я оплошал)) Вынесу модель и перезалью

@hunker · 07.04.2013, 14:01 **[ТС]**

Бета-версия:

@PlumBoom · 07.04.2013, 16:59

Все модули MVC независимы друг от друга, т.е. они автономны

Например, проект с использованием MVC могут писать 3 программиста, каждый из которых пишет отдельный модуль от паттерна (Model или Controller, или Viewer). Причем, программиста не волнует, какой код в других модулях

Как-то так)))
Посмотри на изображения (ниже) и увидишь, что писать с MVC очень легко

Новые блоги и статьи Все статьи Все блоги /
PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .
Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .	Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .	Создание Single Page Application на фреймах krapotkin 16.11.2025 Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .

@hunker 86 / 65 / 4 Регистрация: 03.02.2013 Сообщений: 176
	31.03.2013, 23:01 [ТС]
	Да, tmp - это шаблон, в нём выводятся эти переменные. А как, по-вашему, будет лучше? 0

@hunker 86 / 65 / 4 Регистрация: 03.02.2013 Сообщений: 176
	31.03.2013, 23:21 [ТС]
	Хух, даже не ожидал) Про callable забыл, каюсь. И как, всё-таки, понятней передавать переменные в шаблон? 0

KOPOJI
	01.04.2013, 20:35
	Не по теме: casual_visitor, я Вас неправильно понял наверное..? render() 0

@PlumBoom :) 134 / 134 / 31 Регистрация: 30.03.2013 Сообщений: 428
	07.04.2013, 05:02
	Хорошо бы использовать паттерн MVC 1

@hunker 86 / 65 / 4 Регистрация: 03.02.2013 Сообщений: 176
	07.04.2013, 12:18 [ТС]
	Ух, спасибо, что-то я оплошал)) Вынесу модель и перезалью 0

@PlumBoom :) 134 / 134 / 31 Регистрация: 30.03.2013 Сообщений: 428
	07.04.2013, 16:59
	Все модули MVC независимы друг от друга, т.е. они автономны Например, проект с использованием MVC могут писать 3 программиста, каждый из которых пишет отдельный модуль от паттерна (Model или Controller, или Viewer). Причем, программиста не волнует, какой код в других модулях Как-то так))) Посмотри на изображения (ниже) и увидишь, что писать с MVC очень легко Миниатюры 1