问题如图，js代码可读性差。想知道差在哪里。请各位指出。不胜感激

第一，变量名要能体现所表示的内容，让他人能一看就明白什么意思。
第二，就是 if else 逻辑控制的问题，上面有人解释得比较清楚了。
第三，像 "未确认数据不能编制单元号!" "宗地代码不足19位不能编制单元号!" "已生成的不动产单元号不能重复编制!"
此类的返回消息应定义为常量。或者用一个变量保存，最后统一返回。

因为你每个判断都是return
所以用不着把下面的if写在else里。

你的判断并没有包括所有的情况，试试讲所有布尔数值在所有判断前声明，并使用if(){} elseif(){} ……else(){}

else里包着if 感觉怪怪的，还有就是希望题主养成变量名尽量别用拼音代替，代替的时候注释一下下…

你判断里面有用return,就没必要用else，在后面直接做你想做的逻辑就好了。如果有嵌套的if，在使用else,尽量把代码简化。

楼主干嘛要用else呢。。直接写一个个的if(){return}不就可以了。。而且那个bool变量也没有必要定义。。
等等。。js变量声明不是var么。。bool也可以了？？

 if(!_FDGLRepository.CheckSFQR(Zddms)){
    return "未确认数据不能编制单元号!";
}
if(!_FDGLRepository.CheckZDDM(Zddms)){
    return "宗地代码不足19位不能编制单元号!";
}
if(_FDGLRepository.IsExistsDYH(Zddms)){
    return "已生成的不动产单元号不能重复编制!";
}
return _FDGLRepository.CreateDYH(Zddms,"NEW").ToString();

首先谢谢各位热心解答。这段代码是一个朋友写的。因为有人说代码风格不好。截图后转给我让我在CSDN上贴出来，看看大家有什么修改意见。其实我自己的话，更喜欢
if()

{

return;
}
if()

{

return;
}
谢谢各位啦

代码可读性就是说你的代码给别人看别人能很容易看出你的思路。谁告诉我那段代码 有人看不懂吗？ 什么变量命名 什么有没有必要都屁话，所答非所问，
人家问可读性差在哪！在这问题上纠结的没一个是大神。