You are maintaining a Python script that calculates team bonuses based on sales data. The script reads a dictionary where keys are employee names and values are total sales (float). It then applies a 10% bonus if sales exceed 5000. The code snippet is:

def calculate_bonus(sales):
for name, value in sales.items():
if value 5000:
print(f"{name} gets bonus")

However, the manager wants the script to return a list of employees who qualify, not just print them. They also want to avoid side effects. What is the best way to modify this function?

Question

You are maintaining a Python script that calculates team bonuses based on sales data. The script reads a dictionary where keys are employee names and values are total sales (float). It then applies a 10% bonus if sales exceed 5000. The code snippet is:

def calculate_bonus(sales):
    for name, value in sales.items():
        if value > 5000:
            print(f"{name} gets bonus")

However, the manager wants the script to return a list of employees who qualify, not just print them. They also want to avoid side effects. What is the best way to modify this function?

Accepted Answer

Build a list inside the function and return it at the end.. It modifies the function to build a list of qualifying employee names inside the function and returns that list. This avoids side effects (no global variables, no mutation of the input dictionary) and follows the principle of returning results rather than printing them, making the function reusable and testable.

Answer

Create a global list variable at the top of the script and append each qualifying name to it.

Answer

Keep the function as is and have the caller capture the printed names by redirecting stdout.

Answer

Use the dictionary's update method to mark bonus status in the original sales dictionary.

PCEP Computer Programming and Python Fundamentals Practice Question

Why each option matters

Option-by-option breakdown

Related to this question