Expand Minimize

Consider to use built in helper method, instead of type checking null, and undefined

Consider to use built in helper method from SharePoint (SP.ScriptUtility isNullOrEmptyString, isNullorUndefined, isUndefined and truncateToInt), instead of type checking null, and undefined

CheckId SPC038904
TypeName ConsiderToNotTypeCheckDirectlyNullOrUndefined
Severity Warning
Type JavaScriptFile

Bad Practice

if(typeof foo == 'undefined' || foo === null) {
  return false;
}

Good Practice
if(SP.ScriptUtility.isNullOrEmptyString(foo)) {
  return false';
}

Disclaimer: The views and opinions expressed in this documentation and in SPCAF do not necessarily reflect the opinions and recommendations of Microsoft or any member of Microsoft. SPCAF and RENCORE are registered trademarks of Rencore. All other trademarks, service marks, collective marks, copyrights, registered names, and marks used or cited by this documentation are the property of their respective owners.