oka, lo tendré en cuenta, en lo personal me gusta salir apenas algo sale
mal :)


On Mon, Apr 28, 2014 at 11:03 PM, haden9 <notifications@github.com> wrote:

> In src/main/java/com/gangfive/sima/services/CaloriaService.java:
>
> > @@ -182,4 +192,59 @@ public OperationResult crearNuevas(List<Caloria> calorias, Paciente paciente) {
> >          bitacoraService.createBitacoraLog(Bitacora.getClassName(this), Bitacora.getMethodName(), oResult.message());
> >          return oResult;
> >      }
> > +
> > +    @Override
> > +    public Map<String, Integer> getBalanceDiario(int idPaciente, String fecha) {
> > +        Map<String, Integer> balanceDeCalorias;
> > +        Date fechaLimite = new Date();
> > +
> > +        if (Utils.isValidString(fecha)) {
> > +            fechaLimite = DateUtils.convertStringToDate(fecha);
> > +        }
> > +
> > +        Paciente paciente = pacienteService.findOne(idPaciente);
> > +        CalendarioCita cita = citaService.getUltimaCitaCompletaAntesDeFecha(paciente, fechaLimite);
> > +
> > +        if (paciente == null || cita == null || cita.getCalorias() == null || cita.getCalorias() < 0) {
>
> igual leo que un comentario arriba
>
> "if (balance != null || balance.size() > 0) para poner la condición que
> mas acciones tiene de primero. es solo un consejo, pero no es necesario
> cambiarlo"
> y ojala tambien que se pueda remover el return y dejarlo como:
> balanceDeCalorias = null;
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/GangOfFive/falafel/pull/168/files#r12079693>
> .
>



-- 
Leo Picado, UI Guy.
www.leonardopicado.com  |  me@leonardopicado.com