Ordinamento di un vettore in ordine decrescente in due intervalli

Di 'che ho un vettore di numeri interi:

std::vector<int> indices;
for (int i=0; i<15; i++) indices.push_back(i);

Quindi lo ordino in ordine decrescente:

sort(indices.begin(), indices.end(), [](int first, int second) -> bool{return indices[first] > indices[second];})
for (int i=0; i<15; i++) printf("%i\n", indices[i]);

Questo produce quanto segue:

14
13
12
11
10
9
8
7
6
5
4
3
2
1
0

Ora voglio avere i numeri 3, 4, 5 e 6 da spostare fino alla fine e mantenere l'ordine discendente per loro (preferibilmente senza doverlo usare sortper la seconda volta). Cioè, ecco quello che voglio:

14
13
12
11
10
9
8
7
2
1
0
6
5
4
3

Come devo modificare la funzione di confronto di std::sortper ottenere ciò?

La tua funzione di confronto è sbagliata poiché i valori che ottieni come firste secondsono gli elementi di std::vector. Pertanto, non è necessario utilizzarli come indici. Quindi, devi cambiare

return indices[first] > indices[second];

per

return first > second;

Ora, riguardo al problema che provi a risolvere ...

Puoi lasciare 3, 4, 5 e 6 fuori dal confronto con altri elementi e comunque confrontarli tra loro:

std::sort(
    indices.begin(), indices.end(),
    [](int first, int second) -> bool {
        bool first_special = first >= 3 && first <= 6;
        bool second_special = second >= 3 && second <= 6;
        if (first_special != second_special)
            return second_special;
        else
            return first > second;
    }
);

dimostrazione

Funzioni dalla libreria di algoritmi standard come iota, sort, find, rotatee copyrenderebbe la vita più facile. Il tuo esempio si riduce a:

#include <iostream>
#include <vector>
#include <numeric>
#include <algorithm>
#include <iterator>


int main()
{
  std::vector<int> indices(15);
  std::iota(indices.begin(), indices.end(), 0);
  std::sort(indices.begin(), indices.end(), std::greater<>());

  auto a = std::find(indices.begin(), indices.end(), 6);
  auto b = std::find(indices.begin(), indices.end(), 3);
  std::rotate(a, b + 1, indices.end());

  std::copy(indices.begin(), indices.end(), std::ostream_iterator<int>(std::cout, "\n"));
  return 0;
}

Produzione:

14
13
12
11
10
9
8
7
2
1
0
6
5
4
3

auto a = std::lower_bound(indices.begin(), indices.end(), 6, std::greater<int>{});
auto b = a + 4;

Soluzione 1

Approccio diretto con un comparatore non lineare .

inline constexpr bool SpecialNumber(const int n) noexcept {
  return n < 7 && 2 < n;
}

void StrangeSortSol1(std::vector<int>* v) {
  std::sort(v->begin(), v->end(), [](const int a, const int b) noexcept {
    const bool aSpecial = SpecialNumber(a);
    const bool bSpecial = SpecialNumber(b);

    if (aSpecial && bSpecial) return b < a;
    if (aSpecial) return false;
    if (bSpecial) return true;
    return b < a;
  });
}

Soluzione 2

Usando std::algorithms (partizione)!

inline constexpr bool SpecialNumber(const int n) noexcept {
  return n < 7 && 2 < n;
}

void StrangeSortSol2(std::vector<int>* v) {
  auto pivot = std::partition(v->begin(), v->end(), std::not_fn(SpecialNumber));
  std::sort(v->begin(), pivot, std::greater{});
  std::sort(pivot, v->end(), std::greater{});
}

Considerazioni sulle prestazioni

Potrebbe sembrare che la seconda soluzione sia più lenta a causa del sovraccarico della partizione. Probabilmente no, a causa della previsione cache e miss-branch nei processori moderni.

Prova delle prestazioni