use parameters instead of string