Skip to content

Commit 3b65598

Browse files
committed
add more detailed logs in infrastructure methods (GetLabel, GetImage, GetSupertip). #83
1 parent 05e59b2 commit 3b65598

File tree

1 file changed

+50
-4
lines changed

1 file changed

+50
-4
lines changed

NavfertyExcelAddIn/NavfertyRibbon.cs

Lines changed: 50 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
using System;
22
using System.Collections.Generic;
3+
using System.Diagnostics;
34
using System.Diagnostics.CodeAnalysis;
45
using System.Drawing;
56
using System.IO;
@@ -64,7 +65,13 @@ public class NavfertyRibbon : IRibbonExtensibility, IDisposable
6465
#region IRibbonExtensibility
6566
public string GetCustomUI(string ribbonID)
6667
{
67-
return GetResourceText();
68+
var sw = Stopwatch.StartNew();
69+
logger.Debug($"{nameof(GetCustomUI)} - {ribbonID}");
70+
71+
var customUiXml = GetResourceText();
72+
73+
logger.Debug($"{nameof(GetCustomUI)} got xml for {sw.Elapsed}");
74+
return customUiXml;
6875
}
6976
#endregion
7077

@@ -391,15 +398,54 @@ public void ValidateXml(IRibbonControl ribbonControl)
391398
#region Utils
392399
public string GetLabel(IRibbonControl ribbonControl)
393400
{
394-
return RibbonLabels.ResourceManager.GetString(ribbonControl.Id);
401+
var sw = Stopwatch.StartNew();
402+
logger.Debug($"{nameof(GetLabel)} - {ribbonControl?.Id}");
403+
try
404+
{
405+
var label = RibbonLabels.ResourceManager.GetString(ribbonControl.Id);
406+
logger.Debug($"{nameof(GetLabel)} - {ribbonControl?.Id}: '{label}'. Elapsed {sw.Elapsed}");
407+
return label;
408+
}
409+
catch (Exception ex)
410+
{
411+
logger.Error($"{nameof(GetLabel)} - {ribbonControl?.Id} error after {sw.Elapsed}:");
412+
logger.Error(ex);
413+
return ribbonControl?.Id;
414+
}
395415
}
396416
public Bitmap GetImage(string imageName)
397417
{
398-
return (Bitmap)RibbonIcons.ResourceManager.GetObject(imageName);
418+
var sw = Stopwatch.StartNew();
419+
logger.Debug($"{nameof(GetImage)} - {imageName}");
420+
try
421+
{
422+
var bitmap = (Bitmap)RibbonIcons.ResourceManager.GetObject(imageName);
423+
logger.Debug($"{nameof(GetImage)} - {imageName}: '{bitmap?.Size}'. Elapsed {sw.Elapsed}");
424+
return bitmap;
425+
}
426+
catch (Exception ex)
427+
{
428+
logger.Error($"{nameof(GetImage)} - {imageName} error after {sw.Elapsed}:");
429+
logger.Error(ex);
430+
return null;
431+
}
399432
}
400433
public string GetSupertip(IRibbonControl ribbonControl)
401434
{
402-
return RibbonSupertips.ResourceManager.GetString(ribbonControl.Id);
435+
var sw = Stopwatch.StartNew();
436+
logger.Debug($"{nameof(GetSupertip)} - {ribbonControl?.Id}");
437+
try
438+
{
439+
var superTip = RibbonSupertips.ResourceManager.GetString(ribbonControl.Id);
440+
logger.Debug($"{nameof(GetSupertip)} - {ribbonControl?.Id}: '{superTip}'. Elapsed {sw.Elapsed}");
441+
return superTip;
442+
}
443+
catch (Exception ex)
444+
{
445+
logger.Error($"{nameof(GetSupertip)} - {ribbonControl?.Id} error after {sw.Elapsed}:");
446+
logger.Error(ex);
447+
return ribbonControl?.Id;
448+
}
403449
}
404450

405451
private T GetService<T>()

0 commit comments

Comments
 (0)