Добро пожаловать на Pro Pawn - Портал о PAWN-скриптинге.
Страница 2 из 2 ПерваяПервая 1 2
Показано с 11 по 16 из 16
  1. #11
    Аватар для Daniel_Cortez
    "Это не хак, это фича"

    Статус
    Оффлайн
    Регистрация
    06.04.2013
    Адрес
    Novokuznetsk, Russia
    Сообщений
    2,192
    Репутация:
    2589 ±
    Цитата Сообщение от Unreal Посмотреть сообщение
    Это я сам, просто на другом портале
    Вы находитесь на Pro-Pawn, а не на каком-то "другом портале", поэтому будьте добры, указывайте ник, который у вас на этом форуме.
    Либо можете попросить меня или Osetin'а сменить вам ник, здесь это бесплатно.

    Теперь по поводу кода.
    Если ваш код критикуют, старайтесь вникнуть в суть критики, понять, в чём заключается недостаток, на который вам указывают - возможно он в вашей работе не один.
    TheMallard уже правильно подметил, что название "seller" не уникальное - видимо рассчитывал, что вы поймёте, что у других переменных точно такие же неуникальные названия (в чужом моде наверняка могут найтись и какие-нибудь другие "PlayerMenuTD" или "BuyMenu"), но вы послушно сделали только то, о чём вас попросили, и ни грамма больше.

    Вот вам небольшая задача. В моде есть 5 переменных:
    PHP код:
    new action[MAX_PLAYERS];
    new 
    pickup;
    new 
    selected[MAX_PLAYERS char];
    new 
    uses[MAX_PLAYERS char];
    new 
    object[MAX_PLAYERS]; 
    Определите по названию, какая из них (только одна) относится к системе работы лесоруба.

    Надеюсь, сможете сделать из этого какие-нибудь выводы.
    Индивидуально в ЛС по скриптингу не помогаю. Задавайте все свои вопросы здесь (click).

  2. #12
    Аватар для arzumanov
    Пользователь

    Статус
    Оффлайн
    Регистрация
    24.10.2015
    Сообщений
    3
    Репутация:
    0 ±
    Цитата Сообщение от Unreal Посмотреть сообщение
    PHP код:
    // ===================== [ by Scooter ] =================
    stock ReturnPlayerMenu(playerid,status)
    {
        
    // Если он в машине
        
    if(IsPlayerInAnyVehicle(playerid)) return 1;
        
    // Если он не активирован
        
    if(status == 1)
        {
            
    TextDrawShowForPlayer(playerid,u_SkinPlayerMenuTD[0]); // Фон
            
    TextDrawShowForPlayer(playerid,u_SkinPlayerMenuTD[1]); // Фон 2
            
    TextDrawShowForPlayer(playerid,u_SkinPlayerMenuTD[2]); // Кнопка Close
            
    TextDrawShowForPlayer(playerid,u_SkinPlayerMenuTD[3]); // Фон Дропа
            
    TextDrawShowForPlayer(playerid,u_SkinPlayerMenuTD[4]); // Кнопка Дроп

            
    new empty_skin PlayerInfo[playerid][pSex] == 154 140;
            if(
    GetPlayerSkin(playerid) != empty_skin)
            {
                
    u_SkinSlot[playerid][0] = (PlayerInfo[playerid][pMember]) ? (PlayerInfo[playerid][pModel]) : (PlayerInfo[playerid][pChar]);
            }
            else 
    u_SkinSlot[playerid][0] = empty_skin;
            
    PlayerTextDrawSetPreviewModel(playerid,u_SkinPlayerMenuTDforPlayer[playerid][0],u_SkinSlot[playerid][0]);
            
    PlayerTextDrawSetPreviewModel(playerid,u_SkinPlayerMenuTDforPlayer[playerid][1],u_SkinSlot[playerid][1]);
            
    PlayerTextDrawSetPreviewModel(playerid,u_SkinPlayerMenuTDforPlayer[playerid][2],u_SkinSlot[playerid][2]);
            
    PlayerTextDrawSetPreviewModel(playerid,u_SkinPlayerMenuTDforPlayer[playerid][3],u_SkinSlot[playerid][3]);

            
    PlayerTextDrawShow(playerid,u_SkinPlayerMenuTDforPlayer[playerid][0]); // Основной
            
    PlayerTextDrawShow(playerid,u_SkinPlayerMenuTDforPlayer[playerid][1]); // Слот 1
            
    PlayerTextDrawShow(playerid,u_SkinPlayerMenuTDforPlayer[playerid][2]); // Слот 2
            
    PlayerTextDrawShow(playerid,u_SkinPlayerMenuTDforPlayer[playerid][3]); // Слот 3

            
    SelectTextDraw(playerid0xcfcfcfAA);
            
    SetPVarInt(playerid,"u_SkinPlayerMenuTD_Active",1);
            return 
    1;
        }
        
    // Иначе
        
    TextDrawHideForPlayer(playerid,u_SkinPlayerMenuTD[0]); // Фон
        
    TextDrawHideForPlayer(playerid,u_SkinPlayerMenuTD[1]); // Фон 2
        
    TextDrawHideForPlayer(playerid,u_SkinPlayerMenuTD[2]); // Кнопка Close
        
    TextDrawHideForPlayer(playerid,u_SkinPlayerMenuTD[3]); // Фон Дропа
        
    TextDrawHideForPlayer(playerid,u_SkinPlayerMenuTD[4]); // Кнопка Дроп

        
    PlayerTextDrawHide(playerid,u_SkinPlayerMenuTDforPlayer[playerid][0]); // Основной
        
    PlayerTextDrawHide(playerid,u_SkinPlayerMenuTDforPlayer[playerid][1]); // Слот 1
        
    PlayerTextDrawHide(playerid,u_SkinPlayerMenuTDforPlayer[playerid][2]); // Слот 2
        
    PlayerTextDrawHide(playerid,u_SkinPlayerMenuTDforPlayer[playerid][3]); // Слот 3

        
    CancelSelectTextDraw(playerid);
        
    DeletePVar(playerid,"u_SkinPlayerMenuTD_Active");
        return 
    1;

    Думаю так будет лучше
    PHP код:
    // ===================== [ by Scooter ] =================
    stock ReturnPlayerMenu(playerid,status)
    {
        
    // Если он в машине
        
    if(IsPlayerInAnyVehicle(playerid)) return 1;
        
    // Если он не активирован
        
    if(status == 1)
        {
            new 
    empty_skin PlayerInfo[playerid][pSex] == 154 140;
            if(
    GetPlayerSkin(playerid) != empty_skin)
            {
                
    u_SkinSlot[playerid][0] = (PlayerInfo[playerid][pMember]) ? (PlayerInfo[playerid][pModel]) : (PlayerInfo[playerid][pChar]);
            }
            else 
    u_SkinSlot[playerid][0] = empty_skin;

            for(new 
    ii<5i++) 
            {
                
    TextDrawShowForPlayer(playerid,u_SkinPlayerMenuTD[i]);
                if(
    4)
                {
                    
    PlayerTextDrawSetPreviewModel(playerid,u_SkinPlayerMenuTDforPlayer[playerid][i],u_SkinSlot[playerid][i]);
                    
    PlayerTextDrawShow(playerid,u_SkinPlayerMenuTDforPlayer[playerid][i]);
                }
            }

            
    SelectTextDraw(playerid0xcfcfcfAA);
            
    SetPVarInt(playerid,"u_SkinPlayerMenuTD_Active",1);
            return 
    1;
        }
        
    // Иначе
        
    for(new ii<5i++) 
        {
            
    TextDrawHideForPlayer(playerid,u_SkinPlayerMenuTD[i]);
            if(
    4PlayerTextDrawHide(playerid,u_SkinPlayerMenuTDforPlayer[playerid][i]); // Основной
        
    }

        
    CancelSelectTextDraw(playerid);
        
    DeletePVar(playerid,"u_SkinPlayerMenuTD_Active");
        return 
    1;

    P.S. Вообще не похоже на мануал, это скорее просто готовое решение.

  3. #13
    Аватар для TheMallard
    Пользователь

    Статус
    Оффлайн
    Регистрация
    08.12.2015
    Адрес
    San Fierro
    Сообщений
    217
    Репутация:
    39 ±
    У нас готовое решение называется мануалом, а мануал уроком.

  4. #14
    Аватар для Unreal
    Пользователь

    Статус
    Оффлайн
    Регистрация
    24.09.2014
    Сообщений
    133
    Репутация:
    4 ±
    Цитата Сообщение от TheMallard Посмотреть сообщение
    У нас готовое решение называется мануалом, а мануал уроком.
    а разве есть такой раздел ? готовое решение?
    Pro-pawn.ru

  5. #15
    Аватар для arzumanov
    Пользователь

    Статус
    Оффлайн
    Регистрация
    24.10.2015
    Сообщений
    3
    Репутация:
    0 ±
    Цитата Сообщение от Unreal Посмотреть сообщение
    а разве есть такой раздел ? готовое решение?
    тут это называется "Разработки".

    Цитата Сообщение от TheMallard Посмотреть сообщение
    У нас готовое решение называется мануалом, а мануал уроком.
    мануал и урок - практически синонимы. отличий нет.

  6. #16
    Аватар для DeimoS
    Модератор?

    Статус
    Оффлайн
    Регистрация
    27.01.2014
    Адрес
    Восточный Мордор
    Сообщений
    5,588
    Репутация:
    1984 ±
    Цитата Сообщение от arzumanov Посмотреть сообщение
    мануал и урок - практически синонимы. отличий нет.
    Как "RP" и "RPG". Только вот в SA-MP они приобрели свой смысл. Так же и с "Мануал/Урок"
    Связаться со мной в VK можно через личные сообщения этой группы
    Заказы не принимаю

    Широко известно, что идеи стоят 0.8333 цента каждая (исходя из рыночной цены 10 центов за дюжину).
    Великих идей полно, на них нет спроса.
    Воплощение идеи в законченную игру требует долгой работы,
    таланта, терпения и креативности, не говоря уж о затратах денег, времени и ресурсов.
    Предложить идею просто, воплотить – вот в чём проблема

    Steve Pavlina

 

 
Страница 2 из 2 ПерваяПервая 1 2

Информация о теме

Пользователи, просматривающие эту тему

Эту тему просматривают: 1 (пользователей: 0 , гостей: 1)

Ваши права

  • Вы не можете создавать новые темы
  • Вы не можете отвечать в темах
  • Вы не можете прикреплять вложения
  • Вы не можете редактировать свои сообщения
  •