Оптимизировать код выбора сложного условного ветвления

[calypso]

Думал-думал,как это замутить.Вроде и case не поможет:

//на форме выбрана комбинация чекбоксов (checkbox1...checkbox6)
if CheckBox1->Checked условие1=TRUE
и т.д.
function Func1
{
if (условие1) if ((условие 1.1) | (условие 1.2) | (условие 1.3))
   {
some code
   }
if (условие2) if ((условие 2.1) | (условие 2.2) | (условие 2.3))
   {
some code
   }
//таких будет 5-6 ветвлений
}

Как можно этот говнокод привести в красивый вид? И оптимизировать

[nick42]

Сообщение от calypso

Думал-думал,как это замутить.Вроде и case не поможет...

Отчего ж не поможет? Из-за шести боксов и напрягаться не стоит - так сойдет. Но вообще-то можно использовать битовую матрицу и ветвление делать по её значению (комбинации чекбоксов). Например:

  int kluch=0;
  for (int c=0,i=0;c<ControlCount;c++)
    if ((String)Controls[c]->Name=="CheckBox"+IntToStr(i+1))
      kluch |= ((TCheckBox*)Controls[c])->Checked ? 1<<i++ : 0<<i++ ;
  switch (kluch) {
    case 0: /*  some code */  break;
    case 2: /*  some code */  break;
    case 5: /*  some code */  break;
  }

[calypso]

Ругается,что метод Controls используется неверно (Embarcadero XE2)

[Smitt&Wesson]

if ((String)Controls[c]->Name

Разыменование не верное. Звёздочку нужно ставить.

if ((String*)Controls[c]->Name

А почему бы Вам вместо кучи CheckBox-ов не применить CheckLictBox. В этом случае получить значения флажков гораздо легче.

[calypso]

Не работает ни со звездочкой,ни без нее:

[BDA]

Хм, а Вы этот код пишете в функции, которая является методом формы?
h:

//---------------------------------------------------------------------------

#ifndef Unit1H
#define Unit1H
//---------------------------------------------------------------------------
#include <Classes.hpp>
#include <Controls.hpp>
#include <StdCtrls.hpp>
#include <Forms.hpp>
//---------------------------------------------------------------------------
class TForm1 : public TForm
{
__published:	// IDE-managed Components
	TButton *Button1;
	void __fastcall Button1Click(TObject *Sender);
private:	// User declarations
public:		// User declarations
	__fastcall TForm1(TComponent* Owner);
};
//---------------------------------------------------------------------------
extern PACKAGE TForm1 *Form1;
//---------------------------------------------------------------------------
#endif

cpp:

// ---------------------------------------------------------------------------

#include <vcl.h>
#pragma hdrstop

#include "Unit1.h"
// ---------------------------------------------------------------------------
#pragma package(smart_init)
#pragma resource "*.dfm"
TForm1 *Form1;

// ---------------------------------------------------------------------------
__fastcall TForm1::TForm1(TComponent* Owner) : TForm(Owner) {
}

// ---------------------------------------------------------------------------
void __fastcall TForm1::Button1Click(TObject *Sender) {
	for (int c = 0; c < ControlCount; c++)
		ShowMessage(Controls[c]->Name);
}
// ---------------------------------------------------------------------------

Этот код отлично работает в C++ Builder 2010.

[calypso]

Вы этот код пишете в функции, которая является методом формы

Получается что нет,это "левая" функция.Например,если я в ней напишу CheckBox1->Checked ,выдаст ошибку что неизвестный оператор

[BDA]

Тогда добавьте ее как метод формы

[calypso]

Если б я знал как это сделать) TForm9::ListFiles дает ошибку:
[bcc32 Error] Unit9.cpp(423): E2316 'TForm9::ListFiles(AnsiString,TStri ngs *,TStrings *,int)' is not a member of 'TForm9'

[BDA]

Примерно:

//h
//---------------------------------------------------------------------------

#ifndef Unit1H
#define Unit1H
//---------------------------------------------------------------------------
#include <Classes.hpp>
#include <Controls.hpp>
#include <StdCtrls.hpp>
#include <Forms.hpp>
//---------------------------------------------------------------------------
class TForm1 : public TForm
{
__published:	// IDE-managed Components
	TButton *Button1;
	void __fastcall Button1Click(TObject *Sender);
private:	// User declarations
	void __fastcall ListFiles(AnsiString a, TStrings *b, TStrings *c, int d);
public:		// User declarations
	__fastcall TForm1(TComponent* Owner);
};
//---------------------------------------------------------------------------
extern PACKAGE TForm1 *Form1;
//---------------------------------------------------------------------------
#endif

//cpp:

// ---------------------------------------------------------------------------

#include <vcl.h>
#pragma hdrstop

#include "Unit1.h"
// ---------------------------------------------------------------------------
#pragma package(smart_init)
#pragma resource "*.dfm"
TForm1 *Form1;

// ---------------------------------------------------------------------------
__fastcall TForm1::TForm1(TComponent* Owner) : TForm(Owner) {
}

// ---------------------------------------------------------------------------
void __fastcall TForm1::Button1Click(TObject *Sender) {
	for (int c = 0; c < ControlCount; c++)
		ShowMessage(Controls[c]->Name);
}
// ---------------------------------------------------------------------------
void __fastcall TForm1::ListFiles(AnsiString a, TStrings *b, TStrings *c, int d) {
	for (int c = 0; c < ControlCount; c++)
		ShowMessage(Controls[c]->Name);
}
// ---------------------------------------------------------------------------