Как соединить две функции php в одной?

Question

[Илья]

Всем привет!

Подскажите, пожалуйста, я пока только начинаю осваивать PHP и никак не могу разобраться со следующей проблемой.

Использую 2 функции.

Первая функция:

function my_get_cat( $description ) {
	$description = explode( ' ', $description );
	return reset( $description );
}

Она делает следующее: например, если строка = "женское платье красное", то в итоге получим первое значение "женское"

Есть вторая функция:

function my_map_data_Pol( $data ) {
	if (mb_strpos($data, 'платье') !== false) {
    return 'платья';
}
    return ( array_key_exists( $data, $map ) ) ? $map[ $data ] : $data;
}

Она делает следующее: например, если строка = "платье женское красное", то в итоге получим значение "платья"

И вот мой вопрос - как соединить обе функции? Плюс к этому, чтобы поиск осуществлялся не по первому слову, а по первым двум словам?
Объясню, мне это нужно для определения категории товара. Бывают разные значения, например:
- платье
- платье женское
- женское платье
- воротник для платья

Comments

[FanatPHP]

Сначала надо нацчиться задавать вопросы
В частности, задавать вопрос не про "функции", а конкретно написать, в первую очередь в заголовке, что тебе на самом деле нужно сделать
А потом уже привести свои функции как пример того как ты пытался свою задачу решить

[Георгий Еремеев]

my_map_data_Pol выглядит не рабочей функцией...

[Илья]

FanatPHP, с комментариями согласен, спасибо. А по существу моей задачи можешь помочь или мне ждать только упреки в мой адрес?

[Кирилл Горелов]

А я добавлю следующее.

Функция должна делать только что-то одно
А не содержать внутри себя несколько других.
(хотя конечно же так делают, но не очень хорошая практика). Или используют ООП паттерн фасад.

Если тебе нужно выполнить две, то просты вынеси в одну функцию.

function nameF(){
$description = explode( ' ', $description );
  $data =  reset( $description );

if (mb_strpos($data, 'платье') !== false) {
    return 'платья';
}
    return ( array_key_exists( $data, $map ) ) ? $map[ $data ] : $data;
}

[Илья]

Георгий Еремеев, тем не менее, она работает.
Если ты знаешь легче способ, как найти определенное слово среди предложения и заменить на нужное, то я буду рад увидеть его =)
Но пока вопрос в другом

[FanatPHP]

ну раз согласен, то переписывай вопрос. изложив его по существу, без ненужных и в отвлекающих от сути деталей.
не про "функции" а про то как тебе надо определять категории
вот тогда и поговорим. по существу.

[Илья]

Кирилл Горелов, Спасибо, это уже похоже на что-то нужное =)
А как написать функцию таким образом, чтобы осуществлялся поиск не в первом слове, а в первых двух или трех словах?
Как мне кажется, то, что ты предложил это должно работать, но только по первому слову, т.к. указано reset.

[Илья]

FanatPHP, Тогда обойдусь без твоей помощи. Гораздо приятнее общаться с людьми, которые не язвят, а помогают. Но все равно спасибо за советы, в следующий раз постараюсь более конкретнее задавать вопросы!

[FanatPHP]

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

[Илья]

FanatPHP, тем не менее другие поняли суть вопроса и помогли. А ты ведь понял, наверняка знаешь как решить и, вместо того, чтобы помочь дурачку, какие-то нелепые колкости пишешь. Ты излишне бюрократично подходишь к форумчанам. Не надо так, будь добрее!

[FanatPHP]

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

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

если бы ты не вел тут себя как капризная девочка "красивую меня кто угодно полюбит, а вы полюбите такую как есть!", а сразу сделал что велено, чтобы себе же в первую очередь помочь, то и проблемы бы не было. но вместо этого ты решил другим нотации читать, что они неправильно тебе помонают. Для самооценки этио конечно приятнее, но ты уж тогда определись зачем пришел - за ответом на вопрос, или других жизни учить. И если второе то не надо клянчить у меня первое

[Илья]

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

[FanatPHP]

"на третий день Штирлиц начал что-то подозревать" :)
я тебе даже покажу, когда он приобрел токсичный характер

мне ждать только упреки в мой адрес?

вот с этой вот фразы
в следующий раз рекомендую оставлять её при себе

[ThunderCat]

Илья,

наш разговор приобретает токсичный характер.

Еще одна жертва пропоганды смузи вайтишников... Фанат местами резок в словах, но по сути абсолютно прав. В конкретно данном примере - вместо того чтобы спросить "как решить задачу "А", ты приводишь кривое решение задачи "Б", и спрашиваешь как его за уши притянуть к решению задачи "А". Я это вижу, фанат это видит, видит куча людей из профи, кто реально решает проблемы похожего характера пачками ежедневно, и видит абсурдность и кривость постановки задачи (важно отметить что именно правильная постановка задачи помогает решить ее наиболее эффективно и верно). Никто (в том числе и я) не стал заморачиваться с объяснением тебе кучи проблем твоего кода, начиная от нарушения PSR и заканчивая проблемами с негибкостью кода. Тут таких в день 20-30 человек. И часть из них понимает что проблема в их подходе к задаче, а часть - вроде тебя, думает что всегда найдется добрый дядя, который поправит твой г*внокод до рабочих костылей и все будет ок. Так вот - рано или поздно тебя начнут игнорить коллеги, ибо постоянно кормить тебя решениями никто не будет, а с твоим подходом "вы все токсичное г-но, а я Дартаньян", ты нихрена в программировании не добьешся. Меняй подход, становись лучше, развивайся...

[Илья]

FanatPHP, ну я уже ведь попрощался с тобой, что ты за человек то такой? =)
Нет же, тебе просто необходимо свои пять копеек надо вставить.
А потом жалуешься, что тратишь время на такие вопросы.

[FanatPHP]

какой очаровательный маленький мамин эгоист :)
то есть на полном серьезе думает что "я попрощался" означает что-то другое, кроме "я не буду больше участвовать в дискуссии".
ну ничего, со временем вырастешь, и узнаешь что кроме любящей мамочки бывают и другие собеседники. И придется научиться умерять свои желания относительно других людей

[Илья]

FanatPHP, Ладно, как скажешь)
Будем думать, что ты отстоял свою позицию и из принципа не помог маленькому эгоисту (капризной девочке), который даже не умеет правильно задавать вопросы.
Ты очень крутой! Надеюсь, ты получил удовольствие и этот день прожит не зря =)

[FanatPHP]

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

[Lander]

FanatPHP,

подумать например о времени, которое люди потратят на то чтобы прочитать твой вопрос и разобраться в том что тебе нужно?

А сколько времени вы потратили на написание этого комментария? Был смысл?

[FanatPHP]

Lander, во-первых, я забочусь не о себе, а о других. А во-вторых, как я уже писал выше - время, потреченное на то чтобы сделать Землю чть более привлекательным местом, не может считаться потраченным впустую!
(:

[Илья]

FanatPHP, Кто прочитает этот вопрос и ответы, тот найдет решение. И твои комментарии тут конечно неуместны и никакой смысловой нагрузки не несут. Только потраченное время.
И нет, я не ни в коем случае не упрекаю тебя. Я понимаю, что вопросы нужно задавать грамотно. Но если уж он задан и ты его понял - зачем нос воротить и высказывать недовольство? Неужели не проще просто помочь?

[FanatPHP]

в этом-то и проблема.
тот, кто прочтет ответ в низу, вообще не поймет, какое он имеет отношение к вопросу "Как соединить две функции"

И я не понимаю, почему ты у нас такой особенный - если уж ты понял что задал вопрос криво, то почему бы тебе его сразу и не исправить?
А если уж ты счел это ниже своего достоинства, и решил вместо этого поучить меня жизни, то с какой стати тогда я должен делать то что тебе? Ты уж определись - или ты кладешь на других людей, но тогда не бегай вокруг и не дергай за штанину, "дядь, ну дай копечкеечку". А если хочешь чтобы к тебе относились по-человечески, то будь добр сначала продемонстрировать такое же отношение к другим.

[Илья]

FanatPHP,

тот, кто прочтет ответ в низу, вообще не поймет, какое он имеет отношение к вопросу "Как соединить две функции"

Ну так помоги, если видишь, что тут один дурак неправильно задает вопрос, другой дурак предлагает костыльное решение. С одной половиной вопроса разобрались, с другой разобрались, осталось только вместе соединить. Ну если ты видишь эту картину - возьми, да и покажи, как надо! Я посмотрю - порадуюсь, другие посмотрят - порадуются. Ты молодец, можно будет только спасибо тебе сказать. Но вместо этого ты с пеной у рта доказываешь, что проблема во мне. Да, я это знаю, в следующий раз буду лучше. Но чего гнобить то и подкалывать? Будь добрее!

[FanatPHP]

Я бы показал, но "этот разговор приобретает токсичный характер"

[Илья]

FanatPHP, давай показывай, не выпендривайся. Нечего обиженку строить. Правильное решение с тебя, за твой косяк)

[Илья]

Кирилл Горелов, Спасибо! Без тебя бы не решил.

[Илья]

FanatPHP, Привет =)
Я подготовил новый вопрос по всем правилам, которыми ты меня учил. но мне никто не помогает. Что за дела?))
Как изменить заголовок Woocommerce исходя из выбранных атрибутов?

[FanatPHP]

вукоммерс вообще маргинальная система и специалистов по ней в сто раз меньше чем по РНР
но вопрос задан нормально так что если кто-то появится то сможет ответить

[Илья]

FanatPHP, А можешь ты мне помочь? Там вообще главное не вукомерс, все дело в php. Я сейчас прохожу курсы, но пока до этого совсем не дошел. И не могу понять.
Пожаааааалуйста)

[FanatPHP]

а какая проблема реализовать в коде алгоритм "Если атрибут = Пол или Застежка, то не выводить заголовки атрибута, в остальных случаях выводить."
то именно вызывает затруднения?

[Илья]

FanatPHP, я не знаю, что и куда писать))

[Илья]

FanatPHP, Ну что, сможешь помочь с решением?

[FanatPHP]

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

[Илья]

Ну я вообще-то что только не пробовал, но это не работает.
Вот например вот так:

protected function formatAttributes($attributes)
    {
        $attributeStrings = [];
        /** @var FilterInterface $attribute */
        foreach ($attributes as $attribute) {
            $attribute->init();
            $items = $attribute->getActiveItems();
            $names = array_column($items, 'title');
            if (count($names)) {
                        if ( $attribute == 'Пол' || $attribute == 'Застежка') {
                        $attributeStrings[] = $attribute->init() . ' (' . implode(', ', $names) . ')';
		  }
                $attributeStrings[] = $attribute->getLabel() . ' (' . implode(', ', $names) . ')';

            }
        }

        return implode(', ', $attributeStrings);
    }

[Илья]

FanatPHP, Ну что, есть какие-то предположения?

[FanatPHP]

про else слышал когда-нибудь?

[Илья]

FanatPHP, Так тоже не работает

protected function formatAttributes($attributes)
    {
        $attributeStrings = [];
        /** @var FilterInterface $attribute */
        foreach ($attributes as $attribute) {
            $attribute->init();
            $items = $attribute->getActiveItems();
            $names = array_column($items, 'title');
            if (count($names)) {
                        if ( $attribute == 'Пол' || $attribute == 'Застежка') {
                        $attributeStrings[] = $attribute->init() . ' (' . implode(', ', $names) . ')';
      } else {
                $attributeStrings[] = $attribute->getLabel() . ' (' . implode(', ', $names) . ')';
	  }
            }
        }

        return implode(', ', $attributeStrings);
    }

[FanatPHP]

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

[Илья]

FanatPHP, Мда... Помощник из тебя так себе. Ладно, забудь. Видимо нам не найти общий язык.

[FanatPHP]

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

[Илья]

FanatPHP, Кстати, по поводу твоего вопроса "Что делать с тупыми ответами на Тостере?" - Ответ "Начать с себя" - идеально тебе подходит.
Вообще не понимаю, зачем ты здесь зарегистрировался...

Answer 1

[Илья]

Отредактировано*
Вот правильное и полное решение. Также добавлено приведение к нижнему регистру всех символов.
P.S. Не знаю, насколько грамотно я его составил, но по крайней мере работает.

function my_get_cat_probacat( $data ) {
    $words = explode( ' ', $data );
    $result = array_slice($words, 0, 3);
    $data = implode(' ', $result);
	$data = mb_strtolower ($data);
	$map = array(
);
	if (array_key_exists($data, $map)) {
	return $map[$data];
}

	if (mb_strpos($data, 'четвертый') !== false) {
    return '4';
}
	if (mb_strpos($data, 'третий') !== false) {
    return '3';
}
	if (mb_strpos($data, 'второй') !== false) {
    return '2';
}	
	if (mb_strpos($data, 'первый') !== false) {
    return '1';
}
 return ( array_key_exists( $data, $map ) ) ? $map[ $data ] : $data;

}

echo my_get_cat_probacat ("Первый второй Третий четвертый");

Comments

[FanatPHP]

Хоть бы ссылку дал на ответ, который тебе на блюдечке с голоубой каемочкой написали.
или ты всерёз думаешь что кто-то решит что это типа ты сам догадался?

[Илья]

FanatPHP, зачем мне делать так, чтобы все думали, что я догадался? Я если честно не понимаю тебя и мне странны твои высказывания. Я только начинаю изучать код, с чего и начинаю упоминание в своем вопросе, и не претендую на звание фаната PHP. Вот ссылка

[FanatPHP]

молодец.
ты хоть и не понял как это работает, и думаешь что всё дело в добавлении параметра, ха-ха, но хотя бы справедливость восстановлена.