Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

NLog control to an existing RichTextBox Windows Form

Tags:

c#

nlog

Below is my NLog configuration, I want to load log into existing RichTextBox called rtMessage in Form1, but NLog will create a new windows with log message loaded into the RichTextBox:

 <targets>
    <target xsi:type="RichTextBox" name="m" layout="${longdate}|${level:uppercase=true}|${logger}|${message}" 
            controlName="rtMessage" formName="Form1" />
  </targets>

  <rules>
    <logger name="*" minlevel="Debug" writeTo="m" />
  </rules>

Thank you.

like image 469
Alvin Avatar asked Jul 24 '12 13:07

Alvin


4 Answers

I think you can find the answer to your issue on the NLog Codeplex forum, here.

If you initialize the static logger directly in the field declaration inside your Form1 form, the Form1 instance will not yet exist, and NLog will go on creating a new form for the RichTextBox target.

What you need to do is delay the initialization of the logger to a time when the Form1 instance is already initialized, for example in a Load event handler.

Here is an excerpt of functional code from the Codeplex issue:

public partial class Form1 : Form
{
    private static Logger logger;// = LogManager.GetCurrentClassLogger();

    public Form1()
    {
        InitializeComponent();
    }

    private void Form1_Load(object sender, EventArgs e)
    {
        logger = LogManager.GetCurrentClassLogger();
    }
}

To avoid unnecessary re-initialization, you might want to initialize logger only if it has not already been initialized, i.e.

    private void Form1_Load(object sender, EventArgs e)
    {
        if (logger == null) logger = LogManager.GetCurrentClassLogger();
    }
like image 88
Anders Gustafsson Avatar answered Nov 19 '22 13:11

Anders Gustafsson


1.normally init logger in winform1_Load while it have done InitializeComponent -> has init your own RichTextBox.

2.then makesure your RichTextBoxTarget's FormName and ControlName initialized ok. such as:

RichTextBoxTarget rtbTarget = new RichTextBoxTarget();
logConfig.AddTarget("richTextBox", rtbTarget);
rtbTarget.FormName = "frmScrapeAmazonProduct"; // your winform class name
rtbTarget.ControlName = "rtbLog"; // your RichTextBox control/variable name

more can refer my post

like image 1
crifan Avatar answered Nov 19 '22 14:11

crifan


Here are three tips to help you load log in existing RichTextBox.

  1. Ensure formName and controlName is consistent with the actual use
  2. Set allowAccessoryFormCreation="False" in NLog.config
  3. RichTextBoxTarget.ReInitializeAllTextboxes(this); in Form_Load

PS:Follow this way, you neeed to config your RichTextBoxTarget in NLog.config, please refer RictTextBoxTarget.

like image 1
MoonAche Avatar answered Nov 19 '22 15:11

MoonAche


Don't forget to add

LogManager.ReconfigExistingLoggers();

if setting the richtextbox as a logger programmatically

like image 1
BigChief Avatar answered Nov 19 '22 15:11

BigChief