using variables properly