ClicShopping #335

    // Keep highest priority, cancel others
    $kept = $objectives[0];
    $cancelled = [];
    for ($i = 1; $i < count($objectives); $i++) {
This loop uses a function. To avoid the overhead of executing the function n times, you should precalculate it before the loop.
Time to fix: about 15 minutes
Read doc
                Open Issue
             Permalink Copy Prompt

                                    Last edited Tue, 17 Feb 2026 13:08:46 +0000 by clicshopping
                    
      $obj = $objectives[$i];
      $registry->cancelObjective($obj->getId(), 'Cancelled due to conflict with higher priority objective');
      $cancelled[] = $obj->getId();
    }

    {
        // Find logical inconsistencies
        $inconsistencies = [];
        // Simple contradiction detection
        for ($i = 0; $i < count($statements); $i++) {
This loop uses a function. To avoid the overhead of executing the function n times, you should precalculate it before the loop.
Time to fix: about 15 minutes
Read doc
                Open Issue
             Permalink Copy Prompt

                                    Last edited Tue, 17 Feb 2026 13:08:46 +0000 by clicshopping
                    
            for ($j = $i + 1; $j < count($statements); $j++) {
                if ($this->areContradictory($statements[$i], $statements[$j])) {
                    $inconsistencies[] = [
                        'statement1' => $statements[$i],
                        'statement2' => $statements[$j],

        // Find logical inconsistencies
        $inconsistencies = [];
        // Simple contradiction detection
        for ($i = 0; $i < count($statements); $i++) {
            for ($j = $i + 1; $j < count($statements); $j++) {
This loop uses a function. To avoid the overhead of executing the function n times, you should precalculate it before the loop.
Time to fix: about 15 minutes
Read doc
                Open Issue
             Permalink Copy Prompt

                                    Last edited Tue, 17 Feb 2026 13:08:46 +0000 by clicshopping
                    
                if ($this->areContradictory($statements[$i], $statements[$j])) {
                    $inconsistencies[] = [
                        'statement1' => $statements[$i],
                        'statement2' => $statements[$j],
                        'type' => 'contradiction'