За последние 24 часа нас посетили 16596 программистов и 1297 роботов. Сейчас ищут 1045 программистов ...

Что не так с кодом?

Тема в разделе "PHP для новичков", создана пользователем Sergeyka81, 3 авг 2024.

Метки:
  1. Sergeyka81

    Sergeyka81 Гость

    С нами с:
    3 авг 2024
    Сообщения:
    2
    Симпатии:
    0
    Задание

    Написать функцию формирования sql-запросов (MySQL) из шаблона и значений параметров.

    Места вставки значений в шаблон помечаются вопросительным знаком, после которого может следовать спецификатор преобразования.
    Спецификаторы:
    ?d - конвертация в целое число
    ?f - конвертация в число с плавающей точкой
    ?a - массив значений
    ?# - идентификатор или массив идентификаторов

    Если спецификатор не указан, то используется тип переданного значения, но допускаются только типы string, int, float, bool (приводится к 0 или 1) и null.
    Параметры ?, ?d, ?f могут принимать значения null (в этом случае в шаблон вставляется NULL).
    Строки и идентификаторы автоматически экранируются.

    Массив (параметр ?a) преобразуется либо в список значений через запятую (список), либо в пары идентификатор и значение через запятую (ассоциативный массив).
    Каждое значение из массива форматируется в зависимости от его типа (идентично универсальному параметру без спецификатора).

    Также необходимо реализовать условные блоки, помечаемые фигурными скобками.
    Если внутри условного блока есть хотя бы один параметр со специальным значением, то блок не попадает в сформированный запрос.
    Специальное значение должно возвращаться методом skip. Нужно выбрать подходящее значение на своё усмотрение.
    Условные блоки не могут быть вложенными.

    При ошибках в шаблонах или значениях выбрасывать исключения.

    Вот моё решение:

    PHP:
    1. namespace FpDbTest;
    2.  
    3. use Exception;
    4. use mysqli;
    5.  
    6. class Database implements DatabaseInterface
    7. {
    8.     private mysqli $mysqli;
    9.  
    10.     public function __construct(mysqli $mysqli)
    11.     {
    12.         $this->mysqli = $mysqli;
    13.     }
    14.     private function swap(&$val1, &$val2)
    15.     {
    16.     $tmp=$val1;
    17.     $val1=$val2;
    18.     $val2=$tmp;
    19.     }
    20.     private function getFunc($query, &$pos, $max, &$suffix)
    21.     {
    22.         $func = 'unknown_param';
    23.         $param_len = 0;
    24.     do
    25.     {
    26.         $pos++;
    27.         switch($query[$pos])
    28.         {
    29.         case ' ':
    30.             $suffix .= ' ';
    31.             break;
    32.         case 'd':
    33.                 $func = 'd_param';
    34.             break;
    35.         case 'f':
    36.             $func = 'f_param';
    37.             break;
    38.         case 'a':
    39.             $func = 'a_param';
    40.             break;
    41.         case '#':
    42.             $func = 'dies_param';
    43.             break;
    44.         default:
    45.             throw new \Exception("Unknown param type: ".$query[$pos]." in ".$query);
    46.         }
    47.         $param_len++;
    48.     }
    49.     while($pos<$max && !$query[$pos]===' ');
    50.    
    51.     switch($param_len)
    52.     {
    53.         case 0:
    54.         $func = 'unknown_param';
    55.         break;
    56.         case 1:
    57.             break;
    58.         default:
    59.             throw new Exception("Bad param: ".$param_len);
    60.     }
    61.     return $func;
    62.     }
    63.  
    64.     public function buildQuery(string $query, array $args = []): string
    65.     {
    66.     //debug print "template: ".$query."\n";
    67.     $result = '';
    68.     $buf = '';
    69.     $pos = 0;
    70.     $max = strlen($query);
    71.     $param = false;
    72.     $index_param = 0;
    73.     $skip = false;
    74.     $block = '';
    75.     while($pos<$max)
    76.     {  
    77.         //debug print "scan: [".$query[$pos]."]\n";
    78.         switch($query[$pos])
    79.         {
    80.         case '}':
    81.             //debug print "clear skip: [".$buf."]\n";
    82.             $result .= $buf;
    83.             $buf = '';
    84.             $this->swap($result, $block);
    85.             if($skip)
    86.             {
    87.                 $skip = false;
    88.                 $block = '';
    89.                 break;
    90.             }
    91.             //debug print "conncat block: [".$block."]\n";
    92.             $result .= $block;
    93.             break;
    94.         case '{':
    95.             $result .= $buf;
    96.             $buf='';
    97.             $skip = $args[$index_param]===$this->skip();
    98.             $this->swap($result, $block);
    99.             //debug print "read block: [".$block."], concat: [".$result."]\n";
    100.             break;
    101.         case '?':
    102.             //debug print "case ?, buf: [".$buf."] param^".$args[$index_param]."\n";
    103.             $suffix = '';
    104.             $func = $this->getFunc($query, $pos, $max, $suffix);
    105.             if($args[$index_param]===$this->skip() || $skip)
    106.             {
    107.             //debug print "set skip: [".$buf."]\n";
    108.             $skip = true;
    109.             $index_param++;
    110.             break;
    111.             }
    112.             //debug print "insert param: ".$args[$index_param]."\n";
    113.             $p = $this->$func($args[$index_param]);
    114.             $result .= $buf.$p.$suffix;
    115.             $index_param++;
    116.             $buf = '';
    117.             //debug print "param: [".$p."] concat [".$result."]\n";
    118.             break;
    119.         case ' ':
    120.             //debug print "case space\n";
    121.             $keyword = strtolower($buf);
    122.             switch($keyword)
    123.             {
    124.             case "select":
    125.             case "from":
    126.             case "where":
    127.             case "update":
    128.             case "set":
    129.             case "and":
    130.             case "in":
    131.                 //debug print "case keyword: [".$keyword."]\n";
    132.                 $buf = strtoupper($buf);
    133.             }
    134.             $result .= $buf.' ';
    135.             //debug print "keyword: [".$keyword."], buf: [".$buf."], concat: [".$result."].\n";
    136.             $buf = '';
    137.             break;
    138.         default:
    139.             $buf.= $query[$pos];
    140.             //debug print "case default, buf:[".$buf."]\n";
    141.  
    142.         }
    143.         $pos++;
    144.     }
    145.     //debug print "pos:".$pos." max:".$max."\n";
    146.     //debug print "result: [".$result.$buf."]\n";
    147.     return $result.$buf;
    148.     }
    149.  
    150.     private function d_param($arg)
    151.     {
    152.     if(is_null($arg))
    153.         return 'NULL';
    154.  
    155.     return sprintf('%d',$arg);
    156.     }
    157.  
    158.     private function f_param($arg)
    159.     {
    160.     if(is_null($arg))
    161.         return 'NULL';
    162.  
    163.     return sprintf("%f",$arg);
    164.     }
    165.  
    166.     private function a_param($arg)
    167.     {
    168.     if(count($arg)===0)
    169.         throw new Exception("Bad param value");
    170.     $ret='';
    171.     $i=0;
    172.     if(is_int(array_key_first($arg)))
    173.     {
    174.         foreach($arg as $val)
    175.         {
    176.         $ret .= $i===0 ? '' : ', ';
    177.         $ret .= $this->unknown_param($val);
    178.         $i++;
    179.         }
    180.     }
    181.     else
    182.     {
    183.         reset($arg);
    184.         $i = 0;
    185.         foreach($arg as $key => $val)
    186.         {
    187.         $ret .= $i===0 ? '' : ', ';
    188.         $ret .= '`'.$key . '` = '.$this->unknown_param($val);
    189.         $i++;
    190.         }
    191.     }
    192.     return $ret;
    193.  
    194.     }
    195.  
    196.     private function dies_param($arg)
    197.     {
    198.     //debug print ("# param: ".$arg."\n");
    199.     $ret = '';
    200.     $i = 0;
    201.     if(is_array($arg))
    202.     {
    203.             foreach($arg as $v)
    204.             {
    205.         $ret .= $i===0 ? '' : ', ';
    206.         $ret .= '`'.$v.'`';
    207.         $i++;
    208.         }
    209.     }
    210.     else
    211.     {
    212.         $ret = '`'.$arg.'`';
    213.     }
    214.    
    215.     return $ret;
    216.  
    217.     }
    218.  
    219.     private function unknown_param($arg)
    220.     {
    221.     if(is_null($arg))
    222.         return 'NULL';
    223.  
    224.     switch(gettype($arg))
    225.     {
    226.         case 'string':
    227.         return '\''.$arg.'\'';
    228.         case 'double':
    229.         return $this->f_param($arg);
    230.         case 'integer':
    231.         return $this->d_param($arg);
    232.         case 'boolean':
    233.         return $arg ? 1 : 0;
    234.         default:
    235.         throw new Exception ("Bad type param");
    236.        
    237.     }
    238.     return null;
    239.     }
    240.  
    241.     public function skip()
    242.     {
    243.     return "skip\n";
    244.     }
    245. }
    Как его можно улучшить?