<ul class="contains-task-list">
<li class="task-list-item"><input type="checkbox" id="" disabled="" class="task-list-item-checkbox" checked=""> I have searched the <a href="https://github.com/apache/incubator-dubbo/issues">issues</a> of this repository and believe that this is not a duplicate.</li>
<li class="task-list-item"><input type="checkbox" id="" disabled="" class="task-list-item-checkbox" checked=""> I have checked the <a href="https://github.com/apache/incubator-dubbo/blob/master/FAQ.md">FAQ</a> of this repository and believe that this is not a duplicate.</li>
</ul>
<h3 dir="auto">Environment</h3>
<ul dir="auto">
<li>Dubbo version: 2.7.0-SNAPSHOT</li>
<li>Operating System version: mac os</li>
<li>Java version: 1.8</li>
</ul>
<h3 dir="auto">Step to reproduce this issue</h3>
<p dir="auto">Pls. provide [GitHub address] to reproduce this issue.</p>
<h3 dir="auto">Expected Result</h3>
<p dir="auto">What do you expected from the above steps？</p>
<h3 dir="auto">Actual Result</h3>
<p dir="auto">What is actually happen?</p>
<p dir="auto">If there is an exception, please attach the exception trace:</p>
<div class="snippet-clipboard-content notranslate position-relative overflow-auto" data-snippet-clipboard-copy-content="Just put your stack trace here!"><pre class="notranslate"><code class="notranslate">Just put your stack trace here!
</code></pre></div>
<ul class="contains-task-list">
<li class="task-list-item"><input type="checkbox" id="" disabled="" class="task-list-item-checkbox"> I have searched the <a href="https://github.com/apache/dubbo/issues">issues</a> of this repository and believe that this is not a duplicate.</li>
<li>[ -] I have checked the <a href="https://github.com/apache/dubbo/blob/master/FAQ.md">FAQ</a> of this repository and believe that this is not a duplicate.</li>
</ul>
<h3 dir="auto">Environment</h3>
<ul dir="auto">
<li>Dubbo version: 2.7.3</li>
</ul>
<h3 dir="auto">ExtensionLoader.getActivateExtension</h3>
<div class="highlight highlight-source-java notranslate position-relative overflow-auto" dir="auto" data-snippet-clipboard-copy-content="
                if (isMatchGroup(group, activateGroup)) {
                    // 这个应该放到 if 里面，会更合适
                    T ext = getExtension(name);
                    if (!names.contains(name)
                            &amp;&amp; !names.contains(Constants.REMOVE_VALUE_PREFIX + name)
                            &amp;&amp; isActive(activateValue, url)) {
                        exts.add(ext);
                    }

"><pre class="notranslate">                <span class="pl-k">if</span> (<span class="pl-en">isMatchGroup</span>(<span class="pl-s1">group</span>, <span class="pl-s1">activateGroup</span>)) {
                    <span class="pl-c">// 这个应该放到 if 里面，会更合适</span>
                    <span class="pl-smi">T</span> <span class="pl-s1">ext</span> = <span class="pl-en">getExtension</span>(<span class="pl-s1">name</span>);
                    <span class="pl-k">if</span> (!<span class="pl-s1">names</span>.<span class="pl-en">contains</span>(<span class="pl-s1">name</span>)
                            &amp;&amp; !<span class="pl-s1">names</span>.<span class="pl-en">contains</span>(<span class="pl-smi">Constants</span>.<span class="pl-c1">REMOVE_VALUE_PREFIX</span> + <span class="pl-s1">name</span>)
                            &amp;&amp; <span class="pl-en">isActive</span>(<span class="pl-s1">activateValue</span>, <span class="pl-s1">url</span>)) {
                        <span class="pl-s1">exts</span>.<span class="pl-en">add</span>(<span class="pl-s1">ext</span>);
                    }
</pre></div>