时间戳

首页 > 解决方案 > 定义一个常量，而不是将这个文字复制 3 次声纳

javascript - 定义一个常量，而不是将这个文字复制 3 次声纳

问题描述

我想知道如何重构以下内容。基本上，在我从谷歌获取用户信息后，它会被验证，因为可能允许特定域。如果它通过了那个，我们会根据后端 api 验证令牌。

Sonar 抱怨使用return displayAlert('LoginFailed')三次。鉴于这些方法具有独特的用途，我想知道可以做些什么来改进它？

import React, {useContext} from 'react';
import {View, Alert} from 'react-native';
import {LocalizationContext} from 'library/localization';
import ServiceAuth from 'services/ServiceAuth';
import {saveLoggedInDetails} from 'library/userInformation';

const MyComponent = (props) => {
  const {translations} = useContext(LocalizationContext);

  const displayAlert = (msg) =>{
      //translations happen here
    Alert.alert("Failed", msg)
    return false;
  }


  const validateToken = async (userGoogleInfo) => {
    const {idToken} = userGoogleInfo;
    let serviceAuth = ServiceAuth.getInstance();
    try {
      return await serviceAuth.validateT(idToken);
    } catch (error) {
      //show alert here
      return displayAlert('LoginFailed')
    }
  };


  const authorize = async (userGoogleInfo) => {
    const allowedDomains = [];
    let reply = {ok: false};
    //check if domain is allowed etc and set to true.
    if (!reply['ok']) {
      return displayAlert('InvalidDomain');
    }

    try {
      const userInfo = await validateToken(userGoogleInfo);

      if (userInfo) {
       console.log("okay")
      } else {
        return displayAlert('LoginFailed')
      }
    } catch (error) {
      return displayAlert('LoginFailed')
    }
  };

  return (
    <>
        <View>
          <Text>Hello</Text>
        </View>
    </>
  );
};
export default MyComponent;

标签： javascriptreact-nativesonarqube

解决方案

该消息有点令人困惑，但声纳只抱怨LoginFailed使用三次作为参数的字符串。

创建一个常量，例如：

const LOGIN_FAILED = 'LoginFailed';

然后在对 displayAlert 的所有三个调用中使用它：

return displayAlert(LOGIN_FAILED);

推荐阅读