Found empty functions JS-0057
Anti-pattern
Minor
8 months ago
—
a year old
Unexpected empty function
2680 input = input || $.trim(self.$control_input.val() || '');
2681
2682 var callback = arguments[arguments.length - 1];
2683 if (typeof callback !== 'function') callback = function() {};2684
2685 if (typeof triggerDropdown !== 'boolean') {
2686 triggerDropdown = true;Unexpected empty function
710 };
711
712
713 var MicroEvent = function() {}; 714 MicroEvent.prototype = {
715 on: function(event, fct){
716 this._events = this._events || {};Unexpected empty function
2044 input = input || $.trim(self.$control_input.val() || '');
2045
2046 var callback = arguments[arguments.length - 1];
2047 if (typeof callback !== 'function') callback = function() {};2048
2049 if (typeof triggerDropdown !== 'boolean') {
2050 triggerDropdown = true;Unexpected empty function
74 };
75
76
77 var MicroEvent = function() {}; 78 MicroEvent.prototype = {
79 on: function(event, fct){
80 this._events = this._events || {};Unexpected empty function 'CloseOnSelect'
4428S2.define('select2/dropdown/closeOnSelect',[
4429
4430], function () {
4431 function CloseOnSelect () { }4432
4433 CloseOnSelect.prototype.bind = function (decorated, container, $container) {
4434 var self = this;
Description
Having empty functions hurts readability, and is considered a code-smell. There's almost always a way to avoid using them. If you must use one, consider adding a comment to inform the reader of its purpose.
Bad Practice
getUser('SwaGaLisTiQuE', () => {})
function f() {}
Recommended
getUser('SwaGaLisTiQuE', () => {
// empty because <reason>
})
function f() {
// intentionally empty. <reason>
}