xpostudio4 / metacommunity

New website of the meta community
3 stars 4 forks source link

Nombrar manejadores de rutas como acciones en lugar de sustantivos #13

Closed francisbrito closed 8 years ago

francisbrito commented 8 years ago

La idea es tener getSpeakers y createSpeaker:

function* getSpeakers() {
  // get speakers somehow.
}

function* createSpeaker() {
  // create speaker somehow.
}

router.get('/speakers', getSpeakers);
router.post('/speakers', createSpeaker);

En lugar de:

function* speakers() {
  if (this.method === 'GET') // get speakers somehow
  if (this.method === 'POST') // create speaker somehow
}

router.all('/speakers', speakers);

La primera forma evita tener que hacer conditional branching de las respuestas y, imho, es más hace que el proposito del manejador de ruta sea evidente.

RaulMonteroC commented 8 years ago

+1