我有以下方法
private void SetProcessDocumentStatus(string status) { var setStatusWith = new Action( statusValue => processDocumentStatusLabel.Text = statusValue); if (processDocumentStatusLabel.InvokeRequired) processDocumentStatusLabel.Invoke( (MethodInvoker)(() => setStatusWith(status))); else setStatusWith(status); }
从上面的代码中,我将动作封装到setStatusWith
.该行为是否应重构为另一种方法如下?
private void SetProcessDocumentStatusWith(string status) { processDocumentStatusLabel.Text = status; } private void SetProcessDocumentStatus(string status) { if (processDocumentStatusLabel.InvokeRequired) processDocumentStatusLabel.Invoke( (MethodInvoker)(() => SetProcessDocumentStatusWith(status))); else SetProcessDocumentStatusWith(status); }
我想知道是否应该在代码中谨慎使用"Action"委托.
你所拥有的东西对我来说似乎非常清楚.将lambda移动到单独的函数会增加代码行但不清晰.
我可能会把第一行写成:
ActionsetStatusWith = statusValue => processDocumentStatusLabel.Text = statusValue;
但我不知道哪种方式通常是首选.