Incorrect Loop for Positioning Linear Actuator

Looks like it works to me:

The targets position = 72 
swcnt:0
The stroke's position = 72 
The targets position = 72 
swcnt:0
The stroke's position = 107 
The targets position = 72 
swcnt:0
The stroke's position = 102 
The targets position = 72 
swcnt:0
The stroke's position = 117 
The targets position = 72 
swcnt:0
The stroke's position = 190 
The targets position = 97 
swcnt:1
The stroke's position = 235 
The targets position = 122 
swcnt:2
The stroke's position = 307 
The targets position = 147 
swcnt:3
The stroke's position = 340 
The targets position = 172 
swcnt:4
The stroke's position = 379 
The targets position = 197 
swcnt:5
The stroke's position = 404 
The targets position = 197 
swcnt:5
The stroke's position = 451 
The targets position = 197 
swcnt:5
The stroke's position = 449 
The targets position = 197 
swcnt:5
The stroke's position = 428 
The targets position = 197 
swcnt:5
The stroke's position = 447 
The targets position = 197 
swcnt:5
The stroke's position = 469 
The targets position = 197 
swcnt:5
The stroke's position = 446 
The targets position = 197 
swcnt:5

Of course, because of the delay(3500), you have to be holding the button down at the exact time the delay(3500) expires and the loop() loops.