use constructors instead of setter functions

Author: tbhaxor

BinaryFile/main.cpp

@@ -17,9 +17,8 @@ int main(int argc, char **argv) {
   ofile->open("student.dat", std::ios::binary | std::ios::trunc);
 
   // setup student object
-  Student *student = new Student();
-  student->setAge((unsigned short)std::stoul(argv[2]));
-  student->setName(argv[1]);
+  Student *student =
+      new Student(argv[1], static_cast<unsigned short>(std::stoul(argv[2])));
 
   // write binary content into student
   ofile->write(reinterpret_cast<char *>(student), sizeof(*student));

BinaryFile/student.cpp

@@ -3,12 +3,15 @@
 #include <sstream>
 #include <string>
 
-void Student::setAge(int age) { this->age = age; }
-void Student::setAge(short age) { this->age = age; }
-void Student::setAge(unsigned short age) { this->age = age; }
+Student::Student() {
+  this->name = "";
+  this->age = 0;
+}
 
-void Student::setName(std::string &name) { this->name = name; }
-void Student::setName(const char *name) { this->name = std::string(name); }
+Student::Student(const char *name, unsigned short age) {
+  this->name = std::string(name);
+  this->age = age;
+}
 
 std::string Student::whoami() const {
   std::stringstream s;

BinaryFile/student.hpp

@@ -6,12 +6,7 @@ class Student {
   unsigned short age;
 
 public:
-  void setAge(int age);
-  void setAge(short age);
-  void setAge(unsigned short age);
-
-  void setName(std::string &name);
-  void setName(const char *name);
-
+  Student();
+  Student(const char *name, unsigned short age);
   std::string whoami() const;
 };