黑马程序员技术交流社区

标题: 求优化代码？ [打印本页]

作者: HM刘俊 时间: 2013-4-8 22:19
标题: 求优化代码？
本帖最后由 HM刘俊于 2013-4-9 08:26 编辑

下面是我刚刚用for嵌套循环写的圣诞树代码。代码太多，求优化。

class Tree{
public static void main(String[] args){
int le=10; //第一层
for(int i=0;i<le;i++){
for(int j=0;j<le-i-1;j++){
System.out.print(" ");
}
for(int j=0;j<=i;j++){
System.out.print("* ");
}
System.out.println();
}
int l=8; //第二层
for(int a=0;a<l;a++){
for(int s=0;s<l-a+1;s++){
System.out.print(" ");
}
for(int s=0;s<=a;s++){
System.out.print("* ");
}
System.out.println();
}
int q=6; //第三层
for(int w=0;w<q;w++){
for(int e=0;e<q-w+3;e++){
System.out.print(" ");
}
for(int e=0;e<=w;e++){
System.out.print("* ");
}
System.out.println();
}
for(int m=0;m<6;m++){ //底层
System.out.print(" ");
for(int n=0;n<5;n++){
System.out.print("*");
}
System.out.println();
}
}
}

复制代码

tree.jpg (21.88 KB, 下载次数: 5)

传说中的圣诞树

作者: 丘凤光 时间: 2013-4-9 01:08
本帖最后由丘凤光于 2013-4-9 01:17 编辑

弄了老久，这样应该差不多了吧.。能封装的都尽量封装了，最主要的是数的大小可控制，树冠层数可以控制，只要改 num=10,level=3,chang=6;就行。

/*
需求：简化打印树形的代码
*/
import static java.lang.System.*;
class Tree
{
private static int num=10,level=3,chang=6; //控制打印树的大小与形状的变量
public static void main(String[] args)
{
for(int i=1;i<=level;i++)
san(i);
ju();
}
//打印三角形
public static void san(int p)
{
for(int i=0;i<num;i++)
{
myFor(num+2*p-i-3," ");//num+2*p-i-3为左边打印的空格数
myFor(i,"* ");
sop("\n");
}
num-=2;
}
//打印矩形
public static void ju()
{
for(int m=0;m<chang;m++)
{
myFor(chang," ");
myFor(chang,"*");
sop("\n");
}
}
//for循环
public static void myFor(int x,Object y)
{
for(int i=0;i<=x;i++)
{
sop(y);
}
}
//打印
public static void sop(Object obj)
{
out.print(obj);
}
}

复制代码

作者: 赵家阳 时间: 2013-4-9 04:27
本帖最后由赵家阳于 2013-4-9 04:33 编辑

基本实现了能动态打印树，可根据自己想要的结果打印，但只能倒着打印。
树干不太容易控制，但基本还能看。
<div class="blockcode"><blockquote>import java.util.ArrayList;
class Tree{
private static ArrayList<Integer> al = new ArrayList<Integer>();
/*
* 用一个集合记录每次打印树尖的位置，主要是要用第一次打印的位置
*/
public static void main(String[] args){
printTree(8);
printTree(6);
printGan(6);
}
public static void printTree(int e){
for(int i=0;i<e;i++){
if(al.size()==0){ //al.size()==0,表示是第一次打印树
for(int j=0;j<e-i-1;j++){
System.out.print(" ");
}
}else{ //如果不是第一次打印，那么就根据第一次打印树的位置来判断这次需要多少空格
int a = al.get(0)-e;
for(int j=0;j<e-i-1+a;j++){
System.out.print(" ");
}
}
for(int j=0;j<=i;j++){
System.out.print("* ");
}
System.out.println();
}
al.add(e); //记录该次打印树尖的位置
}
public static void printGan(int k){
int j = al.get(0) - (k/2);
for(int m=0;m<k;m++){ //底层
for(int i=0;i<j;i++){ //控制树干前面打印的空格数
System.out.print(" ");
}
for(int n=0;n<k;n++){
System.out.print("*");
}
System.out.println();
}
}
}

复制代码

]58O{1GVL(2%N~Y4KHDGHTS.jpg (17.03 KB, 下载次数: 6)

作者: 黄小贝 时间: 2013-4-9 04:29
本帖最后由黄小贝于 2013-4-9 04:32 编辑

咳咳，让专业的来，我点评一下，撸主这完全是C代码的风格，可能是收了某高校谭姓教授的书籍的印象，变量的命令各种a，b，c的这样取，这样在java里面是不科学的，光少的写法单纯使用了extract method的重构办法，虽然一定程度改善了代码，但是感觉还是有些欠妥~阳少的那几个if条件语句让我有一种淡淡的忧伤~~

我下面的这种是面向对象的常规思维~~

public class Tree {
public static void main(String[] args) {
Triangle triangle1 = new Triangle(10,-1);
Triangle triangle2 = new Triangle(8,1);
Triangle triangle3 = new Triangle(6,3);
Rectangle rectangle = new Rectangle(6,5);
triangle1.show();
triangle2.show();
triangle3.show();
rectangle.show();
}
}
class Triangle{
private int size;
private int len;//为了对齐的偏移量，应该有某种数学规律，但是我没有总结出来，就穿死数字进去吧
public Triangle(int size, int len) {
this.size = size;
this.len = len;
}
public void setLevel(int size) {
this.size = size;
}
public void show() {
for (int i = 0; i < this.size; i++) {
for (int j = 0; j < this.size - i + len; j++) {
System.out.print(" ");
}
for (int j = 0; j <= i; j++) {
System.out.print("* ");
}
System.out.println();
}
}
}
class Rectangle{
private int with;
private int height;
public Rectangle(int with, int height) {
this.with = with;
this.height = height;
}
public void show(){
for (int i = 0; i < height; i++) { // 底层
System.out.print(" ");
for (int j = 0; j < with; j++) {
System.out.print("*");
}
System.out.println();
}
}
}

复制代码

这里我插一句，我对于毕老师的关于抽取的System.out.println()方法的做法持保留态度，这样做的优点是可以少写几个字母，可是~~那是你以为~~~~

如果你使用的是eclipse系列的神器的话，手打 sys 然后Alt+/

你会发现我们也只要打三个字母，而且还不影响程序可读性~~

作者: HM刘俊 时间: 2013-4-9 08:30

丘凤光发表于 2013-4-9 01:08
弄了老久，这样应该差不多了吧.。能封装的都尽量封装了，最主要的是数的大小可控制，树冠层数可以控制，只 ...

谢谢哈，看起来，舒服多了。

作者: HM刘俊 时间: 2013-4-9 08:39

黄小贝发表于 2013-4-9 04:29
咳咳，让专业的来，我点评一下，撸主这完全是C代码的风格，可能是收了某高校谭姓教授的书籍的印象，变量的 ...

谢谢。还有，你起来的好早啊。

作者: HM刘俊 时间: 2013-4-9 08:40

赵家阳发表于 2013-4-9 04:27

思路很清晰，谢了。

作者: 刘胜寒 时间: 2013-4-9 09:57
叼炸了........

作者: 刘胜寒 时间: 2013-4-9 10:33
给你奉上代码

public class ChristmasTree {
public static void main(String[] args)
{
printTree(6,10,0);
printTree(8,10,2);
printTree(10,10,4);
printlRec(10,10);
}
public static void printTree(int rol,int offset,int Init)
{
for(int i=0;i<rol;i++)
{
if(i<Init) continue;
for(int j=0;j<offset-i;j++)
{
System.out.print(" ");
}
for(int j=0;j<=i;j++)
{
System.out.print("* ");
}
System.out.println();
}
}
public static void printlRec(int rol,int offset)
{
for(int i=0;i<rol/2;i++)
{
for(int j=0;j<rol/2*1.5;j++)
{
System.out.print(' ');
}
for(int j=0;j<rol/2;j++)
{
System.out.print("*");
}
System.out.println();
}
}
}

复制代码

我就不给你注释解释了哈哈 ....

`VGEJ@LB5]OM@0CEOPB7CS8.jpg (45.93 KB, 下载次数: 2)

作者: 刘胜寒 时间: 2013-4-9 10:36

class Print
{
public static void printTree(int rol,int offset,int Init)
{
for(int i=0;i<rol;i++)
{
if(i<Init) continue;
for(int j=0;j<offset-i;j++)
{
System.out.print(" ");
}
for(int j=0;j<=i;j++)
{
System.out.print("* ");
}
System.out.println();
}
}
public static void printlRec(int rol,int offset)
{
for(int i=0;i<rol/2;i++)
{
for(int j=0;j<rol/2*1.5;j++)
{
System.out.print(' ');
}
for(int j=0;j<rol/2;j++)
{
System.out.print("*");
}
System.out.println();
}
}
}
public class ChristmasTree {
public static void main(String[] args)
{
Print print = new Print();
print.printTree(6,10,0);
print.printTree(8,10,2);
print.printTree(10,10,4);
print.printlRec(10,10);
}
}

复制代码

为了避免有c语言风格....

作者: 陈圳 时间: 2013-4-9 11:08

package exercises;
public class MerryTest {
public static void main(String[] args){
triangle(10,3,5);//给定三个条件，num:打印树形的长度，cas:打印次数，cols：树体长度
}
public static void triangle(int num,int cas,int cols){//
char spa=32;
char star=32;
int count=num;
for(int c=0;c<cas;c++,num-=2){//输出树体部分
for(int i=num,nums=count;i>0;){
for(int s=0;s<nums;s++)
System.out.print(spa);
for(int j=num;j>=i;j--){
System.out.print("*"+star);
if(num==6&&j==1){
num=3;
cas++;
nums=8;//
star='*';
}
}
System.out.println();
if(cas<=3){//打印三个树形，就让这些变量不改动了。
nums--;
i--;
}else cols--;
if(cols<0)//跳出条件，不然会死循环
return ;
}
}
}
}//一个循环体解决，尽可能减少了循环次数。小题目就不考虑面向对向封装类了。里面涉及了较多的条件判断。有点麻烦

复制代码

作者: 赵家阳 时间: 2013-4-9 13:26

似水像火发表于 2013-4-9 10:33
给你奉上代码我就不给你注释解释了哈哈 ....

够屌。。。。。。:lol

作者: 刘胜寒 时间: 2013-4-9 14:30

赵家阳发表于 2013-4-9 13:26
够屌。。。。。。

献丑了....

作者: 黄小贝 时间: 2013-4-9 16:38

HM刘俊发表于 2013-4-9 08:39
谢谢。还有，你起来的好早啊。

那时候还没有睡，我现在才起床~~~

作者: HM刘俊 时间: 2013-4-9 16:46

黄小贝发表于 2013-4-9 16:38
那时候还没有睡，我现在才起床~~~

你这作息时间真吊。传说中的通宵啊。

欢迎光临黑马程序员技术交流社区 (http://bbs.itheima.com/)

黑马程序员IT技术论坛 X3.2