Avoiding branching

avoid_branching.js

// Implementation 1
async function makeRequest({
    method = RequestMethod.GET,
    path,
    body = {},
    tokenStore,
    isLoginRequest = false,
}) {
    
    // We can avoid this branching by using implementation 2
    const url = isLoginRequest
        ? new URL(path, VM_LOGIN_HOST)
        : new URL(path, tokenStore?.api_url);

    await Requests.makeRequest({ method, url, body, headers });
}

// From LognOperator call
makeRequest(...args, (isLoginRequest = True));
// From everywhere else
makeRequest(...args);






// Implementaion 2

// Instead of isLoginRequest - add param host.
async function makeRequest({
    method = RequestMethod.GET,
    host,
    path,
    body = {},
    tokenStore,
}) {
    

    await Requests.makeRequest({ method, url, body, headers });
}

// From LoginOperator
makeRequest(...args, (host = VM_LOGIN_HOST), (path = Login_url));
// From everywhere else, know what should be the host
makeRequest(...args, (host = tokenStore.host));