Loading...
Loading...
Compare original and translation side by side
def calculate_total_price(items):
total = 0
for item in items:
total += item.price
return totaldef calculate_total_price(items):
return sum(item.price for item in items) if items else 0 or reduce(lambda x, y: x + y.price, items, 0)def calculate_total_price(items):
total = 0
for item in items:
total += item.price
return totaldef calculate_total_price(items):
return sum(item.price for item in items) if items else 0 or reduce(lambda x, y: x + y.price, items, 0)def process_user_registration():
user_data = validate_input()
user = create_user(user_data)
send_welcome_email(user)
return userdef process_user_registration():
if not email or '@' not in email: # Low-level validation
raise ValueError("Invalid email")
user = User.create(email) # High-level operation
smtp.send(email, "Welcome!") # Medium-level operationdef process_user_registration():
user_data = validate_input()
user = create_user(user_data)
send_welcome_email(user)
return userdef process_user_registration():
if not email or '@' not in email: # 低层级验证
raise ValueError("Invalid email")
user = User.create(email) # 高层级操作
smtp.send(email, "Welcome!") # 中层级操作undefinedundefinedundefinedundefinedundefinedundefinedundefinedundefinedclass ConfigManager:
def __init__(self, config_file):
with open(config_file) as f:
self.config = json.load(f)
def get(self, key):
return self.config.get(key)class AbstractConfigurationStrategy(ABC):
@abstractmethod
def load_configuration(self): pass
class JSONConfigurationStrategy(AbstractConfigurationStrategy):
# ... complex factory pattern implementation
class ConfigurationManager:
def __init__(self, strategy: AbstractConfigurationStrategy):
self.strategy = strategy
# ... unnecessary abstractions for a simple config readerclass ConfigManager:
def __init__(self, config_file):
with open(config_file) as f:
self.config = json.load(f)
def get(self, key):
return self.config.get(key)class AbstractConfigurationStrategy(ABC):
@abstractmethod
def load_configuration(self): pass
class JSONConfigurationStrategy(AbstractConfigurationStrategy):
# ... 复杂的工厂模式实现
class ConfigurationManager:
def __init__(self, strategy: AbstractConfigurationStrategy):
self.strategy = strategy
# ... 针对简单配置读取器的不必要抽象def calculate_discount(price: float, customer_type: str) -> float:
"""Calculate discount based on customer type.
Args:
price: Original price
customer_type: One of 'regular', 'premium', 'vip'
Returns:
Discounted price
"""
discounts = {'regular': 0, 'premium': 0.1, 'vip': 0.2}
return price * (1 - discounts.get(customer_type, 0))def calculate_discount(price: float, customer_type: str) -> float:
"""根据客户类型计算折扣。
参数:
price: 原价
customer_type: 可选值为'regular'、'premium'、'vip'
返回:
折扣后的价格
"""
discounts = {'regular': 0, 'premium': 0.1, 'vip': 0.2}
return price * (1 - discounts.get(customer_type, 0))user_count = len(users)
is_valid_email = validate_email(email)
for user_index in range(user_count):uc = len(u)
flag = check(e)
for i in range(uc):user_count = len(users)
is_valid_email = validate_email(email)
for user_index in range(user_count):uc = len(u)
flag = check(e)
for i in range(uc):is_has_can_is_has_can_def process_order(order):
if not order.is_valid():
return None
if not order.has_payment():
return None
return fulfill_order(order)def process_order(order):
if order.is_valid():
if order.has_payment():
# deep nesting continues...
return fulfill_order(order)
else:
return None
else:
return Nonedef process_order(order):
if not order.is_valid():
return None
if not order.has_payment():
return None
return fulfill_order(order)def process_order(order):
if order.is_valid():
if order.has_payment():
# 深层嵌套持续...
return fulfill_order(order)
else:
return None
else:
return Noneundefinedundefinedundefinedundefinedundefinedundefined
**Avoid obvious comments:**
```python
**应避免显而易见的注释:**
```python
**Avoid emoji decorations:**
```python
**应避免表情符号装饰:**
```pythonundefinedundefineddef test_should_reject_user_with_invalid_email():
invalid_email = "not-an-email"
with pytest.raises(InvalidEmailError):
create_user(email=invalid_email)def test_should_reject_user_with_invalid_email():
invalid_email = "not-an-email"
with pytest.raises(InvalidEmailError):
create_user(email=invalid_email)project/
├── src/
│ ├── main.py # Main code
│ └── utils.py # Helper functions
├── tests/
│ └── test_main.py # Tests
└── README.md # One README onlyproject/
├── src/
│ ├── main.py
│ └── utils.py
├── docs/
│ ├── CHANGES.md # Unnecessary
│ ├── SUMMARY.md # Unnecessary
│ └── API_GUIDE.md # Unnecessary
├── README.md
└── NOTES.md # Unnecessaryproject/
├── src/
│ ├── main.py # 主代码
│ └── utils.py # 辅助函数
├── tests/
│ └── test_main.py # 测试代码
└── README.md # 仅保留一个READMEproject/
├── src/
│ ├── main.py
│ └── utils.py
├── docs/
│ ├── CHANGES.md # 不必要
│ ├── SUMMARY.md # 不必要
│ └── API_GUIDE.md # 不必要
├── README.md
└── NOTES.md # 不必要