2

「DownloadManagerWrapper」というクラスを作成しました。これは、実行時の引数に基づいて呼び出されます。以下のコードの設計パターンに従って、クリーンで構造化されたコードを作成できますか。可能であれば、どの設計パターンを使用できますか?私はデザインパターンに慣れていないので、可能であればデザインパターンコードと簡単な説明のアドバイスとサンプルを提供してください。

public class DownloadManagerWrapper {

    public static void main(String[] args) {
    String downloadId = args[0];
    String bankId =args[1];
    String serviceId =args[2];
    readInterfaceConfig(serviceId,bankId,downloadId);
    }

    public static void readInterfaceConfig(String intfCode,String bankId,String downloadId) {

    Connection conn = CRMConnection.getConnection();

    String statusOfDownload;
    DownloadManager dwlManager = new DownloadManager();

    ResultSet rs=null;

    if(downloadId.equals("A"))
    {
    if(intfCode.equals("-1")) //all download
    {
        String query = "select INTF_CODE from CRMUSER.TABLE1 where bank_id ="+"'"+bankId+"'"+"order by INTF_CODE";
        System.out.println("query>"+query);
        try
        {
        Statement st = conn.createStatement();
        rs = st.executeQuery(query);
                while(rs.next())
             {
                    String intfCodeFromDB = rs.getString("INTF_CODE");
                    statusOfDownload = new DownloadManager().downloadDataFromCore(intfCodeFromDB,bankId,downloadId);
             }
        }
        catch(SQLException e)
        {
        e.printStackTrace();
        }
        finally
        {
        try {
            rs.close();
        } catch (SQLException e) {
            e.printStackTrace();
        }
        }
    }
    else  //specific download
    {
          statusOfDownload = dwlManager.downloadDataFromCore(intfCode,bankId,downloadId);
    }
    }
    else if(downloadId.equals("I"))
    {
           if(intfCode.equals("-1")) //all download
        {
            String query = "select INTF_CODE from CRMUSER.TABLE1 where bank_id ="+"'"+bankId+"'"+"order by INTF_CODE";
            System.out.println("query>"+query);
            try
            {
            Statement st = conn.createStatement();
            rs = st.executeQuery(query);
                    while(rs.next())
                 {
                        String intfCodeFromDB = rs.getString("INTF_CODE");
                        statusOfDownload = new DownloadManager().downloadDataFromCore(intfCodeFromDB,bankId,downloadId);
                 }
            }
            catch(SQLException e)
            {
            e.printStackTrace();
            }
            finally
            {
            try {
                rs.close();
            } catch (SQLException e) {
                e.printStackTrace();
            }
            }
        }
        else  //specific download
        {
              statusOfDownload = dwlManager.downloadDataFromCore(intfCode,bankId,downloadId);
        }
    }
    }
}
4

2 に答える 2

0

いくつかのポインタ:

  • DB へのアクセスは、Data Access Object (DAO) パターンを使用する必要があります

  • if 文が多い場合は、 Strategy Patternを参照してください。

于 2013-02-25T12:47:39.877 に答える
0

前に述べたように、戦略パターンは良い方法かもしれません。あなたのクラスにはメソッドが 1 つしかないことに気付きました。これは、コマンド パターンを実装する良い機会になる可能性があります。

コマンド パターンを使用すると、機能を必要とするクラスから機能を分離できます。したがって、この関数を頻繁に実行していることに気付いた場合、これは望ましくないコードを回避するのに非常に役立ちます。

于 2013-02-25T13:39:15.920 に答える