Parser php oracle не пишет в БД?

Question

[Sanya899]

Выдает ошибку:

Warning: oci_execute() expects parameter 1 to be resource, string given in

require_once "simple_html_dom.php";

        error_reporting(E_ALL);
        ini_set('display_errors', '1');
        $conn = oci_connect('ddd', 'ddd', '127.0.0.1/orcl', 'AL32UTF8');


    $brands = array();

    $html = str_get_html(file_get_contents('https://kolesa.kz/cars/'));
    $insert = oci_parse($conn, 'INSERT INTO kolesa_brands (id, name) VALUES');
    $select = $html->find('select[id=auto-car-mm-0]', 0);
    foreach($select->find('option') as $opt)
    {
        if($opt->value == '') continue;
        $insert .= '('.$opt->value.', \''.$opt->plaintext.'\'),';
        $brands[$opt->value] = $opt->plaintext; 
    }
    oci_execute(rtrim($insert, ','));

Comments

[idShura]

можешь показать что находится в переменной $insert перед тем как выполняешь oci_execute.

[Sanya899]

В переменной $insert находится таблица с двумя полями id и name , а до этого собирается массив с брендами автомобилей который по идее должен вставить в эту таблицу kolesa_brands. Я уже задолбался весь гугл перерыл.

[idShura]

Sanya899, судя по коду insert собирается правильно, ошибка скорее все в подставляемых значениях $opt->value или $opt->plaintext - поэтому и прошу запрос показать.

[Дмитрий]

Sanya899, массив брендов ни где не используется

[Sanya899]

idShura,

<?php 
require_once "simple_html_dom.php";

error_reporting(E_ALL);
    ini_set('display_errors', '1');
    $conn = oci_connect('dev', 'def', '127.0.0.1/orcl');


$brands = array();

$html = str_get_html(file_get_contents('https://kolesa.kz/cars/'));
$insert = oci_parse($conn, 'INSERT INTO kolesa_brands (id, name) VALUES');
$select = $html->find('select[id=auto-car-mm-0]', 0);
foreach($select->find('option') as $opt)
{
    if($opt->value == '') continue;
    $insert .= '('.$opt->value.', \''.$opt->plaintext.'\'),';
    $brands[$opt->value] = $opt->plaintext; 
}
//echo $brands;
echo 
'<pre>',
print_r($brands),
'</pre>';

Если откинуть insert b прочее на выходе получаем вот что:

Array
(
[4] => ARO
[1] => Acura
[2] => Alfa Romeo
[3] => Alpina
[6] => Aston Martin
[7] => Audi
[150] => BAW
[11] => BMW
[15] => BYD
[9] => Bentley
[140] => Brilliance
[14] => Buick
[16] => Cadillac
[17] => Chana
[122] => Changan
[18] => Chery
[19] => Chevrolet
[20] => Chrysler
[21] => Citroen
[141] => DFM
[137] => Dacia
[22] => Dadi
[23] => Daewoo
[24] => Daihatsu
[186] => Daimler
[25] => Datsun
[143] => Derways
[26] => Dodge
[27] => DongFeng
[29] => FAW
[30] => Ferrari
[31] => Fiat
[133] => Fisker
[146] => Force
[32] => Ford
[124] => Foton
[35] => GMC
[33] => Geely
[125] => Gonow
[36] => Great Wall
[37] => Hafei
[149] => Haima
[40] => Honda
[41] => Huanghai
[42] => Hummer
[43] => Hyundai
[44] => Infiniti
[152] => Iran Khodro
[45] => Isuzu
[138] => Iveco
[46] => JAC
[49] => JMC
[47] => Jaguar
[48] => Jeep
[128] => Jin Bei
[50] => Kia
[127] => King Long
[51] => Lamborghini
[52] => Lancia
[53] => Land Rover
[54] => Lexus
[55] => Lifan
[56] => Lincoln
[57] => Lotus
[119] => Luxgen
[129] => MG
[173] => Marussia
[59] => Maserati
[60] => Maybach
[61] => Mazda
[62] => Mercedes-Benz
[154] => Mercedes-Maybach
[63] => Mercury
[64] => Mini
[65] => Mitsubishi
[66] => Nissan
[68] => Opel
[69] => Peugeot
[72] => Pontiac
[73] => Porsche
[162] => Puch
[166] => Ravon
[76] => Renault
[80] => Renault Samsung
[77] => Rolls-Royce
[78] => Rover
[79] => Saab
[81] => Saturn
[82] => Sceo
[83] => Scion
[84] => Seat
[153] => Shuanghuan
[86] => Skoda
[87] => Smart
[89] => SsangYong
[90] => Subaru
[91] => Suzuki
[93] => Tatra
[123] => Tesla
[94] => Tianma
[96] => Toyota
[98] => Volkswagen
[99] => Volvo
[147] => Vortex
[130] => Wuling
[101] => Xinkai
[106] => ZX
[107] => ВАЗ (Lada)
[108] => ВИС
[109] => ГАЗ
[136] => ЕрАЗ
[110] => ЗАЗ
[112] => ИЖ
[113] => ЛуАЗ
[114] => Москвич
[117] => РАФ
[145] => Ретро-автомобили
[163] => СМЗ
[118] => ТагАЗ
[116] => УАЗ
)
1

[idShura]

Sanya899, я $insert просил показать, а не $brands. Кстати посмотри ответ Kerm, похоже он правильно ответил. Единственное я не понимаю что такое oci_parse.

[Sanya899]

idShura, только что попробовал

Warning: oci_execute(): ORA-00933: SQL command not properly ended

[Sanya899]

Sanya899, в перемнную $insert приходит вот что:

Resource id #6(4, 'ARO'),(1, 'Acura'),(2, 'Alfa Romeo'),(3, 'Alpina'),(6, 'Aston Martin'),(7, 'Audi'),(150, 'BAW'),(11, 'BMW'),(15, 'BYD'),(9, 'Bentley')

ну и т.д

[Дмитрий]

Sanya899, в конце нет запятой?

[Kerm]

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

Answer 1

[Kerm]

Попробуй вот так:

require_once "simple_html_dom.php";

error_reporting(E_ALL);
ini_set('display_errors', '1');
$conn = oci_connect('ddd', 'ddd', '127.0.0.1/orcl', 'AL32UTF8');


$brands = array();

$html = str_get_html(file_get_contents('https://kolesa.kz/cars/'));
$i = 'INSERT INTO kolesa_brands (id, name) VALUES';
$select = $html->find('select[id=auto-car-mm-0]', 0);

foreach($select->find('option') as $opt)
{
	if($opt->value == '') continue;
	$i .= ' ('.$opt->value.', \''.$opt->plaintext.'\'),';
	$brands[$opt->value] = $opt->plaintext; 
}

$insert = oci_parse($conn, rtrim($i, ',').';');

oci_execute($insert);

Comments

[Sanya899]

Пишет вот что:
Warning: oci_execute(): ORA-00933: SQL command not properly ended

[Kerm]

Sanya899, точку с запятой в конце нужно поставить!

[Sanya899]

Так все же стоит где именно то ?

[Kerm]

Sanya899, в конце любого SQL запроса должна быть точка с запятой, я исправил:

$insert = oci_parse($conn, rtrim($i, ',').';');

[Sanya899]

опять та же ошибка. неверное sql предложение.

[Kerm]

Sanya899, тут уже смотри ответ другого юзера, про то что оракл не поддерживает такого вида запросы!

[Sanya899]

нуууууууууууууууууууууууууууууууууууу нетттт

[Sanya899]

Друзья,товарищи подскажите как сделать так ,чтобы поддерживал пожалуйста

[Kerm]

Sanya899, окей попробуй вот так:

require_once "simple_html_dom.php";

error_reporting(E_ALL);
ini_set('display_errors', '1');
$conn = oci_connect('ddd', 'ddd', '127.0.0.1/orcl', 'AL32UTF8');


$brands = array();

$html = str_get_html(file_get_contents('https://kolesa.kz/cars/'));
$i = 'INSERT ALL';
$select = $html->find('select[id=auto-car-mm-0]', 0);

foreach($select->find('option') as $opt)
{
	if($opt->value == '') continue;
	$i .= ' INTO kolesa_brands (id, name) VALUES('.$opt->value.', \''.$opt->plaintext.'\')';
	$brands[$opt->value] = $opt->plaintext; 
}

$insert = oci_parse($conn, $i.' SELECT * FROM dual;');

oci_execute($insert);

[Sanya899]

Спасибо большое все получилось))!

[Kerm]

Sanya899, рад помочь, поставь там тогда что мой ответ является решением

[Sanya899]

Как это "отметить решением" ?

[Sanya899]

только сверху я так понимаю ну да ладно. Могу рассчитывать на вашу помощь если возникнут вопросы?

[Kerm]

Sanya899, da

[Sanya899]

Kerm, Код работает все гуд парсит в базу, но возник вопрос как сделать проверку?
Добавить записи если не существуют или обновить если существуют . в oracle нашел функцию merge но она добавляет по одной записи а мне нужно много записей судя по моему коду. Куда копать ?

Answer 2

[idShura]

Oracle не поддерживает такой формат вставки:

INSERT INTO kolesa_brands (id, name) VALUES (4, 'ARO'), (1, 'Acura'), (2, 'Alfa Romeo')

Я сейчас не могу ответить более подробно... гугли про INSERT ALL

Comments

[Sanya899]

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

[idShura]

Только освободился и вижу что Kerm уже помог тебе, не забудь ему отметить решение.