Skip to content

#35 Exception Handling - Manipulação de Arquivos CSV#50

Open
fran-olv wants to merge 14 commits intodevs-javagirl:mainfrom
fran-olv:main
Open

#35 Exception Handling - Manipulação de Arquivos CSV#50
fran-olv wants to merge 14 commits intodevs-javagirl:mainfrom
fran-olv:main

Conversation

@fran-olv
Copy link

@fran-olv fran-olv commented Oct 4, 2023

Criação de um csv exemplo
Criação da classe Pessoa e da ListaPessoas
Na ListaPessoas é onde esta a implementação solicitada no Issue #35

@fran-olv fran-olv marked this pull request as draft October 4, 2023 02:10
@fran-olv fran-olv closed this Oct 4, 2023
@fran-olv fran-olv reopened this Oct 4, 2023
@fran-olv fran-olv marked this pull request as ready for review October 4, 2023 02:13
@fran-olv fran-olv marked this pull request as draft October 4, 2023 02:14
@fran-olv fran-olv marked this pull request as ready for review October 4, 2023 02:14
Copy link
Collaborator

@isdrchagas isdrchagas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fran, faltou criar o readme falando um pouquinho sobre exceções em Java.

fran-olv and others added 4 commits October 19, 2023 16:57
Co-authored-by: isdrchagas <85903568+isdrchagas@users.noreply.github.com>
Co-authored-by: isdrchagas <85903568+isdrchagas@users.noreply.github.com>
Co-authored-by: isdrchagas <85903568+isdrchagas@users.noreply.github.com>
Copy link
Collaborator

@isdrchagas isdrchagas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oi Fran, falta o readme!! Assim que você fizer isso pede meu review aqui que dou uma olhada para conseguirmos fazer o merge :D

@fran-olv
Copy link
Author

fran-olv commented Nov 1, 2023

@isdrchagas criei o arquivo readme e tive que resolver um conflito de merge. ve se ta certinho?

@fran-olv fran-olv requested a review from isdrchagas November 1, 2023 14:05
Copy link
Collaborator

@isdrchagas isdrchagas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fran, o readme ficou bem bacana! Só fazendo os ajustes que comentei e resolvendo o conflito, já conseguimos mergear :D

Comment on lines +35 to +41
public static double calcularMediaIdade(List<Pessoa> pessoaList){
return pessoas.stream()
.mapToDouble(Pessoa::getIdade)
.average()
.orElse(0.0);
}

Copy link
Collaborator

@isdrchagas isdrchagas Nov 1, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

aqui não está compilando porquê pessoas não existe, o nome do parametro é pessoaList


public static void main(String[] args) {
String pathArquivoCsv = "Exception Handling/ManipulacaoCSV/com.manipulacao-csv.java/dados.csv";
List<Pessoa> pessoas = PreencheLista(pathArquivoCsv);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mesma coisa aqui, como mudou o nome do metodo, o PreencheLista não existe mais

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants