は、同じような関数定義の重複コードを排除します。

Question

私はいくつかのロジック+基本システムコールで構成されたメソッドを持っています。まったく同じロジックを含み、基盤となるシステムコールの変更だけを含むもう1つの方法を実装する必要があります。

共通のコードを再利用する方法を考えています。readとrecvの呼び出しが異なるため、呼び出し元のシステムコールを呼び出しても成功しなかった別の方法を実装しようとしています。

同じような優雅なソリューションを見つけることは素晴らしいことでしょう。方法は次のようになり -

---

最初の関数

std::string Socket::read(const int bufSize) const 
{ 
    auto buffer = std::make_unique<char[]>(bufSize + 1); 
    auto recvd = 0, count = 0; 

    std::string str; 
    str.reserve(bufSize); 

    do { 

     // ONLY THIS PART IS DIFFERENT 
     recvd = ::read(sockfd, buffer.get() + count, bufSize - count); 
     // ONLY THIS PART IS DIFFERENT 

     count += recvd; 
     if (count == bufSize) { 
      str.append(buffer.get()); 
      str.reserve(str.length() + bufSize); 
      std::memset(buffer.get(), 0, bufSize); 
      count = 0; 
     } 
    } while (recvd > 0); 

    str.append(buffer.get(), count); 

    if (recvd == -1) { 
     // TODO: Check for recvd == EAGAIN or EWOULDBLOCK and 
     // don't throw exception in that case. 
     throw std::runtime_error("Error occurred while writing message"); 
    } 

    return str; 
} 

---

第二の機能

std::string Socket::recv(const int bufSize, SF::recv flags) const 
{ 
    auto buffer = std::make_unique<char[]>(bufSize + 1); 
    auto recvd = 0, count = 0; 

    std::string str; 
    str.reserve(bufSize); 

    do { 

     // ONLY THIS PART IS DIFFERENT 
     const auto f = static_cast<int>(flags); 
     recvd = ::recv(sockfd, buffer.get() + count, bufSize - count, f); 
     // ONLY THIS PART IS DIFFERENT 

     count += recvd; 
     if (count == bufSize) { 
      str.append(buffer.get()); 
      str.reserve(str.length() + bufSize); 
      std::memset(buffer.get(), 0, bufSize); 
      count = 0; 
     } 
    } while (recvd > 0); 

    str.append(buffer.get(), count); 

    if (recvd == -1) { 
     // TODO: Check for recvd == EAGAIN or EWOULDBLOCK and 
     // don't throw exception in that case. 
     throw std::runtime_error("Error occurred while writing message"); 
    } 

    return str; 
} 

Answer 1

のデフォルト値を提供することができます。これはあなたの正確なニーズに合わせて設計されていませんが、より柔軟なソリューションです。その結果、表現力が低下することがあります。

#include <utility> 

namespace detail { 

template <class Fn, class... Args> 
auto DuplicatedCode(Fn &&fn, Args&&... args) { 
    // some code 

    // auto result = 
    std::forward<Fn>(fn)(std::forward<Args>(args)...); 

    // more code 

    // return 
} 

} 

void foo() { 
    detail::DuplicatedCode([](){return 0;}); 
} 

void bar() { 
    detail::DuplicatedCode([](){return 1;}); 
} 

あなたはfooとbarにいくつかのローカル変数を宣言することができ、かつDucplicatedCodeはfnにそれらを転送するか、あなたは、単にこれらの変数をキャプチャすることができます。

Answer 2

私はstd::string Socket::read(const int bufSize) constとのあなたのバージョンを比較するつもりです

唯一の違いは、このようにあなたの第一バージョンがflagsの特定のセットを使用して第二版の呼び出しにリファクタリングすることができ

const auto f = static_cast<int>(flags); 

と

recvd = ::recv(sockfd, buffer.get() + count, bufSize - count, f); 

です。

それとも、あなたは、このようにそれを行うことができますC++ 14ではflags

よう

std::string Socket::recv(const int bufSize, SF::recv flags = DefaultFlags) const 

Answer 3

最も簡単な解決策は、読み取りメッセージが呼び出された場合、メソッドを合計し、無効なフラグを渡すことです：

if(flags==INVALID) 
     recvd = ::read(sockfd, buffer.get() + count, bufSize - count); 
    else 
     recvd = ::recv(sockfd, buffer.get() + count, bufSize - count, f); 

しかし、この方法は現在2人の責任と二つの理由を持っているので、これは、Single Responsibility Principleに違反することになります変更する。

もっと良い解決策は、両方の方法の共通部分を抽出することです。

read() { 
commonMethod1(); 
::read(); 
commonMethod2(); 
} 
write() { 
commonMethod1(); 
::read(); 
commonMethod2(); 
} 

あなたの質問はおそらく異なる意見に基づいていますが、これは私のものです。;-)

Answer 4

1つの簡単な方法は、

class Socket 
{ 
    // everything else you already have 

    private: 

     std::string recv_helper(int bufSize, const SF::recv *flags = nullptr) const; 
       // note the second argument is a pointer 
};  

Socketクラス内のプライベートヘルパー関数を追加し、

std::string Socket::recv_helper(int bufSize, const SF::recv *flags) const 
{ 
    // All the code preceding your first // ONLY THIS PART IS DIFFERENT 

    if (flags) 
    { 
     recvd = ::recv(sockfd, buffer.get() + count, bufSize - count, 
         static_cast<int>(*flags)); 
    } 
    else 
    { 
     recvd = ::read(sockfd, buffer.get() + count, bufSize - count); 
    } 

     // All the code following your second // ONLY THIS PART IS DIFFERENT  
} 

としてそれを実装するために次にあなたがする必要があるすべてはあなたの二つの機能を再実装されていますヘルパーに電話する。

std::string Socket::read(int bufSize) const 
{ 
    return recv_helper(bufSize); 
} 

std::string Socket::recv(int bufSize, SF::recv flags) const 
{ 
    return recv_helper(bufSize, &flags); 
} 

また、冗長const修飾子を値渡しの引数から削除しました。