Martin Edgar Albarracin

Hacker Rank for challenge Rappi

# install dependencies
npm install

# serve with hot reload at localhost:8080
npm run dev

# build for production with minification
npm run build

# build for production and view the bundle analyzer report
npm run build --report

# run unit tests
npm run unit

# run all tests
npm test

For a detailed explanation on how things work, check out the guide and docs for vue-loader.

public function post_confirm(){
    $id = Input::get('service_id');
    $servicio = Service::find($id);
    //dd($Servicio);
    if ($servicio != NULL){
        if  ($servicio->status_id == '6'){
            return Response::json(array('error' => '2'));
        }
        if ($servicio->driver_id == NULL && $servicio->status_id == '1'){
            $servicio = Service::update($id, array(
                'driver_id' => Input::get('driver_id'),
                'status_id' => '2'
                //Up Carro
                //,'pwd' => md5(Input::get('pwd'))
            ));
            Driver::update(Input::get('driver_id'), array(
                'available' => '0'
            ));
            $driverTmp = Driver::find(Input::get('driver_id'));
            Service::update($id, array(
                'car_id'=>$driverTmp->car_id
                //Up Carro
                //,'pwd' => md5(Input::get('pwd'))
            ));
            //Notificar a usuario!!
            $pushMessage = 'Tu servicio ha sido confirmado!';
            /*$servicio = Service::find($id);
            $push = Push::make();
            if ($servicio->user->type=='1'){//iPhone
                $pushAns = $push->ios($servicio->user->uuid, $pushMessage);
            }else{
                $pushAns = $push->android($servicio->user->uuid, $pushMessage);
            }*/
            $servicio = Service::find($id);
            $push = Push::make();
            if ($servicio->user->uuid == ''){
                return Response::json(array('error'=>'0'));
            }
            if($servicio->user->type == '1'){//iPhone
                $result = $push->ios($servicio->user->uuid, $pushMessage, 1, 'honk.wav',
                                        'Open', array('service_id'=>$servicio->id));
            } else{
                $result = $push->android2($servicio->user->uuid, $pushMessage, 1, 'default',
                                        'Open', array('service_id'=>$servicio->id));
            }
            return Response::json(array('error'=>'0'));
        }else{
            return Response::json(array('error'=>'1'));
        }
    }else{
        return Response::json(array('error'=>'3'));
    }
}

const ERROR = 'Error';
const ERROR_INVALID_REQUEST =3
const ERROR_PARAMETERS_CONFLICT = 2
const ERROR_PARAMETERS_CONFLICT = 2
const ERROR_DRIVER = 1
const ERROR_UUI_EMPTY = 0


public function post_confirm(){
$service_id = Input::get('service_id');
$service = Service::find($service_id);
if ($service != NULL) {
    if ($service->status_id == '6') {
        return Response::json(array(ERROR => ERROR_PARAMETERS_CONFLICT));
    }
    if ($service->driver_id == NULL && $service->status_id == '1')
    {
            $driver_id = Input::get('driver_id');
            $driverTmp = Driver::find($driver_id);
            $servicio = Service::update($service_id, array(
                'driver_id' => $driver_id,
                'status_id' => '2',
                car_id' => $driverTmp->car_id
            ));
            Driver::update($driver_id, array(
            'available' => '0'
            ));	
            //Notificar a usuario!!
            $pushMessage = 'Tu servicio ha sido confirmado!';
            $push = Push::make();
            if ($servicio->user->uuid == ''){
                return Response::json(array(ERROR=>ERROR_UUI_EMPTY));
            }
            //Notification platform IOS
            if ($service->user->type == '1') {
                $push->ios($servicio->user->uuid, $pushMessage, 1, 'honk.wav', 'Open', array('service_id' => $servicio->id));
            }
            //Notification platform ANDROID
            else{
                $push->android2($servicio->user->uuid, $pushMessage, 1, 'default', 'Open', array('service_id' => $servicio>id));
            }
        }
    }
    else{
            return Response::json(array(ERROR=> ERROR_DRIVER));
        }
}
else{
        return Response::json(array(ERROR=> ERROR_INVALID_REQUEST));
    }
}

• Se recomienda que los nombres de las variabled sean dicientes, es decir que reflejen el contenido, dado que en el refactor de codigo se encuentran variables que son nombradas como "id", que puede hacer referencia a cualquier id, y no beneficia la mantenibilidad del software

• Debe existir un standard de codigo que brinde las herramientas de nombramiento de variables, ya que en el codigo se encuentran variables en ingles, variables en español

• Los comentarios en codigo son muy utilizados en cuanto aporte a la mantenibilidad del software y serviran de ayuda a futuros desarrolladores para comprender la funcionalidad del codigo, a la vez tambien pueden servir de ayuda para el desarrollador al establecer "ToDo" que son comentarios de codigo de funcionalidades esperadas que no se han terminado, sin embargo en el codigo presentado se evidencian partes de codigo que estan comentadas, que no aportan un valor significativo a los desarrolladores

• En ocasiones es necesario declarar constantes con valores string que permitan utilizarlas en las diferentes partes del codigo, por ejemplo en varias partes del codigo se utiliza el string 'Error', lo que se sugiere es que como se trata de un concepto de error tendra que ser generico por tal motivo se decide declarar una constante con dicho valor, con el fin de que si se desea cambiar solo se deba hacer la modificación a la constante declarada

• Se evidencia que en el codigo se devuelve un json "key" "valor", sin embargo por temas de mantenibilidad se recomienda que dichos valores se cambien por constantes que sean lo suficientemente dicientes del error ocurrido, por ejemplo el error 3 representa que no se recibieron parametros en la petición por lo que se puede reemplazar por una constante llamada ERROR_INVALID_REQUEST que tenga el valor de 3

• Se evidencia que en codigo a analizar se esta llamando dos veces al metodo update, en el primer caso envia dos parametros $driver_id y $status_id, y en el segundo caso envia car_id, por lo que se sugiere unificar los metodos mencionados en el que solo se llame una unica vez con los tres parametros

• Se evidencia que las respuestas de los metodos de las notificaciones no se utilizan, lo cual hace guardar en memeoria varibles innecesarias, por lo cual la sugerencia es llamar los metodos sin tener un variable almacenando el retorno

El principio de responsabilidad unica establece que cada clase debe usarse unica y solo unicamente para un fin, es decir cada clase debe responder solo por una funcionalidad de la aplicación, los metodos que estan dicha clase serviran para soportar la funcionalidad, por ejemplo suponiendo que se tiene un software que tiene varias formas de autenticar usuarios(redes sociales, formulario inicio de sesion, sesion invitado), entonces debe existir una clase que se encargue de manejar la autenticación de usuarios, dentro de dicha clase debera existir un metodo que se encargue de validar la autenticacion de usuarios que ingresan con su red social

• Pienso que una de las caracterisicas mas importantes en el desarrollo del software deberia ser la mantenibilidad, que es la capacidad de un software a modificar parcial o total su funcionalidad sin demandar mucho tiempo.    - De la mano de la mantenibilidad pueden ir atados muchas caracteristicas que pueden hacer que nuestro softwae sea de alta calidad, como el tema de una documentación de la arquitectura y documentacion de codigo que pueda apoyar a los futuros desarrolladores a dismunuir el tiempo de entendimiento del software
• La segunda caracteristica que deberia tener la mayoria de aplicativos con diseño por componentes que beneficie la reutiliacion de codigo, ue comunmente se llamaria DRY, dont repeat yourself , no te repitas
• Otra de las practicas que pienso se deberia hacer en la implementación de un software es la captura y tratamiento de errores no controlados , que va de la mano con sistemas de auditoría, que permita registrar cada una de las acciones de los usuarios con el fin de llevar una trazabilidad y lograr detectar errores en el sistema
• No es recomendado que alguno de los metodos que se tenga reciba mas de 3 parametros, para ello se deberia crear una clase con el fin de que el metodo reciba la clase, y analizar bien la funcionalidad porque en ciertos casos se puede dividir en metodos con una unica responsabilidad los cuales cada uno retorne su valor a un metodo grande que sea el encargado de toamr las decisiones en base a la respuesta de los metodos de responsabilidad unica.