From 0f8bffdc16f2318011e90d0a523352f592b84696 Mon Sep 17 00:00:00 2001 From: Lucas dos Santos Abreu Date: Fri, 28 Jun 2019 13:41:31 -0300 Subject: [PATCH] =?UTF-8?q?Erro=20ao=20Compilar=20no=20Linux=20e=20Problem?= =?UTF-8?q?a=20ao=20Logar=20Erros,=20por=20falta=20uso=20de=20=C3=8Dndice?= =?UTF-8?q?=20n=C3=A3o=20existente=20(#9)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * do not use String.Format * be linux friendly --- .../CurriculumVitaeProcessorController.cs | 20 +- ...romRestServiceCurriculumVitaeController.cs | 31 +-- ...FromWebServiceCurriculumVitaeController.cs | 18 +- ...portCurriculumVitaeFromFolderController.cs | 11 +- LattesExtractor/DAO/ProfessorDAOService.cs | 198 +++++++++--------- LattesExtractor/LattesExtractor.csproj | 2 +- LattesExtractor/LattesModule.cs | 10 +- .../DownloadCurriculumVitaeWebService.cs | 14 +- 8 files changed, 132 insertions(+), 172 deletions(-) diff --git a/LattesExtractor/Controller/CurriculumVitaeProcessorController.cs b/LattesExtractor/Controller/CurriculumVitaeProcessorController.cs index d58d266..6e3d530 100644 --- a/LattesExtractor/Controller/CurriculumVitaeProcessorController.cs +++ b/LattesExtractor/Controller/CurriculumVitaeProcessorController.cs @@ -71,11 +71,7 @@ private void ProcessCurriculumVitae(CurriculoEntry curriculoEntry, ManualResetEv curriculoEntry.NomeProfessor = curriculumVitae.DADOSGERAIS.NOMECOMPLETO; var professorDAOService = new ProfessorDAOService(new LattesDatabase()); - Logger.Debug(String.Format( - "Iniciando processamento currículo {0} do Professor {1}...", - curriculoEntry.NumeroCurriculo, - curriculumVitae.DADOSGERAIS.NOMECOMPLETO - )); + Logger.Debug($"Iniciando processamento currículo {curriculoEntry.NumeroCurriculo} do Professor {curriculumVitae.DADOSGERAIS.NOMECOMPLETO}..."); if (professorDAOService.ProcessCurriculumVitaeXML(curriculumVitae, curriculoEntry)) { @@ -86,23 +82,13 @@ private void ProcessCurriculumVitae(CurriculoEntry curriculoEntry, ManualResetEv } catch (Exception ex) { - Logger.Error(String.Format( - "Erro durante a leitura do XML {0}: {1}\n{2}", - curriculoEntry.NumeroCurriculo, - ex.Message, - ex.StackTrace - )); + Logger.Error($"Erro durante a leitura do XML {curriculoEntry.NumeroCurriculo}: {ex.Message}\n{ex.StackTrace}"); int sequencia = 1; while (ex.InnerException != null) { ex = ex.InnerException; - Logger.Error(String.Format( - "Excessão Interna [{0}]: {1}\n{2}", - sequencia++, - ex.Message, - ex.StackTrace - )); + Logger.Error($"Excessão Interna [{sequencia++}]: {ex.Message}\n{ex.StackTrace}"); } } finally diff --git a/LattesExtractor/Controller/DownloadFromRestServiceCurriculumVitaeController.cs b/LattesExtractor/Controller/DownloadFromRestServiceCurriculumVitaeController.cs index d5944ca..9edadd4 100644 --- a/LattesExtractor/Controller/DownloadFromRestServiceCurriculumVitaeController.cs +++ b/LattesExtractor/Controller/DownloadFromRestServiceCurriculumVitaeController.cs @@ -72,7 +72,7 @@ public void DownloadUpdatedCurriculums(ManualResetEvent doneEvent) { doneEvent.Set(); } - Logger.Info(String.Format("Download terminou")); + Logger.Info("Download terminou"); } } @@ -100,10 +100,7 @@ private void DownloadCurriculumVitae(RetryMessage retryMessage, ManualResetEvent if (response.CodRhCript == null || response.CodRhCript.Trim().Length == 0) { - Logger.Error(String.Format( - "Não foi possível baixar o currículo de número {0}", - curriculumVitae.NumeroCurriculo - )); + Logger.Error($"Não foi possível baixar o currículo de número {curriculumVitae.NumeroCurriculo}"); return; } @@ -111,7 +108,7 @@ private void DownloadCurriculumVitae(RetryMessage retryMessage, ManualResetEvent if (NeedsToBeUpdated(curriculumVitae, response) == false) { - Logger.Info(String.Format("Currículo {0} - {1} já esta atualizado.", curriculumVitae.NumeroCurriculo, curriculumVitae.NomeProfessor)); + Logger.Info($"Currículo {curriculumVitae.NumeroCurriculo} - {curriculumVitae.NomeProfessor} já esta atualizado."); return; } @@ -119,21 +116,17 @@ private void DownloadCurriculumVitae(RetryMessage retryMessage, ManualResetEvent if (curriculumVitae.NomeProfessor == null || curriculumVitae.NomeProfessor.Trim().Length == 0) { - Logger.Info(String.Format("Curriculo {0} baixado", curriculumVitae.NumeroCurriculo)); + Logger.Info($"Curriculo {curriculumVitae.NumeroCurriculo} baixado"); return; } - Logger.Info(String.Format("Curriculo {0} - {1} baixado", curriculumVitae.NumeroCurriculo, curriculumVitae.NomeProfessor)); + Logger.Info($"Curriculo {curriculumVitae.NumeroCurriculo} - {curriculumVitae.NomeProfessor} baixado"); } catch (WebException exception) { - Logger.Error(String.Format( - "Erro ao realizar requisão do Currículo {2} (Tentativas Sobrando {3}): {0}\n{1}", - exception.Message, - exception.StackTrace, - curriculumVitae.NumeroCurriculo, - retryMessage.PendingRetries - )); + Logger.Error( + $"Erro ao realizar requisão do Currículo {curriculumVitae.NumeroCurriculo} (Tentativas Sobrando {retryMessage.PendingRetries}): {exception.Message}\n{exception.StackTrace}" + ); retryMessage.PendingRetries--; if (retryMessage.PendingRetries > 0) { @@ -156,11 +149,7 @@ private void DownloadCurriculumVitae(RetryMessage retryMessage, ManualResetEvent private void DownloadXml(CurriculoEntry curriculumVitae, MetadataResponse response, WebClient wc) { var link = String.Format(_urlDownloadXml, response.CodRhCript); - Logger.Debug(String.Format( - "Currículo {0} marcado para download ({1})...", - curriculumVitae.NumeroCurriculo, - link - )); + Logger.Debug($"Currículo {curriculumVitae.NumeroCurriculo} marcado para download ({link})..."); var stream = wc.OpenRead(link); if (File.Exists(_lattesModule.GetCurriculumVitaeFileName(curriculumVitae.NumeroCurriculo))) @@ -197,7 +186,7 @@ private bool NeedsToBeUpdated(CurriculoEntry curriculumVitae, MetadataResponse r } var dataAtualizacaoLattes = DateTime.ParseExact( - String.Format("{0} {1}", response.Document.DataAtualizacao, horaAtualizacao), + $"{response.Document.DataAtualizacao} {horaAtualizacao}", "ddMMyyyy %Hmmss", null ); diff --git a/LattesExtractor/Controller/DownloadFromWebServiceCurriculumVitaeController.cs b/LattesExtractor/Controller/DownloadFromWebServiceCurriculumVitaeController.cs index 0dc439f..47dec93 100644 --- a/LattesExtractor/Controller/DownloadFromWebServiceCurriculumVitaeController.cs +++ b/LattesExtractor/Controller/DownloadFromWebServiceCurriculumVitaeController.cs @@ -43,7 +43,7 @@ public void DownloadUpdatedCurriculums(ManualResetEvent doneEvent) finally { doneEvent.Set(); - Logger.Info(String.Format("Download terminou")); + Logger.Info("Download terminou"); } } @@ -53,10 +53,7 @@ private void DownloadCurriculumVitae(CurriculoEntry curriculumVitae) { if (curriculumVitae.NumeroCurriculo == null || curriculumVitae.NumeroCurriculo.Trim().Length == 0) { - Logger.Error(String.Format( - "O número do curríuculo Lattes do professor {0} não foi encontrado", - curriculumVitae.NomeProfessor - )); + Logger.Error($"O número do curríuculo Lattes do professor {curriculumVitae.NomeProfessor} não foi encontrado"); return; } @@ -88,20 +85,15 @@ private void DownloadCurriculumVitae(CurriculoEntry curriculumVitae) if (curriculumVitae.NomeProfessor == null || curriculumVitae.NomeProfessor.Trim().Length == 0) { - Logger.Info(String.Format("Curriculo {0} baixado", curriculumVitae.NumeroCurriculo)); + Logger.Info($"Curriculo {curriculumVitae.NumeroCurriculo} baixado"); return; } - Logger.Info(String.Format("Curriculo {0} - {1} baixado", curriculumVitae.NumeroCurriculo, curriculumVitae.NomeProfessor)); + Logger.Info($"Curriculo {curriculumVitae.NumeroCurriculo} - {curriculumVitae.NomeProfessor} baixado"); } catch (Exception exception) { - Logger.Error(String.Format( - "Erro ao buscar o currículo {0}, mensagem: {1}\n{2}", - curriculumVitae.NumeroCurriculo, - exception.Message, - exception.StackTrace - )); + Logger.Error($"Erro ao buscar o currículo {curriculumVitae.NumeroCurriculo}, mensagem: {exception.Message}\n{exception.StackTrace}"); } } } diff --git a/LattesExtractor/Controller/ImportCurriculumVitaeFromFolderController.cs b/LattesExtractor/Controller/ImportCurriculumVitaeFromFolderController.cs index db85576..4ca7bea 100644 --- a/LattesExtractor/Controller/ImportCurriculumVitaeFromFolderController.cs +++ b/LattesExtractor/Controller/ImportCurriculumVitaeFromFolderController.cs @@ -38,13 +38,13 @@ public void LoadCurriculums(ManualResetEvent doneEvent) { if (!Directory.Exists(_importFolder)) { - Logger.Info(String.Format("Pasta de trabalho não foi encontrado ({0})", _importFolder)); + Logger.Info($"Pasta de trabalho não foi encontrado ({_importFolder})"); return; } if (Directory.GetFiles(_importFolder).Length == 0) { - throw new Exception(String.Format("Não foram encontrados currículos na pasta {0} !", _importFolder)); + throw new Exception($"Não foram encontrados currículos na pasta {_importFolder} !"); } var unzipDoneEvent = new ManualResetEvent(false); @@ -100,12 +100,7 @@ private void UnzipAndCopy(ManualResetEvent doneEvent, string filename, Curriculo } catch (ZipException exception) { - Logger.Error(String.Format( - "Erro ao importar currículo {2}: {0}\n{1}", - exception.Message, - exception.StackTrace, - curriculumVitae.NumeroCurriculo - )); + Logger.Error($"Erro ao importar currículo {curriculumVitae.NumeroCurriculo}: {exception.Message}\n{exception.StackTrace}"); } finally { diff --git a/LattesExtractor/DAO/ProfessorDAOService.cs b/LattesExtractor/DAO/ProfessorDAOService.cs index 063a25a..ff0c486 100644 --- a/LattesExtractor/DAO/ProfessorDAOService.cs +++ b/LattesExtractor/DAO/ProfessorDAOService.cs @@ -171,7 +171,7 @@ public bool ProcessCurriculumVitaeXML(CurriculoVitaeXml cvXml, CurriculoEntry cu { if (cvXml.NUMEROIDENTIFICADOR != null && !cvXml.NUMEROIDENTIFICADOR.Equals("") && cvXml.NUMEROIDENTIFICADOR != curriculumVitae.NumeroCurriculo) { - Logger.Error(String.Format("Curriculo {0} solicitado não corresponde ao recebido {1} !", curriculumVitae.NumeroCurriculo, cvXml.NUMEROIDENTIFICADOR)); + Logger.Error($"Curriculo {curriculumVitae.NumeroCurriculo} solicitado não corresponde ao recebido {cvXml.NUMEROIDENTIFICADOR} !"); return false; } @@ -183,7 +183,7 @@ public bool ProcessCurriculumVitaeXML(CurriculoVitaeXml cvXml, CurriculoEntry cu if (curriculumVitae.DataUltimaAtualizacao == null) { curriculumVitae.DataUltimaAtualizacao = DateTime.ParseExact( - String.Format("{0} {1}", cvXml.DATAATUALIZACAO, cvXml.HORAATUALIZACAO), + $"{cvXml.DATAATUALIZACAO} {cvXml.HORAATUALIZACAO}", Utils.ParseDateFormat(cvXml.FORMATODATAATUALIZACAO) + " %Hmmss", null ); @@ -193,11 +193,7 @@ public bool ProcessCurriculumVitaeXML(CurriculoVitaeXml cvXml, CurriculoEntry cu if (professor != null && professor.DataUltimaAtualizacaoCurriculo >= curriculumVitae.DataUltimaAtualizacao) { - Logger.Info(String.Format( - "Currículo {0} do Professor {1} já esta na última versão.", - curriculumVitae.NumeroCurriculo, - curriculumVitae.NomeProfessor - )); + Logger.Info($"Currículo {curriculumVitae.NumeroCurriculo} do Professor {curriculumVitae.NomeProfessor} já esta na última versão."); return true; } @@ -216,7 +212,7 @@ public bool ProcessCurriculumVitaeXML(CurriculoVitaeXml cvXml, CurriculoEntry cu // dados do curriculo professor.NumeroCurriculo = curriculumVitae.NumeroCurriculo; - professor.LinkParaCurriculo = String.Format("http://lattes.cnpq.br/{0}", professor.NumeroCurriculo); + professor.LinkParaCurriculo = $"http://lattes.cnpq.br/{professor.NumeroCurriculo}"; professor.DataUltimaAtualizacaoCurriculo = (DateTime)curriculumVitae.DataUltimaAtualizacao; @@ -939,15 +935,15 @@ private void ComplementarBancaJulgadora(BancaJulgadora banca, string tituloIngle switch (tipo) { case "ACADEMICO": - natureza = String.Format("{0} (Acadêmico)", natureza); + natureza = $"{natureza} (Acadêmico)"; break; case "PROFISSIONALIZANTE": - natureza = String.Format("{0} (Profissionalizante)", natureza); + natureza = $"{natureza} (Profissionalizante)"; break; case "NAO_INFORMADO": break; default: - natureza = String.Format("{0} ({1})", natureza, tipo); + natureza = $"{natureza} ({tipo})"; break; } } @@ -1304,15 +1300,15 @@ private void ComplementarBancaDeTrabalho(BancaDeTrabalho banca, string tituloIng switch (tipo) { case "ACADEMICO": - natureza = String.Format("{0} (Acadêmico)", natureza); + natureza = $"{natureza} (Acadêmico)"; break; case "PROFISSIONALIZANTE": - natureza = String.Format("{0} (Profissionalizante)", natureza); + natureza = $"{natureza} (Profissionalizante)"; break; case "NAO_INFORMADO": break; default: - natureza = String.Format("{0} ({1})", natureza, tipo); + natureza = $"{natureza} ({tipo})"; break; } } @@ -1413,9 +1409,9 @@ private void ProcessarOrientacoesESupervisoes(Professor professor, CurriculoVita { lock (logLocker) { - Logger.Error(String.Format("A Orientação ou Supervisão Concluída de nome \"{0}\" {1} - {2} não possui Ano de Ocorrência, somente serão considerados Orientações/Supervições com Ano de Ocorrência", - mestrado.DADOSBASICOSDEORIENTACOESCONCLUIDASPARAMESTRADO.TITULO, - professor.NumeroCurriculo, professor.NomeProfessor)); + Logger.Error( + $"A Orientação ou Supervisão Concluída de nome \"{mestrado.DADOSBASICOSDEORIENTACOESCONCLUIDASPARAMESTRADO.TITULO}\" {professor.NumeroCurriculo} - {professor.NomeProfessor} não possui Ano de Ocorrência, somente serão considerados Orientações/Supervições com Ano de Ocorrência" + ); } } else @@ -1497,9 +1493,9 @@ private void ProcessarOrientacoesESupervisoes(Professor professor, CurriculoVita { lock (logLocker) { - Logger.Error(String.Format("A Orientação ou Supervisão Concluída de nome \"{0}\" {1} - {2} não possui Ano de Ocorrência, somente serão considerados Orientações/Supervições com Ano de Ocorrência", - dout.DADOSBASICOSDEORIENTACOESCONCLUIDASPARADOUTORADO.TITULO, - professor.NumeroCurriculo, professor.NomeProfessor)); + Logger.Error( + $"A Orientação ou Supervisão Concluída de nome \"{dout.DADOSBASICOSDEORIENTACOESCONCLUIDASPARADOUTORADO.TITULO}\" {professor.NumeroCurriculo} - {professor.NomeProfessor} não possui Ano de Ocorrência, somente serão considerados Orientações/Supervições com Ano de Ocorrência" + ); } } else @@ -1571,9 +1567,9 @@ private void ProcessarOrientacoesESupervisoes(Professor professor, CurriculoVita { lock (logLocker) { - Logger.Error(String.Format("A Orientação ou Supervisão Concluída de nome \"{0}\" {1} - {2} não possui Ano de Ocorrência, somente serão considerados Orientações/Supervições com Ano de Ocorrência", - dout.DADOSBASICOSDEORIENTACOESCONCLUIDASPARAPOSDOUTORADO.TITULO, - professor.NumeroCurriculo, professor.NomeProfessor)); + Logger.Error( + $"A Orientação ou Supervisão Concluída de nome \"{dout.DADOSBASICOSDEORIENTACOESCONCLUIDASPARAPOSDOUTORADO.TITULO}\" {professor.NumeroCurriculo} - {professor.NomeProfessor} não possui Ano de Ocorrência, somente serão considerados Orientações/Supervições com Ano de Ocorrência" + ); } } else @@ -1645,9 +1641,9 @@ private void ProcessarOrientacoesESupervisoes(Professor professor, CurriculoVita { lock (logLocker) { - Logger.Error(String.Format("A Orientação ou Supervisão Concluída de nome \"{0}\" {1} - {2} não possui Ano de Ocorrência, somente serão considerados Orientações/Supervições com Ano de Ocorrência", - outras.DADOSBASICOSDEOUTRASORIENTACOESCONCLUIDAS.TITULO, - professor.NumeroCurriculo, professor.NomeProfessor)); + Logger.Error( + $"A Orientação ou Supervisão Concluída de nome \"{outras.DADOSBASICOSDEOUTRASORIENTACOESCONCLUIDAS.TITULO}\" {professor.NumeroCurriculo} - {professor.NomeProfessor} não possui Ano de Ocorrência, somente serão considerados Orientações/Supervições com Ano de Ocorrência" + ); } } else @@ -1660,9 +1656,7 @@ private void ProcessarOrientacoesESupervisoes(Professor professor, CurriculoVita if (outras.DADOSBASICOSDEOUTRASORIENTACOESCONCLUIDAS.TIPO != null && outras.DADOSBASICOSDEOUTRASORIENTACOESCONCLUIDAS.TIPO != "") - orien.NaturezaOrientacaoSupervicao = String.Format("{0} ({1})", - orien.NaturezaOrientacaoSupervicao, - outras.DADOSBASICOSDEOUTRASORIENTACOESCONCLUIDAS.TIPO); + orien.NaturezaOrientacaoSupervicao = $"{orien.NaturezaOrientacaoSupervicao} ({outras.DADOSBASICOSDEOUTRASORIENTACOESCONCLUIDAS.TIPO})"; orien.InstituicaoEmpresa = GetInstituicaoEmpresa(outras.DETALHAMENTODEOUTRASORIENTACOESCONCLUIDAS.CODIGOINSTITUICAO, outras.DETALHAMENTODEOUTRASORIENTACOESCONCLUIDAS.NOMEDAINSTITUICAO); @@ -1731,9 +1725,9 @@ private void ProcessarOrientacoesESupervisoes(Professor professor, CurriculoVita { lock (logLocker) { - Logger.Error(String.Format("A Orientação ou Supervisão Em Andamento de nome \"{0}\" {1} - {2} não possui Ano de Ocorrência, somente serão considerados Orientações/Supervições com Ano de Ocorrência", - mestrado.DADOSBASICOSDAORIENTACAOEMANDAMENTODEMESTRADO.TITULODOTRABALHO, - professor.NumeroCurriculo, professor.NomeProfessor)); + Logger.Error( + $"A Orientação ou Supervisão Em Andamento de nome \"{mestrado.DADOSBASICOSDAORIENTACAOEMANDAMENTODEMESTRADO.TITULODOTRABALHO}\" {professor.NumeroCurriculo} - {professor.NomeProfessor} não possui Ano de Ocorrência, somente serão considerados Orientações/Supervições com Ano de Ocorrência" + ); } } else @@ -1815,9 +1809,9 @@ private void ProcessarOrientacoesESupervisoes(Professor professor, CurriculoVita { lock (logLocker) { - Logger.Error(String.Format("A Orientação ou Supervisão Em Andamento de nome \"{0}\" {1} - {2} não possui Ano de Ocorrência, somente serão considerados Orientações/Supervições com Ano de Ocorrência", - dout.DADOSBASICOSDAORIENTACAOEMANDAMENTODEDOUTORADO.TITULODOTRABALHO, - professor.NumeroCurriculo, professor.NomeProfessor)); + Logger.Error( + $"A Orientação ou Supervisão Em Andamento de nome \"{dout.DADOSBASICOSDAORIENTACAOEMANDAMENTODEDOUTORADO.TITULODOTRABALHO}\" {professor.NumeroCurriculo} - {professor.NomeProfessor} não possui Ano de Ocorrência, somente serão considerados Orientações/Supervições com Ano de Ocorrência" + ); } } else @@ -1889,9 +1883,9 @@ private void ProcessarOrientacoesESupervisoes(Professor professor, CurriculoVita { lock (logLocker) { - Logger.Error(String.Format("A Orientação ou Supervisão Em Andamento de nome \"{0}\" {1} - {2} não possui Ano de Ocorrência, somente serão considerados Orientações/Supervições com Ano de Ocorrência", - dout.DADOSBASICOSDAORIENTACAOEMANDAMENTODEPOSDOUTORADO.TITULODOTRABALHO, - professor.NumeroCurriculo, professor.NomeProfessor)); + Logger.Error( + $"A Orientação ou Supervisão Em Andamento de nome \"{dout.DADOSBASICOSDAORIENTACAOEMANDAMENTODEPOSDOUTORADO.TITULODOTRABALHO}\" {professor.NumeroCurriculo} - {professor.NomeProfessor} não possui Ano de Ocorrência, somente serão considerados Orientações/Supervições com Ano de Ocorrência" + ); } } else @@ -1963,9 +1957,9 @@ private void ProcessarOrientacoesESupervisoes(Professor professor, CurriculoVita { lock (logLocker) { - Logger.Error(String.Format("A Orientação ou Supervisão Em Andamento de nome \"{0}\" {1} - {2} não possui Ano de Ocorrência, somente serão considerados Orientações/Supervições com Ano de Ocorrência", - espec.DADOSBASICOSDAORIENTACAOEMANDAMENTODEAPERFEICOAMENTOESPECIALIZACAO.TITULODOTRABALHO, - professor.NumeroCurriculo, professor.NomeProfessor)); + Logger.Error( + $"A Orientação ou Supervisão Em Andamento de nome \"{espec.DADOSBASICOSDAORIENTACAOEMANDAMENTODEAPERFEICOAMENTOESPECIALIZACAO.TITULODOTRABALHO}\" {professor.NumeroCurriculo} - {professor.NomeProfessor} não possui Ano de Ocorrência, somente serão considerados Orientações/Supervições com Ano de Ocorrência" + ); } } else @@ -2040,9 +2034,9 @@ private void ProcessarOrientacoesESupervisoes(Professor professor, CurriculoVita { lock (logLocker) { - Logger.Error(String.Format("A Orientação ou Supervisão Em Andamento de nome \"{0}\" {1} - {2} não possui Ano de Ocorrência, somente serão considerados Orientações/Supervições com Ano de Ocorrência", - dados.TITULODOTRABALHO, - professor.NumeroCurriculo, professor.NomeProfessor)); + Logger.Error( + $"A Orientação ou Supervisão Em Andamento de nome \"{dados.TITULODOTRABALHO}\" {professor.NumeroCurriculo} - {professor.NomeProfessor} não possui Ano de Ocorrência, somente serão considerados Orientações/Supervições com Ano de Ocorrência" + ); } } else @@ -2117,9 +2111,9 @@ private void ProcessarOrientacoesESupervisoes(Professor professor, CurriculoVita { lock (logLocker) { - Logger.Error(String.Format("A Orientação ou Supervisão Em Andamento de nome \"{0}\" {1} - {2} não possui Ano de Ocorrência, somente serão considerados Orientações/Supervições com Ano de Ocorrência", - dados.TITULODOTRABALHO, - professor.NumeroCurriculo, professor.NomeProfessor)); + Logger.Error( + $"A Orientação ou Supervisão Em Andamento de nome \"{dados.TITULODOTRABALHO}\" {professor.NumeroCurriculo} - {professor.NomeProfessor} não possui Ano de Ocorrência, somente serão considerados Orientações/Supervições com Ano de Ocorrência" + ); } } else @@ -2194,9 +2188,9 @@ private void ProcessarOrientacoesESupervisoes(Professor professor, CurriculoVita { lock (logLocker) { - Logger.Error(String.Format("A Orientação ou Supervisão Em Andamento de nome \"{0}\" {1} - {2} não possui Ano de Ocorrência, somente serão considerados Orientações/Supervições com Ano de Ocorrência", - dados.TITULODOTRABALHO, - professor.NumeroCurriculo, professor.NomeProfessor)); + Logger.Error( + $"A Orientação ou Supervisão Em Andamento de nome \"{dados.TITULODOTRABALHO}\" {professor.NumeroCurriculo} - {professor.NomeProfessor} não possui Ano de Ocorrência, somente serão considerados Orientações/Supervições com Ano de Ocorrência" + ); } } else @@ -2375,9 +2369,9 @@ private void ProcessarParticipacaoEventos(Professor professor, CurriculoVitaeXml { if (partCongresso.DETALHAMENTODAPARTICIPACAOEMCONGRESSO == null) { - Logger.Error(String.Format("Não foram informados os detalhes da Participação de Evento \"{0}\" do Professor(a) {1} ({2})", - partCongresso.DADOSBASICOSDAPARTICIPACAOEMCONGRESSO.TITULO, - professor.NomeProfessor, professor.NumeroCurriculo)); + Logger.Error( + $"Não foram informados os detalhes da Participação de Evento \"{partCongresso.DADOSBASICOSDAPARTICIPACAOEMCONGRESSO.TITULO}\" do Professor(a) {professor.NomeProfessor} - {professor.NumeroCurriculo})" + ); } else { @@ -2441,9 +2435,9 @@ private void ProcessarParticipacaoEventos(Professor professor, CurriculoVitaeXml { if (partFeira.DETALHAMENTODAPARTICIPACAOEMFEIRA == null) { - Logger.Error(String.Format("Não foram informados os detalhes da Participação de Evento \"{0}\" do Professor(a) {1} ({2})", - partFeira.DADOSBASICOSDAPARTICIPACAOEMFEIRA.TITULO, - professor.NomeProfessor, professor.NumeroCurriculo)); + Logger.Error( + $"Não foram informados os detalhes da Participação de Evento \"{partFeira.DADOSBASICOSDAPARTICIPACAOEMFEIRA.TITULO}\" do Professor(a) {professor.NomeProfessor} ({professor.NumeroCurriculo})" + ); } else { @@ -2507,9 +2501,9 @@ private void ProcessarParticipacaoEventos(Professor professor, CurriculoVitaeXml { if (partSeminario.DETALHAMENTODAPARTICIPACAOEMSEMINARIO == null) { - Logger.Error(String.Format("Não foram informados os detalhes da Participação de Evento \"{0}\" do Professor(a) {1} ({2})", - partSeminario.DADOSBASICOSDAPARTICIPACAOEMSEMINARIO.TITULO, - professor.NomeProfessor, professor.NumeroCurriculo)); + Logger.Error( + $"Não foram informados os detalhes da Participação de Evento \"{partSeminario.DADOSBASICOSDAPARTICIPACAOEMSEMINARIO.TITULO}\" do Professor(a) {professor.NomeProfessor} - {professor.NumeroCurriculo})" + ); } else { @@ -2573,9 +2567,9 @@ private void ProcessarParticipacaoEventos(Professor professor, CurriculoVitaeXml { if (partSimposio.DETALHAMENTODAPARTICIPACAOEMSIMPOSIO == null) { - Logger.Error(String.Format("Não foram informados os detalhes da Participação de Evento \"{0}\" do Professor(a) {1} ({2})", - partSimposio.DADOSBASICOSDAPARTICIPACAOEMSIMPOSIO.TITULO, - professor.NomeProfessor, professor.NumeroCurriculo)); + Logger.Error( + $"Não foram informados os detalhes da Participação de Evento \"{partSimposio.DADOSBASICOSDAPARTICIPACAOEMSIMPOSIO.TITULO}\" do Professor(a) {professor.NomeProfessor} ({professor.NumeroCurriculo})" + ); } else { @@ -2639,9 +2633,9 @@ private void ProcessarParticipacaoEventos(Professor professor, CurriculoVitaeXml { if (partOficina.DETALHAMENTODAPARTICIPACAOEMOFICINA == null) { - Logger.Error(String.Format("Não foram informados os detalhes da Participação de Evento \"{0}\" do Professor(a) {1} ({2})", - partOficina.DADOSBASICOSDAPARTICIPACAOEMOFICINA.TITULO, - professor.NomeProfessor, professor.NumeroCurriculo)); + Logger.Error( + $"Não foram informados os detalhes da Participação de Evento \"{partOficina.DADOSBASICOSDAPARTICIPACAOEMOFICINA.TITULO}\" do Professor(a) {professor.NomeProfessor} ({professor.NumeroCurriculo})" + ); } else { @@ -2705,9 +2699,9 @@ private void ProcessarParticipacaoEventos(Professor professor, CurriculoVitaeXml { if (partEncontro.DETALHAMENTODAPARTICIPACAOEMENCONTRO == null) { - Logger.Error(String.Format("Não foram informados os detalhes da Participação de Evento \"{0}\" do Professor(a) {1} ({2})", - partEncontro.DADOSBASICOSDAPARTICIPACAOEMENCONTRO.TITULO, - professor.NomeProfessor, professor.NumeroCurriculo)); + Logger.Error( + $"Não foram informados os detalhes da Participação de Evento \"{partEncontro.DADOSBASICOSDAPARTICIPACAOEMENCONTRO.TITULO}\" do Professor(a) {professor.NomeProfessor} ({professor.NumeroCurriculo})" + ); } else { @@ -2771,9 +2765,9 @@ private void ProcessarParticipacaoEventos(Professor professor, CurriculoVitaeXml { if (partExposicao.DETALHAMENTODAPARTICIPACAOEMEXPOSICAO == null) { - Logger.Error(String.Format("Não foram informados os detalhes da Participação de Evento \"{0}\" do Professor(a) {1} ({2})", - partExposicao.DADOSBASICOSDAPARTICIPACAOEMEXPOSICAO.TITULO, - professor.NomeProfessor, professor.NumeroCurriculo)); + Logger.Error( + $"Não foram informados os detalhes da Participação de Evento \"{partExposicao.DADOSBASICOSDAPARTICIPACAOEMEXPOSICAO.TITULO}\" do Professor(a) {professor.NomeProfessor} ({professor.NumeroCurriculo})" + ); } else { @@ -2837,9 +2831,9 @@ private void ProcessarParticipacaoEventos(Professor professor, CurriculoVitaeXml { if (partOlimpiada.DETALHAMENTODAPARTICIPACAOEMOLIMPIADA == null) { - Logger.Error(String.Format("Não foram informados os detalhes da Participação de Evento \"{0}\" do Professor(a) {1} ({2})", - partOlimpiada.DADOSBASICOSDAPARTICIPACAOEMOLIMPIADA.TITULO, - professor.NomeProfessor, professor.NumeroCurriculo)); + Logger.Error( + $"Não foram informados os detalhes da Participação de Evento \"{partOlimpiada.DADOSBASICOSDAPARTICIPACAOEMOLIMPIADA.TITULO}\" do Professor(a) {professor.NomeProfessor} ({professor.NumeroCurriculo})" + ); } else { @@ -2903,9 +2897,9 @@ private void ProcessarParticipacaoEventos(Professor professor, CurriculoVitaeXml { if (outras.DETALHAMENTODEOUTRASPARTICIPACOESEMEVENTOSCONGRESSOS == null) { - Logger.Error(String.Format("Não foram informados os detalhes da Participação de Evento \"{0}\" do Professor(a) {1} ({2})", - outras.DADOSBASICOSDEOUTRASPARTICIPACOESEMEVENTOSCONGRESSOS.TITULO, - professor.NomeProfessor, professor.NumeroCurriculo)); + Logger.Error( + $"Não foram informados os detalhes da Participação de Evento \"{outras.DADOSBASICOSDEOUTRASPARTICIPACOESEMEVENTOSCONGRESSOS.TITULO}\" do Professor(a) {professor.NomeProfessor} ({professor.NumeroCurriculo})" + ); } else { @@ -3657,14 +3651,9 @@ private ProducaoBibliografica GetProducaoBibliografica(Professor professor, stri } catch (Exception exception) { - Logger.Error(String.Format( - "A Publicação \"{0}\" informado no currículo {1} do Professor {2} possui ano de publicação inválido ({3}), favor revisar o mesmo. Mensagem de erro: {4}", - titulo, - professor.NumeroCurriculo, - professor.NomeProfessor, - ano, - exception.Message - )); + Logger.Error( + $"A Publicação \"{titulo}\" informado no currículo {professor.NumeroCurriculo} do Professor {professor.NomeProfessor} possui ano de publicação inválido ({ano}), favor revisar o mesmo. Mensagem de erro: {exception.Message}" + ); } lock (saveLocker) @@ -4788,8 +4777,9 @@ private void ProcessarAtuacoesProfissionaisEProjetos(Professor professor, Curric { lock (logLocker) { - Logger.Error(String.Format("O Projeto \"{0}\" do Professor {1} - {2} não possui Ano de Início, somente serão considerados projetos com Ano de Início", - pdp.NOMEDOPROJETO, professor.NumeroCurriculo, professor.NomeProfessor)); + Logger.Error( + $"O Projeto \"{pdp.NOMEDOPROJETO}\" do Professor {professor.NumeroCurriculo} - {professor.NomeProfessor} não possui Ano de Início, somente serão considerados projetos com Ano de Início" + ); } } else @@ -4864,7 +4854,7 @@ private void ProcessarAtuacoesProfissionaisEProjetos(Professor professor, Curric } } - ativProf.InformacoesAdicionaisAtividadeProfissional = Utils.SetMaxLength(String.Format("Projetos: {0}", projetos), 2000); + ativProf.InformacoesAdicionaisAtividadeProfissional = Utils.SetMaxLength($"Projetos: {projetos}", 2000); professor.AtividadeProfissional.Add(ativProf); } @@ -4893,7 +4883,7 @@ private void ProcessarAtuacoesProfissionaisEProjetos(Professor professor, Curric else ativProf.DataTerminoAtividadeProfissional = Utils.ParseMonthAndYear("12", ccc.ANOFIM); - ativProf.InformacoesAdicionaisAtividadeProfissional = Utils.SetMaxLength(String.Format("Especificação: {0}", ccc.ESPECIFICACAO), 2000); + ativProf.InformacoesAdicionaisAtividadeProfissional = Utils.SetMaxLength($"Especificação: {ccc.ESPECIFICACAO}", 2000); professor.AtividadeProfissional.Add(ativProf); } @@ -4922,7 +4912,7 @@ private void ProcessarAtuacoesProfissionaisEProjetos(Professor professor, Curric else ativProf.DataTerminoAtividadeProfissional = Utils.ParseMonthAndYear("12", outra.ANOFIM); - ativProf.InformacoesAdicionaisAtividadeProfissional = Utils.SetMaxLength(String.Format("Atividade Realizada: {0}", outra.ATIVIDADEREALIZADA), 2000); + ativProf.InformacoesAdicionaisAtividadeProfissional = Utils.SetMaxLength($"Atividade Realizada: {outra.ATIVIDADEREALIZADA}", 2000); professor.AtividadeProfissional.Add(ativProf); } @@ -4960,7 +4950,7 @@ private void ProcessarAtuacoesProfissionaisEProjetos(Professor professor, Curric } } - ativProf.InformacoesAdicionaisAtividadeProfissional = Utils.SetMaxLength(String.Format("Treinamentos Ministrados: {0}", treinamentos), 2000); + ativProf.InformacoesAdicionaisAtividadeProfissional = Utils.SetMaxLength($"Treinamentos Ministrados: {treinamentos}", 2000); professor.AtividadeProfissional.Add(ativProf); } @@ -4989,7 +4979,7 @@ private void ProcessarAtuacoesProfissionaisEProjetos(Professor professor, Curric else ativProf.DataTerminoAtividadeProfissional = Utils.ParseMonthAndYear("12", eu.ANOFIM); - ativProf.InformacoesAdicionaisAtividadeProfissional = Utils.SetMaxLength(String.Format("Atividade Realizada: {0}", eu.ATIVIDADEDEEXTENSAOREALIZADA), 2000); + ativProf.InformacoesAdicionaisAtividadeProfissional = Utils.SetMaxLength($"Atividade Realizada: {eu.ATIVIDADEDEEXTENSAOREALIZADA}", 2000); professor.AtividadeProfissional.Add(ativProf); } @@ -5018,7 +5008,7 @@ private void ProcessarAtuacoesProfissionaisEProjetos(Professor professor, Curric else ativProf.DataTerminoAtividadeProfissional = Utils.ParseMonthAndYear("12", se.ANOFIM); - ativProf.InformacoesAdicionaisAtividadeProfissional = Utils.SetMaxLength(String.Format("Serviço Realizado: {0}", se.SERVICOREALIZADO), 2000); + ativProf.InformacoesAdicionaisAtividadeProfissional = Utils.SetMaxLength($"Serviço Realizado: {se.SERVICOREALIZADO}", 2000); professor.AtividadeProfissional.Add(ativProf); } @@ -5047,7 +5037,7 @@ private void ProcessarAtuacoesProfissionaisEProjetos(Professor professor, Curric else ativProf.DataTerminoAtividadeProfissional = Utils.ParseMonthAndYear("12", estagio.ANOFIM); - ativProf.InformacoesAdicionaisAtividadeProfissional = Utils.SetMaxLength(String.Format("Estágio Realizado: {0}", estagio.ESTAGIOREALIZADO), 2000); + ativProf.InformacoesAdicionaisAtividadeProfissional = Utils.SetMaxLength("Estágio Realizado: " + estagio.ESTAGIOREALIZADO, 2000); professor.AtividadeProfissional.Add(ativProf); } @@ -5084,10 +5074,10 @@ private void ProcessarAtuacoesProfissionaisEProjetos(Professor professor, Curric } } - ativProf.InformacoesAdicionaisAtividadeProfissional = Utils.SetMaxLength(String.Format("Curso Ministrado: {0} - {1}; Disciplinas Ministradas: {2}", - ensino.CODIGOCURSO, - ensino.NOMECURSO, - disciplinas), 2000); + ativProf.InformacoesAdicionaisAtividadeProfissional = Utils.SetMaxLength( + $"Curso Ministrado: {ensino.CODIGOCURSO} - {ensino.NOMECURSO}; Disciplinas Ministradas: {disciplinas}", + 2000 + ); professor.AtividadeProfissional.Add(ativProf); } @@ -5147,9 +5137,10 @@ private void ProcessarAtuacoesProfissionaisEProjetos(Professor professor, Curric else ativProf.DataTerminoAtividadeProfissional = Utils.ParseMonthAndYear("12", dea.ANOFIM); - ativProf.InformacoesAdicionaisAtividadeProfissional = Utils.SetMaxLength(String.Format("Cargo Ou Função: {0}", - TraduzirCargoOuFuncao(dea.FORMATOCARGOOUFUNCAO.ToString(), - dea.CARGOOUFUNCAO)), 2000); + ativProf.InformacoesAdicionaisAtividadeProfissional = Utils.SetMaxLength( + "Cargo Ou Função: " + TraduzirCargoOuFuncao(dea.FORMATOCARGOOUFUNCAO.ToString(), dea.CARGOOUFUNCAO), + 2000 + ); professor.AtividadeProfissional.Add(ativProf); } @@ -5164,8 +5155,9 @@ private void ProcessarAtuacoesProfissionaisEProjetos(Professor professor, Curric { lock (logLocker) { - Logger.Error(String.Format("O vínculo profissional do Professor {1} - {2} com a empresa \"{0}\" não possui Ano de Início, somente serão considerados vínculos com Ano de Início", - ie.NomeInstituicaoEmpresa, professor.NumeroCurriculo, professor.NomeProfessor)); + Logger.Error( + $"O vínculo profissional do Professor {professor.NumeroCurriculo} - {professor.NomeProfessor} com a empresa \"{ie.NomeInstituicaoEmpresa}\" não possui Ano de Início, somente serão considerados vínculos com Ano de Início" + ); } } else diff --git a/LattesExtractor/LattesExtractor.csproj b/LattesExtractor/LattesExtractor.csproj index 436f542..b2730f9 100644 --- a/LattesExtractor/LattesExtractor.csproj +++ b/LattesExtractor/LattesExtractor.csproj @@ -88,7 +88,7 @@ True - ..\packages\LinqToCsv.1.5.0.0\lib\net35\LINQtoCSV.dll + ..\packages\LINQtoCSV.1.5.0.0\lib\net35\LINQtoCSV.dll True diff --git a/LattesExtractor/LattesModule.cs b/LattesExtractor/LattesModule.cs index 249093e..f859c36 100644 --- a/LattesExtractor/LattesModule.cs +++ b/LattesExtractor/LattesModule.cs @@ -150,7 +150,7 @@ private void LoadCurriculums() var loadFromTempDirectory = new LoadFromTempDirectory(this, TempDirectory, CurriculumVitaeForProcess); if (loadFromTempDirectory.HasPendingResumes()) { - Logger.Info(String.Format("Foram encontrados XMLs pendentes na pasta '{0}' !'", TempDirectory)); + Logger.Info($"Foram encontrados XMLs pendentes na pasta '{TempDirectory}' !'"); QueueThreadGetCurriculumVitae(loadFromTempDirectory.LoadCurriculums); return; } @@ -158,7 +158,7 @@ private void LoadCurriculums() if (ImportFolder != null) { - Logger.Info(String.Format("Lendo Currículos do diretório '{0}'...", ImportFolder)); + Logger.Info($"Lendo Currículos do diretório '{ImportFolder}'..."); var importFromFolder = new ImportCurriculumVitaeFromFolderController( this, ImportFolder, @@ -258,14 +258,14 @@ public void Extract() private void ShowException(Exception ex) { - Logger.Error(String.Format("Erros durante a execução: {0}\n{2}", ex.Message, ex.StackTrace)); + Logger.Error($"Erros durante a execução: {ex.Message}\n{ex.StackTrace}"); if (ex.InnerException != null) { int sequencia = 1; while (ex.InnerException != null) { ex = ex.InnerException; - Logger.Error(String.Format("Excessão Interna [{0}]: {1}\n{2}", sequencia++, ex.Message, ex.StackTrace)); + Logger.Error($"Excessão Interna [{sequencia++}]: {ex.Message}\n{ex.StackTrace}"); } } } @@ -286,7 +286,7 @@ public void UpdateJCRImpactFactorDataBase(string inputJCRFile) public string GetCurriculumVitaeFileName(string curriculumVitaeNumber) { - return String.Format("{0}{1}{2}.xml", this.TempDirectory, Path.DirectorySeparatorChar, curriculumVitaeNumber); + return $"{this.TempDirectory}{Path.DirectorySeparatorChar}{curriculumVitaeNumber}.xml"; } } } diff --git a/LattesExtractor/Service/DownloadCurriculumVitaeWebService.cs b/LattesExtractor/Service/DownloadCurriculumVitaeWebService.cs index b3ab412..14ddad0 100644 --- a/LattesExtractor/Service/DownloadCurriculumVitaeWebService.cs +++ b/LattesExtractor/Service/DownloadCurriculumVitaeWebService.cs @@ -31,13 +31,17 @@ public MemoryStream GetCurriculumVitaeIfUpdated(CurriculoEntry curriculumVitae) if (curriculumVitae.NumeroCurriculo == null || curriculumVitae.NumeroCurriculo == "") { - Logger.Info(String.Format("Buscando Número do Currículo do Professor {0}", curriculumVitae.NomeProfessor)); + Logger.Info( + $"Buscando Número do Currículo do Professor {curriculumVitae.NomeProfessor}" + ); curriculumVitae.NumeroCurriculo = ws.getIdentificadorCNPq(curriculumVitae.CPF, curriculumVitae.NomeProfessor, curriculumVitae.DataNascimento); if (curriculumVitae.NumeroCurriculo == null || curriculumVitae.NumeroCurriculo == "" || curriculumVitae.NumeroCurriculo.Contains("ERRO")) return null; - Logger.Info(String.Format("Número do Currículo do Professor {0} encontrado: {1}", curriculumVitae.NomeProfessor, curriculumVitae.NumeroCurriculo)); + Logger.Info( + $"Número do Currículo do Professor {curriculumVitae.NomeProfessor} encontrado: {curriculumVitae.NumeroCurriculo}" + ); } // verificar se a data de atualizacao do CV é maior que a do sistema @@ -64,7 +68,9 @@ public MemoryStream GetCurriculumVitaeIfUpdated(CurriculoEntry curriculumVitae) if (zip == null || zip.Length == 0) { - Logger.Error(String.Format("Aconteceu um erro ao tentar buscar o currículo de Número {0}, favor verificar o mesmo", curriculumVitae.NumeroCurriculo)); + Logger.Error( + $"Aconteceu um erro ao tentar buscar o currículo de Número {curriculumVitae.NumeroCurriculo}, favor verificar o mesmo" + ); return null; } @@ -87,7 +93,7 @@ private Nullable GetDataAtualizacaoProfessor(string numeroCurriculo) { var db = new LattesDatabase(); var p = db.Professor.FirstOrDefault(prof => prof.NumeroCurriculo == numeroCurriculo); - + if (p == null) { return null;