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