Награда по пати

[L2J]

Написал код, все работает, но теперь делаю другое, если человек в пати, то награда выдается рендомному человеку из группы, сделал так:
[SRC="java"] public void giveHonorForPvP(Player pk)
{
if(pk.getParty() == null)
{
if(pk.getTarget().getPlayer().getFame() >= 0 && pk.getTarget().getPlayer().getFame() < 50)
{
pk.addFame(1);
}
else if(pk.getTarget().getPlayer().getFame() >= 50 && pk.getTarget().getPlayer().getFame() < 100)
{
pk.addFame(4);
}
else if(pk.getTarget().getPlayer().getFame() >= 100 && pk.getTarget().getPlayer().getFame() < 300)
{
pk.addFame(5);
}
else if(pk.getTarget().getPlayer().getFame() >= 300 && pk.getTarget().getPlayer().getFame() < 600)
{
pk.addFame(6);
}
else if(pk.getTarget().getPlayer().getFame() >= 600)
{
pk.addFame(7);
}
}
else
{
List<Player> ret = null;
ret = new ArrayList<Player>(getParty().getPartyMembers().size());
for(Player member : getParty().getPartyMembers())
if(member.isInRangeZ(pk, 1000))
ret.add(member);
}
ranksystem(pk);
factioncolor(pk);
}[/SRC]
Сделал добавление людей, теперь как выдавать награду рендомному из пати?

[kingzor]

Написал код, все работает, но теперь делаю другое, если человек в пати, то награда выдается рендомному человеку из группы, сделал так:
[SRC="java"] public void giveHonorForPvP(Player pk)
{
if(pk.getParty() == null)
{
if(pk.getTarget().getPlayer().getFame() >= 0 && pk.getTarget().getPlayer().getFame() < 50)
{
pk.addFame(1);
}
else if(pk.getTarget().getPlayer().getFame() >= 50 && pk.getTarget().getPlayer().getFame() < 100)
{
pk.addFame(4);
}
else if(pk.getTarget().getPlayer().getFame() >= 100 && pk.getTarget().getPlayer().getFame() < 300)
{
pk.addFame(5);
}
else if(pk.getTarget().getPlayer().getFame() >= 300 && pk.getTarget().getPlayer().getFame() < 600)
{
pk.addFame(6);
}
else if(pk.getTarget().getPlayer().getFame() >= 600)
{
pk.addFame(7);
}
}
else
{
List<Player> ret = null;
ret = new ArrayList<Player>(getParty().getPartyMembers().size());
for(Player member : getParty().getPartyMembers())
if(member.isInRangeZ(pk, 1000))
ret.add(member);
}
ranksystem(pk);
factioncolor(pk);
}[/SRC]
Сделал добавление людей, теперь как выдавать награду рендомному из пати?

Хз как в оверах, в лыже есть метод getCheckedRandomMember

[L2J]

Рендомный человек сделан, Player randomPlayerFromParty = pk.getParty().getPartyMembers().get(Rnd.get(getParty().getMemberCount()-1));
Теперь как ему выдавать награду?

Добавлено через 2 минуты
[SRC="java"] public void giveHonorForPvP(Player pk)
{
int fame = pk.getTarget().getPlayer().getFame();
if(pk.getParty() == null)
{
if(fame >= 0 && fame < 50)
{
pk.addFame(1);
}
else if(fame >= 50 && fame < 100)
{
pk.addFame(4);
}
else if(fame >= 100 && fame < 300)
{
pk.addFame(5);
}
else if(fame >= 300 && fame < 600)
{
pk.addFame(6);
}
else if(fame >= 600)
{
pk.addFame(7);
}
}
else
{
List<Player> ret = null;
Player randomPlayerFromParty = pk.getParty().getPartyMembers().get(Rnd.get(getParty().getMemberCount()-1));
ret = new ArrayList<Player>(getParty().getPartyMembers().size());
for(Player member : getParty().getPartyMembers())
if(fame >= 0 && fame < 50)
{
randomPlayerFromParty.addFame(1);
}
else if(fame >= 50 && fame < 100)
{
randomPlayerFromParty.addFame(4);
}
else if(fame >= 100 && fame < 300)
{
randomPlayerFromParty.addFame(5);
}
else if(fame >= 300 && fame < 600)
{
randomPlayerFromParty.addFame(6);
}
else if(fame >= 600)
{
randomPlayerFromParty.addFame(7);
}
}
ranksystem(pk);
factioncolor(pk);
}[/SRC]
Сделал так, но почему то не работает

[kingzor]

Рендомный человек сделан, Player randomPlayerFromParty = pk.getParty().getPartyMembers().get(Rnd.get(getParty().getMemberCount()-1));
Теперь как ему выдавать награду?

Добавлено через 2 минуты
[SRC="java"] public void giveHonorForPvP(Player pk)
{
int fame = pk.getTarget().getPlayer().getFame();
if(pk.getParty() == null)
{
if(fame >= 0 && fame < 50)
{
pk.addFame(1);
}
else if(fame >= 50 && fame < 100)
{
pk.addFame(4);
}
else if(fame >= 100 && fame < 300)
{
pk.addFame(5);
}
else if(fame >= 300 && fame < 600)
{
pk.addFame(6);
}
else if(fame >= 600)
{
pk.addFame(7);
}
}
else
{
List<Player> ret = null;
Player randomPlayerFromParty = pk.getParty().getPartyMembers().get(Rnd.get(getParty().getMemberCount()-1));
ret = new ArrayList<Player>(getParty().getPartyMembers().size());
for(Player member : getParty().getPartyMembers())
if(fame >= 0 && fame < 50)
{
randomPlayerFromParty.addFame(1);
}
else if(fame >= 50 && fame < 100)
{
randomPlayerFromParty.addFame(4);
}
else if(fame >= 100 && fame < 300)
{
randomPlayerFromParty.addFame(5);
}
else if(fame >= 300 && fame < 600)
{
randomPlayerFromParty.addFame(6);
}
else if(fame >= 600)
{
randomPlayerFromParty.addFame(7);
}
}
ranksystem(pk);
factioncolor(pk);
}[/SRC]
Сделал так, но почему то не работает

Пробуйте

Код:

Player randomPlayerFromParty = pk.getParty().getPartyMembers().get(Rnd.get(pk.getParty().getMemberCount()));

if(fame >= 0 && fame < 50)
{
    randomPlayerFromParty.addFame(1);
}
else if(fame >= 50 && fame < 100)
{
    randomPlayerFromParty.addFame(4);
}
else if(fame >= 100 && fame < 300)
{
    randomPlayerFromParty.addFame(5);
}
else if(fame >= 300 && fame < 600)
{
    randomPlayerFromParty.addFame(6);
}
else if(fame >= 600)
{
    randomPlayerFromParty.addFame(7);
}

[WoWan-SM]

L2J, Сделай отдельный метод для рассчета выдачи fame

Код:

private static int rewardFame(int fame){
           if(fame >= 0 && fame < 50)
           {
               return 1;
           }
           else if(fame >= 50 && fame < 100)
           {
               return 4;
           }
           else if(fame >= 100 && fame < 300)
           {
               return 5;
           }
           else if(fame >= 300 && fame < 600)
           {
               return 6;
           }

           return 7;
}

Выдача рандомному человеку в группе тогда будет выглядеть так:

Код:

Player randomPlayerFromParty = pk.getParty().getPartyMembers().get(Rnd.get(getParty().getMemberCount()-1)); //Рандомный игрок из пати

int fame = randomPlayerFromParty.getFame(); //Узнаем его fameб чтобы потом знать кол-во награды , которую ему выдавать

randomPlayerFromParty.addFame(rewardFame(fame)); //Рассчитываем кол-во награды и выдаем её

[Visor]

Метод для пати написать, там сделать скажем отбор игроков в онлайне и чтобы были в радиусе пати, потом из списка - если индекс - то так как вы и писали то Rnd.get(list.size())

И соответственно не городить все в кучу, потом обрабатывать только этого чара в пати

[L2J]

kingzor

Данный метод работает, теперь последний вопрос. Как сделать чтобы выдавалось только рендомному человеку из пати который в ренже 1200 например.
Я сделал так
[SRC="java"] Player randomPlayerFromParty = pk.getParty().getPartyMembers().get(Rnd.get(pk.getParty().getMemberCount()));
if(randomPlayerFromParty.isInRangeZ(pk,1200))
{

if(fame >= 0 && fame < 50)
{
randomPlayerFromParty.addFame(1);
}
else if(fame >= 50 && fame < 100)
{
randomPlayerFromParty.addFame(4);
}
else if(fame >= 100 && fame < 300)
{
randomPlayerFromParty.addFame(5);
}
else if(fame >= 300 && fame < 600)
{
randomPlayerFromParty.addFame(6);
}
else if(fame >= 600)
{
randomPlayerFromParty.addFame(7);
}
}[/SRC]
В принципе работает, но если нет ниодного чара рядом с убийцей, то награда не дается убийце и вообще никому.

[kingzor]

kingzor

Данный метод работает, теперь последний вопрос. Как сделать чтобы выдавалось только рендомному человеку из пати который в ренже 1200 например.
Я сделал так
[SRC="java"] Player randomPlayerFromParty = pk.getParty().getPartyMembers().get(Rnd.get(pk.getParty().getMemberCount()));
if(randomPlayerFromParty.isInRangeZ(pk,1200))
{

if(fame >= 0 && fame < 50)
{
randomPlayerFromParty.addFame(1);
}
else if(fame >= 50 && fame < 100)
{
randomPlayerFromParty.addFame(4);
}
else if(fame >= 100 && fame < 300)
{
randomPlayerFromParty.addFame(5);
}
else if(fame >= 300 && fame < 600)
{
randomPlayerFromParty.addFame(6);
}
else if(fame >= 600)
{
randomPlayerFromParty.addFame(7);
}
}[/SRC]
В принципе работает, но если нет ниодного чара рядом с убийцей, то награда не дается убийце и вообще никому.

Как-то так

Код:

Player randomPlayerFromParty = pk.getParty().getPartyMembers().get(Rnd.get(pk.getParty().getMemberCount()));
if (!randomPlayerFromParty.isInRangeZ(pk,1200))
{
    randomPlayerFromParty = pk;
}

if(fame >= 0 && fame < 50)
{
    randomPlayerFromParty.addFame(1);
}
else if(fame >= 50 && fame < 100)
{
    randomPlayerFromParty.addFame(4);
}
else if(fame >= 100 && fame < 300)
{
    randomPlayerFromParty.addFame(5);
}
else if(fame >= 300 && fame < 600)
{
    randomPlayerFromParty.addFame(6);
}
else if(fame >= 600)
{
    randomPlayerFromParty.addFame(7);
}

Вообще это говнокод от которого глаза ломит

По хорошему надо запилить метод в Party.java

Код:

public Player getRandomPlayer()
{
    return getPartyMembers().get(Rnd.get(getMemberCount()));
}

Так же сделать отдельный метод выдачи как посоветовал WoWan-SM