some improvements.

Set an attribute is a void process I think.