[Sonic X]

DeimoS (10 Апрель 2018 - 19:26) писал:

Её и правда пока нет.

Меня тут другое заинтересовало. Почему инклуд будет называться Weapon FIX? Что он фиксит? Ты только расширяешь функционал ведь и не более

Самое главное - фикс перезарядки, но называть recharge fix тоже будет не правильно. Код скоро выложу, чтобы вы принцип поняли. Хотелось бы сделать ВСЁ, а потом выложить, чтоб уже исправлять.

Сообщение отредактировал Sonic X: 10 Апрель 2018 - 20:36

[Romzes]

Sonic X (10 Апрель 2018 - 20:28) писал:

Самое главное - фикс перезарядки

Речь идет о колесике мышки? То есть когда его прокручиваешь, оружие само перезаряжается.

[Sonic X]

Romzes (10 Апрель 2018 - 20:52) писал:

Речь идет о колесике мышки? То есть когда его прокручиваешь, оружие само перезаряжается.

Да, изначально хотел решить эту проблему. Потом же потихоньку функции начали сами собой вытекать. Поэтому сейчас довожу всё до ума.

Сообщение отредактировал Sonic X: 15 Апрель 2018 - 12:33

[Sonic X]

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

[SooBad]

Вообще, все глобальные переменные, которые объявлены вначале инклюда можно было заменить на константы (макросы), все-таки это постоянные значения. Я бы вообще все на препроцессор положил.
Далее. Неверно построена структура перехватов внутри вызываемого коллбека. Посмотри любой урок в инете и исправь ошибку.
Ну и по мелочи, всё логирование передается с "префиксом" Weapon Fix.

[Sonic X]

SooBad (17 Апрель 2018 - 08:17) писал:

Нажмите сюда, чтобы прочитать это сообщение. [Показать]

Вообще, все глобальные переменные, которые объявлены вначале инклюда можно было заменить на константы (макросы), все-таки это постоянные значения. Я бы вообще все на препроцессор положил.
Далее. Неверно построена структура перехватов внутри вызываемого коллбека. Посмотри любой урок в инете и исправь ошибку.
Ну и по мелочи, всё логирование передается с "префиксом" Weapon Fix.

По поводу переменных, сделал так, чтобы можно было изменить их во время игры, "гибкость".

Исправил

---

Вообще, как код построен? Более менее приемлемо? Я думал насчёт массива, который будет хранить в себе значения максимального количества патрон, но это пришлось бы всё переписывать и тогда не получилось бы, наверное, сделать независимое количество патрон для двух оружий в руках (не просто ammo[№]*2).

Сообщение отредактировал Sonic X: 17 Апрель 2018 - 12:47

[SooBad]

Sonic X (17 Апрель 2018 - 08:20) писал:

По поводу переменных, сделал так, чтобы можно было изменить их во время игры, "гибкость".

С макросами прокатывает абсолютно то же самое. Правда, придётся писать чуть больше кода.
Практический пример (если я правильно тебя понял):

Скрытый текст [Показать]

#define TEST_MACROS 30

main()
{
    new playerid = 1;
	
    CallLocalFunction("OnPlayerCommandText", "ds", playerid, "/test");

    //console: result - 40 (если не понял - мы просто банально сымитировали вызов команды)
    
}

public OnPlayerCommandText(playerid, cmdtext[])
{
    if(!strcmp(cmdtext, "/test", true))
    {
	#if defined TEST_MACROS
	    #if TEST_MACROS == 30
	        #undef TEST_MACROS
		#define TEST_MACROS 40
	    #else
	        #undef TEST_MACROS
		#define TEST_MACROS 30
	    #endif
	#endif
		
	printf("%d", TEST_MACROS);
		
	return 1;
    }
    return 0;
}

Sonic X (17 Апрель 2018 - 08:20) писал:

Вообще, как код построен? Более менее приемлемо?

Код построен нормально (моё субъективное мнение). В микрооптимизации нет смысла.

Сообщение отредактировал SooBad: 17 Апрель 2018 - 14:59

[Sonic X]

SooBad (17 Апрель 2018 - 14:56) писал:

Нажмите сюда, чтобы прочитать это сообщение. [Показать]

С макросами прокатывает абсолютно то же самое. Правда, придётся писать чуть больше кода.
Практический пример (если я правильно тебя понял):

Скрытый текст [Показать]

#define TEST_MACROS 30

main()
{
    new playerid = 1;
	
    CallLocalFunction("OnPlayerCommandText", "ds", playerid, "/test");
    
}

public OnPlayerCommandText(playerid, cmdtext[])
{
    if(!strcmp(cmdtext, "/test", true))
    {
	#if defined TEST_MACROS
	    #if TEST_MACROS == 30
	        #undef TEST_MACROS
		#define TEST_MACROS 40
	    #else
	        #undef TEST_MACROS
		#define TEST_MACROS 30
	    #endif
	#endif
		
	printf("%d", TEST_MACROS);
		
	return 1;
    }
    return 0;
}

Код построен нормально (моё субъективное мнение). В микрооптимизации нет смысла.

Понял тебя, спасибо за пример

[Nexius]

Из беглого просмотра кода наткнулся на пару небольших опечаток

stock WP_SetPlayerWeaponAmmo(id, wp_weapon, wp_ammo)
{
    if(wp_weapon < 0 || wp_weapon > 46 || wp_weapon == 39 || wp_weapon == 40) return printf("[ERROR Weapon PLUS] INVALID WEAPON ID \"%i\"", wp_weapon);
    if(wp_weapon < 0) return printf("[ERROR Weapon PLUS] INVALID AMMO AMOUNT \"%i\"", wp_ammo); //Здесь (wp_ammo < 0)
 
    wp[id][ammo][GetWeaponSlot(wp_weapon)] += wp_ammo;
 
    return 1;
 
}
 
stock WP_SetPlayerWeaponMag(id, wp_weapon, wp_ammo)
{
    if(wp_weapon < 0 || wp_weapon > 46 || wp_weapon == 39 || wp_weapon == 40) return printf("[ERROR Weapon PLUS] INVALID WEAPON ID \"%i\"", wp_weapon);
    if(wp_weapon < 0) return printf("[ERROR Weapon PLUS] INVALID AMMO AMOUNT \"%i\"", wp_ammo); //И здесь
 
    wp[id][mag][GetWeaponSlot(wp_weapon)] += wp_ammo;
 
    return 1;
 
}

//И ещё в некоторых похожих функциях ниже

SooBad (17 Апрель 2018 - 14:56) писал:

С макросами прокатывает абсолютно то же самое. Правда, придётся писать чуть больше кода.
Практический пример (если я правильно тебя понял):

Скрытый текст [Показать]

#define TEST_MACROS 30

main()
{
    new playerid = 1;
	
    CallLocalFunction("OnPlayerCommandText", "ds", playerid, "/test");

    //console: result - 40 (если не понял - мы просто банально сымитировали вызов команды)
    
}

public OnPlayerCommandText(playerid, cmdtext[])
{
    if(!strcmp(cmdtext, "/test", true))
    {
	#if defined TEST_MACROS
	    #if TEST_MACROS == 30
	        #undef TEST_MACROS
		#define TEST_MACROS 40
	    #else
	        #undef TEST_MACROS
		#define TEST_MACROS 30
	    #endif
	#endif
		
	printf("%d", TEST_MACROS);
		
	return 1;
    }
    return 0;
}

Не прокатит, потому что работают они совершенно иначе: изменённое значение макроса распространяется только на код, который идёт дальше этого изменения, и более того, изменения произойдёт независимо от обычных "runtime-условий" в коде.

Вот как пример два простых куска кода, которые ломают весь алгоритм с твоим редефайном вместо нормальной переменной:
1) Выведем значения макроса ещё раз до его фактического изменения, но после выполнения функции, в которой он изменяется:

Скрытый текст [Показать]

#define TEST_MACROS 30

main()
{
    new playerid = 1;

    CallLocalFunction("OnPlayerCommandText", "ds", playerid, "/test");

    printf("2: %d", TEST_MACROS); //Вот что мы добавили
}

public OnPlayerCommandText(playerid, cmdtext[])
{
    if(!strcmp(cmdtext, "/test", true))
    {
        #if defined TEST_MACROS
            #if TEST_MACROS == 30
                #undef TEST_MACROS
                #define TEST_MACROS 40
            #else
                #undef TEST_MACROS
                #define TEST_MACROS 30
            #endif
        #endif

        printf("1: %d", TEST_MACROS);

        return 1;
    }
	return 0;
}

Результаты:

1: 40
2: 30

2) Вызовем вообще совершенно другую команду, чтобы никакие обычные условия для редефайна не сработали:

Скрытый текст [Показать]

#define TEST_MACROS 30

main()
{
    new playerid = 1;

    CallLocalFunction("OnPlayerCommandText", "ds", playerid, "/ghkdfhg");
}

public OnPlayerCommandText(playerid, cmdtext[])
{
    if(!strcmp(cmdtext, "/test", true))
    {
        #if defined TEST_MACROS
            #if TEST_MACROS == 30
                #undef TEST_MACROS
                #define TEST_MACROS 40
            #else
                #undef TEST_MACROS
                #define TEST_MACROS 30
            #endif
        #endif

        return 1;
    }
    printf("%d", TEST_MACROS); //Переместили вывод сюда, чтобы вывело итог для любой команды
	return 0;
}

Результаты:

40

Сообщение отредактировал Nexius: 17 Апрель 2018 - 21:14

[Sonic X]

Забирайте себе эти наработки, если вдруг захотите допилить систему. У меня в планах другая. В принципе, её уже можно использовать, только с транспортом нет взаимодействия.

По поводу TD [Показать]

Нужно сделать 2 функции. Первая будет показывать счётчик, вторая убирать.
В OnPlayerUpdate:

if(wp[playerid][weapon] >= 22)
{
    ShowTD(playerid);
}
else
{
    DestroyTD(playerid);
}

В функции ShowTD сделать проверки, если же это гранаты или дробовик, то показывать только 1 TD, переменная wp[playerid][ammo][slot], если какая-нибудь бита, то ничего не показывать.

Сообщение отредактировал Sonic X: 18 Апрель 2018 - 21:49