saleOutBounds.sort((a, b) => { if (a.bill_date > b.bill_date) { return -1 // eslint-disable-next-line no-self-compare } else if (a.bill_date === a.bill_date) { if (a.id < b.id) { return -1 } else if (a.id === b.id) { return 1 } else { return 0 } } else { return 1 } }) 排序失效

这段代码排序失效的原因可能是比较函数中的第二个判断条件有误，应该是 `a.bill_date === b.bill_date` 而不是 `a.bill_date === a.bill_date`，因为 a.bill_date 等于自己本身是一定成立的。如果条件错误，会导致 id 的排序不正确，从而使整个排序失效。你可以将代码修改为如下：

saleOutBounds.sort((a, b) => {
  if (a.bill_date > b.bill_date) {
    return -1
  } else if (a.bill_date === b.bill_date) {
    if (a.id < b.id) {
      return -1
    } else if (a.id === b.id) {
      return 0
    } else {
      return 1
    }
  } else {
    return 1
  }
})

这样修改之后，应该就能够正确地排序了。

相关问题

saleOutBounds.sort((a, b) => { if (a.bill_date > b.bill_date) { return -1 // eslint-disable-next-line no-self-compare } else if (a.bill_date === a.bill_date) { if (a.id < b.id) { return -1 } else if (a.id === b.id) { return 1 } else { return 0 } } else { return 1 } })

这段代码是什么意思？

这段代码实现了一个数组对象的排序，数组中的元素包含两个属性：bill_date和id。首先按照bill_date降序排列，如果bill_date相等，则按照id升序排列。这段代码使用了Array.sort()函数和一个比较函数作为参数。比较函数中使用了条件语句和比较运算符实现了元素的排序逻辑。

const processSaleBoundArr = async (saleBoundArr, tokenData, accountData) => { let saleOutBounds = []; if (saleBoundArr && saleBoundArr.length) { const customerIds = [...new Set(saleBoundArr.map(item => item.customer_id))]; const customerPromises = customerIds.map(id => { return getCustomerDetail({ id }, { "app-token": tokenData['app-token'], "X-GW-Router-Addr": accountData?.domain }); }); const customerDetails = await Promise.all(customerPromises); saleOutBounds = saleBoundArr.map(saleBound => { const customerDetail = customerDetails.find(detail => detail.data.errcode === 0 && detail.data.data.id === saleBound.customer_id); if (customerDetail) { const customerData = customerDetail.data.data; saleBound.account_open_addr = customerData.account_open_addr; saleBound.addr = customerData.addr; saleBound.bank = customerData.bank; saleBound.bank_account = customerData.bank_account; saleBound.taxpayer_no = customerData.taxpayer_no; saleBound.orgName = customerData.sale_dept_name; saleBound = Object.assign(saleBound, customerData.custom_field); } return saleBound; }); saleOutBounds.sort((a, b) => { if (a.bill_date > b.bill_date) { return -1 } else if (a.bill_date === b.bill_date) { if (a.id < b.id) { return -1 } else if (a.id === b.id) { return 0 } else { return 1 } } else { return 1 } }) } return saleOutBounds; }; 优化这段代码

这段代码可以进行以下优化：

1. 使用对象解构语法简化代码，可以将 tokenData 和 accountData 的属性直接解构出来，方便后面的使用。

2. 使用 Promise.allSettled() 方法替换 Promise.all() 方法，可以在其中使用 try-catch 捕获错误，确保即使其中某一个 Promise 出错也不会影响其他 Promise 的执行。

3. 在 Promise.allSettled() 方法的结果中遍历每一个 Promise 的结果，判断结果是否成功，根据结果来确定是否需要将该元素加入 saleOutBounds 数组。

4. 使用 const 替换 let 声明 saleOutBounds 变量，因为该变量在函数中只被赋值一次，不需要修改。

5. 简化代码逻辑，将排序的回调函数中的相同日期的判断语句改为比较运算符，可以减少代码量，提高可读性。

优化后的代码如下所示：

const processSaleBoundArr = async (saleBoundArr, { "app-token": appToken }, { domain } = {}) => {
  const saleOutBounds = [];

  if (saleBoundArr?.length) {
    const customerIds = [...new Set(saleBoundArr.map(({ customer_id }) => customer_id))];
    const customerPromises = customerIds.map(id => getCustomerDetail({ id }, { "app-token": appToken, "X-GW-Router-Addr": domain }));

    const customerDetails = await Promise.allSettled(customerPromises);

    saleBoundArr.forEach((saleBound, index) => {
      const customerDetail = customerDetails[index].status === 'fulfilled' && customerDetails[index].value.data.errcode === 0 && customerDetails[index].value.data.data.id === saleBound.customer_id;

      if (customerDetail) {
        const { data: customerData } = customerDetails[index].value;

        saleOutBounds.push({
          ...saleBound,
          account_open_addr: customerData.account_open_addr,
          addr: customerData.addr,
          bank: customerData.bank,
          bank_account: customerData.bank_account,
          taxpayer_no: customerData.taxpayer_no,
          orgName: customerData.sale_dept_name,
          ...customerData.custom_field
        });
      }
    });

    saleOutBounds.sort((a, b) => {
      if (a.bill_date > b.bill_date) {
        return -1;
      } else if (a.bill_date === b.bill_date) {
        return a.id - b.id;
      }
      return 1;
    });
  }

  return saleOutBounds;
};