Форум программистов, компьютерный форум, киберфорум
PHP: ООП
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.50/4: Рейтинг темы: голосов - 4, средняя оценка - 4.50
 Аватар для GOLD
124 / 37 / 6
Регистрация: 14.01.2012
Сообщений: 264

Переделать функцию согласно ООП

14.06.2017, 11:27. Показов 769. Ответов 3
Метки нет (Все метки)

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

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

Возникла следующая задача.

Есть функция

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
function remove_objects ($objList, $attrArr) {
 
  if (!empty($objList->Error)) {
    return $objList->Error;
  }
 
  $size = $objList->{'@size'};
  $counter = $objList->{'@size'};
 
  if ( !empty($attrArr['minPrice'] || !empty($attrArr['maxPrice'])) ) {
    for ($i = 0; $i < $size - 1; $i++) { 
      $objElementPrice = $objList->ItemsArray[$i]->price;
 
      if (intval($attrArr['minPrice']) > intval($objElementPrice) || 
          intval($attrArr['maxPrice']) < intval($objElementPrice)) {
        unset($objList->ItemsArray[$i]);
        $counter--;
        $i--;
      }
    }
    $size = $counter;
  }
  if ( !empty($attrArr['someAttr']) ) {
    foreach ($GLOBALS['itemsList'] as $key => $value) { 
      $objElementProperty = $objList->ItemsArray[$i]->someProperty;
 
      if (in_array($objElementProperty, $value) ) {
        unset($objList->ItemsArray[$i]);
        $counter--;
        $i--;
      }
    }
    $size = $counter;
  }
  $objList->{'@size'} = $counter;
}
В этой функции ещё много подобных условий по удалению элементов из массива объектов. Функция разрослась уже на 100+ строчек кода и ещё будет расти, если будут добавляться условия для удаления. Возникла необходимость переписать эту функцию так чтобы было удобно добавлять новые элементы с условиями. Я попытался упростить задачу и создал класс Remover, но опять таки возникает загвоздка в том, что похожее в этих функциях только несколько параметров. Все остальные действия зависят от условий и я в замешательстве что лучше плодить в одной функции условия, или плодить классы наследники с переопределенным функциями, кол-во которых будет почти всегда равняться количеству условий для удаления объекта.

Пример базового класса:

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
Class Remover {
  protected $size;
  protected $counter;
  protected $obj;
 
  public function __construct(&$obj = array(), $size = '', $counter = '') {
    $this->size = $size;
    $this->counter = $counter;
    $this->obj = $obj;
    // Exit if error
    $this->checkForErors();
  }
 
  function checkForErrors() {
    if (!empty($this->obj->objList->Error)) {
      return $this->obj->objList->Error;
    }
  }
}
0
IT_Exp
Эксперт
34794 / 4073 / 2104
Регистрация: 17.06.2006
Сообщений: 32,602
Блог
14.06.2017, 11:27
Ответы с готовыми решениями:

переделать код в ООП
Подскажите пожалуйста, как переделать это в ООП? Создать класс time. using System; using System.Collections.Generic; ...

Переделать код в ООП
Помогите переделать код в ООП #include &quot;stdafx.h&quot; #include &lt;stdlib.h&gt; #include&lt;iostream&gt; #include &lt;conio.h&gt; #include...

Переделать код в ООП
Здравствуйте! Есть код на Delphi с использованием DirectX: Как сделать с него обьектно-ориентированый код? Ну чтобы было...

3
123 / 77 / 33
Регистрация: 05.03.2013
Сообщений: 449
14.06.2017, 16:32
А зачем тебе этот класс? Может лучше просто разбить код на несколько разных функций?
0
 Аватар для GOLD
124 / 37 / 6
Регистрация: 14.01.2012
Сообщений: 264
15.06.2017, 13:05  [ТС]
Может и так, я просто хотел узнать мнение других разработчиков как тут лучше поступить для дальнейшей расширяемости.
0
Эксперт PHP
3899 / 3237 / 1353
Регистрация: 01.08.2012
Сообщений: 10,904
15.06.2017, 13:55
Просто разбить удаление на этапы.

PHP
1
2
3
4
5
6
7
8
9
10
public function remove()
{
    $this->cleanByPrice();
    $this->cleanByName();
    $this->cleanBySomeAttr();
}
 
public function cleanByPrice(){...}
public function cleanByName(){...}
public function cleanBySomeAttr(){...}
1
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
BasicMan
Эксперт
29316 / 5623 / 2384
Регистрация: 17.02.2009
Сообщений: 30,364
Блог
15.06.2017, 13:55
Помогаю со студенческими работами здесь

Переделать программу в ООП
Здравствуйте, помогите пожалуйста переделать программу под ООП с применением класса. Все функции алгоритма должны быть его методами....

Помогите пожалуйста переделать код в ООП
Здравствуйте. Помогите пожалуйста переделать данный код в ООП. Начал только учить js, к тому ООП почти не изучал, а тут такая задача. ...

Как переделать игру без ООП
Помогите пожалуйста переделать игру без использования Объектно-ориентированного программирования.

Переделать строчный калькулятор через ООП
Друзья, имею отличный строчный калькулятор, сделанный через процедурное программирование. #define _CRT_SECURE_NO_WARNINGS ...

Из консольного проекта с++ переделать в ООП с интерфейсом QT
Добрый день! Делал на 1 заочном курсе курсовую работу по матрицам на с++ в консольном режиме, все работает. Однако сейчас,...


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

Или воспользуйтесь поиском по форуму:
4
Ответ Создать тему
Новые блоги и статьи
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 . Быстренько разберем подход "на фреймах". Мы делаем одну. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru