发一个小程序，感觉代码超乱。怎么优化下

黄玉昆

/*
三个学生英语，数学，语文成绩，总成绩排序输出

分析：
1. 创建一个学生类有姓名和学号的属性 private.
   设置姓名和属性的set、get方法.
   sum方法——求总成绩，并进行排序
2. 主函数.
1，设置姓名学号
2, 获取成绩的随机数
3, 求总成绩，并进行排序
3. 调用print方法

*/
import java.util.*;
class Test1
{

public static void main(String[] args)
{

Person p1 = new Person();
Person p2 = new Person();
Person p3 = new Person();

//设置张三和id，并获取三科成绩
p1.setName("张三");
p1.setId(1);
Random ran = new Random();
p1.english=ran.nextInt(61)+40;
p1.chinese=ran.nextInt(61)+40;
p1.math=ran.nextInt(61)+40;

//设置李四和id，并获取三科成绩
p2.setName("李四");
p2.setId(2);
p2.english=ran.nextInt(61)+40;
p2.chinese=ran.nextInt(61)+40;
p2.math=ran.nextInt(61)+40;

//设置王五和id，并获取三科成绩
p3.setName("王五");
p3.setId(3);
p3.english=ran.nextInt(61)+40;
p3.chinese=ran.nextInt(61)+40;
p3.math=ran.nextInt(61)+40;

//判断他们的总成绩并进行排序

if( p1.sum() < p2.sum()){
Person p=p1;
p1 = p2;
p2 = p;
}

if( p1.sum() < p3.sum()){
Person p=p1;
p1 = p3;
p3 = p;
}

if( p2.sum() < p3.sum()){
Person p=p2;
p2 = p3;
p3 = p;
}

//排序后调用print方法打印
p1.print();
p2.print();
p3.print();

}
}

import java.util.*;
class Person
{
private String name; //成员属性——姓名
private int id; //成员属性——学号
int english; //成员属性——英语成绩
int math; //成员属性——数学成绩
int chinese; //成员属性——语文成绩
int sum; //成员属性——总成绩

//无参构造函数
Person(){

}

//有参构造函数
void Score(String name, int id){
this.name=name;
this.id=id;
}

//姓名的set方法
public void setName(String name){
this.name=name;
}

//学号的set方法
public void setId(int id){
this.id=id;
}

//姓名的get方法
public String getName(){
return name;
}

//学号的get方法
public int getId(){
return id;
}

//总成绩sum方法
public int sum(){
return sum = english + math +chinese;
}

public void print(){

System.out.println("姓名:"+getName()+"  学号："+getId()+"  英语："+ english +"  数学："+ math
+"  语文："+chinese+" 总成绩:"+sum());

}

}

山西_李帅 · 山西_李帅

/*
三个学生英语，数学，语文成绩，总成绩排序输出
分析：
1. 创建一个学生类有姓名和学号的属性 private.
   设置姓名和属性的set、get方法.
   sum方法——求总成绩，并进行排序
2. 主函数.
1，设置姓名学号
2, 获取成绩的随机数
3, 求总成绩，并进行排序
3. 调用print方法
*/
import java.util.*;
class Test1
{
public static void main(String[] args)
{

Person p1 = new Person();
Person p2 = new Person();
Person p3 = new Person();

//设置张三和id，并获取三科成绩
p1.setName("张三");
p1.setId(1);
p1.english();
p1.chinese();
p1.math();

//设置李四和id，并获取三科成绩
p2.setName("李四");
p2.setId(2);
p2.english();
p2.chinese();
p2.math();

//设置王五和id，并获取三科成绩
p3.setName("王五");
p3.setId(3);
p3.english();
p3.chinese();
p3.math();

//判断他们的总成绩并进行排序，

if( p1.sum() < p2.sum()){
Person p=p1;
p1 = p2;
p2 = p;
}

if( p1.sum() < p3.sum()){
Person p=p1;
p1 = p3;
p3 = p;
}

if( p2.sum() < p3.sum()){
Person p=p2;
p2 = p3;
p3 = p;
}

//排序后调用print方法打印
p1.print();
p2.print();
p3.print();

}
}

import java.util.*;
class Person
{
private String name; //成员属性——姓名
private int id; //成员属性——学号
int english; //成员属性——英语成绩
int math; //成员属性——数学成绩
int chinese; //成员属性——语文成绩
int sum; //成员属性——总成绩

//无参构造函数
Person(){

}

//有参构造函数
void Score(String name, int id){
this.name=name;
this.id=id;
}

//姓名的set方法
public void setName(String name){
this.name=name;
}

//学号的set方法
public void setId(int id){
this.id=id;
}

//姓名的get方法
public String getName(){
return name;
}

//学号的get方法
public int getId(){
return id;
}

//总成绩sum方法
public int sum(){
return sum = english + math +chinese;
}

//返回英语成绩值english方法
public int english(){
Random ran = new Random();
return(english=ran.nextInt(61)+40);
}

//返回数学成绩值math方法
public int math(){
Random ran = new Random();
return(math=ran.nextInt(61)+40);
}

//返回语文成绩值chinese方法
public int chinese(){
Random ran = new Random();
return(chinese=ran.nextInt(61)+40);
}

//显示姓名，学号，成绩print方法
public void print(){
System.out.println("姓名:"+getName()+"  学号："+getId()+"  英语："+ english +"  数学："+ math
+"  语文："+chinese+" 总成绩:"+sum());
}

}

山西_李帅 · 山西_李帅

上面我自己改了下，是不是代码太长了。

邵震 · 邵震

额好长

我先占楼

顺便问下你学到第几天的视频了

王亚东 · 王亚东

排序时不可以改变原来的对象，我写的不知道你有没有学到，应该正规些：

package Test;
import java.util.ArrayList;
import java.util.Collections;
import java.util.Comparator;
import java.util.List;
import java.util.Random;
/*
三个学生英语，数学，语文成绩，总成绩排序输出
分析：
1. 创建一个学生类有姓名和学号的属性 private.
设置姓名和属性的set、get方法.
sum方法——求总成绩，并进行排序
2. 主函数.
1，设置姓名学号
2, 获取成绩的随机数
3, 求总成绩，并进行排序
3. 调用print方法
*/
public class Demo01 {
public static void main(String[] args) throws Exception {
List<Student> list = new ArrayList<Student>();
Random random = new Random();
Student s1 = new Student("张三",1);
s1.setChinese(random.nextInt(61)+40);
s1.setMath(random.nextInt(61)+40);
s1.setEnglish(random.nextInt(61)+40);
list.add(s1);
Student s2 = new Student("李四",2);
s2.setChinese(random.nextInt(61)+40);
s2.setMath(random.nextInt(61)+40);
s2.setEnglish(random.nextInt(61)+40);
list.add(s2);
Student s3 = new Student("王五",3);
s3.setChinese(random.nextInt(61)+40);
s3.setMath(random.nextInt(61)+40);
s3.setEnglish(random.nextInt(61)+40);
list.add(s3);
//对list中的Student对象按照总分降序排序
Collections.sort(list, new Comparator<Student>(){
public int compare(Student o1, Student o2) {
int i = o2.getSum().compareTo(o1.getSum());
if(i == 0) //总分相同时，按姓名升序排序
return o1.getName().compareTo(o2.getName());
return i;
}
});
//打印出学生信息
System.out.println("姓名\t"+"学号\t"+"总分\t"+"语文\t"+"数学\t"+"英语\t");
for(Student s:list){
System.out.println(s);
}
}
}
class Student {
private String name;
private int id;
private double chinese;
private double math;
private double english;
//构造方法
public Student(String name,int id){
this.name = name;
this.id = id;
}
//获取总成绩的方法
public Double getSum(){
return chinese+math+english;
}
//重写toString方法
@Override
public String toString() {
return name+"\t"+id+"\t"+getSum()+"\t"+chinese+"\t"+math+"\t"+english;
}
//getter、setter
public String getName() {
return name;
}
public void setName(String name) {
this.name = name;
}
public int getId() {
return id;
}
public void setId(int id) {
this.id = id;
}
public double getChinese() {
return chinese;
}
public void setChinese(double chinese) {
this.chinese = chinese;
}
public double getMath() {
return math;
}
public void setMath(double math) {
this.math = math;
}
public double getEnglish() {
return english;
}
public void setEnglish(double english) {
this.english = english;
}
}

复制代码

邵震 · 邵震

我擦超崩溃

哥们你能单独发一下你的需求吗

你这个我完全看不懂但是我知道你肯定有重复的代码

山西_李帅 · 山西_李帅

王亚东发表于 2013-4-1 12:30
排序时不可以改变原来的对象，我写的不知道你有没有学到，应该正规些：

这改变的是引用呐

山西_李帅 · 山西_李帅

邵震发表于 2013-4-1 12:31
我擦超崩溃

哥们你能单独发一下你的需求吗

对啊，你看我第二遍发的。就是修改了以后的，但是还有重复代码。哭啊！！！

黄小贝 · 黄小贝

王亚东发表于 2013-4-1 12:30
排序时不可以改变原来的对象，我写的不知道你有没有学到，应该正规些：

重复代码主要在main里面，你看下我改的吧，最开始我重构是这样的

然后我觉得实在没有办法了，只能改变排序的办法了，于是我实现了Comparable接口

还有一个重构的小细节

完整代码如下

import java.util.ArrayList;
import java.util.Collections;
import java.util.List;
/*
三个学生英语，数学，语文成绩，总成绩排序输出
分析：
1. 创建一个学生类有姓名和学号的属性 private.
设置姓名和属性的set、get方法.
sum方法——求总成绩，并进行排序
2. 主函数.
1，设置姓名学号
2, 获取成绩的随机数
3, 求总成绩，并进行排序
3. 调用print方法
*/
public class Test1 {
public static void main(String[] args) throws CloneNotSupportedException {
List<Person> persons = new ArrayList<Person>();
Person p1 = new Person("张三",1);
Person p2 = new Person("李四",2);
Person p3 = new Person("王五",3);
persons.add(p1);
persons.add(p2);
persons.add(p3);
getGrades(p1);
getGrades(p2);
getGrades(p3);
Collections.sort(persons);
System.out.println(persons);
}
private static void getGrades(Person p1) {
p1.english();
p1.chinese();
p1.math();
}
}

复制代码

import java.util.Random;
public class Person implements Comparable{
private String name; // 成员属性——姓名
private int id; // 成员属性——学号
int english; // 成员属性——英语成绩
int math; // 成员属性——数学成绩
int chinese; // 成员属性——语文成绩
int sum; // 成员属性——总成绩
// 无参构造函数
Person() {
}
// 有参构造函数
Person(String name, int id) {
this.name = name;
this.id = id;
}
// 姓名的set方法
public void setName(String name) {
this.name = name;
}
// 学号的set方法
public void setId(int id) {
this.id = id;
}
// 姓名的get方法
public String getName() {
return name;
}
// 学号的get方法
public int getId() {
return id;
}
// 总成绩sum方法
public int sum() {
return sum = english + math + chinese;
}
// 返回英语成绩值english方法
public int english() {
return english = getGrades();
}
// 返回数学成绩值math方法
public int math() {
return math = getGrades();
}
// 返回语文成绩值chinese方法
public int chinese() {
return chinese = getGrades();
}
private int getGrades() {
Random ran = new Random();
return ran.nextInt(61) + 40;
}
// 显示姓名，学号，成绩print方法
public String toString() {
return "姓名:" + getName() + " 学号：" + getId() + " 英语："
+ english + " 数学：" + math + " 语文：" + chinese + " 总成绩:"
+ sum() + "\n";
}
@Override
public int compareTo(Object o) {
if(this.sum() > ((Person) o).sum()){
return -1;
}
if(this.sum() < ((Person) o).sum()){
return 1;
}
return 0;
}
}

复制代码

王亚东 · 王亚东

黄小贝发表于 2013-4-1 17:06
重复代码主要在main里面，你看下我改的吧，最开始我重构是这样的

从技术角度来说，你这样抽取不错，很简洁，但是从面向对象的方面考虑，这样就不理想了，每个学生的成绩是要传递进去的，这个随机只是为了测试方便，还有一个细节，各个成绩最好使用double，java作为一个扩展性很强的语言，考虑到成绩需要求平均分等，使用double可提高程序的扩展性，也符合生活中的实际情况
其实说这么多，只是大家一起学习交流，提高自我的编程认识

黄小贝 · 黄小贝

王亚东发表于 2013-4-1 17:36
从技术角度来说，你这样抽取不错，很简洁，但是从面向对象的方面考虑，这样就不理想了，每个学生的成绩是 ...

我实在想尽量不改变撸主源码的情况下重构，我这样搞都觉得改多了~~练手的小东西不要在意这些细节。。。

陈圳 · 陈圳

package exercises;
import java.util.Arrays;
import java.util.Set;
import java.util.TreeSet;
public class StudentSort {
/** 三个学生英语，数学，语文成绩，总成绩排序输出
* @param args
*/
public static void main(String[] args) {
// TODO Auto-generated method stub
Set<Student> student=new TreeSet<Student>();
student.add(new Student("张三","A101",58,78,23));
student.add(new Student("李四","A102",38,38,53));
student.add(new Student("王五","B103",18,98,73));
System.out.println(student);
}
}
class Student implements Comparable<Student>{
private String name;
private String num;
private int score[]=new int[3];
public Student(String name, String num, int... socre) {
super();
this.name = name;
this.num = num;
this.score = socre;
}
@Override
public int compareTo(Student s) {
int sumScore1=0;
int sumScore2=0;
for(int i=0;i<this.score.length;i++){
sumScore1+=this.score[i];
sumScore2+=s.score[i];
}
//这里测试一下成绩,顺便看一下结果
System.out.println("姓名:"+this.name+"总成绩为:"+sumScore1);
System.out.println("姓名:"+s.name+"总成绩为:"+sumScore2);
return sumScore2-sumScore1;//这里就不比较俩个人分数一样按名称字称字典排序了
}
@Override
public String toString(){
return "学生:"+name+" 学号:"+num+" 语文:"+Arrays.toString(score);
}
}

复制代码

帐号		自动登录	找回密码
密码			加入黑马

发一个小程序，感觉代码超乱。怎么优化下

点评

评分

11 个回复

评分

评分

评分

黑马骑士勋章

发一个小程序，感觉代码超乱。 怎么优化下

点评

评分

11 个回复

评分

评分

评分

黑马骑士勋章

发一个小程序，感觉代码超乱。怎么优化下