Forum Discussion

chris_rothermel's avatar
chris_rothermel
Contributor
2 years ago

Referencing C# Workspace Assemblies from VB.Net, Warning Access of ... not evaluated

I found a C# code library and I'd like to use this class in VB.Net.  Here are the steps I've taken so far.

1. In my VB.Net Rule I've referenced the C# assemblies.

2. In my VB.Net Rule I've created a new object for the C# class

 'Create an object for the class defined in External Assembly (Toolbox) so we can use methods in the class
Dim XIRRWrapper As New OneStream.BusinessRule.Finance.XIRR.CalculationWrapper

3. Now I'm able to create objects in the C# referenced assemblies in VB.Net

Dim cashFlows = New List(Of OneStream.BusinessRule.Finance.XIRR.CashFlowDates ) From 
{
New OneStream.BusinessRule.Finance.XIRR.CashFlowDates(-1000, New DateTime(2017, 1, 1)),
New OneStream.BusinessRule.Finance.XIRR.CashFlowDates(500, New DateTime(2017, 7, 1)),
New OneStream.BusinessRule.Finance.XIRR.CashFlowDates(507.5, New DateTime(2018, 1, 1))
}

4. And finally able to run a function (LINE 55 which produces a warning)

api.LogMessage( XIRRWrapper.XIRR(cashFlows, 6).ToString() )

I get the correct result.  However I don't like this warning.  How can I write the code better to eliminate this warning?

SUCCESS
Business Rule 'XIRRusage' was compiled successfully with warnings.

1) Warning at line 55: Access of shared member, constant member, enum member or nested type through an instance; qualifying expression will not be evaluated.

 

NOTE:  The library being used is for the XIRR function.  https://github.com/klearlending/XIRR#xirr

 

Finance
  • RobbSalzmann's avatar
    RobbSalzmann
    2 years ago

    Hi Chris,

    It doesn't matter that your library is written in C#.  Once compiled, .net is .net.
    The error is telling you that you are accessing the shared (static) method XIRR through in instance of CalculationWrapper  - created when you used the New keyword.
     Try this:

     

     

     'Create an object for the class defined in External Assembly (Toolbox) so we can use methods in the class
'** Commented out.  Don't instantiate to use a static method.
'Dim XIRRWrapper As New OneStream.BusinessRule.Finance.XIRR.CalculationWrapper
Dim cashFlows = New List(Of OneStream.BusinessRule.Finance.XIRR.CashFlowDates ) From 
{
   New OneStream.BusinessRule.Finance.XIRR.CashFlowDates(-1000, New DateTime(2017, 1, 1)),
   New OneStream.BusinessRule.Finance.XIRR.CashFlowDates(500, New DateTime(2017, 7, 1)),
   New OneStream.BusinessRule.Finance.XIRR.CashFlowDates(507.5, New DateTime(2018, 1, 1))
}
api.LogMessage(OneStream.BusinessRule.Finance.XIRR.CalculationWrapper.XIRR(cashFlows, 6).ToString())

     

     

     

  • chris_rothermel's avatar
    chris_rothermel
    2 years ago

    Hi Robb,

    Thank you!  The warning is gone.  Business Rule 'XIRRusage' was compiled successfully.

    api.LogMessage( OneStream.BusinessRule.Finance.XIRR.CalculationWrapper.XIRR(cashFlows, 6).ToString() )

     

  • RobbSalzmann's avatar
    RobbSalzmann
    Valued Contributor II
    2 years ago

    Hi Chris,

    It doesn't matter that your library is written in C#.  Once compiled, .net is .net.
    The error is telling you that you are accessing the shared (static) method XIRR through in instance of CalculationWrapper  - created when you used the New keyword.
     Try this:

     

     

     'Create an object for the class defined in External Assembly (Toolbox) so we can use methods in the class
'** Commented out.  Don't instantiate to use a static method.
'Dim XIRRWrapper As New OneStream.BusinessRule.Finance.XIRR.CalculationWrapper
Dim cashFlows = New List(Of OneStream.BusinessRule.Finance.XIRR.CashFlowDates ) From 
{
   New OneStream.BusinessRule.Finance.XIRR.CashFlowDates(-1000, New DateTime(2017, 1, 1)),
   New OneStream.BusinessRule.Finance.XIRR.CashFlowDates(500, New DateTime(2017, 7, 1)),
   New OneStream.BusinessRule.Finance.XIRR.CashFlowDates(507.5, New DateTime(2018, 1, 1))
}
api.LogMessage(OneStream.BusinessRule.Finance.XIRR.CalculationWrapper.XIRR(cashFlows, 6).ToString())

     

     

     

  • chris_rothermel's avatar
    chris_rothermel
    Contributor
    2 years ago

    Hi Robb,

    Thank you!  The warning is gone.  Business Rule 'XIRRusage' was compiled successfully.

    api.LogMessage( OneStream.BusinessRule.Finance.XIRR.CalculationWrapper.XIRR(cashFlows, 6).ToString() )

     

  • cbriscoe's avatar
    cbriscoe
    Contributor
    12 months ago

    Hi Chris_Rothermel

    Have you done a lot of these refs in your application? Have you seen performance issues as a result of reference C# in VB.NET codes? Cheers

    • chris_rothermel's avatar
      chris_rothermel
      Contributor
      12 months ago

      Hi cbriscoe,

      I've used the direct reference to functions to simplify my code in other calculations.  I've seen no performance differences between coding in C# versus VB.Net.  The biggest single performance issue I've seen is when people code and then loop through multiple hierarchies looking for data intersections.  Much better is to use and understand Data Buffer calculations (including unbalanced data buffer arithmetic which needs some improvements in usability).

      I went another direction on this XIRR journey -- one that was simpler to code and uses the Microsoft IRR function as the solution basis.  I posted that code in the forum.

      OneStream.BusinessRule.Finance.XIRR.Function here...

       OneStream business rules is my favorite part of the platform and I'm learning more about it everyday.

      • GregHertling's avatar
        GregHertling
        New Contributor III
        8 months ago

        hi Chris, I am in need of similar code for XIRR, can you point me to the post that you mention? I'm not seeing the link.  thanks, Greg

        "I went another direction on this XIRR journey -- one that was simpler to code and uses the Microsoft IRR function as the solution basis.  I posted that code in the forum."

  • cbriscoe's avatar
    cbriscoe
    Contributor
    8 months ago

    Chris, I like your code, Is all good now? Yes? Cheers to Jack.