Codice di refactoring con solo poche righe diverse

Prima di entrare nella borsa del "moderno C ++", manteniamo le cose semplici e cominciamo con alcune tecniche classiche, agnostiche del linguaggio. Una soluzione è creare una funzione del modulo

   void myfunc(int i,bool upward)
   {
        identical_lines
        identical_lines
        identical_lines
        if(upward)
           first_for_specific_line
        else
           second_for_specific_line
        //...
    }

e chiamalo come

 for (int i=0; i < MAX; ++i)
    myfunc(i,true);

  for (int i=MAX - 4; i > 0; --i) 
    myfunc(i,false);

Si noti che questa non è sempre la soluzione migliore dal momento che tende a rendere myfunc facendo "troppo", forse violando il principio di responsabilità unica. Spesso è meglio rifattorizzare ogni blocco che hai contrassegnato come identical_lines in una funzione piccola da sola (chiamiamolo block1(int i) , block2(int i) e crea 2 funzioni

   myfunc1(int i)
   {
        block1(i);
        first_for_specific_line;
        block2(i);
        // ...
   }

e

   myfunc2(int i)
   {
        block1(i);
        second_for_specific_line;
        block2(i);
        // ...
   }

e usalo come

  for (int i=0; i < MAX; ++i)
    myfunc1(i);

  for (int i=MAX - 4; i > 0; --i) 
    myfunc2(i);

Tuttavia, decidere quale sia il modo "migliore" per ridurre i blocchi di codice a piccole funzioni dipende da ciò che i blocchi sono veramente facendo. È meglio costruire astrazioni non principalmente su criteri formali come "i blocchi di codice sembrano simili", ma su "qual è il compito o lo scopo di questo blocco. Chiediti quali delle linee appartengono insieme per adempiere a tale scopo e se puoi caratterizzare quell'attività con un nome unico e accurato.