Project

General

Profile

Actions

Task #4793

closed

duplicate code

Added by Alexey Demakov about 10 years ago. Updated about 9 years ago.

Status:
Closed
Priority:
Normal
Category:
-
Target version:
Start date:
03/25/2014
Due date:
% Done:

100%

Estimated time:
Detected in build:
svn
Published in build:
0.15.197

Description

методы isEnabled в
requality\src\java\com\unitesk\requality\eclipse\handlers\copypaste\EclipseCopyHandler.java
requality\src\java\com\unitesk\requality\eclipse\handlers\copypaste\EclipsePasteHandler.java

отличаются только константами "org.eclipse.ui.edit.copy" и "org.eclipse.ui.edit.paste". Эти же константы используются при создании хэндлеров в классах ReqMarker, JSEditor и ReportViewer:

handlerService
.activateHandler("org.eclipse.ui.edit.copy", new EclipseCopyHandler());
handlerService
.activateHandler("org.eclipse.ui.edit.paste", new EclipsePasteHandler());

Предлагаю:
1. Добавить в классы EclipseCopyHandler и EclipsePasteHandler публичные именованные константы для этих строк и именно их использовать в проекте.
2. Создать общий базовый класс, где объявить абстрактный виртуальный метод типа getCommandId(), который в наследниках возвращает необходимое значение.
3. Перенести метод isEnabled в общий базовый класс, заменив использование констант на вызов getCommandId()

Actions #1

Updated by Denis Kildishev almost 10 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100

Applied in changeset r3510.

Actions #2

Updated by Denis Kildishev almost 10 years ago

  • Published in build set to 0.15.197
Actions #3

Updated by Alexey Khoroshilov almost 10 years ago

  • Target version set to 0.16
Actions #4

Updated by Alexey Khoroshilov about 9 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF