Code conventions for controllers

vituchon commented 5 years ago

https://github.com/cerobpm/wmlclient/blob/8042bdbb070e78dd1c4f96419b37e42f9711ddb9/app/index.js#L42

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

cerobpm commented 5 years ago

Gracias oh Maestro! Será hecho!

vituchon commented 5 years ago

Mi querido joven "Ya-no-más-Padawan", veo que el código es fuerte en ti!!

cerobpm / wmlclient

Code conventions for controllers #2