重构此功能

Question

收到声纳问题，因为可以在功能下进行重构。以下重构功能的任何方法。

function getBodyFields(request:UpdateCustomerRequest, tmsGuid:string) {
  return {
    tmsGuid: tmsGuid,
    ...((request.organizationName && request.organizationName !== "") && {'organizationName': request.organizationName}),
    ...((request.firstName && request.firstName !== "") && {'firstName': request.firstName}),
    ...((request.middleInitial && request.middleInitial !== "") && {'middleInitial': request.middleInitial}),
    ...((request.lastName && request.lastName !== "") && {'lastName': request.lastName}),
    ...((request.email && request.email !== "") && {'email': request.email}),
    ...((request.communicationEmail && request.communicationEmail !== "") && {'communicationEmail': request.communicationEmail}),
    ...((request.areaCode && request.areaCode !== "") && {'areaCode': request.areaCode}),
    ...((request.phoneNumber && request.phoneNumber !== "") && {'phoneNumber': request.phoneNumber}),
    ...((request.phoneType && request.phoneType !== "") && {'phoneType': request.phoneType}),
    ...((request.alternatePhoneAreaCode && request.alternatePhoneAreaCode !== "") && {'alternatePhoneAreaCode': request.alternatePhoneAreaCode}),
    ...((request.alternatePhoneNumber && request.alternatePhoneNumber !== "") && {'alternatePhoneNumber': request.alternatePhoneNumber}),
    ...((request.alternatePhoneType && request.alternatePhoneType !== "") && {'alternatePhoneType': request.alternatePhoneType}),
    ...((request.address1 && request.address1 !== "") && {'address1': request.address1}),
    ...((request.address2 && request.address2 !== "") && {'address2': request.address2}),
    ...((request.city && request.city !== "") && {'city': request.city}),
    ...((request.state && request.state !== "") && {'state': request.state}),
    ...((request.zip && request.zip !== "") && {'zip': request.zip}),
  }
}

Answer 1

自从您以相同的方式测试所有道具，因此您可以使用循环。

const props = [
    'organizationName',
    'firstName',
    'middleInitial',
    'lastName',
    'email',
    'communicationEmail',
    'areaCode',
    'phoneNumber',
    'phoneType',
    'alternatePhoneAreaCode',
    'alternatePhoneNumber',
    'alternatePhoneType',
    'address1',
    'address2',
    'city',
    'state',
    'zip'
];

function getBodyFields(request:UpdateCustomerRequest, tmsGuid:string) {
    var obj = {
        tmsGuid: tmsGuid
    };
    props.forEach(p => {
        if (typeof request[p] === 'string' && request[p] !== '')
            obj[p] = request[p];
    });
    return obj;
}

最好检查该值是否为字符串，因为您的检查request.zip＆amp;＆amp; request.zip！==“”即使zip是true或1 1 或[1 ]或任何真实价值。