Discussion :

[vdumont]

Bonjour, supposons le constructeur suivant:

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
 
MyFrame::MyFrame(wxWindow *parent, const wxString& title, const wxPoint& pos,
   const wxSize& size, long style) :  wxFrame(parent, wxID_ANY, title, pos, size, style)
{
   wxSize sizeTreeview(223,400);
   wxSize sizeGlCanvas(757, 527);
 
   /* Menus et status bar */
   m_menuFile = new wxMenu;
   m_menuFile->Append(ID_OPENFILE,wxT("&Ouvrir"));
   m_menuFile->AppendSeparator();
   m_menuFile->Append(ID_QUIT,wxT("Quitter"));
 
   m_menuMode = new wxMenu;
   m_menuMode->Append(ID_WIRE,wxT("&Filaire"));
   m_menuMode->Append(ID_RENDER,wxT("&Rendu"));
 
   m_menuBar = new wxMenuBar();
   m_menuBar->Append(m_menuFile,wxT("&Fichier"));
   m_menuBar->Append(m_menuMode,wxT("&Mode"));
   this->SetMenuBar(m_menuBar);
   this->CreateStatusBar(1);
   m_menuBar->EnableTop(1,false); // D�sactiver le menu Mode
 
 
   this->SetBackgroundColour(wxSystemSettings::GetColour(wxSYS_COLOUR_BTNFACE));
   this->SetIcon(wxIcon(vm_xpm));   
 
 
 
 /* Cr�ation des contr�les */    
   m_btnClose = new wxButton(this,ID_BTNCLOSE,wxT("Quitter"),wxPoint(910,500), wxDefaultSize);
   m_treeCtrl = new MyTreeCtrl(this, ID_TREECTRL, wxPoint(759, 0), sizeTreeview, wxTR_DEFAULT_STYLE | wxTR_HIDE_ROOT);
 
   m_glCanvas =  new GL_MAILLAGE(this, wxID_ANY, wxDefaultPosition, sizeGlCanvas,0,wxT("GLCanvas")); 
   m_glCanvas->m_glCanvasSize = sizeGlCanvas;
 
 
   /* Sizers */
   wxBoxSizer *mainSizer = new wxBoxSizer(wxHORIZONTAL); 
   mainSizer->Add(m_glCanvas, 1, wxEXPAND, 0); 
   wxBoxSizer *tempSizer = new wxBoxSizer(wxVERTICAL); 
   tempSizer->Add(m_treeCtrl, 1, wxEXPAND | wxRIGHT | wxLEFT | wxBOTTOM | wxALIGN_TOP, 5); 
   tempSizer->Add(m_btnClose, 0, wxALIGN_RIGHT | wxALIGN_BOTTOM,  0);               
   mainSizer->Add(tempSizer, 0, wxEXPAND | wxALIGN_RIGHT | wxALL, 2); 
   this->SetSizer(mainSizer); 
}

Est-ce qu'il est pr�f�rable de laisser ca comme ca, ou de faire des fonctions (m�mes si ils vont �tre appel�s juste une seule fois) et de les appeler pour ainsi "diminuer" le nombre de ligne dans le constructeur??<

Merci

[Laurent Gomila]

Bof, pas sp�cialement non. Au contraire mieux vaut ne pas �clater le code (qui n'est globalement que des cr�ations de cont�les) et garder ta classe clean.

[vdumont]

Merci pour ta r�ponse