feat(ErrorLogger): Enhance the global exception component function

[izanhzh]

Is there an existing issue for this?

• I have searched the existing issues

Is your feature request related to a problem? Please describe the problem.

我在一个项目中参照了ErrorLogger的实现，但是最终效果不太符合我预期，我也看到有人遇到了类似的问题#217
我希望能够增强ErrorLogger的全局异常处理能力：

1. 能够自动处理类似OnAfterRenderAsync等这种生命周期中触发的异常，这种异常会导致页面重新渲染，重新触发异常，不断循环
2. 能够自动处理弹窗内的错误，弹窗出现错误也会导致不断循环

Describe the solution you'd like

经过一番折腾，我能够明白这是Blazor自身全局处理不够强导致，ErrorBoundary 基本不可能做到这种支持，最终我选择了一个妥协一点的方案：

1. 检测是否触发循环错误渲染，如果是则采用默认的方案，进入错误页面，打断循环，OnAfterRenderAsync这些生命周期中，应当尽量避免异常，或者手动处理异常，但是如果我们想要全局日志记录等，还是需要封装组件统一处理的支持
2. 检测错误弹窗是否正确渲染出来

以下是我折腾出的代码：

 public class ExceptionHandler : ErrorBoundaryBase
 {
     [Inject]
     private IErrorBoundaryLogger? ErrorBoundaryLogger { get; set; }

     private bool firstRender = true;
     private bool showExceptionMessagePopover = false;
     private int errorRenderCount = 0;

     protected override async Task OnErrorAsync(Exception exception)
     {
         showExceptionMessagePopover = true;
         errorRenderCount++;
         await ErrorBoundaryLogger!.LogErrorAsync(exception);
     }

     protected override void OnAfterRender(bool firstRender)
     {
         this.firstRender = firstRender;
     }

     protected override void BuildRenderTree(RenderTreeBuilder builder)
     {
         int sequence = 0;
         if (CurrentException is null)
         {
             builder.AddContent(sequence++, ChildContent);
         }
         else if (ErrorContent is not null)
         {
             builder.AddContent(sequence++, ErrorContent(CurrentException));
         }
         else
         {
             if (firstRender || errorRenderCount> 2)
             {
                 builder.OpenElement(sequence++, "div");
                 builder.AddAttribute(sequence++, "class", "blazor-error-boundary");
                 builder.CloseElement();
             }
             else
             {
                 builder.AddContent(sequence++, ChildContent);
                 builder.OpenComponent<ExceptionMessagePopover>(3);
                 builder.AddComponentParameter(sequence++, "Open", showExceptionMessagePopover);
                 builder.AddComponentParameter(sequence++, "OpenChanged", EventCallback.Factory.Create<bool>(this, SetShowExceptionMessagePopover));
                 builder.AddComponentParameter(sequence++, "Exception", CurrentException);
                 builder.CloseComponent();
             }
         }
     }
     private void SetShowExceptionMessagePopover(bool value)
     {
         if (showExceptionMessagePopover && !value)//满足此条件，说明错误提示框成功渲染出来了，没有再触发其他异常，这里重置掉相关状态
         {
             errorRenderCount= 0;
             Recover();
         }
         showExceptionMessagePopover = value;
     }
 }

Additional context

No response

[bb-auto[bot]]

@izanhzh Thank you for contacting us. We will give feedback later.

[ArgoZhang]

@izanhzh 请提供一下示例代码啊。上传个工程给我。大佬的意思是让我们的组件用你这套逻辑对吗？

[izanhzh]

没有没有，你才是真大佬，我在上面提供了我折腾的一个代码，是想看看有没有参考价值，或者你们有没有更好的办法，目前blazor的这个全局处理老是差那么点意思，如果大佬觉得这个建议不合适，可以关掉这个feat

[ArgoZhang]

@izanhzh 别紧张，能提高组件性能或者功能，我们都乐意合并的。需要示例比如哪里不满意？你这个代码增强了哪里？

[izanhzh]

我希望能够自动跳出循环错误的情况，让它能够应对全部地方抛出的异常，如果是像初始化时触发的异常，实在没办法弹窗，就退而求其次，进入错误页面，包括弹窗提示的时候有异常，也进入错误页面

[izanhzh]

BlazorApp2.zip

[ArgoZhang]

QQ 群 795206915

[izanhzh]

满员了，我代码压缩发出来了

[ArgoZhang]

代码我看了。问题如下

<ErrorLogger>
    <h1>Counter</h1>

    <p role="status">Current count: @currentCount</p>

    <button class="btn btn-primary" @onclick="IncrementCount">Click me</button>
</ErrorLogger>

不能用原生的 button 这样拦不住