no nullpointers, no data update