Стоит ли использовать штатные методы OpenCart для изменения категорий и товаров при импорте из 1с?

Question

[Александр Синицын]

Хотел использовать методы edit для сохранения категорий и товаров, но что-то напрягает код запросов с его DELETE - INSERT .

$this->db->query("UPDATE " ..... WHERE product_id = '" . (int)$product_id . "'");

	if (isset($data['image'])) {
		$this->db->query("UPDATE " . DB_PREFIX . "product SET image = '" . $this->db->escape($data['image']) . "' WHERE product_id = '" . (int)$product_id . "'");
		}

		$this->db->query("DELETE FROM " . DB_PREFIX . "product_description WHERE product_id = '" . (int)$product_id . "'");

		foreach ($data['product_description'] as $language_id => $value) {
			$this->db->query("INSERT INTO " . DB_PREFIX . "product_description .....);
		}

		$this->db->query("DELETE FROM " . DB_PREFIX . "product_to_store WHERE product_id = '" . (int)$product_id . "'");

		if (isset($data['product_store'])) {
			foreach ($data['product_store'] as $store_id) {
				$this->db->query("INSERT INTO " . DB_PREFIX . "product_to_store ......);
			}
		}

		$this->db->query("DELETE FROM " . DB_PREFIX . "product_attribute WHERE product_id = '" . (int)$product_id . "'");

		if (!empty($data['product_attribute'])) {
			foreach ($data['product_attribute'] as $product_attribute) {
				if ($product_attribute['attribute_id']) {
					foreach ($product_attribute['product_attribute_description'] as $language_id => $product_attribute_description) {
						$this->db->query("INSERT INTO " . DB_PREFIX . "product_attribute .......);
					}
				}
			}
		}

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

Answer 1

[1011]

Можно узнать, каким образом делаете импорт из 1с в опенкарт? через csv?

Comments

[Александр Синицын]

Да. Не штатная обработка.

[1011]

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

[Александр Синицын]

1011: Вопрос в частности о том как быстро переполнится int(11)

[Александр Синицын]

1011: Прикинул... даже если 50 000 товаров по 10 характеристик, при обновлении 2 раза в час, этого хватит на 10 лет, пожалуй магазины столько не живут ))

[1011]

если 2 раза в час то это уже не разовый импорт а односторонняя синхронизация, здесь лучше что то свое придумать. Допустим выборку из таблицы донора и таблицы акцептора, сравнение и только затем sql - запросы. у меня так это реализовано.

[Александр Синицын]

1011: А выборка не может оказаться слишком большой?

[1011]

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

Answer 2

[Григорий Васильков]

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

Например, в Битриксе при добавлении элементов через админку или апи автоматически заполняется целая куча полей и если самому это вручную делать - можно уехать крышей.
В то же время АПИ битрикса не упрощает, а даже усложняет доступ к БД - твой код становится даже сложнее, чем писать SQL запрос. Поэтому все равно прибегаем к хитростям вида "запросить SQL-ом, закачать через Апи"

Касательно твоего примера мы наблюдаем в Опенкарте практически чистый SQL и потому нет разницы - будешь ты его импортировать через собственное подключение к бд, не забывая экранировать спецсимволы, или будешь использовать Опенкарт, который бесплатный, а значит ни за что не несет ответственности и вполне вероятно, что там такие проверки тоже не делаются, что делает потенциальную дыру для взлома.

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

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

И этому радостно подпевает голливуд, рассказывая, что из-за ученых злобные вирусы заражают человечество, поэтому в нашем мире лучше дескать вообще нихрена не делать, а только ждать пока само сделается. А оно, как ни странно, не сделается.

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

Бизнес в целом это всего три понятия:
1) деньги принадлежат хитрым, а не умным. Это разные вещи.
2) никогда не будь виноватым, твой авторитет и внешний вид важнее любой работы, которую никто не увидит, поэтому лучше не делай ничего, а всегда имей крайнего
3) если ты видишь, что кто-то упал, помоги ему сломать побольше конечностей, а всем расскажи, что "он же сам хотел упасть, а ты честно помог"

И это все достижимо только если самому не делать вообще ничего, а всегда иметь крайнего.

И понимая это, какими будут твои действия? С точки зрения логики - не работать на бизнес и на деньги. Но ведь это заставит тебя копать землю и растить для себя продукты, а значит бросить программирование... поэтому просто вливайся в колею и улыбайся, делая вид, что ты глупый и ничего не понимаешь.

Comments

[Александр Синицын]

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

И вообще это уже философия, а она вне тематики данного ресурса ))

[Григорий Васильков]

600... нет погоди, 2100 лет не приходили на смену, но как только Ютуб появился, все бизнесмены подобрели, если кого-то кинули, это потому что он не был позитивен, а раньше потому что бизнесмены были злыми. А сейчас добрые, но кидают также

[Александр Синицын]

Григорий Васильков:
Кидалы, кидали, кидают и будут кидать,
Ютюб как им в этом может мешать?
Потому как кидают они не с проста
К киданью лежит их пустая душа.

Похоже я похоронил в себе стихоплета )))