Форум программистов, компьютерный форум, киберфорум
PHP для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.75/4: Рейтинг темы: голосов - 4, средняя оценка - 4.75
13 / 13 / 2
Регистрация: 01.06.2013
Сообщений: 245

Концепция "Один уровень(абстракции) на функцию/класс

29.08.2022, 13:18. Показов 800. Ответов 8
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Не могу до конца понять данную концепцию:

- функция должна выполнять только одну операцию
- класс должен выполнять только одну задачу.

Как понять что является одной задачей?

например, Есть класс получения POST , обработки и валидации, ошибок. Технически это же разные задачи, но делить их на подклассы кажется странным. при всём при этом просто принять и обработать запрос безсмысленно. Его надо отправить куда следует. Например: маил, бд, файл. И это еще один класс? В чём смысл, разве не удобней обращаться в каждом случае по методу в своей области видимости? нежели давать доступ извне или объявлять глобали.
тоже самое касается и функций.
0
Programming
Эксперт
39485 / 9562 / 3019
Регистрация: 12.04.2006
Сообщений: 41,671
Блог
29.08.2022, 13:18
Ответы с готовыми решениями:

Права доступа, уровень абстракции
Использую Asp.net 4, в качестве провайдера доступа использую пока свой, похожий на membership. В БД есть: 1)таблица Users, в ней есть...

Какой уровень абстракции применять для классов с набором обязательных и своих полей?
Я окончательно запутался :cry: и дабы всё разрешить вопрошаю следующее: Есть общий набор свойств, который обязателен для нескольких...

Создать базовый класс, использовать свойства полиморфизма и абстракции
Создать базовый класс " Транспортное средство" и производные классы: "Автомобиль", "Мопед", "Велосипед". ...

8
Эксперт PHP
4925 / 3920 / 1620
Регистрация: 24.04.2014
Сообщений: 11,441
29.08.2022, 14:20
Повторное использование кода. Когда класс решает одну задачу, его проще использовать повторна как в рамках одного проекта, так и перенести в другой. Так класс получается максимально абстрагирован от проекта.
Тут основа лежит в очень старом принципе - разделяй и властвуй. Решить отдельно одну небольшую задачу проще.

И никаких глобальных переменных и доступов извне. Все необходимые данные передаются через аргументы.
1
 Аватар для sad67man
2600 / 1504 / 689
Регистрация: 23.08.2015
Сообщений: 3,811
29.08.2022, 14:47
VeTal4ik, Это лучше разбирать на примере. Покажите код. Разберем как правильно разделять и нужно ли это вообще.

Добавлено через 22 минуты
Цитата Сообщение от VeTal4ik Посмотреть сообщение
функция должна выполнять только одну операцию
Функция может выполнять и 2 действия, но тогда она должна вызывать внутри себя 2 функции.
Т.е. Каждая из этих подфункций выполняют какое-то одно более маленькое действие. А общую функцию можно рассматривать как одно действие но на более высоком уровне. Это и называется уровень абстракции.

Плохо смешивать уровни абстракции, т.е. высокоуровневые абстракции с низкоуровневыми деталями. Это затрудняет чтение кода. Программисту приходится в голове постоянно переключаться от бизнес-логики предметной области к техническим деталям.
1
13 / 13 / 2
Регистрация: 01.06.2013
Сообщений: 245
29.08.2022, 14:59  [ТС]
sad67man, Я читаю "советы" по чистому и правильному коду. Мотивацией послужило то, что до этого в проектах я делал такую грязь,что сейчас гляжу и проще выкинуть и переписать, чем что-то править.

сейчас я пришёл к такому уровня кода.

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
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
class getApiData
{
    public string $apiUrl;
    public array $apiAuth;
    public mixed $apiParams; //http_build_query
    public string $countryName;
    public array $allCountries;
    public string $countryFilepath;
    public string $tourFilepath;
    public string $filedate;
    public string $tourName;
 
    public function __construct($apiUrl, $apiAuth, $apiParams, $countryName, $allCountries, $countryFilepath, $tourFilepath, $tourName)
    {
        $this->apiUrl = $apiUrl;
        $this->apiAuth = $apiAuth;
        $this->apiParams = http_build_query($apiParams);
        $this->countryName = $countryName;
        $this->allCountries = $allCountries;
        $this->countryFilepath = $countryFilepath;
        $this->tourFilepath = $tourFilepath;
        $this->filedate = date('Y-m-d');
        $this->tourName = $tourName;
 
    }
    public function buildDirectories($filepath): void
    {
        foreach ($this->allCountries as $dirName) {
            if (!is_dir($filepath)) {
                mkdir($filepath,0755);
            }
        }
    }
 
    private function apiAnswer($apiUrl): string
    {
        $curl = curl_init($apiUrl);
        curl_setopt($curl, CURLOPT_HTTPAUTH, CURLAUTH_BASIC);
        curl_setopt($curl, CURLOPT_POSTFIELDS, $this->apiAuth);
        curl_setopt($curl, CURLOPT_RETURNTRANSFER, true);
        curl_setopt($curl, CURLOPT_SSL_VERIFYPEER, false);
        $apiAnswer = curl_exec($curl);
        curl_close($curl);
        return $apiAnswer;
    }
 
    private function deleteFiles($filepath): void 
    {
        if (file_exists($filepath)) {
            foreach (glob($filepath . "[аА-яЯ]*.xml") as $file) {
                unlink($file);
            }
        }
    }
 
    public function buildCoutriesFiles(): void
    {
        $this->deleteFiles($this->countryFilepath); 
        foreach ($this->allCountries as $country_id => $country_name) {
            $xml = $this->apiAnswer($this->apiUrl . $this->apiParams . $country_id);
            $filename = $this->countryFilepath . $country_name . '_' . $this->filedate . '.xml';
            file_put_contents($filename, $xml, LOCK_EX);
        }
    }
 
    private function getHotelsApiParams(): array
    {
        $Countryfiles = array_diff(scandir($this->countryFilepath), array('..', '.', '.htaccess'));
        foreach ($Countryfiles as $hotelsInCountry) {
            $toursArray = simplexml_load_string(file_get_contents($this->countryFilepath . $hotelsInCountry));
            if ($toursArray->hotcount > 0) {
                foreach ($toursArray->tour as $tour) {
                    $hotelsCodeForApi[(string)$tour->countryname][] = (string)$tour->hotelcode;
                }
            }
        }
        return $hotelsCodeForApi;
    }
 
    public function buildHotelFiles(): void
    {
        $this->deleteFiles($this->tourFilepath); 
        $hotelsCodeForApi = $this->getHotelsApiParams();
        foreach ($hotelsCodeForApi as $key => $hotelcode) {
            foreach ($hotelcode as $code) {
                $xml = $this->apiAnswer($this->apiUrl . $this->apiParams . $code);
                $filename = $this->tourFilepath . $key. '_' . $code . '_' . $this->filedate . '.xml';
                  file_put_contents($filename, $xml, LOCK_EX);
            }
        }
 
    }
}
Но чем больше я читаю, тем больше у меня впечатление что это вообще не подходит под "правильный код"
т.к. взять хоть это правило "класс должен выполнять только одну задачу". Но нет, этот класс делает сразу всё что мне нужно. Это весь код всего функционала задачи, который передо мной стоял. Но согласно принципам я сделал всё плохо. Пытаюсь осознать, но пока не выходит.

Хотя с другой стороны, мне сопровождать этот код очень просто и удобно, легко читаю и правлю даже спустя огромное кол-во времени. Всё крайне понятно и исполняется этот код всего в 2 строки (хотя это может быть эффектом автора) =)
0
 Аватар для sad67man
2600 / 1504 / 689
Регистрация: 23.08.2015
Сообщений: 3,811
29.08.2022, 15:21
VeTal4ik, Позже разберу, сейчас времени нет.

Добавлено через 2 минуты
VeTal4ik, Заодно покажите код, как этот класс вызывается.
1
13 / 13 / 2
Регистрация: 01.06.2013
Сообщений: 245
29.08.2022, 15:33  [ТС]
PHP
1
2
3
$countryData = new getApiData($apiUrl, $apiAuth, $apiParams, $countryName, $allCountries, $countryFilepath, $tourFilepath, $tourName); //Естественно, все переменные перед созданием объявлены
//$countryData->buildDirectories($countryFilepath); // Это первый запуск, иначе положить файлы некуда 
$countryData->buildCoutriesFiles(); //создание файлов стран
Подзадача выполняется тем же кодом.(сменив лишь 2 переменных: $apiUrl и $apiParams)

PHP
1
2
3
$tourData = new getApiData($apiUrl, $apiAuth, $apiParams, $countryName, $allCountries, $countryFilepath, $tourFilepath, $tourName);
//$tourData->buildDirectories($tourFilepath); // Это первый запуск, иначе положить файлы некуда
$tourData->buildHotelFiles(); //создание файлов отелей
Описание класса:
обращаемся к API с определенными GET, вытаскиваем оттуда список стран, формируем в файлы и складываем по человекопонятным названиям (т.к. пользователю эти файлы предоставляются по ссылке)
Затем читаем эти файлы и вытаскиваем свойство одной из веток описания каждой страны, на основании этих свойств строим параметры запроса, вытаскиваем название отелей,которые принадлежат этим странам, и так же складываем в файлы.
0
 Аватар для sad67man
2600 / 1504 / 689
Регистрация: 23.08.2015
Сообщений: 3,811
29.08.2022, 16:06
VeTal4ik, Ну это и по конструктору класса было понятно. Хотелось бы больше получить информации, откуда эти параметры берутся.
0
321 / 189 / 78
Регистрация: 04.10.2016
Сообщений: 809
29.08.2022, 16:08
Цитата Сообщение от VeTal4ik Посмотреть сообщение
Я читаю "советы" по чистому и правильному коду. Мотивацией послужило то, что до этого в проектах я делал такую грязь
советы советами, но вы так и будете лепить "грязь", если не начнете изучать и использовать существующие практики.
вам сейчас покажут и подскажут как делать, а в итоге все равно останется при своем.

изучите patterns - php
там все наглядно показано, что и как делается.
0
13 / 13 / 2
Регистрация: 01.06.2013
Сообщений: 245
29.08.2022, 16:11  [ТС]
sad67man, задаются ручками, это статичные данные. Некоторые от поставщика API, типа параметров(по докам), ключа-авторизации и точки входа. Эти
PHP
1
$apiUrl, $apiAuth, $apiParams, $countryName, $allCountries

$countryFilepath, $tourFilepath сам придумал) просто директория на сервере, может быть любой.

$tourName берется по сформированным файлам ранее, при чтении XML страны

Добавлено через 16 секунд
Fart83, потому я и здесь =) разбираюсь
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
inter-admin
Эксперт
29715 / 6470 / 2152
Регистрация: 06.03.2009
Сообщений: 28,500
Блог
29.08.2022, 16:11
Помогаю со студенческими работами здесь

Изящное пользовательское консольное меню используя функцию prompt. концепция powershell модулей тоже ваш друг
Задача: Написать консольное меню в виде А. Сортировка файлов по дню B. Сортировка файлов по месяцу ...

Tr выставить на один уровень
Проблема такая, в tr есть два блока td левый выше чем правый, как исправить ? td_r это правый блок, а td_l левый. .table_td_r { ...

Пропускает один уровень
Есть проблема: пишу игру типа викторина. Практически все закончил. Беда в том что запускаем уровень и должны пройти 3 теста в уровне для...

Тег 'a', подняться на один уровень вверх
Здравствуйте! Подскажите как сделать так, чтобы при нажатии на ссылку 'a' страница переходила на один уровень вверх. Приложение...

Вывести формы в MDI на один уровень
Сижу голову ломаю - есть форма-MDI контейнер. Она запускает дочернюю форму. Из этой дочерней я вызываю еще одну форму и хочу чтобы она была...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
9
Ответ Создать тему
Новые блоги и статьи
Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка.
Programma_Boinc 23.12.2025
Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Рецензия / Мнение Это мой обзор планшета X220 с точки зрения школьника. Недавно я решила попытаться уменьшить свой. . .
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
Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru