Как ускорить производительность?

Question

[wyylame]

Добрый вечер. Есть функция, которая считывает данные с бд для топа и выводит (бот вк).

def vkGetFirstName(id):
	return vkapi.users.get(user_id=id)[0]['first_name']
def vkGetLastName(id):
	return vkapi.users.get(user_id=id)[0]['last_name']

def e_top(chatId):
	table.execute("""SELECT user_id, drop_count FROM vkbottable WHERE chat_id = '{}' ORDER BY drop_count DESC""".format(chatId))
	text = "Топ еб**ноидов:\n"
	for i, row in enumerate(table):
		text += "{}. {} {} - {} раз\n".format(i+1, vkGetFirstName(int(row[0])), vkGetLastName(int(row[0])), row[1])
	return text

print(e_top(2000000004))

Проблема в том, что это работает практически 13 секунд (12.6 в среднем). Можно ускорить этот кусок кода?

Answer 1

[ScriptKiddo]

text += "{}. {} {} - {} раз\n".format(i+1, vkGetFirstName(int(row[0])), vkGetLastName(int(row[0])), row[1])

Очевидно, имя и фамилия меняются не настолько часто, чтобы запрашивать их каждый раз при выводе топа.

Пишите их в БД или пакуйте запросы пачками в метод Execute, если очень нужно каждый раз запрашивать ИФ

Comments

[edward_freedom]

что тут говорить, он имя и фамилию двумя запросами получает

[wyylame]

edward_freedom, я начинающий говнокодер, ацтаньте((9(

[edward_freedom]

Уильям Хорн, но логика же должна быть, если ты программировать начал.

def vkGetUser(id):
  return vkapi.users.get(user_id=id)[0]

user = vkGetUser(1);
userFirst_name = user["first_name"];
userLast_name = user["last_name"];

[wyylame]

edward_freedom, спасибо)

[edward_freedom]

Уильям Хорн, так же в документации посмотри метод execute. Ты сможешь за 1 раз, выполнять 25 запросов к апи

[wyylame]

edward_freedom, хорошо. Но в сейчас сделал, как посоветовал ScriptKiddo, через бд. Время выполнения ~1 секунда :)

Answer 2

[o5a]

users.get может принимать сразу массив user_ids, так что можно сразу получить данные всех пользователей, а не тягать каждого по 1 запросу (тем более, не тягать по 2 запроса на имя и фамилию в отдельности).