issues
search
PedroCamillozzi
/
PaginaPCParts
0
stars
0
forks
source link
Sintáxis/Código
#4
Open
lucasjaviluna
opened
10 months ago
lucasjaviluna
commented
10 months ago
[Middle]
Hay una mezcla de español - inglés en los nombres de las variables y métodos. Lo usual es escribir todo en inglés
Hay muchos console.log
Hay mucho código comentado
Hay muchas comparaciones del tipo ==, siempre es mejor usar === para chequear tipo y valor de los operandos
Hay tags img que no tienen asignado un src
https://vscode.dev/github/PedroCamillozzi/PaginaPCParts/blob/angular/src/app/components/navbar-component/navbar-component.component.html#L93
Hay muchos any, que no aprovechan el potencial de Typescript. Ej:
https://vscode.dev/github/PedroCamillozzi/PaginaPCParts/blob/angular/src/app/services/cliente.service.ts#L32
No está mal la idea de un SharedModule, pero al final no se está reutilizando
https://vscode.dev/github/PedroCamillozzi/PaginaPCParts/blob/angular/src/app/shared/shared.ts#L27
No es necesario poner un callback vacío en el subscribe, si no lo vas a usar después. Con .subscribe() es suficiente
https://vscode.dev/github/PedroCamillozzi/PaginaPCParts/blob/angular/src/app/components/carrito-component/carrito-component.component.ts#L192
[Middle]