Skip to content

Commit d091631

Browse files
refactor code
1 parent 7e6e642 commit d091631

File tree

1 file changed

+12
-15
lines changed

1 file changed

+12
-15
lines changed

electronics/power_factor.py

+12-15
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,8 @@ def calculate_apparent_power(voltage, current):
66
try:
77
return voltage * current
88
except TypeError:
9-
raise ValueError(
10-
"Invalid input types for voltage or current. Both must be numbers."
11-
)
9+
error_msg = "Invalid input types for voltage or current. Both must be numbers."
10+
raise ValueError(error_msg)
1211

1312

1413
def calculate_power_factor(real_power, apparent_power):
@@ -18,9 +17,8 @@ def calculate_power_factor(real_power, apparent_power):
1817
raise ValueError("Apparent power cannot be zero.")
1918
return real_power / apparent_power
2019
except TypeError:
21-
raise ValueError(
22-
"Invalid input types for real power or apparent power. Both must be numbers."
23-
)
20+
error_msg = "Invalid input types for real power or apparent power. Both must be numbers."
21+
raise ValueError(error_msg)
2422

2523

2624
def calculate_reactive_power(real_power, apparent_power):
@@ -32,9 +30,8 @@ def calculate_reactive_power(real_power, apparent_power):
3230
)
3331
return math.sqrt(apparent_power**2 - real_power**2)
3432
except TypeError:
35-
raise ValueError(
36-
"Invalid input types for real power or apparent power. Both must be numbers."
37-
)
33+
error_msg = "Invalid input types for real power or apparent power. Both must be numbers."
34+
raise ValueError(error_msg)
3835
except ValueError as ve:
3936
raise ValueError(f"Calculation error: {ve}")
4037

@@ -44,14 +41,14 @@ def calculate_correction_capacitance(reactive_power, voltage, frequency=60):
4441
try:
4542
if voltage == 0:
4643
raise ValueError("Voltage cannot be zero.")
47-
capacitance = (reactive_power * 1_000_000) / (
48-
2 * math.pi * frequency * voltage**2
44+
capacitance = (
45+
(reactive_power * 1_000_000)
46+
/ (2 * math.pi * frequency * voltage**2)
4947
)
5048
return capacitance
5149
except TypeError:
52-
raise ValueError(
53-
"Invalid input types for reactive power, voltage, or frequency. They must be numbers."
54-
)
50+
error_msg = "Invalid input types for reactive power, voltage, or frequency. They must be numbers."
51+
raise ValueError(error_msg)
5552
except ValueError as ve:
5653
raise ValueError(f"Calculation error: {ve}")
5754

@@ -69,7 +66,7 @@ def main():
6966
reactive_power, voltage
7067
)
7168

72-
print(f"\nResults:")
69+
print("\nResults:")
7370
print(f"Power Factor: {power_factor:.4f}")
7471
print(f"Apparent Power: {apparent_power:.0f} VA")
7572
print(f"Reactive Power: {reactive_power:.0f} VAR")

0 commit comments

Comments
 (0)