Sonar

TimefoldAI · Oct 28, 2024 · 11df734 · 11df734
1 parent c163e9d
commit 11df734
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/core/src/main/java/ai/timefold/solver/core/api/move/SolutionView.java b/core/src/main/java/ai/timefold/solver/core/api/move/SolutionView.java
@@ -43,11 +43,11 @@ public interface SolutionView<Solution_> {
      * @param variableMetaModel Describes the variable whose value is to be read.
      * @param entity The entity whose variable is to be read.
      * @param index >= 0
-     * @return maybe null; the value of the variable on the entity at the index
+     * @return The value at the given index in the list variable.
      * @throws NullPointerException if the value of the list variable is null
      * @throws IndexOutOfBoundsException if the index is out of bounds
      */
-    <Entity_, Value_> @Nullable Value_ getValueAtIndex(
+    <Entity_, Value_> @NonNull Value_ getValueAtIndex(
             @NonNull PlanningListVariableMetaModel<Solution_, Entity_, Value_> variableMetaModel, @NonNull Entity_ entity,
             int index);
 

diff --git a/core/src/main/java/ai/timefold/solver/core/impl/move/director/MoveDirector.java b/core/src/main/java/ai/timefold/solver/core/impl/move/director/MoveDirector.java
@@ -102,7 +102,7 @@ public final <Entity_, Value_> Value_ getValue(
 
     @SuppressWarnings("unchecked")
     @Override
-    public final <Entity_, Value_> Value_ getValueAtIndex(
+    public final <Entity_, Value_> @NonNull Value_ getValueAtIndex(
             @NonNull PlanningListVariableMetaModel<Solution_, Entity_, Value_> variableMetaModel,
             @NonNull Entity_ entity, int index) {
         return (Value_) extractVariableDescriptor(variableMetaModel).getValue(entity).get(index);