Skip to content

Atividade semana 4 - Imersão JS - Prototypes II - Elvira Bruno#28

Open
elvirabl wants to merge 3 commits into
reprograma:masterfrom
elvirabl:master
Open

Atividade semana 4 - Imersão JS - Prototypes II - Elvira Bruno#28
elvirabl wants to merge 3 commits into
reprograma:masterfrom
elvirabl:master

Conversation

@elvirabl

Copy link
Copy Markdown

criação de uma simulação de Clinica Veterinária
utilização de métodos Class, Construtores e Super.

Exercício não completo, pretendo terminá-lo em 20/07 após as 22h.

Obrigada por todo aprendizado e desculpas pelo atraso na entrega.

Elvira Bruno

Comment on lines +17 to +21
consultar() {
const dataConsulta = new Date();
this.consultas.push(dataConsulta);
this.historico.consultas.push(dataConsulta);
}

@EllieKonuma EllieKonuma Jul 21, 2023

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

aqui em consultar poderia ser um pouco mais conciso e sem precisar criar uma const

consultar() {
      this.historico.consultas.push(Date());
    }

e o "this.consultas.push(dataConsulta);" o meu console relacmou disso e se tirar o resto tb funcionaria!

Comment on lines +1 to +7
class HistoricoMedico {
constructor() {
this.consultas = [];
this.vacinas = [];
this.procedimento = [];
}
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Eu gostei muito de colocar todos os metodo de historico medico no historico achei mais organizado mas talvez é so uma preferencia minha mas fica a dica :3

Comment on lines +63 to +68
acariciar() {
super.acariciar();
if (this.social) {
console.log(`${this.nome} está ronronando de felicidade.`);
}
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

fiquei pensando nessa estrutura de ter um acariciar em Animal e depois chamar
eu acho melhor ter criado tudo no gatinho pq animal não vem com acariciar e faz mais sentido para mim ser de Gato ou Cachorro...etc

mas por outro lado acho que poderia ser bom pq todos nao precisaria criar um console.log para todos e assim deixando mais padronizado hauhauha


acariciar() {
super.acariciar();
if (this.social) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

acho que poderia ser

if (this.social == true){console.log(`${this.nome} está ronronando de felicidade.`);}

pq ele compararia se fosse true


alimentar(comida) {
super.alimentar();
const comidasFavoritas = ["frango", "sachê", "peixe"];

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

acho que essa lista poderia ser instanciada como static:

static comidas = ["frango", "sache", "peixe"];

pq nao precisaria ser criado como const e acho que poderia ter mais controle sobre o que o console devolve, assim nao mostrando em "gato.comidasFavoritas"

Comment on lines +70 to +87
alimentar(comida) {
super.alimentar();
const comidasFavoritas = ["frango", "sachê", "peixe"];
let encontrouComidaFavorita = false;

for (let i = 0; i < comidasFavoritas.length; i++) {
if (comidasFavoritas[i] === comida) {
encontrouComidaFavorita = true;
break;
}
}

if (this.social && encontrouComidaFavorita) {
console.log(`${this.nome} está mais sociável agora.`);
this.social = true;
}
}
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

acho que seria legal ter um caso se desse errado tipo colocar algo que nao tem na lista de comidas e retornar um "console.log("gatos nao comem comida errada")"

super(nome, idade, cor, consultas);
this.raça = raça;
this.castrado = false;
this.ferido = false;

@EllieKonuma EllieKonuma Jul 21, 2023

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

acho legal ferido ser #ferido então ficaria:

class Cachorro extends Animal {
   #feridp
  constructor(nome, idade, cor, consultas = [], raça = []) {
    super(nome, idade, cor, consultas);
    this.raça = raça;
    this.castrado = false;
    this.#ferido = false;
  }

usaria assim pq # faz com que as pessoas nao possam acesar com gato.ferido e mudar para true assim fazendo com que só sua função mude

@EllieKonuma

Copy link
Copy Markdown

oioiiiiiii Elvira!
amei o codigo continue assim!
fiz alguns apontamentos de como eu faria ou de como eu penso mas nao precisa levar muito a serio huahuah nao sei se o que eu fiz foi o melhor para um codigo mas se fizer sentido para vc fico feliz! :3

qualquer coisa pode me chamar!

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