UICompass

Вид для печати

UICompass

UICompass v0.7

http://tscars.narod.ru/p-w/new/N.png

Данный инклуд предназначен для реализации компаса с стиле PUBG. Он выводит нынешнее направление включая три предыдущих и последующих.

Все направления в шкале кратны максимальному шагу, MAX_UICOMPASS_STEP. Направление выводится с интервалом. Например при [41 - 47] выведет 45. Поддерживается только нечетное количество TD!

Реализация TD на вашей стороне.

RoundCompassDirection - округляет направление до кратного round.

PHP код:

RoundCompassDirection(Float:angle, round);

Float:angle - нынешнее направление игрока
round - делает число кратным указанному значению

CreateCompassString - создает строку с направлением игрока.

PHP код:

CreateCompassString(angle, round);

angle - направление игрока полученное из RoundCompassDirection
round - делает число кратным указанному значению

CompassHeadingNorth - определяет, в каком из TD будет выведено "N".

PHP код:

CompassHeadingNorth(angle);

angle - направление игрока полученное из RoundCompassDirection

Директивы По умолчанию Описание

MAX_UICOMPASS_TD 7 Количество TD, которое будет использоваться в Вашем компасе

MIN_UICOMPASS_STEP 5 Минимальный шаг компаса

MAX_UICOMPASS_STEP 15 Максимальный шаг компаса

Применение:

Создаем коллбэк для таймера.

http://tscars.narod.ru/p-w/new/compass.png

PHP код:

forward UICompassTimer(playerid); public UICompassTimer(playerid) { static const step_1 = MAX_UICOMPASS_STEP * 1, step_2 = MAX_UICOMPASS_STEP * 2, step_3 = MAX_UICOMPASS_STEP * 3; static north, result, direction, Float:angle; //Узнаем направление игрока GetPlayerFacingAngle(playerid, angle); //Получаем максимальный шаг компаса result = RoundCompassDirection(angle); //Получаем минимальный шаг компаса direction = RoundCompassDirection(angle, MIN_UICOMPASS_STEP); //Узнаем, в каком TD будет выведено "N" north = CompassHeadingNorth(result); //Три предыдущих направления PlayerTextDrawSetString(playerid, td_uicompass[playerid][0], CreateCompassString(result - step_3)); PlayerTextDrawSetString(playerid, td_uicompass[playerid][1], CreateCompassString(result - step_2)); PlayerTextDrawSetString(playerid, td_uicompass[playerid][2], CreateCompassString(result - step_1)); //Нынешнее направление PlayerTextDrawSetString(playerid, td_uicompass[playerid][3], CreateCompassString(result)); //Три последующих направления PlayerTextDrawSetString(playerid, td_uicompass[playerid][4], CreateCompassString(result + step_1)); PlayerTextDrawSetString(playerid, td_uicompass[playerid][5], CreateCompassString(result + step_2)); PlayerTextDrawSetString(playerid, td_uicompass[playerid][6], CreateCompassString(result + step_3)); //Выводим направление кратное минимальному шагу PlayerTextDrawSetString(playerid, td__compass_dir[playerid], CreateCompassString(direction, MIN_UICOMPASS_STEP)); //Устанавливаем всем TD изначальный цвет (Готов выслушать предложения по упрощению) PlayerTextDrawColor(playerid, td_uicompass[playerid][0], 0xFFFFFFFF); PlayerTextDrawShow(playerid, td_uicompass[playerid][0]); PlayerTextDrawColor(playerid, td_uicompass[playerid][1], 0xFFFFFFFF); PlayerTextDrawShow(playerid, td_uicompass[playerid][1]); PlayerTextDrawColor(playerid, td_uicompass[playerid][2], 0xFFFFFFFF); PlayerTextDrawShow(playerid, td_uicompass[playerid][2]); PlayerTextDrawColor(playerid, td_uicompass[playerid][3], 0xFFFFFFFF); PlayerTextDrawShow(playerid, td_uicompass[playerid][3]); PlayerTextDrawColor(playerid, td_uicompass[playerid][4], 0xFFFFFFFF); PlayerTextDrawShow(playerid, td_uicompass[playerid][4]); PlayerTextDrawColor(playerid, td_uicompass[playerid][5], 0xFFFFFFFF); PlayerTextDrawShow(playerid, td_uicompass[playerid][5]); PlayerTextDrawColor(playerid, td_uicompass[playerid][6], 0xFFFFFFFF); PlayerTextDrawShow(playerid, td_uicompass[playerid][6]); //Устанавливаем нужному TD желтый цвет, где будет выведено "N" if (0 <= north < MAX_UICOMPASS_TD) { PlayerTextDrawColor(playerid, td_uicompass[playerid][north], 0xFFFF00FF); PlayerTextDrawShow(playerid, td_uicompass[playerid][north]); } return 1; }

Автор: m1n1vv
Скачать/Репозиторий: https://github.com/m1n1vv/UICompass
Версия: v0.7

13.12.2017, 22:45
Daniel_Cortez
Взглянул на код, вот проблемы, которые сразу бросились в глаза:
- Неудачно названные функции. К примеру, в английском языке первым в предложении ставится дополнительное существительное, а после него основное, поэтому корректным названием будет не "GetDirectionCompass", а "GetCompassDirection".
  Настоятельно рекомендую заглянуть сюда, здесь больше примеров в пунктах 14 и 15.
- Функция названа CompassSetString, но на вход принимает не строку, а число. Это идёт вразрез с логикой, по которой именуются стандартные функции TextDrawSetString и PlayerTextDrawSetString (а на неё ориентируются авторы многих других хорошо выполненных инклудов, так что это не просто пара функций).
- Магические числа.
  
  PHP код:
  
  uic__N = 24 - uic__N + uic__min_N;
  
  Пришлось портатить несколько минут, чтобы понять, что "24" здесь означает "360 / 15". Можно было бы вынести это в отдельную константу ("MAX_UICOMPASS_DIRECTIONS", например).
- Путаница со сравнением целочисленных и логических значений с 0/false.
  
  PHP код:
  
  if (!angle)
  
  Обычно таким образом проверяют логические переменные (или выражения) на равенство false. Из-за этого возникает впечатление, что angle - переменная логического типа, и приходится перемещаться в самый верх инклуда, к объявлению этой переменной, чтобы убедиться в обратном.
  Та же самая проблема, но с проверкой на ненулевое значение:
  
  PHP код:
  
  if (uic__diff)
  
  Не ленитесь дописать "== 0" или "!= 0" - это отнимет полсекунды вашего времени, но зато будет понятно, что производится сравнение именно с 0, а не с true/false.
- Этот отрывок
  
  PHP код:
  
  if (angle > 360) angle -= 360; else if (angle < 0) angle += 360;
  
  можно заменить на
  
  PHP код:
  
  angle %= 360;
- Что помешало объявить это как константу?
  
  PHP код:
  
  static getN; getN = (UICOMPASS_MAX_TD + 1) / 2;
- Табы вперемешку с пробелами. На GitHub это особенно заметно, т.к. там один таб равносилен не 4, а 8 пробелам.
  Бросайте pawno, возьмите нормальный текстовый редактор, который не подсовывает пробелы вместо табов - на форуме есть мануалы по адаптации (1, 2).
- В коде нет лицензии. Отсутствие лицензии означает отсутствие разрешения на использование кода, т.е. использование сего инклуда по сути незаконно (по крайней мере, такой принцип действует в юрисдикциях США и стран Европы, на счёт кривого российского законодательства не уверен).
  Устранить эту проблему довольно просто: добавьте в начало файла копирайт и текст лицензии (к примеру, я обычно выкладываю свой код под лицензией zlib - она довольно простая и накладывает минимум обязательств на пользователя, но вы вправе выбрать любую другую лицензию).
Задумка в целом хорошая, но реализация немного хромает. Хотя, справедливости ради стоит отметить объявление локальных переменных с атрибутом static - для одиночных переменных это выгоднее, чем создание переменных в стеке с ключевым словом new, и позволяет немного снизить требования к используемой памяти, но немногие знают об этом приёме.
14.12.2017, 00:03
m1n1vv

А если CreateCompassString и RoundCompassDirection?

Я уже который год использую сублайм.
14.12.2017, 13:09
Fallen A.

Цитата:

Сообщение от m1n1vv

PHP код:

//Устанавливаем всем TD изначальный цвет (Готов выслушать предложения по упрощению) PlayerTextDrawColor(playerid, td_uicompass[playerid][0], 0xFFFFFFFF); PlayerTextDrawShow(playerid, td_uicompass[playerid][0]); PlayerTextDrawColor(playerid, td_uicompass[playerid][1], 0xFFFFFFFF); PlayerTextDrawShow(playerid, td_uicompass[playerid][1]); PlayerTextDrawColor(playerid, td_uicompass[playerid][2], 0xFFFFFFFF); PlayerTextDrawShow(playerid, td_uicompass[playerid][2]); PlayerTextDrawColor(playerid, td_uicompass[playerid][3], 0xFFFFFFFF); PlayerTextDrawShow(playerid, td_uicompass[playerid][3]); PlayerTextDrawColor(playerid, td_uicompass[playerid][4], 0xFFFFFFFF); PlayerTextDrawShow(playerid, td_uicompass[playerid][4]); PlayerTextDrawColor(playerid, td_uicompass[playerid][5], 0xFFFFFFFF); PlayerTextDrawShow(playerid, td_uicompass[playerid][5]); PlayerTextDrawColor(playerid, td_uicompass[playerid][6], 0xFFFFFFFF); PlayerTextDrawShow(playerid, td_uicompass[playerid][6]);

Не?

PHP код:

for(new i; i < 7; i++) { PlayerTextDrawColor(playerid, td_uicompass[playerid][i], 0xFFFFFFFF); PlayerTextDrawShow(playerid, td_uicompass[playerid][i]); }
14.12.2017, 13:42
m1n1vv

Цитата:

Сообщение от Fallen A.

Не?

PHP код:

for(new i; i < 7; i++) { PlayerTextDrawColor(playerid, td_uicompass[playerid][i], 0xFFFFFFFF); PlayerTextDrawShow(playerid, td_uicompass[playerid][i]); }

Потеря скорости
14.12.2017, 13:47
Fallen A.

Цитата:

Сообщение от m1n1vv

Потеря скорости

В таком случае использовать форич.
14.12.2017, 13:52
m1n1vv
Цитата:

Сообщение от Fallen A.

В таком случае использовать форич.

Любой цикл приведет к потере скорости. Я хотел бы услышать предложение о функции, которая заменит цвет у предыдущего и последующего TD. +-1 здесь не поможет.

Что-то вроде такого:

PHP код:

stock ReturnColorCompass(&previous, &next) { static const max_limit = MAX_UICOMPASS_TD - 1; previous = uic__N - 1; next = uic__N + 1; previous = (previous > max_limit) ? (0) : ((previous < 0) ? (max_limit) : (previous)); next = (next > max_limit) ? (0) : ((next < 0) ? (max_limit) : (next)); return 1; }

Не проверял. Сделал на быструю руку.

UICompass V0.4
- Изменены названия двух функций:
  - GetDirectionCompass > RoundCompassDirection;
  - CompassSetString > CreateCompassString.
- Внесены небольшие изменения в работу инклуда.
UICompass V0.5
- Добавлена возможность изменения минимального шага. По умолчанию было 15.
  За это отвечает директива: UICOMPASS_STEP.
15.12.2017, 06:07
m1n1vv

DEL
15.12.2017, 12:21
Daniel_Cortez

Цитата:

Сообщение от m1n1vv

А если CreateCompassString

Советую вдуматься в смысл слова "Create" и для примера посмотреть, в каких функциях SA-MP это слово используется.

Цитата:

Сообщение от m1n1vv

и RoundCompassDirection?

Либо "ToCompassDirection", ведь функция по сути осуществляет конверсию угла.

Цитата:

Сообщение от Fallen A.

Не?

PHP код:

for(new i; i < 7; i++) { PlayerTextDrawColor(playerid, td_uicompass[playerid][i], 0xFFFFFFFF); PlayerTextDrawShow(playerid, td_uicompass[playerid][i]); }

Цитата:

Сообщение от m1n1vv

Потеря скорости

Куда больше времени уйдёт, к примеру, на вызов нативной функции (т.е. на сохранение состояния виртуальной машины, переход на нативный код и восстановление ВМ, не считая времени выполнения самой функции).
Но ведь лучше же засорять свой код, чтобы (может быть) сэкономить пару пикосекунд в коде, который выполняется 2-3 раза в секунду, правда?
15.12.2017, 13:15
m1n1vv

Значит лучше взять в цикл?

Если грамотно реализовать ту функцию, то функций изменения цвета будет в 4 раза меньше. Правда это скорей всего не возможно (при условии резкой смены угла). И для корректной работы компаса нужен таймер на 100-200 мс.

А как тут будет лучше?

PHP код:

stock CompassHeadingNorth(angle) { uic__N = angle / UICOMPASS_STEP; if (uic__N >= uic__max_N) uic__N = uic__max_directions - uic__N + uic__min_N; else { uic__N -= uic__min_N; uic__N = ~uic__N + 1; } return uic__N; }

PHP код:

stock CompassHeadingNorth(angle) { angle /= UICOMPASS_STEP; return (angle >= uic__max_N) ? (uic__max_directions - angle + uic__min_N) : (~(angle - uic__min_N) + 1); }
17.12.2017, 22:17
Daniel_Cortez

Цитата:

Сообщение от m1n1vv

Значит лучше взять в цикл?

Всяко лучше, чем плодить китайский код.

Цитата:

Сообщение от m1n1vv

А как тут будет лучше?

PHP код:

stock CompassHeadingNorth(angle) {     uic__N = angle / UICOMPASS_STEP;     if (uic__N >= uic__max_N)         uic__N = uic__max_directions - uic__N + uic__min_N;     else     {         uic__N -= uic__min_N;         uic__N = ~uic__N + 1;     }     return uic__N; }

PHP код:

stock CompassHeadingNorth(angle) {     angle /= UICOMPASS_STEP;     return (angle >= uic__max_N) ? (uic__max_directions - angle + uic__min_N) : (~(angle - uic__min_N) + 1); }

Тернарные выражения лучше не использовать, если можно обойтись ветвлением if. Кстати, в первом примере фигурные скобки только в одной из веток if выглядят нелепо (да и зачем вообще нужно было разбивать выражение в той ветке на два?). Обычно скобки ставят либо на обеих ветках if, либо ни на одной.

Цитата:

Сообщение от Daniel_Cortez

[*]Что помешало объявить это как константу?

PHP код:

    static         getN; getN = (UICOMPASS_MAX_TD + 1) / 2;

Цитата:

Сообщение от m1n1vv

PHP код:

    static const getN = (MAX_UICOMPASS_TD + 1) / 2;

Это не константа -_-

P.S.: Также до сих пор не видно никаких улучшений в плане именования функций и лицензии на код. Если что-то непонятно, не бойтесь спросить.
17.12.2017, 22:48
m1n1vv

Цитата:

Сообщение от Daniel_Cortez

Всяко лучше, чем плодить китайский код.

Тернарные выражения лучше не использовать, если можно обойтись ветвлением if. Кстати, в первом примере фигурные скобки только в одной из веток if выглядят нелепо (да и зачем вообще нужно было разбивать выражение в той ветке на два?). Обычно скобки ставят либо на обеих ветках if, либо ни на одной.

Это не константа -_-

P.S.: Также до сих пор не видно никаких улучшений в плане именования функций и лицензии на код. Если что-то непонятно, не бойтесь спросить.

Немного поправил CompassHeadingNorth:

PHP код:

stock CompassHeadingNorth(angle) { angle /= UICOMPASS_STEP; if (angle >= uic__max_N) angle = uic__max_directions - angle + uic__min_N; else angle = ~(angle - uic__min_N) + 1; return angle; }

Лицензию добавлю скорей всего со следующей версии. В смысле не константа? Просто писать const?
17.12.2017, 22:59
Geebrox

Цитата:

Сообщение от m1n1vv

В смысле не константа? Просто писать const?

const (constant) - переводится как "постоянная", то есть зачение который хранится в константе постоянный и его нельзя изменить (можно изменить только переобъявив константу)

Можно объявить:

PHP код:

const CONST_NAME = const_value;

или же

PHP код:

#define CONST_NAME some_value

Обычно объявляется в начале кода, чтобы иметь доступ в любом месте.
17.12.2017, 23:45
Nexius_Tailer

Да проще уже было бы как дефайн её объявить и не париться.
18.12.2017, 03:54
m1n1vv

Почему-то у меня lдикое желание добавить GetDirectionOfAircraft в инклуд...
24.12.2017, 22:33
m1n1vv
v0.7
- Добавлена поддержка вывода минимального шага. Который кратный 5;
- У RoundCompassDirection и CreateCompassString появился новый параметр: round;
- Новые макросы: MIN_UICOMPASS_STEP, MAX_UICOMPASS_STEP.
25.12.2017, 21:00
m1n1vv

Вот что умудрился сделать

http://www.youtube.com/watch?v=BCsdpoDG4Xg
25.12.2017, 21:57
Fallen A.

Цитата:

Сообщение от m1n1vv

Вот что умудрился сделать

http://www.youtube.com/watch?v=BCsdpoDG4Xg

А с остальными городами что?
25.12.2017, 22:49
m1n1vv

Цитата:

Сообщение от Fallen A.

А с остальными городами что?

Место действий только в одном городе. Другие пока будут закрыты.
15.03.2018, 21:11
Daniel_Cortez

Один из зарубежных пользователей жалуется на то, что текстдравы компаса криво размещены:

Цитата:

Сообщение от jensen

https://i.imgur.com/KkuHCDI.png
look? numbers disorder (red)
- i want as green

sorrry my english again :)

Оригинальная тема с обсуждением: http://pro-pawn.ru/showthread.php?16052

Директивы	По умолчанию	Описание
MAX_UICOMPASS_TD	7	Количество TD, которое будет использоваться в Вашем компасе
MIN_UICOMPASS_STEP	5	Минимальный шаг компаса
MAX_UICOMPASS_STEP	15	Максимальный шаг компаса