porper use of map