Выскакивает ворнинг "Compare string with number"

Внимание

Для участия в обсуждениях и оформления подписки на новые сообщения форума вам необходимо зарегистрироваться.
Главная » Основные форумы » Программирование на языке Lua
Страницы: 1
RSS
Выскакивает ворнинг "Compare string with number", А его не должно быть, по идее!
 
#1
24.01.2021 18:00:53
Определяю время сервера так:
 
Код
          HHMMSS = getInfoParam("SERVERTIME")

          if string.len(HHMMSS) == 7 then   hcc, mcc, scc = string.match(HHMMSS,"(%d)%p(%d%d)%p(%d%d)")   else  hcc, mcc, scc = string.match(HHMMSS,"(%d%d)%p(%d%d)%p(%d%d)")
          end
          hcc = tonumber(hcc)   if type(hcc) ~= 'number' then  hcc = 0  end
          mcc = tonumber(mcc)   if type(mcc) ~= 'number' then  mcc = 0  end
          scc = tonumber(scc)   if type(scc) ~= 'number' then  scc = 0  end

потом работаю с часами и минутами:
Код
                if type(hcc) == 'number' and type(mcc) == 'number' then
                       if hcc > 10 and mcc < 5 then
                                                       -- какие-то действия
                       end
                end

И на строку "if hcc <= 10 and hcc >= 9 and mcc < 5 then" периодически вылетает вышеупомянутый ворнинг. Почему он выскакивает? Ведь вроде все преобразования в числа и дополнительные проверки произведены!

На работу скрипта не влияет, но неприятно...
 
 
#2
24.01.2021 18:50:49
Код
   ST = getInfoParam("SERVERTIME") -- Время сервера
   if #ST == 7 then ST = "0"..ST end
   
   if ST > "10:00:00" and ST < "14:00:00" then
      ...
   end
Как вариант
 
 
#3
24.01.2021 18:57:59
Так вы код то покажите нормально.
Сравнение которое вы указали где в коде расположено?
Телепаты в отпуске

PS: SERVERTIME в нерабочее время биржи может показывать все что угодно. Для определения времени уж лучше os.date() использовать
 
 
#4
24.01.2021 19:07:55
Константин Рейм, мне нужно, чтобы в числах были отдельно часы и минуты.

BlaZed, первый фрагмент кода находится в main(), второй - в OnQuote() , ворнинг возникает в рабочее время биржи.
 
 
#5
24.01.2021 19:12:01
Цитата
Александр Волфовиц написал:
Константин Рейм, мне нужно, чтобы в числах были отдельно часы и минуты.

BlaZed, первый фрагмент кода находится в main(), второй - в OnQuote() , ворнинг возникает в рабочее время биржи.
А сама аварийная строка "if hcc <= 10 and hcc >= 9 and mcc < 5 then" где находится?
 
 
#6
24.01.2021 19:12:55
BlaZed, os.date() выдаёт время на компьютере с квиком, а нужно именно время брокерского сервера, которое и выдаёт  getInfoParam("SERVERTIME")
 
 
#7
24.01.2021 19:15:54
Игорь М,в OnQuote(). Никаких действий с hcc и mcc , кроме получения времени и преобразования строк в числа, не производится.  
 
 
#8
24.01.2021 19:24:13
Предположение: В строке if type(hcc) == 'number' and type(mcc) == 'number' then значение глобальной hcc ещё number, а дальше при сравнении в строке "if hcc <= 10 and hcc >= 9 and mcc < 5 then" оно уже string.
Попробуйте перед первым if поставить что-то типа local hcc = hcc (или как вам по коду лучше будет)
 
 
#9
24.01.2021 19:42:36
Лучше перед if объявит локальную hcc, а в конце поставить hcc = hcc или что-то типа этого. Тогда для аварийной строки в OnQuote () hcc всегда будет number.
 
 
#10
24.01.2021 20:25:16
Александр Волфовиц, отличный пример, как прострелить себе ногу в многопоточном окружении. Выше верно написали, мейн меняет hcc на строку как раз после проверки типа в колбеке. Обращение к глобальной переменной из разных потоков должно быть атомарным. Я б посоветовал 1) упаковать все три hcc, mcc, scc в луа-таблицу; 2) в мейне строить локально эту таблицу (а еще лучше специальную функцию написать, ее возвращающую) и сохранять в глобальную переменную одним присвоением; 3) в колбеке создавать локальную копию опять-таки одним присвоением и далее работать только с ней.
 
 
#11
25.01.2021 06:28:14
Цитата
Александр Волфовиц написал:
первый фрагмент кода находится в main(), второй - в OnQuote() , ворнинг возникает в рабочее время биржи.

Но зачем??
Положите весь код в OnQuote - и будет счастье.
 
 
#12
25.01.2021 10:15:25
Я бы сказал, что это просто порочная практика жить в глобальном окружении. Не в смысле плохая - иногда можно, но лучше не надо. Иначе в одном месте написал f = 5 и все сломалось.
https://nick-nh.github.io/
https://github.com/nick-nh/qlua
 
 
#13
25.01.2021 21:53:57
Цитата
swerg написал:
Положите весь код в OnQuote - и будет счастье.
На малоликвидных инструментах OnQuote может и по 10 секунд не срабатывать - а время желательно чтоб было актуальное.

Вроде бы проблема решилась с помощью локальных переменных перед проверкой, спасибо всем за советы!
 
 
#14
25.01.2021 22:26:42
Зачем вам актуальное время вне OnQuote, если время вы используете в OnQuote ?
 
 
#15
25.01.2021 22:55:39
swerg, время используется не только в  OnQuote. В OnQuote просто всплыл вышеупомянутый ворнинг.
 
 
#16
26.01.2021 02:38:31
Цитата
Александр Волфовиц написал:
 На малоликвидных инструментах OnQuote может и по 10 секунд не срабатывать - а время желательно чтоб было актуальное.
Вроде бы проблема решилась с помощью локальных переменных перед проверкой, спасибо всем за советы!
Я, когда вам отвечал, торопился и поэтому что-то невнятное написал. Хорошо, что есть Антон, который все верно порекомендовал. В вашей ситуации проблема не только со строками/числами, но и со временем. Если вы хотите получать актуальное время, то делайте, как Антон написал, чтобы не было последовательных присвоений, а только разом. Очень маловероятна, но реальна ситуация, что в момент присвоения local hour, min, sec = hcc, mcc, scc вы получите час из старого определения времени в main, а минуту из нового. Подзавис основной поток в начале часа и у вас в OnQuote смещение на 59 минут.
 
 
#17
26.01.2021 06:59:26
Цитата
Александр Волфовиц написал:
время используется не только в  OnQuote. В OnQuote просто всплыл вышеупомянутый ворнинг.

Значит сделайте функцию, которая будет возвращать вам нужные 3 значения - и вызывайте ее везде, где требуется.
Иначе добра не будет.
На один прикол вы уже напоролись, еще один вам описал Игорь М.
 
 
#18
26.01.2021 09:20:44
Маленький совет со временем: переходите на unix-time.

Если Вам надо сравнивать времена, то определите строковые представления в конфигурации и переведите их в числа при старте алгоритма, чтобы не делать это постоянно. Тогда уже сравнивать время будет проще, т.к. это будут просто числа. Ну и прибавлять, убавлять время намного проще, т.к. все в секундах.
https://nick-nh.github.io/
https://github.com/nick-nh/qlua
 
 
#19
03.10.2021 09:58:23
Код
local Time = X
function OnCallback()  -- любой колбек
  Time = nil
end

function main()
  if Time and os.time() > Time then
    ...
  end
end

Существует ли вероятность того, что значение Time сбросится сразу же после проверки его существования в main и os.time() > Time вернет ошибку?
 
 
#20
03.10.2021 11:51:45
Цитата
Незнайка написал:
Существует ли вероятность того, что значение Time сбросится сразу же после проверки его существования в main
Проверил: вероятность существует.
 
 
#21
03.10.2021 20:36:23
Цитата
Незнайка написал:
Код
   local  Time  =  X
 function   OnCallback ()   -- любой колбек 
  Time  =   nil 
 end 

 function   main ()
   if  Time  and   os.time ()  >  Time  then 
     .. .
   end 
 end  

Существует ли вероятность того, что значение Time сбросится сразу же после проверки его существования в main и os.time() > Time вернет ошибку?
если ядро одно,то не сбросится. Если больше одного, то вероятность =1.
 
 
#22
05.10.2021 13:45:24
Цитата
nikolz написал:
если ядро одно,то не сбросится.
Зачем смуту сеете в «неокрепшие» умы  :smile: ?
---
 Даже если ядро одно, то все равно может сброситься. Потоки и при одном ядре переключаются по времени из-за выделения им квантов процессорного времени. Переключение на выполнение потока колбека может случиться сразу после проверки в main: os.time ()  >  Time. И тогда Time может сброситься при выполнении колбека.
 
 
Страницы: 1
Читают тему
Техническая поддержка
+7 383 219 1606
© 2000–2024,
ARQA Technologies
Наверх