troubleshooting Question

Remove this useless object instantiation of class in C# or use it -- Help

Avatar of Natavia Finnie
Natavia FinnieFlag for United States of America asked on
C#* SonarQube
1 Comment1 Solution73 ViewsLast Modified:
I am not directly following why I am getting this as a Major Bug from my Sonar scan at both lines where this (new SynchronizationTokenMonitor...) is being called.

Result from SonarQube scan:
Either remove this useless object instantiation of class 'SynchronizationTokenMonitor' or use it.

here is the call that is coming from another method:
 AddSynchronizationTokenToOutboundPages(context);
Screenshot of bug
private static void AddSynchronizationTokenToOutboundPages(HttpContext context) {
           
            if (context.Handler != null) {
                if (context.Handler is Page){
                    new SynchronizationTokenMonitor(context.Handler as Page);
                }
                else if (context.Handler is TASK.Web.PageHandlerBase) {
                   new SynchronizationTokenMonitor(((TASK.Web.PageHandlerBase)context.Handler).PageControl as UserControl);
                }
            }
        }

       public class SynchronizationTokenMonitor 
      {

             public SynchronizationTokenMonitor(Page page)
             {
                     if (page != null) {
                          page.PreRender += PagePreRender;
                          page.Load += PageLoad;
                    }
              }

              public SynchronizationTokenMonitor(UserControl control)
             {
                    if (control != null)
                    {
                          control.PreRender += ControlPreRender;
                          control.Load += ControlLoad;                
                     }
              }
       }
        
Join the community to see this answer!
Join our exclusive community to see this answer & millions of others.
Unlock 1 Answer and 1 Comment.
Join the Community
Learn from the best

Network and collaborate with thousands of CTOs, CISOs, and IT Pros rooting for you and your success.

Andrew Hancock - VMware vExpert
See if this solution works for you by signing up for a 7 day free trial.
Unlock 1 Answer and 1 Comment.
Try for 7 days

”The time we save is the biggest benefit of E-E to our team. What could take multiple guys 2 hours or more each to find is accessed in around 15 minutes on Experts Exchange.

-Mike Kapnisakis, Warner Bros