Est-il possible de réécrire cela comme un seul paquebot?

Question

Ceci est le code actuel.Ici j'appelle for_each pour exécuter la fonction montrée dans sum.Can cela peut être réduit à l'écriture de la fonction à l'intérieur de l'instruction for_each elle-même?

int S; 
struct sum 
{ 
sum(int& v): value(v){} 
void operator()(int data)const {value+=(int)((data+(data%S==0?0:S))/S)+2;} 
int& value; 
}; 
int main() 
{ 
    int cnt=0; 
    S=5; 
    for_each(flights.begin(),flights.end(),sum(cnt)); 
    cout<<cnt-2; 
    return 0; 
} 

Answer 1

En C++ 11, vous pouvez, en utilisant lambdas avec une capture:

int main() 
{ 
    int cnt = 0; 
    S = 5; 
    for_each(
    flights.begin(), flights.end(), 
    [&] (int data) { 
     cnt += (data + (data % S == 0 ? 0 : S))/S + 2; 
    }); 
    cout << cnt - 2; 
    return 0; 
} 

La façon dont cela se lit comme suit est la suivante: vous avez une fonction anonyme prenant un argument int, qui capture ([&]) le contexte environnant par référence. Notez que c'est aussi efficace que votre solution puisque le compilateur crée effectivement la structure du lambda pour vous.

- Comme Navaz l'a noté dans ses commentaires, la distribution est en réalité inutile. De plus, les lancers de style C sont généralement considérés comme obsolètes - utilisez plutôt les distributions C++.

Answer 2

Vous pouvez utiliser std::accumulate de <numeric> qui exprime parfaitement votre intention de construire une somme.

int main() 
{ 
    int flights[] = {1, 2, 3}; 
    int S = 5; 
    std::cout << std::accumulate(
     begin(flights), end(flights), 0, 
     [S] (int sum, int data) { return sum + (int)((data+(data%S==0?0:S))/S)+2; }) 
     - 2; 
} 

Bien que vous ayez demandé une seule doublure, je préfère donner à ce lambda non trivial un nom pour augmenter la lisibilité.

int main() 
{ 
    int flights[] = {1, 2, 3}; 
    auto theFooBarSum = [] (int sum, int data) 
     { return sum + (int)((data+(data%5==0?0:5))/5)+2; }; 
    int initialValue = 0; 

    std::cout << std::accumulate(
     begin(flights), end(flights), initialValue , theFooBarSum) - 2; 
}