21

Can you use a if statement in JSX like this?

    var chartGraphContent =
        <div className={"chartContent"}>
            if(this.state.modalityGraph['nca'] > 0){
                <div className={"chart-container"}>
                    <Chart
                        chartType="ColumnChart"
                        data = { this.state.modalityGraph?this.state.modalityGraph.chartData['units']:emptyDataRows }
                        options={chartOptions}
                        graph_id="modalitiesChart"
                        width="100%"
                        height="250px"
                    /> 
                </div>
            }
        </div>;

Something like above? Is it possible to have JSX based on a condition?

Andrew Li
  • 55,805
  • 14
  • 125
  • 143
morne
  • 4,035
  • 9
  • 50
  • 96

4 Answers4

24

Use conditional rendering, and since you have no else-case, you can use && instead of a ternary operator for brevity:

It works because in JavaScript, true && expression always evaluates to expression, and false && expression always evaluates to false.

Therefore, if the condition is true, the element right after && will appear in the output. If it is false, React will ignore and skip it.

Thus:

   <div className={"chartContent"}>
        {this.state.modalityGraph['nca'] > 0 &&
            <div className={"chart-container"}>
                <Chart
                    chartType="ColumnChart"
                    data = { this.state.modalityGraph?this.state.modalityGraph.chartData['units']:emptyDataRows }
                    options={chartOptions}
                    graph_id="modalitiesChart"
                    width="100%"
                    height="250px"
                /> 
            </div>
        }
    </div>

This will only render JSX if a condition is true. If it is false, React won't render anything. Remember, you have to wrap inline JavaScript expressions in JSX with { … }, you can't just have it inside JSX.

Using if/else statements directly in JSX will cause it to be rendered literally as text, which isn't desired. You also can't use them in inline JavaScript expressions because if statements are not expressions, so this won't work:

{
  if(x) y
}
Pang
  • 9,564
  • 146
  • 81
  • 122
Andrew Li
  • 55,805
  • 14
  • 125
  • 143
9

As per DOC:

if-else statements don't work inside JSX. This is because JSX is just syntactic sugar for function calls and object construction.


We can't use if-else statement or any other statement directly inside JSX, only expressions are allowed.

Expressions inside JSX:

Wou can embed any JavaScript expression in JSX by wrapping it in curly braces.

To put any expression we need to use {}, so instead of if use && operator or ternary operator for conditional rendering.

By using ternary operator:

var chartGraphContent =
<div className={"chartContent"}>
    {
        this.state.modalityGraph['nca'] > 0 ?
            <div className={"chart-container"}>
                <Chart
                    chartType="ColumnChart"
                    data = { this.state.modalityGraph?this.state.modalityGraph.chartData['units']:emptyDataRows }
                    options={chartOptions}
                    graph_id="modalitiesChart"
                    width="100%"
                    height="250px"
                /> 
            </div>
        :null
    }
</div>;

By using && operator:

var chartGraphContent =
<div className={"chartContent"}>
    {
        this.state.modalityGraph['nca'] > 0 &&
            <div className={"chart-container"}>
                <Chart
                    chartType="ColumnChart"
                    data = { this.state.modalityGraph?this.state.modalityGraph.chartData['units']:emptyDataRows }
                    options={chartOptions}
                    graph_id="modalitiesChart"
                    width="100%"
                    height="250px"
                /> 
            </div>
    }
</div>;
Mayank Shukla
  • 100,735
  • 18
  • 158
  • 142
  • 1
    Did not down vote you, your answer is also correct but @Andrew Li had a similar answer first. Many thanks though. (PS. This is sometimes a over purist site and people dovn vote for no GOOD reason ) – morne Sep 11 '17 at 13:49
4

Better to use with ternary Operator, By doing so you can also add else block to your code.

Try this:

var chartGraphContent =
        <div className={"chartContent"}>
            {this.state.modalityGraph['nca'] > 0 ?
                <div className={"chart-container"}>
                    <Chart
                        chartType="ColumnChart"
                        data = { this.state.modalityGraph?this.state.modalityGraph.chartData['units']:emptyDataRows }
                        options={chartOptions}
                        graph_id="modalitiesChart"
                        width="100%"
                        height="250px"
                    /> 
                </div>
                : "<span>Else Block</span>"
            }
        </div>;

Update (Another method)

and in case for more complex and large condition you can call inline functions too to return your template , in this way you can avoid your code to become messy. here is an example.

var ifBlockCode = function ifBlockCode(){
    return (
        <div className={"chart-container"}>
            <Chart
                chartType="ColumnChart"
                data = { this.state.modalityGraph?this.state.modalityGraph.chartData['units']:emptyDataRows }
                options={chartOptions}
                graph_id="modalitiesChart"
                width="100%"
                height="250px"
            /> 
        </div>
    )
}

var elseBlockCode = function elseBlockCode(){
    return (
        <span>Else Block</span>
    )
}
var chartGraphContent =
<div className={"chartContent"}>
    {this.state.modalityGraph['nca'] > 0 ?
        {this.ifBlockCode} : {this.elseBlockCode}
    }
</div>;
Pardeep Jain
  • 84,110
  • 37
  • 165
  • 215
  • Thanks Pardeep Jain. Would have worked fine, but Andrew Li's first answer was also correct. Upvoted though. – morne Sep 11 '17 at 13:59
0

To work around the limitation that JSX supports expressions only: (if control blocks are statements)

  • Just use multiple return statements wrapped by your control block.

The keyword return introduces a statement. You can simply wrap you JSX logic into control structure scopes, where each case is its own return statement (nobody ever said you can't use two return statements!)

if (case) {
  return <JSX prop=true/>
else {
  return <JSX prop=false/>
}

Side note about expressions vs statements

Expressions must be pure, which is probably why JSX (function calls) accept them. The if-blocks are not pure in the sense that they can introduce side effects (they can wrap multiple expressions, some of which must have nothing to do with the resulting value). Expressions cannot do this because they must return a value. In that sense, they are pure. They could introduce effects, but these would be effects and not side effects.

e.g. this silly example of an expression that has effects but I would argue that they are not side effects because the expression does exactly one thing in each case.

const a: void = v ? console.log("a") : console.log("b");
Jonathan Komar
  • 2,678
  • 4
  • 32
  • 43