Рефакторинг кода #1

New Issue

amorozov · 2024-11-13T14:35:56+01:00

amorozov commented

2024-11-13 14:35:56 +01:00

Красивости

Константы

У тебя дублируются "магические строки". Замени их константами. Например тут и тут, а также тут.

Используй массивы

Тут у тебя очень некрасивое выражение в коде:

if (operation != a && operation != b && operation != c && operation != d && operation != e)

Ты еще не прошел коллекции и списки, так что используй массив. Пусть доступные операции будут храниться массивом, а ты при получении знака будешь проверять его наличие в этом самом массиве.

# Красивости ## Константы У тебя дублируются "магические строки". Замени их константами. Например [тут](https://git.tswf.io/dima-java-learn/calculator/src/commit/bf305d907e28675cf721d5ccb0824098144fa135/src/Main.java#L15-L19) и [тут](https://git.tswf.io/dima-java-learn/calculator/src/commit/bf305d907e28675cf721d5ccb0824098144fa135/src/Main.java#L26), а также [тут](https://git.tswf.io/dima-java-learn/calculator/src/commit/bf305d907e28675cf721d5ccb0824098144fa135/src/Main.java#L35). ## Используй массивы [Тут](https://git.tswf.io/dima-java-learn/calculator/src/branch/master/src/Main.java#L22) у тебя очень некрасивое выражение в коде: ```java if (operation != a && operation != b && operation != c && operation != d && operation != e) ``` Ты еще не прошел коллекции и списки, так что используй массив. Пусть доступные операции будут храниться массивом, а ты при получении знака будешь проверять его наличие в этом самом массиве.

dkamenski was assigned by amorozov

2024-11-13 14:39:51 +01:00

Sign in to join this conversation.

No Label

No Milestone

No project

No Assignees

1 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: dima-java-learn/calculator#1