有什么办法避免url硬编码吗 private String getPageContentSync(String url) { // 参数判断，未输入参数则直接返回 if (StringUtils.isBlank(url)) { return null; } //2.定义一个request Request request = new Request.Builder().url(url).build(); //3.使用client去请求 Call call = okHttpClient.newCall(request); String result = null; try { //4.获得返回结果 result = call.execute().body().string(); System.out.println("call " + url + " , content's size=" + result.length()); } catch (IOException e) { System.out.println("request " + url + " error . "); e.printStackTrace(); } return result; } public String pick() { String content = getPageContentSync("http://www.weather.com.cn/data/sk/101210106.html"); return content; }

可以使用配置文件或者常量类来避免URL硬编码。比如，将所有的URL保存在一个配置文件中，然后在代码中通过读取配置文件的方式来获取URL。这样做的好处是，如果需要修改URL，只需要修改配置文件，而不需要修改代码。另外，也可以将URL保存在一个常量类中，然后在代码中通过常量类来获取URL。这样做的好处是，可以将所有的常量集中管理，方便修改和维护。例如：

public class Constants {
    public static final String WEATHER_URL = "http://www.weather.com.cn/data/sk/101210106.html";
}

然后在代码中使用常量：

public String pick() {
    String content = getPageContentSync(Constants.WEATHER_URL);
    return content;
}

相关问题

String singleSend = (String) map.get("singleSend"); String expressProductCode = (String) map.get("expressProductCode"); LinkedHashMap<String,String> senderInformation = (LinkedHashMap<String,String>)map.get("senderInformation"); String senderName = senderInformation.get("label"); String senderPhone = senderInformation.get("value"); String productName = (String) map.get("productName"); if("1".equals(singleSend)) { for(Map<String, String> oo : list ) { String receiverName = oo.get("receiverName"); String receiverPhone = oo.get("receiverPhone"); String contactAddress = oo.get("contactAddress"); String orderNos = oo.get("orderNo"); OrderApiCreate orderApiCreate = genParam(expressProductCode,productName,senderName,senderPhone,receiverName,receiverPhone,contactAddress,"1", orderNos); String ret = apiUtil.createOrder(orderApiCreate); LoginUser loginUser = SecurityUtils.getLoginUser(); OrderInfo orderinfo = new OrderInfo(); orderinfo.setUpdateBy(loginUser.getUsername()); orderinfo.setApiOrderNo(orderApiCreate.getOrderNumber()); if(StringUtils.isNotEmpty(ret)) { orderinfo.setSendError(ret); orderinfo.setSendState("3"); orderinfo.setSendCount(0); } else { orderinfo.setSendState("2"); } orderService.updateApiSendInfo(new String[]{String.valueOf(oo.get("uuid"))}, orderinfo); } } else { String receiverName = (String) map.get("receiverName"); String receiverPhone = (String) map.get("receiverPhone"); String contactAddress = (String) map.get("contactAddress"); String productNumber = String.valueOf(uuids.length); String orderNos = list.stream().map(item->String.valueOf(item.get("orderNo"))).collect(Collectors.joining(",")); OrderApiCreate orderApiCreate = genParam(expressProductCode,productName,senderName,senderPhone,receiverName,receiverPhone,contactAddress,productNumber, orderNos); String ret = apiUtil.createOrder(orderApiCreate); LoginUser loginUser = SecurityUtils.getLoginUser(); OrderInfo orderinfo = new OrderInfo(); orderinfo.setUpdateBy(loginUser.getUsername()); orderinfo.setApiOrderNo(orderApiCreate.getOrderNumber()); if(StringUtils.isNotEmpty(ret)) { orderinfo.setSendError(ret); orderinfo.setSendState("3"); orderinfo.setSendCount(0); } else { orderinfo.setSendState("2"); } orderService.updateApiSendInfo(uuids, orderinfo); } apiUtil.getOrderSendState(); return AjaxResult.success(result);优化这段代码

这段代码可以进行如下优化：

1. 将重复的代码封装成方法，避免重复编写代码。

2. 将 map 中的一些常量值抽离出来，避免硬编码。

3. 将字符串拼接改为使用 StringBuilder，提高性能。

4. 将 if else 语句改为三目运算符，简化代码。

5. 将一些不必要的变量去掉，减少内存占用。

重构后的代码如下所示：

String singleSend = (String) map.get("singleSend");
String expressProductCode = (String) map.get("expressProductCode");
LinkedHashMap<String,String> senderInformation = (LinkedHashMap<String,String>)map.get("senderInformation");
String senderName = senderInformation.get("label");
String senderPhone = senderInformation.get("value");
String productName = (String) map.get("productName");

if("1".equals(singleSend)) {
    for(Map<String, String> oo : list ) {
        String receiverName = oo.get("receiverName");
        String receiverPhone = oo.get("receiverPhone");
        String contactAddress = oo.get("contactAddress");
        String orderNos = oo.get("orderNo");
        OrderApiCreate orderApiCreate = genParam(expressProductCode,productName,senderName,senderPhone,receiverName,receiverPhone,contactAddress,"1", orderNos);
        String ret = apiUtil.createOrder(orderApiCreate);
        updateApiSendInfo(ret, orderApiCreate, oo);
    }
} else {
    String receiverName = (String) map.get("receiverName");
    String receiverPhone = (String) map.get("receiverPhone");
    String contactAddress = (String) map.get("contactAddress");
    String productNumber = String.valueOf(uuids.length);
    String orderNos = list.stream().map(item->String.valueOf(item.get("orderNo"))).collect(Collectors.joining(","));
    OrderApiCreate orderApiCreate = genParam(expressProductCode,productName,senderName,senderPhone,receiverName,receiverPhone,contactAddress,productNumber, orderNos);
    String ret = apiUtil.createOrder(orderApiCreate);
    updateApiSendInfo(ret, orderApiCreate, uuids);
}

apiUtil.getOrderSendState();
return AjaxResult.success(result);

// 封装的方法
private void updateApiSendInfo(String ret, OrderApiCreate orderApiCreate, Object uuids) {
    LoginUser loginUser = SecurityUtils.getLoginUser();
    OrderInfo orderinfo = new OrderInfo();
    orderinfo.setUpdateBy(loginUser.getUsername());
    orderinfo.setApiOrderNo(orderApiCreate.getOrderNumber());
    if(StringUtils.isNotEmpty(ret)) {
        orderinfo.setSendError(ret);
        orderinfo.setSendState("3");
        orderinfo.setSendCount(0);
    } else {
        orderinfo.setSendState("2");
    }
    orderService.updateApiSendInfo(uuids instanceof Object[] ? (String[]) uuids : new String[]{String.valueOf(uuids)}, orderinfo);
}

// 抽离的常量
private static final String ONE = "1";

public void prepatEdit(PrePatrol prepatrol) { Map<String,String> map = new HashMap<>(); String str = StringUtils.join(prepatrol.getCheckbox(),","); prepatrol.setValue(str); prepatrol.setCheckbox(null); List<PrePatrol> prePatrols = this.baseMapper.dictList(); for (PrePatrol entity : prePatrols) { map.put(entity.getIds(), entity.getIdsName()); } if ("8".equals(prepatrol.getIds())){ // 1:交班信息填写完成 PreEmployment preEmployment = new PreEmployment(); preEmployment.setId(prepatrol.getPid()); preEmployment.setEndhandover(1); preEmploymentService.edit(preEmployment); } prepatrol.setIdsName(map.get(prepatrol.getIds())) ; this.updateById(prepatrol); }

这段代码看起来已经相当简洁了。不过，我注意到其中一些可以进行优化的地方：

1. 在循环中使用`this.baseMapper.dictList()`获取`prePatrols`列表可能会导致性能问题，特别是在数据量很大的情况下。如果可能的话，可以考虑将该列表缓存起来，或者使用更高效的方式获取数据。

2. 在循环中使用`map.put(entity.getIds(), entity.getIdsName())`将实体对象的`ids`和`idsName`属性放入map中。如果`ids`属性是唯一的，并且你只需要使用`idsName`属性，可以考虑将map的键值对反转，以便更快地根据`idsName`查找对应的`ids`值。

3. 在条件判断中使用硬编码的字符串"8"进行比较，不利于代码的可读性和维护性。可以将该值定义为一个具有描述性名称的常量，并使用常量进行比较。

下面是优化后的代码示例：

public void prepatEdit(PrePatrol prepatrol) {
    Map<String,String> map = new HashMap<>();
    String str = StringUtils.join(prepatrol.getCheckbox(),",");
    prepatrol.setValue(str);
    prepatrol.setCheckbox(null);
    
    List<PrePatrol> prePatrols = getCachedPrePatrols(); // 从缓存中获取数据
    
    for (PrePatrol entity : prePatrols) {
        map.put(entity.getIdsName(), entity.getIds());
    }
    
    final String TARGET_IDS = "8"; // 定义常量
    
    if (TARGET_IDS.equals(prepatrol.getIds())){
        // 1:交班信息填写完成
        PreEmployment preEmployment = new PreEmployment();
        preEmployment.setId(prepatrol.getPid());
        preEmployment.setEndhandover(1);
        preEmploymentService.edit(preEmployment);
    }
    
    prepatrol.setIdsName(map.get(prepatrol.getIds())) ;
    this.updateById(prepatrol);
}

private List<PrePatrol> getCachedPrePatrols() {
    // 从缓存中获取数据，如果缓存不存在则从数据库中获取并放入缓存中
    // 可以使用工具类或框架来实现缓存逻辑，例如Spring的Cache或Ehcache等
}

请注意，上述代码示例仅供参考，并未在实际环境中测试。确保在进行任何更改之前进行适当的测试，并根据实际需求进行调整。