The code you posted [imediatly] above is fine, as far as I can see, it's just It's in your if statements...

you.skill1 == 1 && counter == pos_pick

In some cases, this is not always working out to well. I'm guessing that skill1 is used to tell what type of entity it is, but I'm not understanding why you are also needing counter to equal pos_pick...