Logging Out Users

As a whole, by using actions you're:

• Giving the operation a name & scope, which improves the ability to scan your controllers.

• Gives us the ability to easily reuse the code if needed. It is considered bad practice to call one controller method from another, as controllers are meant to handle routes.

• On top of reusability, if we need to update an operation later on, we only have to perform that update in one spot rather than each spot it is performed in the controllers.

• As you said, it gives us a separation of concerns. If we're performing an action, we know we can find it within our actions folder, under the resource/section it is for.

Note that actions are very similar to services. To compare, you can think of each subfolder within actions as a service class and each file within the subfolder as a service class method. The main difference here though is actions allow us to easily define steps within the actions whereas that'd need to be additional methods inside a service, which can make things a little messy.

A good example, though the code for this is from a future lesson releasing soon, is the account deletion controller & action. The controller is easy to scan. The operations have a designated location & scope and can make use of subactions. We're reusing our webLogout method, meaning we can easily update just the webLogout method if we ever need to adjust how we handle logouts. For example, if you're tracking user sessions, you might need to mark that session record in the database as logged out in addition to actually logging out the user.

@inject()
async destroy({ request, response, session, auth }: HttpContext, webLogout: WebLogout) {
  const user = auth.use('web').user!
  const validator = vine.compile(
    vine.object({
      email: vine.string().in([user.email]),
    })
  )

  await request.validateUsing(validator)

  await DestroyUserAccount.handle({ user })

  await webLogout.handle()

  session.flash('success', 'Your account has been deleted')

  return response.redirect().toRoute('register.show')
}

  

    

      Copied!
      

        

      
    
  
  
    
    
    
    
  

I've worked with code, and written my fair share, where the controller method gets unwieldy large because all the operations are done directly in the controller. It might seem small at first, but requirements change, time gets crunched, and things slowly grow over time.

Hope this helps!