Please cherry-pick:

- renamed local variable
- added comments

diff --git a/src/org/mxchange/jjobs/model/jobskill/JobPositionSkill.java b/src/org/mxchange/jjobs/model/jobskill/JobPositionSkill.java
index 872a383bcd84853bf60b8e979a4c5c88c9e5c206..e154a7a65025f9cb3e5300b5c75eed4cf48e1993 100644 (file)
--- a/src/org/mxchange/jjobs/model/jobskill/JobPositionSkill.java
+++ b/src/org/mxchange/jjobs/model/jobskill/JobPositionSkill.java
@@ -142,18 +142,21 @@ public class JobPositionSkill implements SkillableJobPosition {
                        return false;
                }
 
-               final SkillableJobPosition jobPosition = (SkillableJobPosition) object;
+               // Cast to wanted type
+               final SkillableJobPosition position = (SkillableJobPosition) object;
 
-               if (!Objects.equals(this.getSkillImportance(), jobPosition.getSkillImportance())) {
+               // Check all false conditions
+               if (!Objects.equals(this.getSkillImportance(), position.getSkillImportance())) {
                        return false;
-               } else if (!Objects.equals(this.getJobPosition(), jobPosition.getJobPosition())) {
+               } else if (!Objects.equals(this.getJobPosition(), position.getJobPosition())) {
                        return false;
-               } else if (!Objects.equals(this.getJobSkill(), jobPosition.getJobSkill())) {
+               } else if (!Objects.equals(this.getJobSkill(), position.getJobSkill())) {
                        return false;
-               } else if (!Objects.equals(this.getJobPositionSkillId(), jobPosition.getJobPositionSkillId())) {
+               } else if (!Objects.equals(this.getJobPositionSkillId(), position.getJobPositionSkillId())) {
                        return false;
                }
 
+               // Okay, entities are matching (not objects)
                return true;
        }