aureliobeo / cartao-credito

0 stars 0 forks source link

app.component.html is useless #1

Open maugustosemperfi opened 1 year ago

maugustosemperfi commented 1 year ago

atualmente a gente prefere componentes menores e mais limpos, consequentemente menos arquivos:

utilizar dessa forma:

import { Component } from '@angular/core';

@Component({
  selector: 'app-root',
  templateUrl: './app.component.html',
  styleUrls: ['./app.component.css']
})
export class AppComponent {
  isCollapsed = false;
}

induz uma quantidade de codigo maior por arquivo. tem uma alternativa:

import { Component } from '@angular/core';

@Component({
  selector: 'app-root',
  template: `
//html deve ficar aqui
  `,
  styleUrls: ['./app.component.css']
})
export class AppComponent {
  isCollapsed = false;
}

dessa forma, tem como perceber quando o arquivo ficara grande

aureliobeo commented 1 year ago

E quando eu fizer dessa forma, eu só escrevo no ts mesmo e quando acabar eu crio um component html e passo para la ou só deixo no ts mesmo?

maugustosemperfi commented 1 year ago

Nop, dessa forma você não precisa criar um arquivo .html, baste escrever no html dentro dessa propriedade template.

aureliobeo commented 1 year ago

Estou removendo o compoente HTML e colocando o codigo no ts como vc falou e estou começando a criar novos componentes para "aliviar" o componente pai

maugustosemperfi commented 1 year ago

boa! @aureliobeo tem PR pra isso?

aureliobeo commented 1 year ago

Agora tem

maugustosemperfi commented 1 year ago

associa ele aqui. pode inserir o link pra facilitar 👍

aureliobeo commented 1 year ago

Acabei de atualizar o projeto com novas melhoria na leitura do codigo e coloquei em merge para deixar algum comentario https://github.com/aureliobeo/cartao-credito/pull/7

aureliobeo commented 1 year ago

Eu comecei a melhorar o codigo mais ainda, criando um model para poder tipar o objeto que eu compartilho entre os componentes e quando eu terminar essa minha task, vou diminuir o tanto de codigo que tem no comproent front

maugustosemperfi commented 1 year ago

fiz alguns comentarios no PR. mas acho que ja em um progresso. depois fizer o merge, ja da pra fechar essa issue e a gente consegue abrir outra