use assign properly