Программа на Java

@SLAVIKIWI · Регистрация: 29.09.2013

Author24 — интернет-сервис помощи студентам

С горем пополам( с гуглом и без) я таки написал свою первую типа программу на Java. Выкладываю сейчас здесь эту прогу, чтобы более опытные люди посмотрели и указали мои серьезные ошибки, недочеты и прочее. Буду очень благодарен им.

Java

import java.awt.*;
import javax.swing.*;
import java.awt.event.*;
 
 
 
class Kalkul{
    
 
 
public class Box {
    public double Value;
    public String Name;
 
    public Box(double val, String name){
        Value = val;
        Name = name;
    }
 
    public String toString(){
       return Name;
    }
}
 
 
public class Kof {
 
 
    public Box[] baseTarif(){
        return new Box[]{
                new Box(1215,"Струйный"),
                new Box(2965, "Лазерный"),
 
        };
 
    }
}
 
Kof _Koff = new Kof();
 
JFrame form1, form2, form3;
JPanel pan1, pan2, pan3; 
JTabbedPane tp, tp1, tp2;
JLabel lab1, lab2, lab3, lab4,lab5,lab6,lab7,lab8,lab9,lab10, lab11,lab12,lab13,lab14,lab15,lab16;
JTextField tf1,tf2,tf3,tf4,tf5,tf6,tf7,tf8,tf9,tf10,tf11,tf12,tf13,tf14;
JButton but1, but2, but3, but4, but5, but6; 
JComboBox <?> baseTarif = extracted();
 
private JComboBox<?> extracted() {
    return new JComboBox<Object>(_Koff.baseTarif());
}
 
private String ComboBox(JComboBox <?>  baseTarif) {
    
    return ((Box)baseTarif.getSelectedItem()).Name;
    }
    
 
 
 
Kalkul() {
form1=new JFrame("Расчет загрузки принтера");
form2=new JFrame("Расчет загрузки принтера");
form3=new JFrame("Расчет загрузки принтера");
pan1=new JPanel(new GridLayout(11,9));
pan2=new JPanel(new GridLayout(11,9));
pan3=new JPanel(new GridLayout(11,9));
 
pan1.setBackground(Color.GRAY);
pan2.setBackground(Color.GRAY);
pan3.setBackground(Color.GRAY);
 
 
tp=new JTabbedPane();
tp1=new JTabbedPane();
tp2=new JTabbedPane();
lab1=new JLabel("Здравствуте!Вас приветствует программа по расчету расхода чернил (тонера) ");
lab2=new JLabel("офисным принтером и годовых затрат, связанных с заправкой принтера.");
lab3=new JLabel("Емкость картриджа (мг или миллилитр):");
lab4=new JLabel("Количество печатаемых в день листов А4 (среднее,штук):");
lab5=new JLabel("Стоимость  картриджа для вашего принтера(в руб.):");  
lab6=new JLabel("Стоимость  материала для заправки(в руб.):");
lab7=new JLabel("Стоимость одной заправки (диагностики) в салоне(в руб.):");
lab8=new JLabel("Оставшееся время до новой заправки(дней):");
lab9=new JLabel("Затраты на новые картриджи(в руб,если потребуется):");
lab10=new JLabel("Годовые затраты на чернила или тонер(в руб):");
lab11=new JLabel("Годовые затраты на заправку(диагностику) в салоне(в руб):");
lab12=new JLabel("Суммарные годовые  затраты(в руб):");
lab13=new JLabel("Количество замен картриджа в год (если потребуется):");
lab14=new JLabel("Количество заправок в год(каждую 6-ую лучше проводить в салоне):");
lab15=new JLabel("Выбирете тип используемого принтера:");
lab16=new JLabel("");
 
 
tf3=new JTextField(12);
tf4=new JTextField(12);
tf5=new JTextField(12);
tf6=new JTextField(12);
tf7=new JTextField(12);
tf8=new JTextField(12);
tf9=new JTextField(12);
tf10=new JTextField(12);
tf11=new JTextField(12);
tf12=new JTextField(12);
tf13=new JTextField(12);
tf14=new JTextField(12);
 
 
 
 
but1=new JButton("Далее");
but2=new JButton("Назад");
but3=new JButton("Рассчитать");
but4=new JButton("Завершить");
but5=new JButton("В начало");
but6=new JButton("Очистить");
 
but1.addActionListener(new ActionListener() {
public void actionPerformed(ActionEvent e)
{
    
    form1.setVisible(false);
    
    form2.setVisible(true);
}
});
 
but2.addActionListener(new ActionListener() {
public void actionPerformed(ActionEvent e)
{
    
    form2.setVisible(false);
    form1.setVisible(true);
}
});
 
but3.addActionListener(new ActionListener() {
public void actionPerformed(ActionEvent e)
{
int Y,L,G,D,H,R;
double X,N,C,T,S,Z,W,P;
double A = 0;
 
X=Double.parseDouble(tf3.getText());
Y=(int)Double.parseDouble(tf4.getText());
T=Double.parseDouble(tf5.getText());
N=Double.parseDouble(tf6.getText());
C=Double.parseDouble(tf7.getText());
 
if (ComboBox(baseTarif) == "Струйный") 
    A = 0.6;
if (ComboBox(baseTarif) == "Лазерный") 
    A = 55;
 
L= (int) ((int)X/(Y*A));
G= 250/L;
D = G/6;
H = G-D;
S = H*N;
P = D*C;
R = (250*Y)/15000;
Z = R*T;
W = S+P+Z;
String s=String.valueOf(L);
tf8.setText(s);
String s14=String.valueOf(G);
tf14.setText(s14);
String s1=String.valueOf(Z);
tf9.setText(s1);
String s2=String.valueOf(S);
tf10.setText(s2);
String s3=String.valueOf(P);
tf11.setText(s3);
String s4=String.valueOf(W);
tf12.setText(s4);
String s5=String.valueOf(R);
tf13.setText(s5);
 
 
form2.setVisible(false);
form3.setVisible(true);
}
});
 
 
 
but5.addActionListener(new ActionListener() {
public void actionPerformed(ActionEvent e)
{
    
    form3.setVisible(false);
    form1.setVisible(true);
}
});
 
but6.addActionListener(new ActionListener() {
public void actionPerformed(ActionEvent e)
{ tf3.setText("");
tf4.setText("");
tf5.setText("");
tf6.setText("");
tf7.setText("");
}
});
but4.addActionListener(new ActionListener() {
public void actionPerformed(ActionEvent e)
{
    
    form3.setVisible(false);
}
});
 
 
 
pan2.add(lab3);
pan2.add(tf3);
pan2.add(lab4);
pan2.add(tf4);
pan2.add(lab5);
pan2.add(tf5);
pan2.add(lab6);
pan2.add(tf6);
pan2.add(lab7);
pan2.add(tf7);
pan2.add(but2);
pan2.add(but3);
pan2.add(but6);
pan1.add(lab1);
pan1.add(lab2);
pan1.add(lab15);
pan1.add(baseTarif);
pan1.add(lab16);
pan1.add(but1);
pan3.add(lab8);
pan3.add(tf8);
pan3.add(lab14);
pan3.add(tf14);
pan3.add(lab9);
pan3.add(tf9);
pan3.add(lab10);
pan3.add(tf10);
pan3.add(lab11);
pan3.add(tf11);
pan3.add(lab12);
pan3.add(tf12);
pan3.add(lab13);
pan3.add(tf13);
pan3.add(but5);
pan3.add(but4);
 
 
 
}
 
 
 
void dis()
{
form1.getContentPane().add(tp);
tp.addTab("Приветствие",pan1);
 
 
form2.getContentPane().add(tp1);
tp1.addTab("Введите следующие показатели",pan2);
 
form3.getContentPane().add(tp2);
tp2.addTab("Получены следующие результаты",pan3);
 
 
form1.setSize(900,550); //размер программы (ширина*длина)
form1.setVisible(true);
form1.setResizable(true);
 
form2.setSize(900,550); 
form2.setVisible(false);
form2.setResizable(true);
 
form3.setSize(900,550); 
form3.setVisible(false);
form3.setResizable(true);
 
}
public static void main(String z[]) {
Kalkul pro=new Kalkul();
pro.dis();
}
 
}

@Gibby · 01.11.2013, 14:04

1. Отсутствие тэгов [JAVA] в посте
2. абсолютно произвольные отступы в коде
3. неинформативное название переменных
4. Сравнение объектов через "==" вместо equals()
5. Отсутствие @Override там, где необходимо
6. бизнес-логика не отделена от GUI
7. все запихано в один класс, в итоге каша

~~verylazy~~ · 01.11.2013, 14:32

8. переменные с заглавной буквы (или состоящие из одной заглавной)

пардон, что вмешиваюсь, я сам-то учусь, но как-же пройти мимо когда в интернете кто-то не прав

@SLAVIKIWI · 06.11.2013, 19:28 **[ТС]**

Gibby, спасибо за подсказки. И кстати что такое @Override??? И с чем едят?

@easybudda · 06.11.2013, 20:32

Сообщение от SLAVIKIWI

import javax.swing.*;

Сообщение от SLAVIKIWI

public class Box {

В javax.swing уже есть такой:
http://docs.oracle.com/javase/... g/Box.html

@SLAVIKIWI · 06.11.2013, 20:39 **[ТС]**

easybudda, то есть я чего то лишнего понаписал???

@Gibby · 06.11.2013, 20:55

Сообщение от SLAVIKIWI

И кстати что такое @Override??? И с чем едят?

@Override - это аннотация, которая говорит JVM - "этот метод был переопределен в наследнике. Проверь - правильно ли это сделано". Таким образом вы избежите казуса, когда из-за неправильной сигнатуры метода вы не переопределяете его, а создаете новый.

@SLAVIKIWI · 10.11.2013, 16:46 **[ТС]**

А кто-нибудь сможет помочь написать хорошие комментарии к следующим строкам?
public class Box {
}

Java

    public double Value;
    public String Name;
 
    public Box(double val, String name){
        Value = val;
        Name = name;
    }
 
    public String toString(){
       return Name;
    }
}
 
 
public class Kof {
 
 
    public Box[] baseTarif(){
        return new Box[]{
                new Box(1215,"Струйный"),
                new Box(2965, "Лазерный"),
 
        };
 
    }

Java

private JComboBox<?> extracted() {
    return new JComboBox<Object>(_Koff.baseTarif());
}
 
private String ComboBox(JComboBox <?>  baseTarif) {
return ((Box)baseTarif.getSelectedItem()).Name;
    }

@Gibby 154 / 154 / 10 Регистрация: 16.10.2012 Сообщений: 354 Записей в блоге: 1
	01.11.2013, 14:04	2
	1. Отсутствие тэгов [JAVA] в посте 2. абсолютно произвольные отступы в коде 3. неинформативное название переменных 4. Сравнение объектов через "==" вместо equals() 5. Отсутствие @Override там, где необходимо 6. бизнес-логика не отделена от GUI 7. все запихано в один класс, в итоге каша 1

~~verylazy~~ Заблокирован
	01.11.2013, 14:32	3
	8. переменные с заглавной буквы (или состоящие из одной заглавной) пардон, что вмешиваюсь, я сам-то учусь, но как-же пройти мимо когда в интернете кто-то не прав 0

@SLAVIKIWI 2 / 2 / 1 Регистрация: 29.09.2013 Сообщений: 57
	06.11.2013, 19:28 [ТС]	4
	Gibby, спасибо за подсказки. И кстати что такое @Override??? И с чем едят? 0

@easybudda Модератор 12458 / 7482 / 1753 Регистрация: 25.07.2009 Сообщений: 13,762
	06.11.2013, 20:32	5
	Сообщение от SLAVIKIWI import javax.swing.; Сообщение от SLAVIKIWI* public class Box { В javax.swing уже есть такой: http://docs.oracle.com/javase/... g/Box.html 0

@SLAVIKIWI 2 / 2 / 1 Регистрация: 29.09.2013 Сообщений: 57
	06.11.2013, 20:39 [ТС]	6
	easybudda, то есть я чего то лишнего понаписал??? 0

@Gibby 154 / 154 / 10 Регистрация: 16.10.2012 Сообщений: 354 Записей в блоге: 1
	06.11.2013, 20:55	7
	Сообщение от SLAVIKIWI И кстати что такое @Override??? И с чем едят? @Override - это аннотация, которая говорит JVM - "этот метод был переопределен в наследнике. Проверь - правильно ли это сделано". Таким образом вы избежите казуса, когда из-за неправильной сигнатуры метода вы не переопределяете его, а создаете новый. 1

	10.11.2013, 16:46