Skip to content

Refactored design and implementation smells #612

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
roshni-joshi wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Refactored design and implementation smells #612

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
f8ae51b
refactor: Resolved Feature Envy design smell
Nov 21, 2023
00f457d
refactor: Resolved cyclic hierarchy smell, applied push down method r…
Nov 25, 2023
5b05dfe
refactor: Introduced explaining variable
Nov 25, 2023
0d6ba30
refactor: applied decompose conditional refactoring
Nov 26, 2023
c15c1bc
refactor: Renamed variable to more meaningful name
Nov 26, 2023
22d63f0
refactor: Applied extract method refactoring
Nov 26, 2023
7aac7e0
refactor: Applied extract class refactoring to move all generate days…
Nov 27, 2023
4dba1b3
refactor: Added required comments for CronParser
Nov 28, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
refactor: applied decompose conditional refactoring
  • Loading branch information
Roshni Joshi committed Nov 26, 2023
commit 0d6ba30343546848e602493260e281eff1abc6e1
8 changes: 7 additions & 1 deletion ...n/java/com/cronutils/model/field/expression/visitor/ValidationFieldExpressionVisitor.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ public Between visit(final Between between) {
this.checkUnsupportedChars(between);
preConditions(between);

if ((constraints.isStrictRange()) && between.getFrom() instanceof IntegerFieldValue && between.getTo() instanceof IntegerFieldValue) {
if (isIntegerStrictRange(between)) {
final int from = ((IntegerFieldValue) between.getFrom()).getValue();
final int to = ((IntegerFieldValue) between.getTo()).getValue();
if (from > to) {
Expand Down Expand Up @@ -120,4 +120,10 @@ protected boolean isDefault(final FieldValue<?> fieldValue) {
protected boolean isSpecialCharNotL(final FieldValue<?> fieldValue) {
return fieldValue instanceof SpecialCharFieldValue && !SpecialChar.L.equals(fieldValue.getValue());
}

private boolean isIntegerStrictRange(final Between between) {
return constraints.isStrictRange()
&& between.getFrom() instanceof IntegerFieldValue
&& between.getTo() instanceof IntegerFieldValue;
}
}