Прицел (без сведения) средствами XVM

[HEKPOMAHT 2,168]

@ktulho добавь двухстволки в {{py:sight.isFullClipShells}} 

[ktulho 5,790]

@HEKPOMAHT А с перезарядкой у двухстволки все нормально?

[HEKPOMAHT 2,168]

2 минуты назад, ktulho сказал:

А с перезарядкой у двухстволки все нормально?

на цифры внимания не обращал. Применил {{py:isDualGunVehicle}} - работает, {{py:sight.isFullClipShells}} - не работает. Сыграю бой - гляну как оно.

[HEKPOMAHT 2,168]

@ktulho да, перезарядка шалит

 

вот для теста реплей:

replay_last_battle.wotreplay

----

там же по идее {{py:sight.isAutoReload}} должен срабатывать  , а он не срабатывает

----

{{py:sight.leftTimeShot%3.1f}} - пусто отдаёт 

[ktulho 5,790]

1 час назад, HEKPOMAHT сказал:

там же по идее {{py:sight.isAutoReload}} должен срабатывать  , а он не срабатывает

Там все по другому. Надо подумать, какие макросы, что должны возвращать.

9 часов назад, HEKPOMAHT сказал:

@ktulho добавь двухстволки в {{py:sight.isFullClipShells}} 

Поправил. Скрипт подходит для тестового клиента.

 

quantityShells.py

Edited January 25, 2020 by ktulho

[HEKPOMAHT 2,168]

57 минут назад, ktulho сказал:

Поправил. Скрипт подходит для тестового клиента.

бодрячком 

 

57 минут назад, ktulho сказал:

Надо подумать, какие макросы, что должны возвращать.

так там тоже ж авто. Может сделать типо такого:

{{py:sight.isAutoReload}} возвращает 'auto' для техники с механизмом дозарядки, 'dual' для друхстволки, иначе пусто.

 

[ktulho 5,790]

6 минут назад, HEKPOMAHT сказал:

Может сделать типо такого:

{{py:sight.isAutoReload}} возвращает 'auto' для техники с механизмом дозарядки, 'dual' для друхстволки, иначе пусто.

 

Это надо? auto не достаточно?

[ktulho 5,790]

Исправлен макрос {{py:sight.quantityInClipShells}} для двухсвтольной техники.

Скрипт подходит для тестового клиента.

quantityShells.py

[HEKPOMAHT 2,168]

4 минуты назад, ktulho сказал:

Это надо? auto не достаточно?

ну хуже от этого не будет, хотя и {{py:isDualGunVehicle}} можно применить, чтобы двухстволки (если кто захочет) выделить, например для текстовых подстановок. Но это будет усложнение конструкции. Для тех, кто особо не вникает, то и одного auto хватит  Хотя как по мне, лучше разделить

 

почини ещё для двухстволки {{py:sight.leftTimeShot}} и с {{py:sight.isAutoReload}} нужно что-то решить

 

----

и всё таки лучше разделить 

[ktulho 5,790]

4 минуты назад, HEKPOMAHT сказал:

Но это будет усложнение конструкции.

Такая подстановка не проще:

{{py:sight.isAutoReload=auto?один|{{py:sight.isAutoReload=dual?два|три}}}}

Плюс придется менять название макроса.

Edited January 25, 2020 by ktulho

[HEKPOMAHT 2,168]

1 минуту назад, ktulho сказал:

Такая подстановка не проще:

{{py:sight.isAutoReload=auto?один|{{py:sight.isAutoReload=dual?два|три}}}}

 

так а я тебе про что толкую:

27 минут назад, HEKPOMAHT сказал:

Может сделать типо такого:

{{py:sight.isAutoReload}} возвращает 'auto' для техники с механизмом дозарядки, 'dual' для друхстволки, иначе пусто.

 

[ktulho 5,790]

1 минуту назад, HEKPOMAHT сказал:

так а я тебе про что толкую:

Про что?

[HEKPOMAHT 2,168]

3 минуты назад, ktulho сказал:

Про что?

 

32 минуты назад, HEKPOMAHT сказал:

Может сделать типо такого:

{{py:sight.isAutoReload}} возвращает 'auto' для техники с механизмом дозарядки, 'dual' для друхстволки, иначе пусто.

 

----

 

9 минут назад, ktulho сказал:

Плюс придется менять название макроса.

может так:

{{py:sight.isMultiReload}}

[ktulho 5,790]

Во-первых, is в названии как бы намекает на два варианта ответа да или нет, а не на три варианта.

Во-вторых, повторяю вопрос, это надо? Чем конструкция 

{{py:sight.isAutoReload=auto?один|{{py:sight.isAutoReload=dual?два|три}}}}

лучше

{{py:sight.isAutoReload?один|{{py:sight.isDualGunVehicle?два|три}}}}

?

[HEKPOMAHT 2,168]

6 минут назад, ktulho сказал:

?

на один скрипт меньше (не понадобится infoMyVehicle.py)

 

[ktulho 5,790]

1 минуту назад, HEKPOMAHT сказал:

на один скрипт меньше (не понадобится infoMyVehicle.py)

 

А лучше перенести макрос {{py:sight.isAutoReload}} в  infoMyVehicle.py

[HEKPOMAHT 2,168]

  

1 минуту назад, ktulho сказал:

А лучше перенести макрос {{py:sight.isAutoReload}} в  infoMyVehicle.py

тут уже сам смотри, как проще будет 

 

это ещё:

31 минуту назад, HEKPOMAHT сказал:

почини ещё для двухстволки {{py:sight.leftTimeShot}}

[ktulho 5,790]

1 минуту назад, HEKPOMAHT сказал:

 

это ещё:

Там много чего надо сделать.

[HEKPOMAHT 2,168]

1 минуту назад, ktulho сказал:

Там много чего надо сделать.

ладно тогда. Отложим на потом 

[aster 53]

тогда уж ".ReloadType"

 

Кто  знает, что ещё накрутят разрабы в будущем.

Edited January 25, 2020 by aster