This part does not look coherent, usually the setter methods are created to set the attribute values but in this case you are using them to get the values from the attributes, we need to correct this