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

Question

[R3VERS3R_2020]

Я так понимаю ошибка в sql запроса но не могу понять какое

<?php

 
$conn = oci_connect("login", "password", "DB");

	
$g1 = $_POST['value_1'];
$g2 = $_POST['value_2'];
/*$g3 = $_POST['value_3'];
$g4 = $_POST['value_4'];
$g5 = $_POST['value_5'];
$g6 = $_POST['value_6'];*/
$g7 = $_POST['value_7'];
$g8 = $_POST['value_8'];
$g9 = $_POST['value_9'];


$query = "select ";
if ($g1 = 1) {
$query .= "sum(case when t.subs_subs_id = 244 then t.sum_real_volume/60 end),"; }
if ($g2 = 1) {
$query .=  " sum(case when t.subs_subs_id = 246 then t.sum_real_volume/60 end),"; }
$query .= " trunc(t.start_date, '$g9') 
 from trafic  t,
       SUBSCRIBERS c
      where c.subs_id = t.subs_subs_id 
   and t.start_date >= to_date('$g7', 'yyyy/mm/dd')
   and t.start_date <= to_date('$g8', 'yyyy/mm/dd')
   group by trunc(t.start_date, '$g9');";
	
$stid = oci_parse($conn, $query);
if (!$stid) {
    $e = oci_error($conn);
    trigger_error(htmlentities($e['message'], ENT_QUOTES), E_USER_ERROR);
}

$r = oci_execute($stid);
while ($row = oci_fetch_array($stid, OCI_ASSOC+OCI_RETURN_NULLS))
 {
    print "<tr>\n";
    foreach ($row as $item) {
	 echo '<td>' . ($item.  "") . "</td>\n";
    }   
	}

oci_free_statement($stid);
oci_close($conn);

?>

Comments

[ThunderCat]

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

[ComodoHacker]

Выведите фактический текст запроса. Все станет понятно.

Answer 1

[ThunderCat]

var_dump($query); exit;
$stid = oci_parse($conn, $query);

И руками в консоль бд вставьте, посмотрите что не работает, да и чисто глазами проверьте все ли сформировалось правильно.

Comments

[R3VERS3R_2020]

сделал, вроде выводит все корректно

string(468) "select sum(case when t.subs_subs_id = 244 then t.sum_real_volume/60 end), sum(case when t.subs_subs_id = 246 then t.sum_real_volume/60 end), trunc(t.start_date, 'dd') from dwh.fact_itc_trafic t, dwh.ITC_DIM_SUBSCRIBERS c where c.subs_id = t.subs_subs_id and t.start_date >= to_date('2020-09-02', 'yyyy/mm/dd') and t.start_date <= to_date('2020-09-10', 'yyyy/mm/dd') group by trunc(t.start_date, 'dd')"

Answer 2

[Vitsliputsli]

if ($g1 = 1) {

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

trunc(t.start_date, '$g9')

не вставляйте значения напрямую в запрос, используйте подготовленные выражения https://www.php.net/manual/ru/function.oci-bind-by.... Скорее всего значения вставленные напрямую ломают синтаксис SQL из-за наличия кавычек.

Ну и ; не нужна в конце SQL запроса, это не PL.

Comments

[R3VERS3R_2020]

$g1 это переменная от чек-бокса, чтоб в случаи выбора этого чекбокса строка селекта была добавлена.

trunc(t.start_date, '$g9') можете написать пример как можно вводить переменная, а то от статьи как то запутался

[Vitsliputsli]

Примерно так:

$g7='2020/09/11';

$query= oci_parse($conn,"SELECT * FROM traffic t WHERE t.start_date >= to_date(:g7, 'yyyy/mm/dd')");
oci_bind_by_name($query, ":g7", $g7);
oci_execute($query);

разве что g7 лучше переименовать во чтото вразумительное.