Чітка визначеність та одиничність мети методу

Потрібно намагатись уникати написання в одному методі блоків коду, що мають різне по суті призначення. Зокрема, комбінування в одному методі введення даних через засоби інтерфейсу користувача, обробку даних та виведення.

Некоректний підхід

Коректний підхід

class StringProcessing{ public static void countLetters(char letter) { String initialString = textIn.getText(); int counter=0; for(int i=0; i<initialString.length();i++) if (initialString.charAt(i)==letter) counter++; textOut.setText(""+counter); } public static void main(String[] args){ JFrame frame=new JFrame("Laba 1 "); frame.setDefaultCloseOperation( JFrame.EXIT_ON_CLOSE); JButton btn=new JButton("Start"); Label infoLabel=new Label("Lababoratorna1", Label.CENTER); JTextField textIn=new JTextField(); JTextField textOut=new JTextField(); frame.getContentPane().add(infoLabel); frame.getContentPane().add(textIn, BorderLayout.NORTH); frame.getContentPane().add(textOut, BorderLayout.SOUTH); frame.getContentPane().add(btn,BorderLayout.EAST); frame.setSize(450,100); frame.setLocation(300,250); frame.setVisible(true); frame.setResizable(false); btn.addActionListener(new ActionListener(){ public void actionPerformed(ActionEvent e){ countLetters('a'); } }); } }

class StringProcessing{ public static int countLetters(String initialString, char letter) { int counter=0; for(int i=0; i<initialString.length();i++) if (initialString.charAt(i)==letter) counter++; return counter; } public static void main(String[] args){ JFrame frame=new JFrame("Laba 1 "); frame.setDefaultCloseOperation( JFrame.EXIT_ON_CLOSE); JButton btn=new JButton("Start"); Label infoLabel=new Label("Lababoratorna1", Label.CENTER); JTextField textIn=new JTextField(); JTextField textOut=new JTextField(); frame.getContentPane().add(infoLabel); frame.getContentPane().add(textIn, BorderLayout.NORTH); frame.getContentPane().add(textOut, BorderLayout.SOUTH); frame.getContentPane().add(btn,BorderLayout.EAST); frame.setSize(450,100); frame.setLocation(300,250); frame.setVisible(true); frame.setResizable(false); btn.addActionListener(new ActionListener(){ public void actionPerformed(ActionEvent e){ String initialString = textIn.getText(); textOut.setText(""+countLetters(initialString,'a')); } }); } }

Читайте також:

<== попередня сторінка	\|	наступна сторінка ==>
Принцип DRY (Don't Repeat Yourself)	\|	Інкапсуляція

Не знайшли потрібну інформацію? Скористайтесь пошуком google:

Генерація сторінки за: 0.005 сек.