title: Cache load Method call in constructor to set timer, is it bad approach?
body: <p>In a C# project I have called a cache load method in constructor to set a timer. Is it a bad idea? </p>
  
  <pre><code>      public MyConstructor()
              {
                  cacheDataManager = new MemoryCacheManager();
  
                  var timer = new System.Threading.Timer(
                             e =&gt; LoadMyCacheData(),
                            null,
                            TimeSpan.Zero,
                           TimeSpan.FromSeconds(30));
              }
  
  //cache data method LoadMyCacheData
  </code></pre>