文章详情

背景

在计算机专业的面试中,面试官往往会通过一些具体的业务逻辑来考察者的编程能力、逻辑思维和解决能力。是一个典型的业务逻辑BUG我们将对其进行详细的分析和解答。

假设你正在开发一个电商网站的商品搜索功能,用户可以通过输入关键词来搜索商品。系统需要根据用户输入的关键词返回包含该关键词的商品列表。是一个简单的搜索函数实现,但存在一个BUG,请找出并解释这个BUG。

python

def search_products(products, keyword):

results = []

for product in products:

if keyword.lower() in product['name'].lower():

results.append(product)

return results

# 测试数据

products = [

{'name': 'Red T-shirt', 'price': 19.99},

{'name': 'Blue Jeans', 'price': 29.99},

{'name': 'Running Shoes', 'price': 49.99},

{'name': 'Dress', 'price': 39.99}

]

# 测试搜索

search_results = search_products(products, 'blue')

print(search_results)

分析

在上述代码中,`search_products` 函数接受一个商品列表 `products` 和一个关键词 `keyword`。函数的目标是返回一个包含所有包含关键词的商品的列表。代码创建一个空列表 `results`,遍历每个商品,检查商品名称是否包含关键词(不区分大小写)。包含,则将该商品添加到结果列表中。

BUG定位

在上述代码中,存在一个潜在的错误。当关键词为空字符串时,函数仍然会遍历整个商品列表,检查每个商品名称是否包含空字符串,这将导致不必要的计算和可能的性能。

解答过程

为了修复这个BUG,我们可以在函数开始时检查关键词是否为空。为空,则直接返回一个空列表,避免遍历商品列表。是修改后的代码:

python

def search_products(products, keyword):

if not keyword: # 检查关键词是否为空

return []

results = []

for product in products:

if keyword.lower() in product['name'].lower():

results.append(product)

return results

# 测试搜索

search_results = search_products(products, 'blue')

print(search_results)

测试与验证

修改后的代码在输入空字符串作为关键词时,将直接返回一个空列表,避免了不必要的遍历。下面是测试结果:

python

# 测试搜索空关键词

search_results_empty = search_products(products, '')

print(search_results_empty) # 输出:[]

# 测试搜索非空关键词

search_results = search_products(products, 'blue')

print(search_results) # 输出:[{'name': 'Blue Jeans', 'price': 29.99}]

通过以上分析和修改,我们成功修复了原代码中的BUG,并优化了函数的性能。在面试中,这样的可以帮助面试官了解者对编程细节的关注程度和对代码优化的理解。