separate event handler functions for an event attached to different inputs or one event handler function with lot of if statements/ switch statement? #1783

unitedsol · 2018-04-12T11:59:57Z

separate event handler functions for an event attached to different inputs or one event handler function with lot of if statements/ switch statement?

// event handler 1
function validateEmail() {
	// validate email here
}

// event handler 2
function validatePassword() {
	//validate password here
}

// or

// generic event handler
function validateFields() {
	/*
	if(is email){
		validate email
	}

	if(is password) {
		validate password
	}
	*/
}

ljharb · 2018-04-14T01:43:12Z

This really isn’t the kind of thing this guide currently addresses, but personally I’d tend towards separate ones - it’s easier to unit test, and when something goes wrong, you have way less code to debug.

ljharb added the question label Apr 14, 2018

airbnb / javascript

separate event handler functions for an event attached to different inputs or one event handler function with lot of if statements/ switch statement? #1783

separate event handler functions for an event attached to different inputs or one event handler function with lot of if statements/ switch statement? #1783

unitedsol commented Apr 12, 2018 •

edited by ljharb

ljharb commented Apr 14, 2018

airbnb / javascript

Join GitHub today

separate event handler functions for an event attached to different inputs or one event handler function with lot of if statements/ switch statement? #1783

separate event handler functions for an event attached to different inputs or one event handler function with lot of if statements/ switch statement? #1783

Comments

unitedsol commented Apr 12, 2018 • edited by ljharb

ljharb commented Apr 14, 2018

unitedsol commented Apr 12, 2018 •

edited by ljharb