susha

بهینه سازی کد

2 ارسال در این موضوع قرار دارد

اقا من یک برنامه با جاوا برای اندروید میخوام بنویسم به این صورت که 2 تا ارایه داریم کلمات انگلیسی و ترجمه فارسیشون ... کلمه انگلیسی رندوم انتخاب میشه در یک کادر قرار میگیره، جوابش هم در بین 4 کادر به صورت رندوم قرار میگیره، بعد 3 کلمه دیگه از ارایه ترجمه فارسی انتخاب میشه و بعد در کادر گزینه های خالی به صورت رندوم پر میشه

کدش این هست که البته جواب میده و درست هست:

[code]public class ActivityEnglish extends ActivityEnhanced {

    private boolean select1 = false;
    private boolean select2 = false;
    private boolean select3 = false;
    private boolean select4 = false;


    private boolean part1 = false;
    private boolean part2 = false;
    private boolean part3 = false;
    private boolean part4 = false;


    private String currentwordEnglish;
    private String currentwordFarsi;


    private String answer1;
    private String answer2;
    private String answer3;
    private String answer4;


    private TextView txtSelect1;
    private TextView txtSelect2;
    private TextView txtSelect3;
    private TextView txtSelect4;
    private TextView txtTimer;


    private int randomNumber;


    private String[] wordsEnglish = new String[] { "Hello", "Majid", "Water",
            "Air", "Apple", "Football", "Glass", "Monitor", "Nokia" };


    private String[] wordsFarsi = new String[] { "سلام", "مجید", "آب", "هوا",
            "سیب", "فوتبال", "لیوان", "مانیتور", "نوکیا" };


 @override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.form_english);


        txtTimer = (TextView) findViewById(R.id.txtTimer);


        txtSelect1 = (TextView) findViewById(R.id.txtSelect1);
        txtSelect2 = (TextView) findViewById(R.id.txtSelect2);
        txtSelect3 = (TextView) findViewById(R.id.txtSelect3);
        txtSelect4 = (TextView) findViewById(R.id.txtSelect4);


        start();
    }


    private void start() {
        if (!part1) {
            randomNumber = (int) (Math.random() * wordsEnglish.length);
            currentwordEnglish = wordsEnglish[randomNumber];
            currentwordFarsi = wordsFarsi[randomNumber];
            answer1 = wordsFarsi[randomNumber];
            txtTimer.setText("" + currentwordEnglish);
            putSelect();
            part1 = true;


        }
        if (!part2) {
            randomNumber = (int) (Math.random() * wordsFarsi.length);
            currentwordFarsi = wordsFarsi[randomNumber];


            if (answer1 == currentwordFarsi) {
                start();
            }
            answer2 = wordsFarsi[randomNumber];
            putSelect();
            part2 = true;
        }
        if (!part3) {
            randomNumber = (int) (Math.random() * wordsFarsi.length);
            currentwordFarsi = wordsFarsi[randomNumber];


            if (answer1 == currentwordFarsi) {
                start();
            } else if (answer2 == currentwordFarsi) {
                start();
            }
            answer3 = wordsFarsi[randomNumber];
            putSelect();
            part3 = true;
        }
        if (!part4) {
            randomNumber = (int) (Math.random() * wordsFarsi.length);
            currentwordFarsi = wordsFarsi[randomNumber];


            if (answer1 == currentwordFarsi) {
                start();
            } else if (answer2 == currentwordFarsi) {
                start();
            } else if (answer3 == currentwordFarsi) {
                start();
            }
            answer4 = wordsFarsi[randomNumber];
            putSelect();
            part4 = true;
        }


    }


    private void putSelect() {


        int randomNumberSelect = (int) ((Math.random() * 4) + 1);


        switch (randomNumberSelect) {
        case 1:
            if (!select1) {
                select1 = true;
                txtSelect1.setText(currentwordFarsi);
                break;
            } else if (!select2) {
                select2 = true;
                txtSelect2.setText(currentwordFarsi);
                break;
            } else if (!select3) {
                select3 = true;
                txtSelect3.setText(currentwordFarsi);
                break;
            } else if (!select4) {
                select4 = true;
                txtSelect4.setText(currentwordFarsi);
                break;
            }


        case 2:
            if (!select2) {
                select2 = true;
                txtSelect2.setText(currentwordFarsi);
                break;
            } else if (!select1) {
                select1 = true;
                txtSelect1.setText(currentwordFarsi);
                break;
            } else if (!select3) {
                select3 = true;
                txtSelect3.setText(currentwordFarsi);
                break;
            } else if (!select4) {
                select4 = true;
                txtSelect4.setText(currentwordFarsi);
                break;
            }


        case 3:
            if (!select3) {
                select3 = true;
                txtSelect3.setText(currentwordFarsi);
                break;
            } else if (!select2) {
                select2 = true;
                txtSelect2.setText(currentwordFarsi);
                break;
            } else if (!select1) {
                select1 = true;
                txtSelect1.setText(currentwordFarsi);
                break;
            } else if (!select4) {
                select4 = true;
                txtSelect4.setText(currentwordFarsi);
                break;
            }


        case 4:
            if (!select4) {
                select4 = true;
                txtSelect4.setText(currentwordFarsi);
                break;
            } else if (!select2) {
                select2 = true;
                txtSelect2.setText(currentwordFarsi);
                break;
            } else if (!select3) {
                select3 = true;
                txtSelect3.setText(currentwordFarsi);
                break;
            } else if (!select1) {
                select1 = true;
                txtSelect1.setText(currentwordFarsi);
                break;
            }


        }
    }
}

این کد خیلی پیچ در پیچ هست و من خودم هم کلی بالا پایینش کردم تا درست شد .. میخواستم بدونم راه حلی هست که راه کوتاه تر کنه ؟

یا اگر بخوام به جای اینکه در یک کلاس این کار صورت بگیره در چند گلاس صورت بگیره چی کار باید بکنم ؟

[/code]

به اشتراک گذاری این پست


لینک به ارسال
به اشتراک گذاری در سایت های دیگر

می تونی اون قسمت های کدت رو که تکراریه تو قالب یه متد قرار بدی و اون متد رو هر جای برنامه که خواستی فراخوانی کنی، این جوری بهینه میشه.

به اشتراک گذاری این پست


لینک به ارسال
به اشتراک گذاری در سایت های دیگر

برای ارسال نظر یک حساب کاربری ایجاد کنید یا وارد حساب خود شوید

برای اینکه بتوانید نظر ارسال کنید نیاز دارید که کاربر سایت شوید

ایجاد یک حساب کاربری

برای حساب کاربری جدید در انجمن ما ثبت نام کنید. عضویت خیلی ساده است !


ثبت نام یک حساب کاربری جدید

ورود به حساب کاربری

دارای حساب کاربری هستید؟ از اینجا وارد شوید


ورود به حساب کاربری