Как красиво и вообще правильно организовать запросы

@Roman_rc · Регистрация: 04.02.2013

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

Сохраняю большое количество данных в разных связанных, зависимых и т.п. таблицах инсертами. Но выглядит это просто УЖАСНО!

Вот например один из фрагментов:

PHP

      $this->db->trans_start();
      
      $this->db->query($this->onDubKeyUpdate('catigory', 'name_ru', $goodsCatigory));
      
      $this->db->query("SET @catig_id = (SELECT `id` FROM catigory WHERE `name_ru` = '$goodsCatigory' LIMIT 1);");
         
      $this->db->query($this->onDubKeyUpdate('brand', 'name', $goodsBrand)); 
       
      $this->db->query("SET @brand_id = (SELECT `id` FROM brand WHERE `name` = '$goodsBrand' LIMIT 1);");
 
      $goodsVisibility = $goodsVisibility?1:0;
      $goodsDelete= $goodsDelete?1:0;
      $this->db->query("INSERT INTO product 
                         (`name`, `regNumber`, `formFactor`, 
                         `portion_count`, `weight`, `measure`, 
                         `priceRetail`, `priceGross`, `PriceBonus`, 
                      `currentDiscountRetail`, `catigoryFolder`, 
                         `fileName`, `visibility`, `delete`, 
                      `catigory_id`, `brand_id`) 
                          VALUES 
                       ('$goodsName', '$goodsRegnumber','$goodsFormfactor', 
                          $goodsPortions, $goodsWeight, '$goodsMeasureWeightVal', 
                          $goodsPriceRetail, $goodsPriceGross, $goodsPriceBonus,
                          $goodsPriceDiscount, 'test', 
                          'filename2121012', $goodsVisibility, $goodsDelete, 
                       @catig_id, @brand_id);"); 
 ...
 
     $this->db->trans_complete();

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

@V@D!k · 10.10.2014, 13:18

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

PHP

1	$this->db->query("INSERT INTO product (".implode(',',$keys).") VALUES (".implode(',',$values).")");

где $keys -названия полей, $values их значения

@Roman_rc · 10.10.2014, 15:30 **[ТС]**

Спасибо! Согласен такой подход получше) Никогда не работал с большим количеством таблиц, которые нужно добавлять одновременно.

@noname_club · 10.10.2014, 23:58

$this->db->query("INSERT INTO product
(`name`, `regNumber`, `formFactor`,
`portion_count`, `weight`, `measure`,
`priceRetail`, `priceGross`, `PriceBonus`,
`currentDiscountRetail`, `catigoryFolder`,
`fileName`, `visibility`, `delete`,
`catigory_id`, `brand_id`)
VALUES
('$goodsName', '$goodsRegnumber','$goodsFormfactor',
$goodsPortions, $goodsWeight, '$goodsMeasureWeightVal',
$goodsPriceRetail, $goodsPriceGross, $goodsPriceBonus,
$goodsPriceDiscount, 'test',
'filename2121012', $goodsVisibility, $goodsDelete,
@catig_id, @brand_id);");

PHP

$data = array(
    'name' =>  $goodsRegnumber, 
    'regNumber' =>  $goodsRegnumber,
    'formFactor' => $goodsFormfactor,  
    'portion_count' => $goodsPortions,
    'weight' => $goodsWeight,
    'measure' => $goodsMeasureWeightVal, 
    'priceRetail' => $goodsPriceRetail, 
    'priceGross' => $goodsPriceGross, 
    'PriceBonus' => $goodsPriceBonus, 
    'currentDiscountRetail' => $goodsPriceDiscount,
    'catigoryFolder' => 'test',
    'fileName' => 'filename2121012',
    'visibility' => $goodsVisibility, 
    'delete' => $goodsDelete,
    'catigory_id' => '@catig_id', 
    'brand_id' => '@brand_id'
);
 
// вставка
 
$db->insert('product',$data);
 
// обновление данных
 
$db->update('product',$data, 'id ='.$id);

Естественно при наличии этих "функций" внутри интерфейса бд.
Код функций предоставить не могу так как он является частью кода класса бд который все равно вам не подойдет.

Но вот сама идея вполне.

@Roman_rc · 11.10.2014, 20:14 **[ТС]**

Естественно при наличии этих "функций" внутри интерфейса бд.

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

Кстати говоря я такой массив экстактирую extract($arr), то есть наоборот перевожу в контексте на переменные. В общем думаю сделать класс который будет обрабатывать такие данные при этом сами запросы определить глобально и по необходимости заменять значения переменными.

@noname_club · 12.10.2014, 09:59

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

в цикле не?

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

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

@Roman_rc 26 / 26 / 12 Регистрация: 04.02.2013 Сообщений: 250
	10.10.2014, 15:30 [ТС]	3
	Спасибо! Согласен такой подход получше) Никогда не работал с большим количеством таблиц, которые нужно добавлять одновременно. 0

@Roman_rc 26 / 26 / 12 Регистрация: 04.02.2013 Сообщений: 250
	11.10.2014, 20:14 [ТС]	5
	Естественно при наличии этих "функций" внутри интерфейса бд. Думал о функциях и процедурах если я правильно Вас понял, но вот у меня от пользователя может придти массив с любым количеством и как передать такой аргумент процедуре или функции в бд не понятно. Кстати говоря я такой массив экстактирую extract($arr), то есть наоборот перевожу в контексте на переменные. В общем думаю сделать класс который будет обрабатывать такие данные при этом сами запросы определить глобально и по необходимости заменять значения переменными. 0

@noname_club 112 / 103 / 12 Регистрация: 01.05.2013 Сообщений: 603
	12.10.2014, 09:59	6
	у меня от пользователя может придти массив с любым количеством в цикле не? В общем думаю сделать класс который будет обрабатывать такие данные при этом сами запросы определить глобально и по необходимости заменять значения переменными. вообще когда данные "предметной области" превращаются в кашу, то можно их обобщить в JSON поле котором может лежать что угодно. 0

Опции темы