yes agree. Your function is much cleaner and probably faster? I
think the reason I used
was reading this: https://docs.mql4.com/marketinformation/symbolinfostring
The value of string type. In case of execution failure, information about the error can be obtained using GetLastError() function:
4106 — symbol is not selected in "Market Watch" (not found in the list of available ones),
4051 — invalid identifier of a symbol property,
4024 — internal error.
Does a call to GetLastError() add anything iif we alert for the error:
I dont wish to protract this discussion but I think rewritten with GetLastError would add something.
Anyhow, thanks for your input here. I learned something from you.