add default for data in constructor