Плагин на аммо для сервера, раздача автономно

Counter-Strike 1.6 / Zombie Plague Enterprise / Дополнения

yellow47reg от 28-11-2014, 13:07

Похожее

Предыдущая публикация Следующая публикация

Комментарии 15

strahh

strahh от 28 ноября 2014 14:59
Жалоба
Вип работает совместно с zm_vip ?

--------------------
Рекомендую хостинг
multiplay.ru/?ref=3723
Felix_Font

Felix_Font от 28 ноября 2014 15:46
Жалоба
public NewRound()
{
for(new i=1; i < get_maxplayers(); i++)
{
if(is_user_connected(i))
{
set_task(2.0, "GiveAP", i);
}
}
}

1. Плагин пропустит 32 игрока, т.к. проверка идет на то что индекс меньше чем максимальное количество игроков (допустим 32), а значит цикл пойдет только на 31 игроков.
(Вместо "<" нужно "<=")

2. get_maxplayers() каждый раз будет вызываться по ~ 32 раза
Лучше будет при старте сервера создавать переменную и записать в нее значение get_maxplayers() в plugin_init()

3. Зачем запускать ~ 32 таска на 2 секунды? Если можно запустить один таск на 2 секунды и в нем сделать цикл на всех игрока.

Да, и что если игрок выйдет в течение этих двух секунд?
- Ошибки в логах о Invalid Player

public GiveAP(id)

Лучше было бы так сделать
new UserAP = zp_get_user_ammo_packs(id);
new flags = get_user_flags(id);

А там уже просто подставлять переменные ниже.

Да и почему бы пробелы не делать через TAB?
Так ведь намного удобнее.

--------------------
test]
Pechenechko

Pechenechko от 28 ноября 2014 15:53
Жалоба
[quote=Felix_Font][/quote]
Да и вместить все можно в 5 строк. Сам проверял :D
yellow47reg

yellow47reg от 28 ноября 2014 17:36
Жалоба
Цитата: strahh
Вип работает совместно с zm_vip ?

Нет работает без него, зачем если можно так написать. так короче и проще. все работает, проверенно 100%.
Цитата: Pechenechko
Цитата: Felix_Font

Да и вместить все можно в 5 строк. Сам проверял :D

Че то ты пи*д*ишь по ходу :08: 5 строк!? хм.... я обойдусь без лишних комментов.

Цитата: Felix_Font
public NewRound()
{
for(new i=1; i < get_maxplayers(); i++)
{
if(is_user_connected(i))
{
set_task(2.0, "GiveAP", i);
}
}
}

1. Плагин пропустит 32 игрока, т.к. проверка идет на то что индекс меньше чем максимальное количество игроков (допустим 32), а значит цикл пойдет только на 31 игроков.
(Вместо "<" нужно "<=")

2. get_maxplayers() каждый раз будет вызываться по ~ 32 раза
Лучше будет при старте сервера создавать переменную и записать в нее значение get_maxplayers() в plugin_init()

3. Зачем запускать ~ 32 таска на 2 секунды? Если можно запустить один таск на 2 секунды и в нем сделать цикл на всех игрока.

Да, и что если игрок выйдет в течение этих двух секунд?
- Ошибки в логах о Invalid Player

public GiveAP(id)

Лучше было бы так сделать
new UserAP = zp_get_user_ammo_packs(id);
new flags = get_user_flags(id);

А там уже просто подставлять переменные ниже.

Да и почему бы пробелы не делать через TAB?
Так ведь намного удобнее.

Феликс, признаюсь честно я недавно начал писать коды, по этому имхо делаю как получаеться) Главное результат и исправный плагин) Этот работает на ура и с таким кодом. В логи не кидает.
santa1306

santa1306 от 28 ноября 2014 17:58
Жалоба
Показать / Скрыть текст
Феликс, признаюсь честно я недавно начал писать коды, по этому имхо делаю как получаеться) Главное результат и исправный плагин) Этот работает на ура и с таким кодом. В логи не кидает.
он не работает на ура, так как цикл неправильный
Pechenechko

Pechenechko от 28 ноября 2014 18:14
Жалоба
Показать / Скрыть текст
http://pastebin.com/mYYnpbhh
D34

D34 от 28 ноября 2014 18:15
Жалоба
Таск зачем?
Цикл зачем?
Можно проще сделать
yellow47reg

yellow47reg от 28 ноября 2014 22:34
Жалоба
Цитата: Pechenechko
Показать / Скрыть текст
http://pastebin.com/mYYnpbhh

Это только для админа флага Н, не стал даже компилировать его что бы проверить. у меня он для 3х видов пользователей.
Pechenechko

Pechenechko от 28 ноября 2014 22:56
Жалоба
Цитата: yellow47reg
Цитата: Pechenechko
Показать / Скрыть текст
http://pastebin.com/mYYnpbhh

Это только для админа флага Н, не стал даже компилировать его что бы проверить. у меня он для 3х видов пользователей.

Код посмотри!
gusaq

gusaq от 29 ноября 2014 00:11
Жалоба
Че за бред, лови начало раунда, и там же обьявляй переменные

--------------------
shel

Ymer(R.I.P)
santa1306

santa1306 от 29 ноября 2014 08:22
Жалоба
нашли что обсуждать, первый плагин =))))))))
Felix_Font

Felix_Font от 29 ноября 2014 17:16
Жалоба
Феликс, признаюсь честно я недавно начал писать коды, по этому имхо делаю как получаеться) Главное результат и исправный плагин) Этот работает на ура и с таким кодом. В логи не кидает.

Тебе никто не запрещает делать плагины с таким кодом.
Но раз ты решил их запостить на сайте, значит он должен быть нормальным.
В данном случае он в плохом состоянию, и имеет серьезные ошибки.
Тебя никто не засирает, тебе просто указывают на ошибки.
Чтобы в будущем ты их не допускал.

Попробуй этот плагин полностью исправить, и сделать его более функциональным, после чего выложи под новой версией.
Но так чтобы этот код считался не большой частью, попробуй сделать что-то на подобии системы бонусов.

--------------------
test]
Rom4Nk

Rom4Nk от 30 ноября 2014 06:32
Жалоба
Давно уже себе делал... вроде норм работало

http://pastebin.com/sj3pCmmj
BoecSpecOPs

BoecSpecOPs от 1 декабря 2014 21:45
Жалоба
Начихуа делать 32 таска, если можно сделать один таск и проверить подключенных игроков в нём?
Inovella

Inovella от 2 декабря 2014 11:08
Жалоба
Цитата: BoecSpecOPs
Начихуа делать 32 таска, если можно сделать один таск и проверить подключенных игроков в нём?

Смысл твоего коммента? Он только начинает, многое не понимает.
Даже опытные скриптеры допускают ошибки и причины этому вовсе недостаток знаний а:
рассеянность, невнимательность, отсутствие платформы для тестирования и самое отстойное - для amxmodx нет нормального режима отладки, в отличие от C.

Информация
Посетители, находящиеся в группе Гости, не могут оставлять комментарии к данной публикации.