Fixing cpp05/ex00-02

1 files changed, 35 insertions, 49 deletions

diff --git a/cpp05/ex00/Bureaucrat.cpp b/cpp05/ex00/Bureaucrat.cpp
index 5737939..927d467 100644
--- a/cpp05/ex00/Bureaucrat.cpp
+++ b/cpp05/ex00/Bureaucrat.cpp
@@ -6,67 +6,66 @@
 /*   By: charles <charles.cabergs@gmail.com>        +#+  +:+       +#+        */
 /*                                                +#+#+#+#+#+   +#+           */
 /*   Created: 2020/04/14 18:08:13 by charles           #+#    #+#             */
-/*   Updated: 2020/10/19 13:26:35 by cacharle         ###   ########.fr       */
+/*   Updated: 2020/11/17 10:23:23 by cacharle         ###   ########.fr       */
 /*                                                                            */
 /* ************************************************************************** */
 
 #include "Bureaucrat.hpp"
 
-Bureaucrat::Bureaucrat(Bureaucrat const& other)
-{
-    *this = other;
-}
+Bureaucrat::Bureaucrat(Bureaucrat const& other) { *this = other; }
 
 Bureaucrat& Bureaucrat::operator=(Bureaucrat const& other)
 {
+    m_name  = other.m_name;
     m_grade = other.m_grade;
     return *this;
 }
 
-Bureaucrat::~Bureaucrat()
-{}
+Bureaucrat::~Bureaucrat() {}
 
 Bureaucrat::Bureaucrat(std::string const& name, int grade)
     : m_name(name), m_grade(grade)
 {
+    checkGrade();
 }
 
-std::string const& Bureaucrat::getName() const
+std::string const& Bureaucrat::getName()  const { return m_name;  }
+int                Bureaucrat::getGrade() const { return m_grade; }
+
+void Bureaucrat::incrementGrade()
 {
-    return m_name;
+    m_grade--;
+    checkGrade();
 }
 
-int Bureaucrat::getGrade() const
+void Bureaucrat::decrementGrade()
 {
-    return m_grade;
+    m_grade++;
+    checkGrade();
 }
 
-void Bureaucrat::incrementGrade()
+void Bureaucrat::checkGrade()
 {
-    if (m_grade <= 1)
+    if (m_grade > 150)
+        throw Bureaucrat::GradeTooLowException();
+    if (m_grade < 1)
         throw Bureaucrat::GradeTooHighException();
-    else
-        m_grade--;
 }
 
-void Bureaucrat::decrementGrade()
+std::ostream& operator<<(std::ostream& out, Bureaucrat const& b)
 {
-    if (m_grade >= 150)
-        throw Bureaucrat::GradeTooLowException();
-    else
-        m_grade++;
+    std::cout << b.getName() << ", bureaucrat grade " << b.getGrade() << std::endl;
+    return out;
 }
 
 ///////////////////////////////////////////////////////////////////////////////
-// Exceptions
+// Exception grade too high
 ///////////////////////////////////////////////////////////////////////////////
 
-Bureaucrat::GradeTooHighException::GradeTooHighException() : std::exception()
-{}
+Bureaucrat::GradeTooHighException::GradeTooHighException() : std::exception() {}
 
 Bureaucrat::GradeTooHighException::GradeTooHighException(GradeTooHighException const& other)
-    : std::exception(other)
-{}
+    : std::exception(other) {}
 
 Bureaucrat::GradeTooHighException&
 Bureaucrat::GradeTooHighException::operator=(GradeTooHighException const& other)
@@ -75,20 +74,18 @@ Bureaucrat::GradeTooHighException::operator=(GradeTooHighException const& other)
     return *this;
 }
 
-Bureaucrat::GradeTooHighException::~GradeTooHighException()
-{}
+Bureaucrat::GradeTooHighException::~GradeTooHighException() throw() {}
 
-char const* Bureaucrat::GradeTooHighException::what() const throw()
-{
-    return "Grade is too high";
-}
+char const* Bureaucrat::GradeTooHighException::what() const throw() { return "Grade is too high"; }
+
+///////////////////////////////////////////////////////////////////////////////
+// Exception grade too low
+///////////////////////////////////////////////////////////////////////////////
 
-Bureaucrat::GradeTooLowException::GradeTooLowException() : std::exception()
-{}
+Bureaucrat::GradeTooLowException::GradeTooLowException() : std::exception() {}
 
 Bureaucrat::GradeTooLowException::GradeTooLowException(GradeTooLowException const& other)
-    : std::exception(other)
-{}
+    : std::exception(other) {}
 
 Bureaucrat::GradeTooLowException&
 Bureaucrat::GradeTooLowException::operator=(GradeTooLowException const& other)
@@ -97,19 +94,8 @@ Bureaucrat::GradeTooLowException::operator=(GradeTooLowException const& other)
     return *this;
 }
 
-Bureaucrat::GradeTooLowException::~GradeTooLowException()
-{}
-
-char const* Bureaucrat::GradeTooLowException::what() const throw()
-{
-    return "Grade is too low";
-}
+Bureaucrat::GradeTooLowException::~GradeTooLowException() throw() {}
 
-Bureaucrat::Bureaucrat()
-{}
+char const* Bureaucrat::GradeTooLowException::what() const throw() { return "Grade is too low"; }
 
-std::ostream& operator<<(std::ostream& out, Bureaucrat const& b)
-{
-    std::cout << b.getName() << ", bureaucrat grade " << b.getGrade() << std::endl;
-    return out;
-}
+Bureaucrat::Bureaucrat() {}