Добро пожаловать на Pro Pawn - Портал о PAWN-скриптинге.
Показано с 1 по 2 из 2
  1. #1
    Аватар для Seviel
    Пользователь

    Статус
    Оффлайн
    Регистрация
    23.12.2016
    Адрес
    Казахстан
    Сообщений
    148
    Репутация:
    26 ±

    "Аптимизация"

    Есть у меня код, не красивый, собственно вот он, можно ли сделать из двух(ну по факту их три, но выполнится только два) один?
    1. stock AddHouseSafeItem(houseid, model, amount)
    2. {
    3. new string[102];
    4. mysql_format(connect, string, sizeof string, "SELECT `amount` FROM `house_safe` WHERE `house`='%d' AND `model`='%d'", houseid, model);
    5. new Cache:cache = mysql_query(connect, string);
    6. if(cache_get_row_count(connect) > 0) {
    7. amount += cache_get_field_content_int(0, "amount");
    8. mysql_format(connect, string, sizeof string, "UPDATE `house_safe` SET `amount`='%d' WHERE `house`='%d' AND `model`='%d'", amount, houseid, model);
    9. } else {
    10. mysql_format(connect, string, sizeof string, "INSERT INTO `house_safe`(`house`,`model`,`amount`) VALUES ('%d','%d','%d')", houseid, model, amount);
    11. }
    12. cache_delete(cache, connect);
    13. return mysql_tquery(connect, string);
    14. }

  2. #2
    Аватар для DeimoS
    Модератор?

    Статус
    Оффлайн
    Регистрация
    27.01.2014
    Адрес
    Восточный Мордор
    Сообщений
    5,588
    Репутация:
    1984 ±
    Можно попробовать сделать запрос по принципу "INSERT ... ON DUPLICATE KEY UPDATE", но не уверен, что в твоём случае это сработает + у такой реализации есть неприятный побочный эффект в виде увеличения значения AUTO_INCREMET даже тогда, когда сработает вторая часть запроса, содержащая UPDATE. При частых вызовах можно получить выход за пределы Integer. Ну и в целом такой запрос будет довольно медленный, ибо сначала MySQL просмотрит содержимое всей таблицы на предмет нахождения подходящего ID, а только потом будет выполнять действие.

    Лучше оставить общий вид таким, каким он есть сейчас. Разве что можно убрать работу с amount, перенеся её на плечи MySQL

    1. stock AddHouseSafeItem(houseid, model, amount)
    2. {
    3. new string[102];
    4. format(string, sizeof string, "SELECT 1 FROM house_safe WHERE house=%d AND model=%d", houseid, model);
    5. Cache:string[0] = mysql_query(connect, string);
    6. if(cache_get_row_count(connect))
    7. {
    8. cache_delete(Cache:string[0], connect);
    9. format(string, sizeof string, "UPDATE house_safe SET amount=amount+%d WHERE house=%d AND model=%d", amount, houseid, model);
    10. }
    11. else
    12. {
    13. format(string, sizeof string, "INSERT INTO house_safe(house,model,amount) VALUES (%d,%d,%d)", houseid, model, amount);
    14. }
    15. return mysql_tquery(connect, string);
    16. }
    Связаться со мной в VK можно через личные сообщения этой группы
    Заказы не принимаю

    Широко известно, что идеи стоят 0.8333 цента каждая (исходя из рыночной цены 10 центов за дюжину).
    Великих идей полно, на них нет спроса.
    Воплощение идеи в законченную игру требует долгой работы,
    таланта, терпения и креативности, не говоря уж о затратах денег, времени и ресурсов.
    Предложить идею просто, воплотить – вот в чём проблема

    Steve Pavlina

  3. Пользователь сказал cпасибо:
    Seviel (10.04.2018)
 

 

Информация о теме

Пользователи, просматривающие эту тему

Эту тему просматривают: 1 (пользователей: 0 , гостей: 1)

Ваши права

  • Вы не можете создавать новые темы
  • Вы не можете отвечать в темах
  • Вы не можете прикреплять вложения
  • Вы не можете редактировать свои сообщения
  •