Во-первых, тут не форум бесплатной оптимизации, а форум для помощи тем, кто хочет изучить Pawn. Если нужна оптимизация - обращайся в стол заказов и оплачивай труд того, кто согласится тебе помочь.
Во-вторых, разберись с аргументами функции format. Никаких 1200 ячеек там никуда не выделяется.
Связаться со мной в VK можно через личные сообщения этой группы
Заказы не принимаю
Широко известно, что идеи стоят 0.8333 цента каждая (исходя из рыночной цены 10 центов за дюжину).
Великих идей полно, на них нет спроса.
Воплощение идеи в законченную игру требует долгой работы,
таланта, терпения и креативности, не говоря уж о затратах денег, времени и ресурсов.
Предложить идею просто, воплотить – вот в чём проблема
Steve Pavlina
А за одно форматирование это сделать нельзя? И что это за говнобд, в которой числа записаны в строковый массив через запятую? Тут нужно вносить изменения в бд.Сообщение от alexei
Верно, 1200 - это всего лишь максимальная длина форматируемой строки, а не указание размера.
Я не прошу Вас делать всё за меня, лишь натолкнуть на более менее оптимальный вариант оптимизации данного участка кода
Deimos, учту
Пойдет?
PHP код:stock SaveCars(playerid)
{
if(p_info[playerid][pLogin] == 0)
return true;
new string[775 + 1] = "UPDATE `accounts` SET";
format(string, sizeof(string), "%s `cModel` = '%d,%d,%d',", string, CarInfo[playerid][cModel][0], CarInfo[playerid][cModel][1], CarInfo[playerid][cModel][2]);
format(string, sizeof(string), "%s `cSost` = '%d,%d,%d',", string, CarInfo[playerid][cSost][0], CarInfo[playerid][cSost][1], CarInfo[playerid][cSost][2]);
format(string, sizeof(string), "%s `cColor1` = '%d,%d,%d',", string, CarInfo[playerid][cColor1][0], CarInfo[playerid][cColor1][1], CarInfo[playerid][cColor1][2]);
format(string, sizeof(string), "%s `cColor2` = '%d,%d,%d',", string, CarInfo[playerid][cColor2][0], CarInfo[playerid][cColor2][1], CarInfo[playerid][cColor2][2]);
format(string, sizeof(string), "%s `cNumber` = '%s,%s,%s',", string, VehicleNumberPlate[playerid][0], VehicleNumberPlate[playerid][1], VehicleNumberPlate[playerid][2]);
format(string, sizeof(string), "%s `cPaintJobs` = '%d,%d,%d',", string, CarInfo[playerid][cPaintJobs][0], CarInfo[playerid][cPaintJobs][1], CarInfo[playerid][cPaintJobs][2]);
format(string, sizeof(string), "%s `cLock` = '%d,%d,%d',", string, CarInfo[playerid][cLock][0], CarInfo[playerid][cLock][1], CarInfo[playerid][cLock][2]);
format(string, sizeof(string), "%s `cFuel` = '%f,%f,%f',", string, CarInfo[playerid][cFuel][0], CarInfo[playerid][cFuel][1], CarInfo[playerid][cFuel][2]);
format(string, sizeof(string), "%s `cCost` = '%d,%d,%d',", string, CarInfo[playerid][cCost][0], CarInfo[playerid][cCost][1], CarInfo[playerid][cCost][2]);
format(string, sizeof(string), "%s `cProbeg` = '%f,%f,%f',", string, CarInfo[playerid][cProbeg][0], CarInfo[playerid][cProbeg][1], CarInfo[playerid][cProbeg][2]);
format(string, sizeof(string), "%s `cWheels` = '%d,%d,%d',", string, CarInfo[playerid][cWheels][0], CarInfo[playerid][cWheels][1], CarInfo[playerid][cWheels][2]);
format(string, sizeof(string), "%s `cSpoiler` = '%d,%d,%d',", string, CarInfo[playerid][cSpoiler][0], CarInfo[playerid][cSpoiler][1], CarInfo[playerid][cSpoiler][2]);
format(string, sizeof(string), "%s `cBumper1` = '%d,%d,%d',", string, CarInfo[playerid][cBumper1][0], CarInfo[playerid][cBumper1][1], CarInfo[playerid][cBumper1][2]);
format(string, sizeof(string), "%s `cBumper2` = '%d,%d,%d',", string, CarInfo[playerid][cBumper2][0], CarInfo[playerid][cBumper2][1], CarInfo[playerid][cBumper2][2]);
format(string, sizeof(string), "%s `cNitro` = '%d,%d,%d',", string, CarInfo[playerid][cNitro][0], CarInfo[playerid][cNitro][1], CarInfo[playerid][cNitro][2]);
format(string, sizeof(string), "%s `cHydrawlic` = '%d,%d,%d',", string, CarInfo[playerid][cHydrawlic][0], CarInfo[playerid][cHydrawlic][1], CarInfo[playerid][cHydrawlic][2]);
format(string, sizeof(string), "%s `cHood` = '%d,%d,%d',", string, CarInfo[playerid][cHood][0], CarInfo[playerid][cHood][1], CarInfo[playerid][cHood][2]);
format(string, sizeof(string), "%s `cRoof` = '%d,%d,%d',", string, CarInfo[playerid][cRoof][0], CarInfo[playerid][cRoof][1], CarInfo[playerid][cRoof][2]);
format(string, sizeof(string), "%s `cSideskirt` = '%d,%d,%d',", string, CarInfo[playerid][cSideskirt][0], CarInfo[playerid][cSideskirt][1], CarInfo[playerid][cSideskirt][2]);
format(string, sizeof(string), "%s `cLamps` = '%d,%d,%d',", string, CarInfo[playerid][cLamps][0], CarInfo[playerid][cLamps][1], CarInfo[playerid][cLamps][2]);
format(string, sizeof(string), "%s `cExhaust` = '%d,%d,%d',", string, CarInfo[playerid][cExhaust][0], CarInfo[playerid][cExhaust][1], CarInfo[playerid][cExhaust][2]);
format(string, sizeof(string), "%s `cStereo` = '%d,%d,%d',", string, CarInfo[playerid][cStereo][0], CarInfo[playerid][cStereo][1], CarInfo[playerid][cStereo][2]);
format(string, sizeof(string), "%s `cVentRight` = '%d,%d,%d',", string, CarInfo[playerid][cVentRight][0], CarInfo[playerid][cVentRight][1], CarInfo[playerid][cVentRight][2]);
format(string, sizeof(string), "%s `cVentLeft` = '%d,%d,%d',", string, CarInfo[playerid][cVentLeft][0], CarInfo[playerid][cVentLeft][1], CarInfo[playerid][cVentLeft][2]);
format(string, sizeof(string), "%s `cHeal` = '%f,%f,%f',", string, CarInfo[playerid][cHeal][0], CarInfo[playerid][cHeal][1], CarInfo[playerid][cHeal][2]);
format(string, sizeof(string), "%s `cDamagePanel` = '%d,%d,%d',", string, CarInfo[playerid][cDamagePanel][0], CarInfo[playerid][cDamagePanel][1], CarInfo[playerid][cDamagePanel][2]);
format(string, sizeof(string), "%s `cDamageDoors` = '%d,%d,%d',", string, CarInfo[playerid][cDamageDoors][0], CarInfo[playerid][cDamageDoors][1], CarInfo[playerid][cDamageDoors][2]);
format(string, sizeof(string), "%s `cDamageLights` = '%d,%d,%d',", string, CarInfo[playerid][cDamageLights][0], CarInfo[playerid][cDamageLights][1], CarInfo[playerid][cDamageLights][2]);
format(string, sizeof(string), "%s `cDamageTires` = '%d,%d,%d'", string, CarInfo[playerid][cDamageTires][0], CarInfo[playerid][cDamageTires][1], CarInfo[playerid][cDamageTires][2]);
format(string, sizeof(string), "%s WHERE `pID` = '%d'", string, GetPlayerAccountID(playerid));
mysql_tquery(dbHandle, string, "", "");
return true;
}
Последний раз редактировалось alexei; 05.09.2020 в 09:44.
Числа должны быть числами, для каждого числа должен быть свой столбец в бд, а "%d,%d,%d" - говнокод. И зачем столько форматирований? Почему нельзя сделать одно большое форматирование?
я не знаю как, покажи в качестве примера
Оптимальный вариант в данном случае - отказаться от сохранения всех данных и сохранять каждую конкретную переменную в момент её изменения.
Но ни один из предоставленных отрезков кода не хранил в себе какой-либо критически неоптимизированный код, правки которого дали бы хоть какой-то ощутимый прирост скорости. Так что ты просто тратишь и своё, и чужое время впустую сейчас.
Связаться со мной в VK можно через личные сообщения этой группы
Заказы не принимаю
Широко известно, что идеи стоят 0.8333 цента каждая (исходя из рыночной цены 10 центов за дюжину).
Великих идей полно, на них нет спроса.
Воплощение идеи в законченную игру требует долгой работы,
таланта, терпения и креативности, не говоря уж о затратах денег, времени и ресурсов.
Предложить идею просто, воплотить – вот в чём проблема
Steve Pavlina
Мне было просто интересно разобраться в этом, ибо 1200 ячеек на каждую строку, это не то, что нужно.. Да и выглядит убого)))
Ещё раз: никаких 1200 ячеек на каждую строку там нет. Изучи описание параметров функции format.
И сам по себе код вполне нормально реализован. Единственная проблема в нём только в том, что конкретно в этом случае есть более удачный вариант реализации. Но если бы его не было - нет ничего плохого от того, что у тебя каждая переменная записывается в строку через отдельный format, так как функция сохранения всё равно вызывалась бы не часто. При этом, подобное написание гораздо сильнее упрощает редактирование запроса, в случае нужды, ибо если ты решишь, например, удалить одну из переменных, тебе достаточно будет удалить format, а не выискивать среди текста нужную переменную.
Связаться со мной в VK можно через личные сообщения этой группы
Заказы не принимаю
Широко известно, что идеи стоят 0.8333 цента каждая (исходя из рыночной цены 10 центов за дюжину).
Великих идей полно, на них нет спроса.
Воплощение идеи в законченную игру требует долгой работы,
таланта, терпения и креативности, не говоря уж о затратах денег, времени и ресурсов.
Предложить идею просто, воплотить – вот в чём проблема
Steve Pavlina
Эту тему просматривают: 1 (пользователей: 0 , гостей: 1)
Ваши права
Ответить с цитированием
