Relátorios com muitas tabelas - Ruby on Br - A comunidade Ruby brasileira

<%elsif mes == 3 >

<%elsif mes == 4 %>

<%elsif mes == 5 %>

<%elsif mes == 6 %>

<%end%>

Pode ser substituido por um helper pra ficar assim:



  <= nome_mes(mes) %>

E no seu helper, vc coloca esse monte de if’s =). Essa estratégia pode ser aplicada a outros pontos do seu código, facilitando o entendimento (pelo menos eu acho que fica mais legível).

Formatações de dados, também ficam melhores “escondidas” em um helper, como esse código:



estagio.est_dti.strftime(“%d/%m/%y”)

Helper:



def formata_data data

  data.strftime(“%d/%m/%y”)

end

View:



 <%= formata_data(estagio.est_dti) >

Outra coisa que pode ser melhorada, é que vc parece não estar usando os relacionamentos entre models. Por exemplo, nesse trecho:



<%@estagio = Estagio.find(:all, :conditions => [“aluno_id=?”, aluno.id])%>

Fica melhor se vc montar o relacionamento corretamente e fazer algo como:



<= @estagio = aluno.estagio %>

Com isso já melhora bastante a legibilidade, pelo menos na minha opinião =).

Março	Abril	Maio	Junho
185 semanas atrás Adriano 117 posts	Bom dia, eu fiz um código que chama um relátorio de alunos, onde se pode escolher se quer um mês ou todos, de qual faculdade pertence o aluno e se quer so alunos desligados ou todos. Enfim se puderem dar uma olhada ^^ (eu entendo o tamanho do código e o tanto que é chato, mas não consegui deixar ele mais seco de forma alguma) <%= stylesheet_link_tag "default" %> <h3>Relatórios de Bolsistas Ordenados por Faculdade</h3> <%busca = params[:busca] %> <% fac = Faculdade.find(busca) %> <% for fac in fac %> <h3> Faculdade <%= fac.fac_sigla %></h3> <%end%> <font size='smaller'>( ) = Justificativa LTS (Licença por Tratamento de Saúde ) </font> <%mes = params[:mes].to_s %> <% mes = mes.to_i %> <table id='tabela2'> <tr> <th colspan='2'> Nome </th> <th> CPF </th> <th> Faculdade do Aluno </th> <th> Curso </th> <th> Inicio Estagio </th> <th colspan='2'> Local Do Estagio </th> <% if mes == 100 %> <th> Março </th> <th> Abril </th> <th> Maio </th> <th> Junho </th> </tr> <%elsif mes == 3 %> <th> Março </th> <%elsif mes == 4 %> <th> Abril </th> <%elsif mes == 5 %> <th> Maio </th> <%elsif mes == 6 %> <th> Junho </th> <%end%> <% al = params[:aluno].to_s %> <% if al == "Todos" %> <% cont = 0%> <% n = 0%> <% for aluno in @aluno%> <%@estagio = Estagio.find(:all, :conditions => ["aluno_id=?", aluno.id])%> <% if mes == 100%> <%@frequencia = Frequencia.find(:all, :conditions => ["aluno_id=?", aluno.id], :order => 'mes' )%> <%else %> <%@frequencia = Frequencia.find(:all, :conditions => ["aluno_id=? and mes=?", aluno.id,mes], :order => 'mes' )%> <% end %> <%@desligamento = Desligamento.find(:all, :conditions => ["aluno_id=?", aluno.id])%> <% n = n + 1 %> <tr> <td><%= n %> </td> <td><%= aluno.nome %></td> <td><%= aluno.cpf %> </td> <td><%= aluno.faculdade.fac_sigla %></td> <td><%= aluno.curso.nome_curso %></td> <%if @desligamento.size > 0%> <% for desligamento in @desligamento%> <td>DESLIGADO </td> <td><%= desligamento.justificativa%><%= desligamento.data_des.strftime("%d/%m/%y")%> </td> <td> </td> <% end %> <%else%> <%if @estagio%> <%for estagio in @estagio%> <td><%= estagio.est_dti.strftime("%d/%m/%y") %></td> <td><%= estagio.entidade.ent_sig + " " + estagio.entidade.dre.dre %></td> <%end%> <td> </td> <%end%> <%end%> <% if @frequencia%> <% if mes == 100 %> <% add = 0 %> <% else %> <% add = 1 %> <% end %> <% for frequencia in @frequencia%> <% if frequencia.faltas_lts %> <% lts = frequencia.faltas_lts %> <% else %> <% lts = 0 %> <% end %> <%if frequencia.mes == 3%> <td><%= h frequencia.frequencia%> <% if lts != 0 %> (<%=h lts %>) </td> <%else %> </td> <%end%> <% add = add +1 %> <%elsif frequencia.mes == 4%> <% if add == 0 %> <td> - </td> <%end%> <% add = add +1 %> <td><%= h frequencia.frequencia%> <% if lts != 0 %> (<%=h lts %>) </td> <%else %> </td> <%end%> <%elsif frequencia.mes == 5%> <% if add == 0 %> <td> - </td><td> - </td> <%end%> <% add = add +1 %> <td><%= h frequencia.frequencia%> <% if lts != 0 %> (<%=h lts %>) </td> <%else %> </td> <%end%> <%elsif frequencia.mes == 6%> <% if add == 0 %> <td> - </td><td> - </td> <%end%> <td><%= h frequencia.frequencia%> <% if lts != 0 %> (<%=h lts %>) </td> <%else %> </td> <%end%> <%else%> <td> </td> <td> </td> <%end%> <%end%> <%end%> </tr> <%end%> <%else %> <% cont = 0%> <% n = 0%> <% for aluno in @aluno%> <%@estagio = Estagio.find(:all, :conditions => ["aluno_id=?", aluno.id])%> <% if mes == 100%> <%@frequencia = Frequencia.find(:all, :conditions => ["aluno_id=?", aluno.id], :order => 'mes' )%> <%else %> <%@frequencia = Frequencia.find(:all, :conditions => ["aluno_id=? and mes=?", aluno.id,mes], :order => 'mes' )%> <% end %> <%@desligamento = Desligamento.find(:all, :conditions => ["aluno_id=?", aluno.id])%> <% if @frequencia.size > 0 && @desligamento.size == 0 %> <% n = n + 1 %> <tr> <td><%= n %> </td> <td><%= aluno.nome %></td> <td><%= aluno.cpf %> </td> <td><%= aluno.faculdade.fac_sigla %></td> <td><%= aluno.curso.nome_curso %></td> <%if @desligamento.size > 0%> <% for desligamento in @desligamento%> <td>DESLIGADO </td> <td><%= desligamento.justificativa%><%= desligamento.data_des.strftime("%d/%m/%y")%> </td> <td> </td> <% end %> <%else%> <%if @estagio%> <%for estagio in @estagio%> <td><%= estagio.est_dti.strftime("%d/%m/%y") %></td> <td><%= estagio.entidade.ent_sig + " " + estagio.entidade.dre.dre %></td> <%end%> <td> </td> <%end%> <%end%> <% if @frequencia%> <% if mes == 100 %> <% add = 0 %> <% else %> <% add = 1 %> <% end %> <% for frequencia in @frequencia%> <% if frequencia.faltas_lts %> <% lts = frequencia.faltas_lts %> <% else %> <% lts = 0 %> <% end %> <%if frequencia.mes == 3%> <td><%= h frequencia.frequencia%> <% if lts != 0 %> (<%=h lts %>) </td> <%else %> </td> <%end%> <% add = add +1 %> <%elsif frequencia.mes == 4%> <% if add == 0 %> <td> - </td> <%end%> <% add = add +1 %> <td><%= h frequencia.frequencia%> <% if lts != 0 %> (<%=h lts %>) </td> <%else %> </td> <%end%> <%elsif frequencia.mes == 5%> <% if add == 0 %> <td> - </td><td> - </td> <%end%> <% add = add +1 %> <td><%= h frequencia.frequencia%> <% if lts != 0 %> (<%=h lts %>) </td> <%else %> </td> <%end%> <%elsif frequencia.mes == 6%> <% if add == 0 %> <td> - </td><td> - </td> <%end%> <td><%= h frequencia.frequencia%> <% if lts != 0 %> (<%=h lts %>) </td> <%else %> </td> <%end%> <%else%> <td> </td> <td> </td> <%end%> <%end%> <%end%> </tr> <% end%> <% end%> <% end%>

185 semanas atrás Fernando Luizão 1863 posts	Isso: <%busca = params[:busca] > < fac = Faculdade.find(busca) %> Ficaria melhor no controller. Uma view ler um parâmetro é bem estranho. Uma coisa importante para limpar sua view, é usar mais helpers, por exemplo, esse trecho: <% if mes == 100 %>	Março	Abril	Maio	Junho

185 semanas atrás Adriano 117 posts	gostei das ideias vou tentar aplicar agora mesmo =)

185 semanas atrás Shairon Toledo 1428 posts	põe partial na parada.

185 semanas atrás Mereghost 1476 posts	Esses elseifs aí podem virar um case. =)