简体   繁体   中英

Getting Closure / Plovr to remove functions from an if that is always false

An if statement that is obviously always false, eg. if (false) , closure removes the statement.

My code looks like this:

if (settings.lang === "en"){
    lib.doSomething();
}

settings.lang is a constant.

/** 
 * @type {string}
 * @const 
 */ 
settings.lang = "fr" ;  

So when it equals "fr" the compiler could remove the if and the definition of lib.doSomething at compile time. But it doesn't. Is there any way to get it to do that?

Before you ask why I don't just delete that code: for other clients, settings.lang is set to en .

1) Make sure "settings" is properly defined:

/** @const */
var settings = {};

2) Make sure "settings" lang is properly defined:

/** @const */ 
settings.lang = "fr" ;

3) Make sure the value is referenced after it is defined:

if (settings.lang == "en") ...

In advanced mode this will be inlined and removed, if settings is not used in a way that prevents property collapsing (for instance, passing "settings" as a parameter to a function will cause the value to escape and be uncollapsible).

This is simplified, if you use @define:

/** @const */
var settings = {};

/** @define {string} */
settings.lang = "fr";

You should get a warning if the definition of the define is not valid in some way.

It sounds like you're using some compilation mode (most likely the 'SIMPLE' compilation mode, as that is the default in plovr ) other than the 'ADVANCED' compilation mode. The advanced compilation mode is the one that does dead code removal .

Is goog.LANG defined with goog.define? Last I checked Plovr was not compatible with goog.define (it doesn't have the latest compiler) make sure you are starting with a compatible version of the Closure Library.

The technical post webpages of this site follow the CC BY-SA 4.0 protocol. If you need to reprint, please indicate the site URL or the original address.Any question please contact:yoyou2525@163.com.

 
粤ICP备18138465号  © 2020-2024 STACKOOM.COM