Skip to content

Code conventions for controllers #2

Description

@vituchon

app.get('/wmlclient', (req, res) => {

Nomas una idea que puede llegar a gustar para ordenar el código del archivo que tiene todos los endpoints (lo cual se suele llamar Controller).

En vez de

app.get('/wmlclient', (req, res)  {
	res.render('wmlclient', { url: gicat_url})
	console.log('wmlclient form displayed')
	return
})

pones esto

func getWmnClient(req, res) {
	res.render('wmlclient', { url: gicat_url})
	console.log('wmlclient form displayed')
	return
}
app.get('/wmlclient',getWmnClient)

Ojo es solo estético, ya que cada endpoint tiene lógica que es propia de la prestación que brinda el mismo. Dificilmente se use una misma función en dos endpoints distintos pues no es la idea, poner en funciones el código te va a permitir nomas poder en linas sucesivas todos los endpoints a forma de mejorar la documentación del código

app.get('/wmlclient',getWmnClient)
app.get('/wmlclient/siteinfo',getWmnClientSiteInfo)

Entonces quedan end points por un lado, y funciones por el otro.
En fin, eso y por favor identa el switch que esta abajo! xD
TAmbien me gsuaría hablar luego de las funciones renderList(...) que recibe como 20 parametros o más!.

Saludos
V

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions