牢记BEM的可扩展React CSS使用``类名''

Question

因此，这里是React新手...我将首先说说我有一个简单的单页面应用程序，其中包含一些简单的页面。

使用react-router，我为组件设置了“自顶向下”功能。 为了让您对我的SPA结构有一个基本的了解，请参见以下内容：

index -- layout(react routers) --
                                |--About Page
                                |--Home Page
                                |--Contact Page

我正在从主页组件中渲染一个名为“ GlobalHero”的组件。

这是GlobalHero.jsx组件。

import React from "react";
var classNames = require('classnames');
import s from '../../../index.scss';

class GlobalHero extends React.Component {

    constructor() {
        super();
        //sets initial state
        this.state = {
            fadeIn: "",
            titleSelected: "",
            subTitleSelected: ""
        };
    }

    // <<========= ON COMPONENT RENDER =========
    componentDidMount = () => {
        console.log("GlobalHero");
        console.log(this.props);
        this.handleClass("fadeIn");
    }
    // =========>>

    // <<========= CONTROLS THE STATE =========
    handleClass = (param) => {
        if (param === "fadeIn" && this.state.fadeIn != "true") {
            this.setState({fadeIn: "true"});
        }
        if (param === "titleSelected" && this.state.titleSelected != "true") {
            this.setState({titleSelected: "true"});
        }
        if (param === "subTitleSelected" && this.state.subTitleSelected != "true") {
            this.setState({subTitleSelected: "true"});
        }
    }
    // =========>>

    render() {

        const heroImg = require(`../../../images/hero${this.props.page}.jpg`);

        //REMOVES CLASS IN REALTIME BASED ON STATE'S VALUE =========
        var containerClasses = classNames({
            [s['text-center']]: true,
            [s["hidden"]]: this.state.fadeIn != "true",
            [s["fadeIn"]]: this.state.fadeIn === "true"
        });
        var titleClasses = classNames({
            [s['blue']]: this.state.titleSelected === "true"
        });
        var subTitleClasses = classNames({
            [s['subTitle']]: true,
            [s['text-center']]: true,
            [s['blue']]: this.state.subTitleSelected === "true"
        });
        // =========>>

        return (
            <div className={s["container-fluid"]}>
                <div className={s["row"]}>
                    <div className={s["col-lg-16"]}>

                        <div className={containerClasses}>
                            <img src={heroImg} className={s["hero__img"]}></img>
                            <h1 onClick={() => this.handleClass("titleSelected")} className={titleClasses}>{this.props.page}!</h1>
                            <p className={subTitleClasses} onClick={() => this.handleClass("subTitleSelected")}>{this.props.name}, {this.props.age}, {this.props.city}</p>
                        </div>

                    </div>
                </div>
            </div>
        );
    }
}

export default GlobalHero;

我注意到，为该组件的元素分配一些简单的类名称存在很多复杂性。

我想知道是否有更好的做法？ 也许使用外部js页面来管理我的类名？

感谢您的任何输入或建议...谢谢。

Answer 1

您的标题提到了BEM，但看起来您正在使用CSS模块 ，它的灵感来自相似的想法，但并非同一件事。

无论如何，这是相当主观的，但是我有一些想法太过分以致无法评论：

• 假设您正在通过Webpack的css-loader使用css模块，则可以使用camelCase使您的样式属性对JS更友好：

   loader: "css-loader?modules&camelCase" 

  现在，对于.text-center css类名，您只需使用s.textCenter而不是s["test-center"] 。

• 您可以更好地实现组件化：首先，您为单个组件做了很多工作，但是您可以将其分解为几个较小的组件，每个组件都负有单个责任 （例如，容器，标题，字幕）。 其次，当您可能只有简单的处理程序调用setState()而不知道类名时，您的handleClass()方法就做了很多事情。 换句话说，组件应该具有道具和状态，只有render()函数处理如何将其转换为类名称以进行渲染。 您还真的不需要在设置状态之前检查状态的当前值。 只需将其设置为应有的值，然后让React为您优化渲染性能。
• 您具有使用字符串"true"和"false"存储的布尔状态标志...这使处理起来很嘈杂，只是存储为布尔值。

• 您有很多[s["class-name"]]: true ，没有必要； 如果您始终希望呈现类名，只需将其作为参数传递给classNames ：

   classNames(s.subTitle, { [s.blue]: this.state.subTitleSelected }) 

• 没有理由在componentDidMount上调用处理程序，只需按需要初始化状态即可。
• 看起来您正在使用引导CSS，而不是React Bootstrap组件。 我强烈建议使用React Bootstrap。

放在一起，我会得到类似的东西：

class GlobalHero extends React.Component {

    state = {
        fadeIn: true,
        titleSelected: false,
        subTitleSelected: false
    };

    handleTitleClick = () => {
        this.setState({titleSelected: true});
    };

    handleSubTitleClick = () => {
        this.setState({subTitleSelected: true});
    };

    render() {
        return (
          <Grid fluid>
            <Row>
              <Col lg={16}>
                <HeroContainer fadeIn={this.state.fadeIn}>
                  <HeroImage page={this.props.page} />
                  <HeroTitle selected={this.state.titleSelected} 
                             onClick={this.handleTitleClick} 
                             page={this.props.page} />
                  <HeroSubTitle selected={this.state.subTitleSelected} 
                                onClick={this.handleSubTitleClick} 
                                name={this.props.name} 
                                age={this.props.age} 
                                city={this.props.city} />
                </HeroContainer>
              </Col>
            </Row>
          </Grid>
        );
    }
}

const HeroContainer = ({fadeIn, children}) => {
  return (
    <div className={classNames(s.textCenter, fadeIn ? s.fadeIn : s.hidden)}>
      {children}
    </div>
  );
};

const HeroImage = ({page}) => {
  const heroImg = require(`../../../images/hero${page}.jpg`);
  return (
    <img src={heroImg} className={s.heroImg} />
  );
};

const HeroTitle = ({onClick, selected, page}) => (
    <h1 onClick={onClick} className={selected ? s.blue : null}>{page}!</h1>
);

const HeroSubTitle = ({onClick, selected, name, age, city}) => (
    <p className={classNames(s.subTitle, s.textCenter, { [s.blue]: selected })} onClick={onClick}>
      {name}, {age}, {city}
    </p>
);

像这样将其分解成较小的组件不是完全必要的，但是请注意，从GlobalHero的角度来看，它如何不处理样式，它只是设置道具和状态，而很少的部分没有状态，它们只是根据道具渲染正确的样式。

PS也许这应该转向代码审查 ？