Параметры запроса

@Senarist · Регистрация: 14.03.2014

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

Есть словарь dic который я заполняю данными из запроса. Затем его передаю в функцию get_product_id. Там делаю запрос к БД где аргумент dic[6] это число (тип int). В результате исполнения запроса вылавливаю исключение: SQL query parameters should be a tuple, list or dict; got 368312
Как оно может быть tuple, list or dict если значение параметра это число типа int ?
Если что то это odoo 12

Python

def download_pdf_file(self, result):
        for row in range(result.numberOfRows):
            dic = {}
            for col in range(result.numberOfColumns):
                if col == 0:  # flname
                    dic[col] = result.ado_results[col][row]
                elif col == 1:  # filebody
                    dic[col] = result.ado_results[col][row].obj
                elif col == 2:  # filesize
                    dic[col] = result.ado_results[col][row]
                elif col == 3:  # filedate
                    dic[col] = result.ado_results[col][row]
                elif col == 4:  # doclist.designatio
                    dic[col] = result.ado_results[col][row]
                elif col == 5:  # articles.designatio
                    dic[col] = result.ado_results[col][row]
                elif col == 6:  # art_id
                    dic[col] = result.ado_results[col][row]
 
            code = self.get_product_id(dic)
 
   def get_product_id(self, dic):
       result = self.env.cr.execute(
           'Select id from product_product where code_search = %s', dic[6])
 
       return result

@Fudthhh · 10.01.2020, 14:01

Тебе пиндоским языком написано: SQL query parameters should be a tuple, list or dict

Python

def get_product_id(self, dic):
       result = self.env.cr.execute(
           'Select id from product_product where code_search = %s', (dic[6], ))
       return result

ИЛИ

Python

def get_product_id(self, dic):
       result = self.env.cr.execute(
           'Select id from product_product where code_search = %s', [dic[6]])
       return result

Добавлено через 1 минуту
И так от себя, лучше использовать это так:

'SELECT id FROM product_product WHERE code_search = %s', [dic[6]])

Рыжий Лис · 10.01.2020, 14:27

Сообщение от Senarist

dic = {}
for col in range(result.numberOfColumns):
if col == 0: # flname
dic[col] = result.ado_results[col][row]
elif col == 1: # filebody
dic[col] = result.ado_results[col][row].obj

Зачем цикл и почему не?

Python

dic = {
   0: result.ado_results[0][row],
   1: result.ado_results[1][row].obj.
   #…
}

@Senarist · 10.01.2020, 14:34 **[ТС]**

DmFat, это я сам затупил, тип этого dic[6] в БД строковый а у меня число.

Добавлено через 42 секунды
Рыжий Лис, цикл по row все равно нужен, с col согласен, можно и без него...

Рыжий Лис · 10.01.2020, 14:57

Правильнее вообще изолировать функцию get_product_id от внешних зависимостей:

Python

1
2
3

def get_product_id(self, code_search):
    sql = 'select id from product_product where code_search=%s'
    return self.env.cr.execute(sql, (code_search,))

И вызывать так:

Python

1	code = self.get_product_id(code_search=result.ado_results[6][row])

@Senarist · 11.01.2020, 20:09 **[ТС]**

Рыжий Лис, спасибо, а что имеется в виду под внешней зависимостью?

Рыжий Лис · 11.01.2020, 20:28

Не совсем правильно выразился. Функция не должна знать тонкости реализации окружающего её кода.

Для работы ей нужна строка - пусть во входных аргументах она принимает строку. Просто рано или поздно захочется переиспользовать эту функцию, и придётся гадать, а почему её надо вызывать func({6:var}) вместо func(var).

@Senarist 58 / 62 / 34 Регистрация: 14.03.2014 Сообщений: 905
	10.01.2020, 14:34 [ТС]	4
	DmFat, это я сам затупил, тип этого dic[6] в БД строковый а у меня число. Добавлено через 42 секунды Рыжий Лис, цикл по row все равно нужен, с col согласен, можно и без него... 0

@Senarist 58 / 62 / 34 Регистрация: 14.03.2014 Сообщений: 905
	11.01.2020, 20:09 [ТС]	6
	Рыжий Лис, спасибо, а что имеется в виду под внешней зависимостью? 0

Рыжий Лис Просто Лис 5710 / 3512 / 1052 Регистрация: 17.05.2012 Сообщений: 10,328 Записей в блоге: 9
	11.01.2020, 20:28	7
	Не совсем правильно выразился. Функция не должна знать тонкости реализации окружающего её кода. Для работы ей нужна строка - пусть во входных аргументах она принимает строку. Просто рано или поздно захочется переиспользовать эту функцию, и придётся гадать, а почему её надо вызывать func({6:var}) вместо func(var). 1

Опции темы