use parameter instead of constant