Issue #21: Fix code style, update README.md, Dockerfile.check (previous commit addition)

Author: taketook34

README.md

@@ -16,6 +16,7 @@ cd ecu_sw_bb
 git submodule init
 git submodule update
 ```
+<b> Necessarily check if your glog version is v0.5.0 ! </b> 
 
 ### Build Docker Image
 ```
@@ -142,7 +143,7 @@ sudo apt install -y locales
 sudo dpkg-reconfigure locales # Choose a locale en_US.UTF-8 = English, United States, UTF8
 sudo apt upgrade -y
 sudo apt install build-essential -y
-sudo apt-get install -y cpufrequtils
+sudo apt-get install -y cpufrequtils libgoogle-glog-dev
 sudo sed -i 's/GOVERNOR="ondemand"/GOVERNOR="performance"/g' /etc/init.d/cpufrequtils
 ```
 

src/kpi_rover_ecu/src/KPIRoverECU.cpp

@@ -11,9 +11,9 @@
 #include <vector>
 
 #include "IMUController.h"
-#include "loggingIncludes.h"
 #include "TCPTransport.h"
 #include "UDPClient.h"
+#include "loggingIncludes.h"
 #include "protocolHandler.h"
 
 KPIRoverECU::KPIRoverECU(ProtocolHanlder *_protocolHandler, TCPTransport *_tcpTransport, UDPClient *_udpClient,
@@ -90,7 +90,7 @@ void KPIRoverECU::IMUThreadFucntion(IMUController *workClass) {
                         send_val.push_back(bytes[j]);
                     }
                 }
-                
+
                 LOG_DEBUG << "use UDP client to send message";
                 udp_client_->Send(send_val);
             }

src/kpi_rover_ecu/src/TCPTransport.cpp

@@ -45,7 +45,6 @@ int TCPTransport::Init() {
     source_address_[0] = '\0';
 
     sockfd_ = socket(AF_INET, SOCK_STREAM, 0);
-    
 
     if (sockfd_ < 0) {
         LOG_ERROR << "Socket()";
@@ -92,7 +91,8 @@ void TCPTransport::Start() {
             if (client_sockfd_ >= 0) {
                 inet_ntop(AF_INET, &client_addr.sin_addr, source_address_, INET_ADDRSTRLEN);
                 source_port_ = static_cast<int>(ntohs(client_addr.sin_port));
-                LOG_INFO << "Client connected, source address: " << std::string(source_address_) << " source port: " << source_port_;
+                LOG_INFO << "Client connected, source address: " << std::string(source_address_)
+                         << " source port: " << source_port_;
                 while (true) {  // Use true instead of 1
                     std::uint8_t buffer[kBufferSize];
                     const ssize_t kBytesReceived = recv(client_sockfd_, buffer, sizeof(buffer), 0);

src/kpi_rover_ecu/src/main.cpp

@@ -1,9 +1,9 @@
 #include <getopt.h>
+#include <glog/logging.h>
 #include <ifaddrs.h>
 #include <netdb.h>
 #include <netinet/in.h>
 #include <sys/socket.h>
-#include <glog/logging.h>
 
 #include <chrono>
 #include <csignal>

src/kpi_rover_ecu/src/motor.cpp

@@ -9,8 +9,8 @@
 #include <iostream>
 #include <ratio>
 
-#include "loggingIncludes.h"
 #include "PIDRegulator.h"
+#include "loggingIncludes.h"
 
 Motor::Motor(int assigned_number, bool is_inverted, std::array<float, 3> _coeficients)
     : motorNumber_(assigned_number), inverted_(is_inverted), currentDutyCycle_(0), setpointRpm_(0), actualRpm_(0) {
@@ -91,7 +91,8 @@ int Motor::GetEncoderCounter() {
     const float kTimeDt = GetTimeSegment();
     actualRpm_ = static_cast<int>(GetActualRpm(pid_encoder_ticks, kTimeDt));
     const int kError = setpointRpm_ - actualRpm_;
-    LOG_INFO << "set point " << setpointRpm_ << " current point " << actualRpm_ << " error " << kError << " for motor" << motorNumber_;
+    LOG_INFO << "set point " << setpointRpm_ << " current point " << actualRpm_ << " error " << kError << " for motor"
+             << motorNumber_;
 
     LOG_DEBUG << "Run PID regulator";
     const int kPidOutput = pidRegulator_.Run(static_cast<float>(kError), kTimeDt);

src/kpi_rover_ecu/src/protocolHandler.cpp

@@ -50,7 +50,7 @@ vector<uint8_t> ProtocolHanlder::HandleGetApiVersion(const vector<uint8_t>& mess
     vector<uint8_t> ret_val;
 
     ret_val.push_back(ProtocolHanlder::kIdGetApiVersion);
-    
+
     const uint8_t kResponse = kApiVersion;
     ret_val.push_back(kResponse);