hehe!
After 45 minutes turning around, your little remark made the trick....
Change for
Value1=
Value2 =
Value3 =
Martin
Martin:
You have to develop good coding habits... otherwise you will spend more time in debugging than in making money.
Here are my notes:
1. DO NOT use generic variable names (ie Value1, Condition1, etc.,). Always create a meaningful name, so that you can trace the origin of your values.
(the only time I use Value1 is if it is a throwaway variable. eg. in a loop)
2. in drawing objects, you don't need the assignments at all.
you could code as follow:
Code: Select all
arw_new_s(Date,Time_s, High+.0004,true);
The assignment to a variable is ONLY needed if you want to modify the object later.
Therefore you can skip Value2 and Value3 altogether:
Code: Select all
Sell.Arw = arw_new_s(Date,Time_s, High+.0004, Arrowdown);
arw_setcolor(Sell.Arw, Blue);
arw_setStyle(Sell.Arw, 0);