0

Всем здравствуйте, подскажите как исправить ошибку Cannot redeclare createQuery(), как я понял нельзя так две функции писать, скорее всего это грубейшая ошибка, помогите решить эту задачу..

                function createQuery($id, $a, $b){
                    $query = "INSERT INTO opisanie (x,text,text_opis) VALUES ";
                    $limit = count($a);
                    for($i=0; $i<$limit; $i++){
                        $query .= "('{$id}','{$a[$i]}', '{$b[$i]}')".($i!=$limit-1?',':'');
                    }
                    $res = mysql_query($query);
                }

                $aText = $_POST['form_text_o'];
                $aOpis = $_POST['form_text_opis'];

                $query = createQuery($id, $aText, $aOpis);



                function createQuery( $e, $s, $c, $d, $id){
                    $queryz = "INSERT INTO menu_zav (name,sostav,gramm,cena,x_zav) VALUES ";
                    $limita = count($e);
                    for($is=0; $is<$limita; $is++){
                        $queryz .= "('{$e[$is]}','{$s[$is]}','{$c[$is]}','{$d[$is]}','{$id}')".($is!=$limita-1?',':'');
                    }
                    $resa = mysql_query($queryz);
                }

                $eName = $_POST['form_name_menu'];
                $eOpisa = $_POST['form_opis_menu'];
                $eGr = $_POST['form_gr_menu'];
                $eCena = $_POST['form_cena_menu'];

                $queryz = createQuery($eName, $eOpisa, $eGr, $eCena, $id);

3 ответа 3

1

Так как это PHP, а не Java или C#, к примеру, где можно писать одинаковые имена методов, но с разной сигнатурой (перегрузка методов), то единственным выходом является объявление метода с другим именем. Как его называть решать уже вам.

Только называйте осмысленно, а не createQuery1 и createQuery2

4
  • точно, спасибо за ответ, теперь буду знать! Commented 12 янв. 2017 в 11:45
  • да ноу проблэмо. Commented 12 янв. 2017 в 11:46
  • А зачем осмысленно, если createQuery1 createQuery2 варианты тоже работаю? Commented 12 янв. 2017 в 11:47
  • 1
    @harbor А потом вдруг будет createQuery3, createQuery4, createQuery5 и т.д.? А вы вспомните через год что это за квари такие и что они делают? А что возвращают? А если кто-то захочет поддерживать продукт ваш или начнет в коде разбираться? Ему что ж, надо в начале изучать ваше функции и методы вдоль и поперек, что б понять логику в месте вызова? Это же кошмар)) Ну и некультурно) Commented 12 янв. 2017 в 11:51
0

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

В заключении хочется добавить, что это не самая лучшая идея.

0

А почему Вы останавливаетесь на середине унификации? соберите все массивы в один двумерный (первая "строка" - имена полей, остальные - значения), и передавайте в функцию имя таблицы и этот массив. А список полей для вставки и блоки значений при формировании текста запроса собирайте implode-ом строки.

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.