我搞了好久，才得到输出结果。可是造成这样的原因在哪？

if 后面要一条执行语句   你方法也有一条吧            就算if没有干嘛                             方法必须有返回值

不加大括号  你直接把if(this.age==p.age) 删了 这句是废话

因为你这个函数:public int compareTo (Person p){} 返回值类型写了int，就肯定要有返回值，所以必须有return。你写的return都是在if语句里面的，也就是说返回值都是有判断条件的。而要是if语句条件都不成立（如p=null。），就不能return值了。所以当你加了{}，第二个if语句就结束了。下面的return就独立出来，就肯定可以返回值。

没有返回语句，图一中的第二个if语句为假时没有返回语句，加个大括号的话if和{}组成了一个语句，这样当if为假时一样能返回

1. 改成这样就行了 ：return (this.age==p.age)?this.name.compareTo(p.name):(this.age-p.age);

复制代码

错了，是这样return (this.age==p.age)?this.name.compareTo(p.name):(this.age-p.age);

你的方法返回值是int，看你第一段代码，程序运行进来，如果发现两个if条件都不满足，程序就没有返回值了肯定出错。第二段代码，在第二个if，

1. if(this.age==p.age){}

复制代码

if后面加个大括号，等于这个if什么事都没干，浪费资源啊。

1. if(this.age==p.age){}
   
2. return this.name.compareTo(p.name)
   
3. //上面这两句直接改为下面代码即可
   
4. else{
   
5. return this.name.compareTo(p.name)
   
6. }
   

复制代码

问题解决。

简单的说     if里的语句有可能执行不到！ if是判断，如果！ 没有如果~则不会返回结果。