Add _ in front of all your private data members. I know there's some variables above this that don't follow that rule (and they should also be changed, actually). For example, this variable should be _attach_file_layout. Clearly indicating data members versus parameters and local variables is extremely important, because changing a data member changes the state of the object.