Это нормально так разрабатывать боты?

Question

[E6APB]

У меня простой бот. Вопрос-ответ. Никаких контекстов, сессий и т. д. У меня код выглядит примерно так:

<?php

if ($query === 'привет') {
    echo 'Привет';
} elseif ($query === 'фыввфывфв') {
    echo 'Ага';
} elseif (preg_match('//', $query) {
    echo 'Ясно';
} else {
    echo 'Я не понял тебя';
}

Не является ли это говнокодом?

Comments

[twobomb]

АХахха, мда..... А если у вас будет 1000 вариантов ответа на слово вы так 1000 раз писать будете? Используйте хотябы объект.{вопрос:ответ}

{
привет:привет,
фыввфывфв:Ага
}

Далее перебираете foreach key in val.., а если уже не нашли то

if (preg_match('//', $query) {
    echo 'Ясно';
} else {
    echo 'Я не понял тебя';

Answer 1

[Lee Masagos]

Лучше храните слова в том же json + один класс который это все будет обрабатывать и давать результат
Конечно говнокод, это же для 1000 слов 1000 таких if ?

Comments

[E6APB]

У меня не только ответы. У меня могут быть и вызовы некоторых методов

[Lee Masagos]

E6apb: даже через switch это грустно
Используйте бд, или для больших объемов elastic search, или отдельно даже простейший txt по которому пройдется функция
можно хранить в формате method\word\answer

Answer 2

[xmoonlight]

1. Смотрите switch-case
2. Используйте рекурсивный спуск по веткам наилучших соответствий для выдачи (отображения/echo) нужных вариантов ответа.

UPD: Если хочется реально круто сделать:
Советую попробовать сравнение через Stumper API.
По наибольшему проценту совпадения с имеющимися у вас фразами - даёте нужный ответ.

Answer 3

[Павел Корнилов]

Конечно, ГК.
Нужно хотя бы использовать регулярные выражения, поскольку 'Привет' !== 'привет!'.