1. SA-MP Forums Archive
  2. Non-English
  3. Languages
  4. Русский/Russian

Threaded Mode
Оптимизация кода
DrSlett
Banned
Posts: 279
Threads: 4
Joined: Sep 2012
#161

27.01.2013, 16:58
PHP код:
strdel(Name[playerid], 0, MAX_PLAYER_NAME); 
и
PHP код:
Name[playerid] = 0; 
не то же ли самое?
Find
Reply
anybox
Little Clucker
**
Posts: 48
Threads: 4
Joined: Jan 2013
Reputation: 0
#162

27.01.2013, 17:00
DrSlett, дельное замечание. В таком случае, действительно можно задать вопрос "зачем всё это", смотря на pLastVeh.
stabker, всегда так делал, только в OnPlayerDisconnect можно даже не очищать Name[playerid].
stepashka, да, затея была действительно глупой, но благодаря ответам форумчан мои знания чуть расширились.
Find
Reply
Stepashka
Godfather
*****
Posts: 5,763
Threads: 19
Joined: Jul 2008
Reputation: 0
#163

27.01.2013, 17:12
Quote:
Originally Posted by anybox
Посмотреть сообщение
Ограничиваем использование GetPlayerVehicleID и IsPlayerInAnyVehicle, тем самым сберегая немного трафика. Это мелочи, но при большом онлайне, когда многие используют команды, связанные с этими функциями, получаются уже какие-никакие, но сетевые затраты. А тут уже всё хранится будет в массиве и достаточно только к элементу обратиться.
Во-первых, вызов команды ещё не означает обращение к клиенту, у сервера есть "пул" данных к каждому клиенту, который периодически обновляется клиентом, и все функции работают с этим "пулом".
Во-вторых, данная оптимизация это бесполезная трата времени и памяти, так что оставьте эту затею.
В-третьих, даже при большом онлайне данная функция не будет вызываться так часто что вызовет лаги, не там вы проблему ищите.

Quote:
Originally Posted by DrSlett
Посмотреть сообщение
PHP код:
strdel(Name[playerid], 0, MAX_PLAYER_NAME); 
и
PHP код:
Name[playerid] = 0; 
не то же ли самое?
да. Только вместо нуля используйте EOS.
Find
Reply
stabker
High-roller
*****
Posts: 1,311
Threads: 39
Joined: Mar 2012
Reputation: 0
#164

27.01.2013, 17:42
Quote:
Originally Posted by anybox
Посмотреть сообщение
stabker, всегда так делал, только в OnPlayerDisconnect можно даже не очищать Name[playerid].
Можно, я не очищаю у себя в моде, т.к. оно там не пылиться. Это скорее от онлайна зависит.
Find
Reply
Maikkk
Big Clucker
***
Posts: 171
Threads: 17
Joined: Jul 2011
Reputation: 0
#165

27.01.2013, 18:29
Quote:
Originally Posted by Stepashka
Посмотреть сообщение
Во-первых, вызов команды ещё не означает обращение к клиенту, у сервера есть "пул" данных к каждому клиенту, который периодически обновляется клиентом, и все функции работают с этим "пулом".
Во-вторых, данная оптимизация это бесполезная трата времени и памяти, так что оставьте эту затею.
В-третьих, даже при большом онлайне данная функция не будет вызываться так часто что вызовет лаги, не там вы проблему ищите.

да. Только вместо нуля используйте EOS.
В чем отличие EOS'a от 0? Ведь по сути EOS и есть тот самый 0.
Find
Reply
dimonml
Gangsta
****
Posts: 688
Threads: 2
Joined: Oct 2008
Reputation: 0
#166

27.01.2013, 18:53
Quote:
Originally Posted by Maikkk
Посмотреть сообщение
В чем отличие EOS'a от 0? Ведь по сути EOS и есть тот самый 0.
Удобством последующего чтения кода программистом.
Find
Reply
eakwarp
High-roller
*****
Posts: 1,912
Threads: 5
Joined: Feb 2007
Reputation: 0
#167

27.01.2013, 19:32
Quote:
Originally Posted by Maikkk
Посмотреть сообщение
В чем отличие EOS'a от 0? Ведь по сути EOS и есть тот самый 0.
Нет, EOS - по сути спецсимвол, означающий конец строки. Хотя в некоторых языках действительно используют #define EOS 0 .
Find
Reply
hub4
Big Clucker
***
Posts: 105
Threads: 2
Joined: Jan 2009
Reputation: 0
#168

01.02.2013, 06:26
приветствую
представим систему с домами, а именно входы в них
вход по команде, или кнопке - не суть, главное что не автоматический.

for(перебор домов)
if(стоит на входе)
входим

если домов много, стоит ли это менять на

onplayerpickuppickup
if(пикап дома)
lastpickup=номер дома

а в команде входа проверять
if(lastpickup==номер дома)
входить в дом

т.е. по сути в исходном случае идет проверка координат, несколько сотен isplayerinrangeofpoint
а во втором всего лишь сравнение простых чисел, ида икапа
Find
Reply
Stepashka
Godfather
*****
Posts: 5,763
Threads: 19
Joined: Jul 2008
Reputation: 0
#169

01.02.2013, 06:31
Quote:
Originally Posted by hub4
Посмотреть сообщение
приветствую
представим систему с домами, а именно входы в них
вход по команде, или кнопке - не суть, главное что не автоматический.

for(перебор домов)
if(стоит на входе)
входим

если домов много, стоит ли это менять на

onplayerpickuppickup
if(пикап дома)
lastpickup=номер дома

а в команде входа проверять
if(lastpickup==номер дома)
входить в дом

т.е. по сути в исходном случае идет проверка координат, несколько сотен isplayerinrangeofpoint
а во втором всего лишь сравнение простых чисел, ида икапа
шило на мыло меняете, во втором случае вам так же придется перебирать массив домов и сравнивать ID поднятого пикапа с ID пикапа привязанного к дому.
Find
Reply
White_116
High-roller
*****
Posts: 1,495
Threads: 20
Joined: Sep 2010
Reputation: 0
#170

01.02.2013, 06:59
Если пикап запоминает ид дома, то второй случай правильный.
Find
Reply
White_116
High-roller
*****
Posts: 1,495
Threads: 20
Joined: Sep 2010
Reputation: 0
#171

08.03.2013, 16:49
( Последний раз редактировалось White_116; 11.02.2014 в 11:35. )
Возьмём классический пример: таймер на 1000 мс и прогон всех игроков. Чтобы не вызывать тормозов можно распределить нагрузку. Разобьём одни большой прогон на 4 части, таймер уменьшим в 4 раза, т.е. не 1000 а 250.

PHP код:
#include <a_samp>
new state;
public OnGameModeInit()
{
    SetTimer("TimerW", 250, 1);
    return 1;
}
public TimerW()
{
    for(new i=state; i<MAX_PLAYERS;i+=4)
    {
//.....
    }
    if(state == 3)state=0;else state++;
} 
Quote:

1) 0,4,8,12
2) 1,5,9,13
3) 2,6,10,14
4) 3,7,11,15
Find
Reply
Alexander_Petrov
Gangsta
*****
Posts: 835
Threads: 6
Joined: Oct 2011
Reputation: 0
#172

08.03.2013, 16:59
( Последний раз редактировалось Alexander_Petrov; 08.03.2013 в 17:34. )
Quote:
Originally Posted by White_116
Посмотреть сообщение
Возьмём классический пример: таймер на 1000 мс и прогон всех игроков. Чтобы не вызывать тормозов можно распределить нагрузку. Разобьём одни большой прогон на 4 части, таймер уменьшим в 4 раза, т.е. не 1000 а 250.

PHP код:
new state;
public TimerE()
{
    for(new i=state; i<MAX_PLAYERS;i+=4)
    {
//.....
    }
    if(state == 3)state=0;else state++;
} 
Смысл классического секундного таймера в том, что для каждого игрока таймер должен проходить раз в секунду. С таким раскладом таймер должен выглядеть так
pawn Код:
#include <a_samp>

public OnGameModeInit()
{
    SetTimer("TimerF", 250, 1);
}

forward TimerF();
public TimerF()
{
    static st;
    for(new plus = st; st < (MAX_PLAYERS/4)+plus; st++)
    {
        printf("%d", st);
    }
    if(st >= MAX_PLAYERS-1) st = 0;
}
Quote:

1) 0, 1, 2, 3
2) 4, 5, 6, 7
3) 8, 9, 10, 11
4) 12, 13, 14, 15
Я вообще не вижу смысла разбивать что-то. Легче и лучше просто создавать таймер для каждого игрока.
upd: Не рассмотрел твой код, исправил свой.
Find
Reply
proff116
Little Clucker
**
Posts: 15
Threads: 1
Joined: Dec 2012
Reputation: 0
#173

08.03.2013, 17:00
Ого, никогда думал об этом, интересно. Так даже проще
Find
Reply
White_116
High-roller
*****
Posts: 1,495
Threads: 20
Joined: Sep 2010
Reputation: 0
#174

08.03.2013, 17:05
Quote:
Originally Posted by Alexander_Petrov
Посмотреть сообщение
Смысл классического секундного таймера в том, что для каждого игрока таймер должен проходить раз в секунду. С таким раскладом таймер должен выглядеть так
pawn Код:
public OnGameModeInit()
{
    SetTimer("TimerE", 250, 0);
}

public TimerE()
{
    static state;
    for(new plus = state; state < (MAX_PLAYERS/4)+plus; state++)
    {
//.....
    }
    if(state == MAX_PLAYERS-1) state = 0;
    SetTimer("TimerE", 250, 0);
}
Я вообще не вижу смысла разбивать что-то. Легче и лучше просто создавать таймер для каждого игрока.
То что вы написали вообще не работает. Прошу внимательно ознакомиться с мои примером.
Quote:
Originally Posted by Alexander_Petrov
Посмотреть сообщение
upd: Не рассмотрел твой код, исправил свой.
Все ровно не работает.
Find
Reply
Alexander_Petrov
Gangsta
*****
Posts: 835
Threads: 6
Joined: Oct 2011
Reputation: 0
#175

08.03.2013, 17:19
Quote:
Originally Posted by White_116
Посмотреть сообщение
То что вы написали вообще не работает. Прошу внимательно ознакомиться с мои примером.
Да, мой код в конечном итоге работает так же, просто не досмотрел.
Свой исправил, теперь все работает.
Find
Reply
White_116
High-roller
*****
Posts: 1,495
Threads: 20
Joined: Sep 2010
Reputation: 0
#176

08.03.2013, 17:25
Сделал более мене правдоподобный код. Метод работы чуть ниже. Каждый игрок будет проверен ровно через секунду!
Find
Reply
Alexander_Petrov
Gangsta
*****
Posts: 835
Threads: 6
Joined: Oct 2011
Reputation: 0
#177

08.03.2013, 17:33
Quote:
Originally Posted by White_116
Посмотреть сообщение
Сделал более мене правдоподобный код. Метод работы чуть ниже. Каждый игрок будет проверен ровно через секунду!
Все работает у меня, правда при MAX_PLAYERS больше 3
Тоже добавил у себя метод работы.
Find
Reply
White_116
High-roller
*****
Posts: 1,495
Threads: 20
Joined: Sep 2010
Reputation: 0
#178

08.03.2013, 17:40
А вот теперь изюминка. Ты каждые 250 мс будешь проверять MAX_PLAYERS а я MAX_PLAYERS/4
Даже если ты static st; вынесешь то у тебя лишних действий много.
Find
Reply
Alexander_Petrov
Gangsta
*****
Posts: 835
Threads: 6
Joined: Oct 2011
Reputation: 0
#179

08.03.2013, 17:45
Quote:
Originally Posted by White_116
Посмотреть сообщение
А вот теперь изюминка. Ты каждые 250 мс будешь проверять MAX_PLAYERS а я MAX_PLAYERS/4
Даже если ты static st; вынесешь то у тебя лишних действий много.
Недостатки в моем коде есть, но я все же остаюсь на стороне таймера для каждого игрока.
Кстати, единственный мною замеченный можно устранить, поместив проверку в цикл.
pawn Код:
#include <a_samp>

#undef MAX_PLAYERS
#define MAX_PLAYERS 20

new timer;

public OnGameModeInit()
{
    timer = SetTimer("TimerF", 250, 1);
}

forward TimerF();
public TimerF()
{
    printf("%i", GetTickCount());
    static st;
    for(new plus = st; st < (MAX_PLAYERS/4)+plus; st++)
    {
        printf("%d", st);
        if(st >= MAX_PLAYERS-1) { KillTimer(timer); return; }
    }
}
В итоге получается:
Quote:

[23:13:49] 6846633
[23:13:49] 0
[23:13:49] 1
[23:13:49] 2
[23:13:49] 3
[23:13:49] 4
[23:13:49] 6846926
[23:13:49] 5
[23:13:49] 6
[23:13:49] 7
[23:13:49] 8
[23:13:49] 9
[23:13:49] 6847219
[23:13:49] 10
[23:13:49] 11
[23:13:49] 12
[23:13:49] 13
[23:13:49] 14
[23:13:50] 6847512
[23:13:50] 15
[23:13:50] 16
[23:13:50] 17
[23:13:50] 18
[23:13:50] 19
Это код для дебага и работает всего 1 раз при запуске мода, потом таймер убивается.
Find
Reply
White_116
High-roller
*****
Posts: 1,495
Threads: 20
Joined: Sep 2010
Reputation: 0
#180

08.03.2013, 18:03
Во всяком случае у меня мозг закипел, как это у вас всё работает когда static st; локальный?
//-------------------
Почитал, разобрался.
Find
Reply
« Next Oldest | Next Newest »


Forum Jump:


Users browsing this thread: 3 Guest(s)
  1. SA-MP Forums Archive
  2. Non-English
  3. Languages
  4. Русский/Russian