Транспорт фракций

[kala4iks]

Раньше было вот так

    if(IsACopCar(vehicleid) && !ispassenger) 
    { 
        if(Player[playerid][pMember] != 1 && Player[playerid][pMember] != 2) 
        { 
            send(playerid,0x6959CDAA,"[ТРАНСПОРТ] {FFFFFF}Вы не работаете в ''Los Santos Police Dept''."); 
            GetPlayerPos(playerid,slx,sly,slz); 
            SetPlayerPos(playerid,slx,sly,slz+0.1); 
        } 
    }  

Теперь же я перенес транспорт фракции в mysql и сделал вот так но теперь в транспорт может садится любой.

    if(!ispassenger) 
    { 
        if(IsVehicleOccupiedEx(vehicleid)) 
        { 
             GameTextForPlayer(playerid, "~r~~h~CAR IS USED", 2000, 4); 
              ClearAnimations(playerid, true); 
             return 1; 
        } 
        if(VehInfo[vehicleid][vFraction] != PI[playerid][pMember] && VehInfo[vehicleid][vFraction] > 0) 
        { 
            SendClientMessage(playerid, COLOR_GREY, "Этот транспорт вам недоступен!"); 
            GetPlayerPos(playerid,slx,sly,slz); 
            SetPlayerPos(playerid,slx,sly,slz+0.1); 
        } 
    }  

Выручите пожалуйста на сервере все могут в транспорт садиться.

[DeimoS]

Покажи что в vFraction хранится и как заполняется

[kala4iks]

Покажи что в vFraction хранится и как заполняется

Это транспорт фракции ее ID

public LoadVeh()
{
    new time = GetTickCount(), rows, fields;
    cache_get_data(rows, fields);
    if(rows)
    {
        for(new idx = 1; idx <= rows; idx++)
        {
            VehInfo[idx][vVehicle] = INVALID_VEHICLE_ID;
            VehInfo[idx][vID] = cache_get_row_int(idx-1, 0, dbHandle); // id
            VehInfo[idx][vModel] = cache_get_row_int(idx-1, 1, dbHandle); // model
            VehInfo[idx][vPos_X] = cache_get_row_float(idx-1, 2, dbHandle); // x
            VehInfo[idx][vPos_Y] = cache_get_row_float(idx-1, 3, dbHandle); // y
            VehInfo[idx][vPos_Z] = cache_get_row_float(idx-1, 4, dbHandle); // z
            VehInfo[idx][vPos_FA] = cache_get_row_float(idx-1, 5, dbHandle); // fa
            VehInfo[idx][vInt] = cache_get_row_int(idx-1, 6, dbHandle); // vint
            VehInfo[idx][vWorld] = cache_get_row_int(idx-1, 7, dbHandle); // world
            cache_get_row(idx-1, 8, VehInfo[idx][vColors], dbHandle, 16); // colors
            VehInfo[idx][vFraction] = cache_get_row_int(idx-1, 9, dbHandle); // fraction
            VehInfo[idx][vStatus] = cache_get_row_int(idx-1, 10, dbHandle); // status
            VehInfo[idx][vCost] = cache_get_row_int(idx-1, 11, dbHandle); // cost
            sscanf(VehInfo[idx][vColors], "p<,>a<i>[2]", VehInfo[idx][vColor]);
            if(VehInfo[idx][vStatus] == 0 || VehInfo[idx][vStatus] == 2)
            {
                VehInfo[idx][vVehicle] = CreateVehicle(VehInfo[idx][vModel], VehInfo[idx][vPos_X], VehInfo[idx][vPos_Y], VehInfo[idx][vPos_Z], VehInfo[idx][vPos_FA], VehInfo[idx][vColor][0], VehInfo[idx][vColor][1], 300);
                LinkVehicleToInterior(VehInfo[idx][vVehicle], VehInfo[idx][vInt]);
                SetVehicleVirtualWorld(VehInfo[idx][vVehicle], VehInfo[idx][vWorld]);
                Fuel[VehInfo[idx][vVehicle]] = 150;
                   Engine[VehInfo[idx][vVehicle]] = false;
                Lights[VehInfo[idx][vVehicle]] = false;
            }
            TOTALVEH++;
        }
    }
    printf("[Загружено орг/раб. автомобилей]: <%i>. Потрачено: <%i ms>.", TOTALVEH, GetTickCount() - time);
    return 1;
}  

[DeimoS]

Ну сделай логирование и смотри какие данные приходят в функцию

if(!ispassenger) 
{ 
    printf("\n\n%d) %d | %d\n\n", vehicleid, VehInfo[vehicleid][vFraction], PI[playerid][pMember])
    if(IsVehicleOccupiedEx(vehicleid)) 
    { 
        GameTextForPlayer(playerid, "~r~~h~CAR IS USED", 2000, 4); 
        ClearAnimations(playerid, true); 
        return 1; 
    } 
    if(VehInfo[vehicleid][vFraction] != PI[playerid][pMember] && VehInfo[vehicleid][vFraction] > 0) 
    { 
        SendClientMessage(playerid, COLOR_GREY, "Этот транспорт вам недоступен!"); 
        GetPlayerPos(playerid,slx,sly,slz); 
        SetPlayerPos(playerid,slx,sly,slz+0.1); 
    } 
}  

Хотя я уже, скорее всего, вижу проблему. Она заключается в том, что данные в vFraction загружаются не по ID автомобиля, а по номеру итерации в цикле, из-за чего в коде выше ты обращаешься не к тому автомобилю

[kala4iks]

Ну сделай логирование и смотри какие данные приходят в функцию

if(!ispassenger) 
{ 
    printf("\n\n%d) %d | %d\n\n", vehicleid, VehInfo[vehicleid][vFraction], PI[playerid][pMember])
    if(IsVehicleOccupiedEx(vehicleid)) 
    { 
        GameTextForPlayer(playerid, "~r~~h~CAR IS USED", 2000, 4); 
        ClearAnimations(playerid, true); 
        return 1; 
    } 
    if(VehInfo[vehicleid][vFraction] != PI[playerid][pMember] && VehInfo[vehicleid][vFraction] > 0) 
    { 
        SendClientMessage(playerid, COLOR_GREY, "Этот транспорт вам недоступен!"); 
        GetPlayerPos(playerid,slx,sly,slz); 
        SetPlayerPos(playerid,slx,sly,slz+0.1); 
    } 
}  

Хотя я уже, скорее всего, вижу проблему. Она заключается в том, что данные в vFraction загружаются не по ID автомобиля, а по номеру итерации в цикле, из-за чего в коде выше ты обращаешься не к тому автомобилю

----------
Loaded log file: "server_log.txt".
----------

SA-MP Dedicated Server
----------------------
v0.3.7-R2, (C)2005-2015 SA-MP Team

[23:17:33] 
[23:17:33] Server Plugins
[23:17:33] --------------
[23:17:33]  Loading plugin: streamer
[23:17:33] 

*** Streamer Plugin v2.8.2 by Incognito loaded ***

[23:17:33]   Loaded.
[23:17:33]  Loading plugin: mysql
[23:17:33]  >> plugin.mysql: R39-5 successfully loaded.
[23:17:33]   Loaded.
[23:17:33]  Loading plugin: sscanf
[23:17:33] 

[23:17:33]  ===============================

[23:17:33]       sscanf plugin loaded.     

[23:17:33]          Version:  2.8.1        

[23:17:33]    (c) 2012 Alex "Y_Less" Cole  

[23:17:33]  ===============================

[23:17:33]   Loaded.
[23:17:33]  Loading plugin: regex
[23:17:33] 


[23:17:33] ______________________________________


[23:17:33]  Regular Expression Plugin v0.2.1 loaded
[23:17:33] ______________________________________


[23:17:33]  By: Fro (c) Copyright <TBG> 2009-2011
[23:17:33] ______________________________________


[23:17:33]   Loaded.
[23:17:33]  Loading plugin: iTD
[23:17:33] 
* iTD Plugin loaded. (Support for textdraw editor mouse/keyboard)

[23:17:33]   Loaded.
[23:17:33]  Loaded 5 plugins.

[23:17:33] 
[23:17:33] Filterscripts
[23:17:33] ---------------
[23:17:33]   Loading filterscript 'iTD.amx'...
[23:17:33]   Unable to load filterscript 'iTD.amx'.
[23:17:33]   Loaded 0 filterscripts.

[23:17:33]  
[23:17:33]  
[23:17:33]  
[23:17:33]  ======================================= 
[23:17:33]  |                                     | 
[23:17:33]  |        YSI version 3.09.0684        | 
[23:17:33]  |        By Alex "Y_Less" Cole        | 
[23:17:33]  |                                     | 
[23:17:33]  ======================================= 
[23:17:33]  
[23:17:33]   MySQL connection: TRUE (local)

[23:17:33] -> Динамических объектов: 2754
[23:17:33] -> Статических объектов: 98
[23:17:33] -> Видимых объектов (стример): 902
[23:17:33] _______________________________________________________
[23:17:33]  development fashion: #Nick                                 
[23:17:33]  motion project © 2015-2016, inc. all rights reserved. 
[23:17:33] _______________________________________________________
[23:17:33] Number of vehicle models: 73
[23:17:33] [Загружено орг/раб. автомобилей]: <226>. Потрачено: <6 ms>.
[23:17:33] [Загружено бизнесов]: <73>. Потрачено: <4 ms>.
[23:17:33] [Внимание] Фракции загружены.
[23:17:33] [Загружено домов]: <463>. Потрачено: <22 ms>.
[23:17:33] [Загружены склады мафий]: Потрачено: <0 ms>.
[23:17:33] [Загружены склады банд]: Потрачено: <1 ms>.
[23:17:33] [Загружено гангзон]: <104>. Потрачено: <1 ms>.
[23:17:33] [Загружено банкоматов]: <24>. Потрачено: <1 ms>.
[23:17:33] [Загружено остальное]: Потрачено: <0 ms>.
[23:17:57] [connection] 127.0.0.1:51974 requests connection cookie.
[23:17:58] [connection] incoming connection: 127.0.0.1:51974 id: 0
[23:17:58] [join] kala4iks has joined the server (0:127.0.0.1)
[23:18:08] Игрок kala4iks только что использовал диалог номер 2
[23:18:14] Игрок kala4iks только что использовал команду "/alogin"
[23:18:18] Игрок kala4iks только что использовал команду "/aduty"
[23:18:20] Игрок kala4iks только что использовал диалог номер 8934
[23:18:24] Игрок kala4iks только что использовал команду "/setleader"
[23:18:26] Игрок kala4iks только что использовал команду "/setleader 0"
[23:18:28] Игрок kala4iks только что использовал диалог номер 49
[23:18:29] Игрок kala4iks только что использовал диалог номер 7436
[23:18:30] Игрок kala4iks только что использовал команду "/setleader 0"
[23:18:42] 

266) 0 | 0  

vFraction это ID фракции.

[vovandolg]

Ну вон логи показали что машина принадлежит бомжам))
Ты походу ID коряво грузишь из БД...

[DeimoS]

Ответ на твой вопрос заключается в этом:

Хотя я уже, скорее всего, вижу проблему. Она заключается в том, что данные в vFraction загружаются не по ID автомобиля, а по номеру итерации в цикле, из-за чего в коде выше ты обращаешься не к тому автомобилю

Как вариант костылезации, можно сделать так:

if(!ispassenger) 
{
    new v = 1;
    for(; v < TOTALVEH; v++)
    {
        if(VehInfo[v][vVehicle] == vehicleid)
            break;
    }
    if(IsVehicleOccupiedEx(vehicleid)) 
    {
        GameTextForPlayer(playerid, "~r~~h~CAR IS USED", 2000, 4); 
        ClearAnimations(playerid, true); 
        return 1; 
    }
    if(VehInfo[v][vFraction] != PI[playerid][pMember] && VehInfo[v][vFraction] > 0) 
    {
        SendClientMessage(playerid, COLOR_GREY, "Этот транспорт вам недоступен!"); 
        GetPlayerPos(playerid,slx,sly,slz); 
        SetPlayerPos(playerid,slx,sly,slz+0.1);
        return 1;
    }
}  

[kala4iks]

Ответ на твой вопрос заключается в этом:



Как вариант костылезации, можно сделать так:

if(!ispassenger) 
{
    new v = 1;
    for(; v < TOTALVEH; v++)
    {
        if(VehInfo[v][vVehicle] == vehicleid)
            break;
    }
    if(IsVehicleOccupiedEx(vehicleid)) 
    {
        GameTextForPlayer(playerid, "~r~~h~CAR IS USED", 2000, 4); 
        ClearAnimations(playerid, true); 
        return 1; 
    }
    if(VehInfo[v][vFraction] != PI[playerid][pMember] && VehInfo[v][vFraction] > 0) 
    {
        SendClientMessage(playerid, COLOR_GREY, "Этот транспорт вам недоступен!"); 
        GetPlayerPos(playerid,slx,sly,slz); 
        SetPlayerPos(playerid,slx,sly,slz+0.1);
        return 1;
    }
}  

Спасибо вроде помогло, а еще не подскажите как можно реализовать чтоб допустим FBI могла садиться в транспорт фракции 1,2,5

[DeimoS]

Спасибо вроде помогло, а еще не подскажите как можно реализовать чтоб допустим FBI могла садиться в транспорт фракции 1,2,5

Достаточно просто избавится от мании беспричинно сокращать код и сделать по нормальному, написав условие для каждой фракции.

- - - Добавлено - - -

+- так

  Открыть/закрыть

if(!ispassenger) 
{
    if(IsVehicleOccupiedEx(vehicleid)) 
    {
        GameTextForPlayer(playerid, "~r~~h~CAR IS USED", 2000, 4); 
        ClearAnimations(playerid, true); 
        return 1; 
    }

    new v = 1;
    for(; v < TOTALVEH; v++)
    {
        if(VehInfo[v][vVehicle] == vehicleid)
            break;
    }

    new member = PI[playerid][pMember],
        vehicle_fract_id = VehInfo[v][vFraction],
        status = 1;
    if(vehicle_fract_id && vehicle_fract_id != member) 
    {
        switch(vehicle_fract_id)
        {
            case 1:
            {
                if(member != 1 && member != /*ID FBI*/)
                    status = 0;
            }
            case 2:
            {
                if(member != 2 && member != /*ID FBI*/)
                    status = 0;
            }
            case 3:
            {
                if(member != 3)
                    status = 0;
            }
            case 4:
            {
                if(member != 4)
                    status = 0;
            }
        }
        if(!status)
        {
            SendClientMessage(playerid, COLOR_GREY, "Этот транспорт вам недоступен!"); 
            GetPlayerPos(playerid,slx,sly,slz); 
            SetPlayerPos(playerid,slx,sly,slz+0.1);
            return 1;
        }
    }
}  

[kala4iks]

Достаточно просто избавится от мании беспричинно сокращать код и сделать по нормальному, написав условие для каждой фракции.

- - - Добавлено - - -

+- так

  Открыть/закрыть

if(!ispassenger) 
{
    if(IsVehicleOccupiedEx(vehicleid)) 
    {
        GameTextForPlayer(playerid, "~r~~h~CAR IS USED", 2000, 4); 
        ClearAnimations(playerid, true); 
        return 1; 
    }

    new v = 1;
    for(; v < TOTALVEH; v++)
    {
        if(VehInfo[v][vVehicle] == vehicleid)
            break;
    }

    new member = PI[playerid][pMember],
        vehicle_fract_id = VehInfo[v][vFraction],
        status = 1;
    if(vehicle_fract_id && vehicle_fract_id != member) 
    {
        switch(vehicle_fract_id)
        {
            case 1:
            {
                if(member != 1 && member != /*ID FBI*/)
                    status = 0;
            }
            case 2:
            {
                if(member != 2 && member != /*ID FBI*/)
                    status = 0;
            }
            case 3:
            {
                if(member != 3)
                    status = 0;
            }
            case 4:
            {
                if(member != 4)
                    status = 0;
            }
        }
        if(!status)
        {
            SendClientMessage(playerid, COLOR_GREY, "Этот транспорт вам недоступен!"); 
            GetPlayerPos(playerid,slx,sly,slz); 
            SetPlayerPos(playerid,slx,sly,slz+0.1);
            return 1;
        }
    }
}  

Вот я переделал и немного добавил своего кода, и теперь снова садится в транспорт любой можно.

  Открыть/закрыть

    if(!ispassenger)
    {
        new v = 1;
        for(; v < TOTALVEH; v++)
        {
            if(VehInfo[v][vVehicle] == vehicleid)
            break;
        }
        if(IsVehicleOccupiedEx(vehicleid))
        {
            GameTextForPlayer(playerid, "~r~~h~CAR IS USED", 2000, 4);
            ClearAnimations(playerid, true);
            return 1;
        }
        new member = PI[playerid][pMember],v_frac_id = VehInfo[v][vFraction],status = 1;
        if(v_frac_id && v_frac_id != member)
        {
            switch(v_frac_id)
            {
                case 1:if(member != 1 && member != 4)status = 0;
                case 2:if(member != 2 && member != 4)status = 0;
                case 3:if(member != 3 && member != 4)status = 0;
                case 4:if(member != 4)status = 0;
                case 5:if(member != 5)status = 0;
                case 6:if(member != 6)status = 0;
                case 7:if(member != 7)status = 0;
                case 8:if(member != 8)status = 0;
                case 9:if(member != 9)status = 0;
                case 10:if(member != 10)status = 0;
                case 11:if(member != 11)status = 0;
                case 12:if(member != 12)status = 0;
                case 13:if(member != 13)status = 0;
                case 14:if(member != 14)status = 0;
                case 15:if(member != 15)status = 0;
                case 16:if(member != 16)status = 0;
                case 17:if(member != 17)status = 0;
                case 18:if(member != 18)status = 0;
                case 19:if(member != 19)status = 0;
                case 20:if(member != 20)status = 0;
                case 21:if(member != 21)status = 0;
                case 22:if(member != 22)status = 0;
                case 23:if(member != 23)status = 0;
            }
            if(!status)
            {
                switch(v_frac_id)
                {
                    case 18,19,20:
                       {
                        static const fmt_1[] = "[ТРАНСПОРТ] {FFFFFF}Вы не служите в \"%s\".";
                        new str_1[sizeof(fmt_1)-2+32];
                        format(str_1,sizeof(str_1),fmt_1,FInfo[v][fName]);
                        SendClientMessage(playerid,0x6959CDAA,str_1);
                    }
                    case 1,2,3,4,5,6,17,22:
                    {
                        static const fmt_2[] = "[ТРАНСПОРТ] {FFFFFF}Вы не работаете в \"%s\"";
                        new str_2[sizeof(fmt_2)-2+32];
                        format(str_2,sizeof(str_2),fmt_2,FInfo[v][fName]);
                         SendClientMessage(playerid,0x6959CDAA,str_2);
                    }
                    case 7,8,9,10,11,12,13,14,15,16,21,23:
                    {
                        static const fmt_3[] = "[ТРАНСПОРТ] {FFFFFF}Вы не состоите в \"%s\"";
                        new str_3[sizeof(fmt_3)-2+32];
                        format(str_3,sizeof(str_3),fmt_3,FInfo[v][fName]);
                        SendClientMessage(playerid,0x6959CDAA,str_3);
                    }
                }
                GetPlayerPos(playerid,slx,sly,slz);
                SetPlayerPos(playerid,slx,sly,slz+0.1);
                return 1;
            }
        }
    }