SWT Für jede gui komponente eigene methode

[schuetzejanett]

Hallo,

bin gerade dabei mir ein swt fenster zu erstellen, habe das erst mit dem visual class editor von eclipse versucht aber das wurde mirr schnell zu unübersichtlich. Also habe ich für jede Gui Komponente ne eigene getmethode geschriben die prüft ob sie schon existiert(nicht null) ist und wenn sie noch nicht existiert sie anlegt und alle eigenschaften setzt und listener usw.
Und dann füge ich in der initializee Methode nur noch durch aufruf der Methoden zusammen ist das so ratsam oder nicht und muss ich wenn ich die helloworld rcp von eclipse als grundlage benutze noch irgendwas zum löschen aufrufen ist ja schon in der Klasse Application drin oder?

 

[byte]

Mal abgesehen davon, dass ich gar nich verstanden habe, was Du uns sagen willst: OK ist, was funktioniert.

Wenns Dir um best practice geht in Sachen SWT (oder doch RCP? geht aus Deinem Post nich klar hervor...) dann würde ich mit ApplicationWindow arbeiten zum erstellen des Fensters. Dort steckt schon ein gutes Konzept drin, um die gängigen Komponenten eines Fensters zu initialisieren. Brauchst nur die geeigneten Methoden zu überschreiben.

 

[schuetzejanett]

Ich meinte das erstellen einer view oder besser gesagt einer speziellen composite die ich in meine view einfüge und ich wollte darauf hinaus ob man das erstellen der komponenten besser in eine methode packt also alles einfach hintereinander anlegt oder ob man es in extra methoden schreibt und diese dann nur zum anlegen nur aufruft

Habe es jetztb so gemacht und wollte halt wissen ob das guter oder schlechter Programiierstil bisher ist.
Die Listener kommen noch hinzu bin halt erstmal nur beim aufbau

public class SearchComposite extends Composite {

	private Label label = null;
	private Group groupSearch = null;
	private Button radioPaperKind = null;
	private Combo comboPaperKind = null;
	private Button radioSearch = null;
	private Combo comboSearchField = null;
	private Text textSearchValue = null;
	private Label labelFiller = null;
	private Group groupFilter = null;
	private Combo comboInland = null;
	private Text textMaxCount = null;
	private Button buttonSearch = null;
	
	
	public SearchComposite(Composite parent, int style) 
	{
		super(parent, style);
		initialize();
	}

	private void initialize() 
	{
		setSize(new Point(700, 200));
		GridLayout gridLayout = new GridLayout(); 
		gridLayout.marginHeight = 20;	
		gridLayout.horizontalSpacing = 30;
		gridLayout.marginRight = 30;
		gridLayout.numColumns = 2;
		
		setLayout(gridLayout);	
		getGroupSearch();
		getButtonSearch();
		getGroupFilter();
		
	}

	/**
	 * This method initializes groupSearch	
	 *
	 */
	private Group getGroupSearch() 
	{
		
		if (radioPaperKind == null)
		{
			GridLayout gridLayout = new GridLayout();
			gridLayout.numColumns =10;
			gridLayout.horizontalSpacing = 10;	
			
			groupSearch = new Group(this, SWT.NONE);		
			groupSearch.setLayout(gridLayout);
			groupSearch.setLayoutData(new GridData(GridData.FILL_HORIZONTAL));			
			
			getRadioPaperKind();		
			getComboPaperKind(); 
			getLabelFiller();		
			getRadioSearch();
			getComboSearchField();
			getTextSearchValue();			
		}		
		return groupSearch;
	}
	
	/**
	 * This method initializes radioPaperkind	
	 *
	 */
	private Button getButtonSearch() 
	{
		if (buttonSearch == null)
		{
			buttonSearch = new Button(this, SWT.NONE);
			buttonSearch.setText("Suchen");		
			GridData gridData = new GridData();
			gridData.verticalSpan = 2;			
			gridData.widthHint = 70;
			gridData.horizontalAlignment = GridData.CENTER;
			buttonSearch.setLayoutData(gridData);			
		}		
		return radioPaperKind;
	}
	
	/**
	 * This method initializes radioPaperkind	
	 *
	 */
	private Button getRadioPaperKind() 
	{
		if (radioPaperKind == null)
		{
			radioPaperKind = new Button(groupSearch, SWT.RADIO);
			radioPaperKind.setText("Papierart");
			radioPaperKind.setSelection(true);				
		}		
		return radioPaperKind;
	}