Как заменить многие заявления if в Java? - PullRequest
Новая
       28

Как заменить многие заявления if в Java?

2 голосов
/ 03 июня 2019

Я пытаюсь сделать мой код "чище". У меня много функций с множеством операторов if. Как я могу заменить их?

Я использую 4 разных Hashtables, поэтому у меня есть эти if. Меняется только имя хеш-таблицы и jList, куда я добавляю информацию. 

if(n.getCategorie().getNum() == Categorie.INTERNATIONAL.getNum())
                {
                    DefaultListModel dlm = (DefaultListModel) jListInternationales.getModel();
                    dlm.addElement(n.getTitre());
                    jListInternationales.setModel(dlm);
                    NewsInter.put(""+nNews, n);
                }
                else if(n.getCategorie().getNum() == Categorie.POLITIQUE.getNum())
                {
                    DefaultListModel dlm = (DefaultListModel) jListViePolitique.getModel();
                    dlm.addElement(n.getTitre());
                    jListViePolitique.setModel(dlm);
            NewsPolitique.put(""+nNews, n);
                }
                else if(n.getCategorie().getNum() == Categorie.RAGOT.getNum())
                {
                    DefaultListModel dlm = (DefaultListModel) jListRagotsEtPotins.getModel();
                    dlm.addElement(n.getTitre());
                    jListRagotsEtPotins.setModel(dlm);
            NewsRagot.put(""+nNews, n);
                }
                else if(n.getCategorie().getNum() == Categorie.SPORT.getNum())
                {
                    DefaultListModel dlm = (DefaultListModel) jListInfosSports.getModel();
                    dlm.addElement(n.getTitre());
                    jListInfosSports.setModel(dlm);
            NewsSport.put(""+nNews, n);
                }

Ответы [ 5 ]

3 голосов
/ 03 июня 2019

Единственная разница между ветвями if исходит из ссылки JList. Поскольку код повторяется, вы должны извлечь его в отдельный метод: 

private void updateList(JList list) {
  DefaultListModel dlm = (DefaultListModel) list.getModel();
  dlm.addElement(n.getTitre());
  list.setModel(dlm);
}

После этого код становится намного чище: 

if (n.getCategorie().getNum() == Categorie.INTERNATIONAL.getNum()) {
  updateList(jListInternationales);
} else if (n.getCategorie().getNum() == Categorie.POLITIQUE.getNum()) {
  updateList(jListViePolitique);
} // etc

NewsInter.put(""+nNews, n);
3 голосов
/ 03 июня 2019

Кажется, вы используете перечисления.Перечисления могут сравниваться по экземплярам.Проверка номеров не требуется.

Они могут использоваться в корпусе коммутатора.Например, 

switch (n.getCategorie()) {
case Categorie.INTERNATIONAL:
...

Это устраняет некоторые помехи, но не существенно количество операторов if.

0 голосов
/ 03 июня 2019

Используйте команду Switch Case, которая занимает меньше памяти при обработке

0 голосов
/ 03 июня 2019

в вашем случае вы можете заменить его на Switch оператор.

Это структура: 

switch(value) {
  case x:
    // code block
    break;
  case y:
    // code block
    break;
  default:
    // code block
}

В вашей конкретной проблеме это будет: 

switch(n.getCategorie()) {
  case International:
    // your code
    break;
  case Politique:
    // code block
    break;
...

  default:
    // code block
}
0 голосов
/ 03 июня 2019

Вы можете заменить структуру if, else if, else if, ..., изменив ее на структуру switch (), она выглядит следующим образом: 

switch(n.getCategorie().getNum()){
    case Categorie.INTERNATIONAL.getNum():
        DefaultListModel dlm = (DefaultListModel) jListInternationales.getModel();
        dlm.addElement(n.getTitre());
        jListInternationales.setModel(dlm);
        NewsInter.put(""+nNews, n);

        break;
    case Categorie.POLITIQUE.getNum():
        ...etc
}
...