X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/1169a91932273bc84c23ed9dbd0a2da064d59d66..daebb84bff054d3833baca58a5dc506de56c64a1:/src/univ/radiobox.cpp diff --git a/src/univ/radiobox.cpp b/src/univ/radiobox.cpp index 47b7c1a399..b51b63aecb 100644 --- a/src/univ/radiobox.cpp +++ b/src/univ/radiobox.cpp @@ -1,5 +1,5 @@ ///////////////////////////////////////////////////////////////////////////// -// Name: univ/radiobox.cpp +// Name: src/univ/radiobox.cpp // Purpose: wxRadioBox implementation // Author: Vadim Zeitlin // Modified by: @@ -17,10 +17,6 @@ // headers // ---------------------------------------------------------------------------- -#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA) - #pragma implementation "univradiobox.h" -#endif - #include "wx/wxprec.h" #ifdef __BORLANDC__ @@ -34,6 +30,7 @@ #include "wx/radiobox.h" #include "wx/radiobut.h" #include "wx/validate.h" + #include "wx/arrstr.h" #endif #include "wx/tooltip.h" @@ -73,7 +70,7 @@ public: { if ( m_radio->OnKeyDown((wxKeyEvent &)event) ) { - return TRUE; + return true; } } @@ -95,32 +92,40 @@ IMPLEMENT_DYNAMIC_CLASS(wxRadioBox, wxControl) // wxRadioBox creation // ---------------------------------------------------------------------------- -wxRadioBox::wxRadioBox() +void wxRadioBox::Init() { - Init(); + m_selection = -1; } -wxRadioBox::wxRadioBox(wxWindow *parent, - wxWindowID id, - const wxString& title, - const wxPoint& pos, - const wxSize& size, - int n, - const wxString *choices, - int majorDim, - long style, - const wxValidator& val, - const wxString& name) +wxRadioBox::wxRadioBox(wxWindow *parent, wxWindowID id, const wxString& title, + const wxPoint& pos, const wxSize& size, + const wxArrayString& choices, + int majorDim, long style, + const wxValidator& val, const wxString& name) { + wxCArrayString chs(choices); + Init(); - (void)Create(parent, id, title, pos, size, n, choices, majorDim, style, val, name); + (void)Create(parent, id, title, pos, size, chs.GetCount(), + chs.GetStrings(), majorDim, style, val, name); } -void wxRadioBox::Init() +bool wxRadioBox::Create(wxWindow *parent, + wxWindowID id, + const wxString& title, + const wxPoint& pos, + const wxSize& size, + const wxArrayString& choices, + int majorDim, + long style, + const wxValidator& val, + const wxString& name) { - m_selection = -1; - m_majorDim = 0; + wxCArrayString chs(choices); + + return Create(parent, id, title, pos, size, chs.GetCount(), + chs.GetStrings(), majorDim, style, val, name); } bool wxRadioBox::Create(wxWindow *parent, @@ -132,7 +137,7 @@ bool wxRadioBox::Create(wxWindow *parent, const wxString *choices, int majorDim, long style, - const wxValidator& val, + const wxValidator& wxVALIDATOR_PARAM(val), const wxString& name) { // for compatibility with the other ports which don't handle (yet?) @@ -159,7 +164,7 @@ bool wxRadioBox::Create(wxWindow *parent, } if ( !wxStaticBox::Create(parent, id, title, pos, size, style, name) ) - return FALSE; + return false; #if wxUSE_VALIDATORS SetValidator(val); @@ -168,7 +173,7 @@ bool wxRadioBox::Create(wxWindow *parent, Append(n, choices); // majorDim default value is 0 which means make one row/column - SetMajorDim(majorDim == 0 ? n : majorDim); + SetMajorDim(majorDim == 0 ? n : majorDim, style); if ( size == wxDefaultSize ) { @@ -183,7 +188,7 @@ bool wxRadioBox::Create(wxWindow *parent, // radiobox should already have selection so select at least one item SetSelection(0); - return TRUE; + return true; } wxRadioBox::~wxRadioBox() @@ -192,10 +197,10 @@ wxRadioBox::~wxRadioBox() // the buttons themselves: this must be done as the user code expects them // to disappear now and not some time later when they will be deleted by // our (common) parent - size_t count = m_buttons.GetCount(); - for ( size_t n = 0; n < count; n++ ) + unsigned int count = m_buttons.GetCount(); + for ( unsigned int n = 0; n < count; n++ ) { - m_buttons[n]->PopEventHandler(TRUE /* delete it */); + m_buttons[n]->PopEventHandler(true /* delete it */); delete m_buttons[n]; } @@ -205,26 +210,6 @@ wxRadioBox::~wxRadioBox() // wxRadioBox init // ---------------------------------------------------------------------------- -void wxRadioBox::SetMajorDim(int majorDim) -{ - wxCHECK_RET( majorDim != 0, _T("major radiobox dimension can't be 0") ); - - m_majorDim = majorDim; - - int minorDim = (GetCount() + m_majorDim - 1) / m_majorDim; - - if ( GetWindowStyle() & wxRA_SPECIFY_COLS ) - { - m_numCols = majorDim; - m_numRows = minorDim; - } - else // wxRA_SPECIFY_ROWS - { - m_numCols = minorDim; - m_numRows = majorDim; - } -} - void wxRadioBox::Append(int count, const wxString *choices) { if ( !count ) @@ -236,7 +221,7 @@ void wxRadioBox::Append(int count, const wxString *choices) { // make the first button in the box the start of new group by giving it // wxRB_GROUP style - wxRadioButton *btn = new wxRadioButton(parent, -1, choices[n], + wxRadioButton *btn = new wxRadioButton(parent, wxID_ANY, choices[n], wxDefaultPosition, wxDefaultSize, n == 0 ? wxRB_GROUP : 0); @@ -263,7 +248,7 @@ void wxRadioBox::SetSelection(int n) btn->SetFocus(); // this will also unselect the previously selected button in our group - btn->SetValue(TRUE); + btn->SetValue(true); } int wxRadioBox::GetSelection() const @@ -297,33 +282,47 @@ void wxRadioBox::OnRadioButton(wxEvent& event) // methods forwarded to the buttons // ---------------------------------------------------------------------------- -wxString wxRadioBox::GetString(int n) const +wxString wxRadioBox::GetString(unsigned int n) const { - wxCHECK_MSG( IsValid(n), _T(""), + wxCHECK_MSG( IsValid(n), wxEmptyString, _T("invalid index in wxRadioBox::GetString") ); return m_buttons[n]->GetLabel(); } -void wxRadioBox::SetString(int n, const wxString& label) +void wxRadioBox::SetString(unsigned int n, const wxString& label) { wxCHECK_RET( IsValid(n), _T("invalid index in wxRadioBox::SetString") ); m_buttons[n]->SetLabel(label); } -void wxRadioBox::Enable(int n, bool enable) +bool wxRadioBox::Enable(unsigned int n, bool enable) +{ + wxCHECK_MSG( IsValid(n), false, _T("invalid index in wxRadioBox::Enable") ); + + return m_buttons[n]->Enable(enable); +} + +bool wxRadioBox::IsItemEnabled(unsigned int n) const +{ + wxCHECK_MSG( IsValid(n), false, _T("invalid index in wxRadioBox::IsItemEnabled") ); + + return m_buttons[n]->IsEnabled(); +} + +bool wxRadioBox::Show(unsigned int n, bool show) { - wxCHECK_RET( IsValid(n), _T("invalid index in wxRadioBox::Enable") ); + wxCHECK_MSG( IsValid(n), false, _T("invalid index in wxRadioBox::Show") ); - m_buttons[n]->Enable(enable); + return m_buttons[n]->Show(show); } -void wxRadioBox::Show(int n, bool show) +bool wxRadioBox::IsItemShown(unsigned int n) const { - wxCHECK_RET( IsValid(n), _T("invalid index in wxRadioBox::Show") ); + wxCHECK_MSG( IsValid(n), false, _T("invalid index in wxRadioBox::IsItemShown") ); - m_buttons[n]->Show(show); + return m_buttons[n]->IsShown(); } // ---------------------------------------------------------------------------- @@ -333,31 +332,31 @@ void wxRadioBox::Show(int n, bool show) bool wxRadioBox::Enable(bool enable) { if ( !wxStaticBox::Enable(enable) ) - return FALSE; + return false; // also enable/disable the buttons - size_t count = m_buttons.GetCount(); - for ( size_t n = 0; n < count; n++ ) + const unsigned int count = m_buttons.GetCount(); + for ( unsigned int n = 0; n < count; n++ ) { Enable(n, enable); } - return TRUE; + return true; } bool wxRadioBox::Show(bool show) { if ( !wxStaticBox::Show(show) ) - return FALSE; + return false; // also show/hide the buttons - size_t count = m_buttons.GetCount(); - for ( size_t n = 0; n < count; n++ ) + const unsigned int count = m_buttons.GetCount(); + for ( unsigned int n = 0; n < count; n++ ) { Show(n, show); } - return TRUE; + return true; } wxString wxRadioBox::GetLabel() const @@ -376,8 +375,8 @@ void wxRadioBox::DoSetToolTip(wxToolTip *tooltip) wxControl::DoSetToolTip(tooltip); // Also set them for all Radio Buttons - size_t count = m_buttons.GetCount(); - for ( size_t n = 0; n < count; n++ ) + const unsigned int count = m_buttons.GetCount(); + for ( unsigned int n = 0; n < count; n++ ) { if (tooltip) m_buttons[n]->SetToolTip(tooltip->GetTip()); @@ -393,11 +392,11 @@ void wxRadioBox::DoSetToolTip(wxToolTip *tooltip) wxSize wxRadioBox::GetMaxButtonSize() const { - int widthMax, heightMax, width, height; + int widthMax, heightMax, width = 0, height = 0; widthMax = heightMax = 0; - int count = GetCount(); - for ( int n = 0; n < count; n++ ) + const unsigned int count = GetCount(); + for ( unsigned int n = 0; n < count; n++ ) { m_buttons[n]->GetBestSize(&width, &height); @@ -414,8 +413,8 @@ wxSize wxRadioBox::DoGetBestClientSize() const { wxSize sizeBtn = GetMaxButtonSize(); - sizeBtn.x *= m_numCols; - sizeBtn.y *= m_numRows; + sizeBtn.x *= GetColumnCount(); + sizeBtn.y *= GetRowCount(); // add a border around all buttons sizeBtn.x += 2*BOX_BORDER_X; @@ -443,15 +442,15 @@ void wxRadioBox::DoMoveWindow(int x0, int y0, int width, int height) int x = x0, y = y0; - int count = GetCount(); - for ( int n = 0; n < count; n++ ) + const unsigned int count = GetCount(); + for ( unsigned int n = 0; n < count; n++ ) { m_buttons[n]->SetSize(x, y, sizeBtn.x, sizeBtn.y); if ( GetWindowStyle() & wxRA_TOPTOBOTTOM ) { // from top to bottom - if ( (n + 1) % m_numRows ) + if ( (n + 1) % GetRowCount() ) { // continue in this column y += sizeBtn.y; @@ -466,7 +465,7 @@ void wxRadioBox::DoMoveWindow(int x0, int y0, int width, int height) else // wxRA_LEFTTORIGHT: mirror the code above { // from left to right - if ( (n + 1) % m_numCols ) + if ( (n + 1) % GetColumnCount() ) { // continue in this row x += sizeBtn.x; @@ -507,7 +506,7 @@ bool wxRadioBox::OnKeyDown(wxKeyEvent& event) break; default: - return FALSE; + return false; } int selOld = GetSelection(); @@ -520,8 +519,7 @@ bool wxRadioBox::OnKeyDown(wxKeyEvent& event) SendRadioEvent(); } - return TRUE; + return true; } #endif // wxUSE_RADIOBOX -