计算机专业面试题：业务上BUG一条及解答

在一家电商公司，负责开发一款在线购物APP。用户反馈在订单结算环节出现了具体表现为：当用户选择使用优惠券进行结算时，系统会错误地显示“优惠券不可用”。经过检查，该优惠券在用户账户中是有效的，符合使用条件。是一个简化的代码片段，用于处理优惠券的使用逻辑：

python

def apply_coupon(order, coupon):

if coupon.is_valid and coupon.is_applicable(order):

discount = coupon.amount

order.total -= discount

return True

return False

# 示例数据

order = {

'total': 100.0,

'items': [{'price': 50.0}, {'price': 50.0}]

}

coupon = {

'is_valid': True,

'is_applicable': lambda order: sum(item['price'] for item in order['items']) >= 100,

'amount': 10.0

}

# 调用函数

success = apply_coupon(order, coupon)

print("优惠券使用成功:", success)

print("订单总金额:", order['total'])

请分析上述代码，找出导致“优惠券不可用”BUG的原因，并给出解决方案。

分析过程

我们需要明确BUG的表现：当优惠券有效且符合使用条件时，系统应正确应用优惠券并减少订单总金额。根据用户反馈，系统错误地显示“优惠券不可用”。

我们分析代码片段：

1. `apply_coupon`函数接收两个参数：`order`（订单信息）和`coupon`（优惠券信息）。

2. 函数内部检查优惠券的有效性，优惠券无效，则直接返回`False`。

3. 函数检查优惠券是否适用于当前订单。这里使用了一个lambda函数来判断订单总金额是否大于等于优惠券的面值。

4. 优惠券有效且适用，则应用优惠券，减少订单总金额，并返回`True`。

从代码逻辑上看，优惠券的有效性和适用性检查似乎是正确的。可能出几个方面：

– 优惠券对象的属性和方法实现可能存在错误。

– 订单对象的数据可能不准确或不符合预期。

– 优惠券的使用条件可能被错误地定义。

为了进一步分析，我们可以逐步检查：

1. 确保`coupon`对象中的`is_valid`属性正确反映了优惠券的有效性。

2. 确保`coupon`对象中的`is_applicable`方确实现了优惠券的使用条件。

3. 确保`order`对象中的数据准确无误。

解决方案

在进一步检查过程中，我们发现`coupon`对象的`is_applicable`方法中的lambda表达式存在一个逻辑错误。具体来说，lambda表达式应该返回一个布尔值，而不是一个条件判断。是修正后的代码：

python

def apply_coupon(order, coupon):

if coupon.is_valid and coupon.is_applicable(order):

discount = coupon.amount

order.total -= discount

return True

return False

# 示例数据

order = {

'total': 100.0,

'items': [{'price': 50.0}, {'price': 50.0}]

}

coupon = {

'is_valid': True,

'is_applicable': lambda order: sum(item['price'] for item in order['items']) >= 100,

'amount': 10.0

}

# 修正优惠券适用性检查

coupon['is_applicable'] = lambda order: sum(item['price'] for item in order['items']) >= 100

# 调用函数

success = apply_coupon(order, coupon)

print("优惠券使用成功:", success)

print("订单总金额:", order['total'])

经过修正后，代码应该能够正确地应用优惠券，并在控制台输出“优惠券使用成功: True”和“订单总金额: 90.0”。

通过这个案例，我们可以看到，即使是简单的逻辑错误也可能导致严重的业务。在开发过程中，我们需要仔细审查代码逻辑，确保每个细节都符合预期，从而避免类似的BUG发生。