[ 
https://issues.apache.org/jira/browse/STONEHENGE-81?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12730606#action_12730606
 ] 

Avantika Agrawal commented on STONEHENGE-81:
--------------------------------------------

Okay great so I have eliminated the mapping and as a result considerably shrunk 
the Utility.cs and Settings.cs classes. Also, it will be a relief that we no 
longer have those switch statements to modify when adding new endpoints.

Another question: I am trying to eliminate the hardcoded assemblyname 
"Trade.DALSQLServer" using an SQLHelper method : 
GetAssemblyNameFromDBName(string DBName) 
What should this return for MySQL? SQLExpress?

> Code Cleanup for the ConfigService
> ----------------------------------
>
>                 Key: STONEHENGE-81
>                 URL: https://issues.apache.org/jira/browse/STONEHENGE-81
>             Project: Stonehenge
>          Issue Type: Improvement
>            Reporter: Avantika Agrawal
>            Priority: Trivial
>
> There is some unused code in Utility.cs, Settings.cs and the config files of 
> .NET StockTrader. This should be removed. In addition -we should try to 
> refactor the code to simplify the process of adding new endpoints - by 
> eliminating some switch statements, etc.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to