Witam
Dotąd sprawdzałem dla osobnego textboxa np.
public bool checkTextBox()
{
if(namee.Text.Length==0) return true;
else return false;
}
I każda funkcja dla każdego textbox-a czy można ładniej?
Witam
Dotąd sprawdzałem dla osobnego textboxa np.
public bool checkTextBox()
{
if(namee.Text.Length==0) return true;
else return false;
}
I każda funkcja dla każdego textbox-a czy można ładniej?
Oczywiście mam tu na myśli jakąś pętelkę for lub while() :-)
Zrób sobie tablice TextBox i przeleć przez nią pętla.
List<TextBox> textBoxy = new List<TextBox> {txtBox1, txtBox2, txtBox3, ..., txtBoxn};
for(int i = 0; i < textBoxy.Length; i++)
{
Funk(textBoxy[i]);
}
Elegancko, dziękuję ;-)
A jak mam 3 textboxy:
Name:
Surname:
I button Search.
Gdy wcisnę search to w zależności czy jedno, drugie jest puste chcę szukać: "Select * From Employees Where namee LIKE 'Name.Text%'" i tutaj jeżeli Surname by nie było puste to jeszcze AND i plus podobny warunek. Można coś takiego zgrabnie generować?
Po co piszesz w taki sposób?
public bool checkTextBox()
{
if(namee.Text.Length==0) return true;
else return false;
}
Przecież ta funkcja pomimo, że wykonuje swoje zadanie, jest do niczego. Po 1 nazewnictwo, a po 2 wielkość.
Poprawnie powinno być tak:
public bool IsEmptyTextBox(TextBox text)
{
return text.Text.Length == 0;
}
I wsio :)
Niby funkcja boolowksa a zwraca 0 tak? mogę zwrócić true.
Co zwraca 0? Nic tutaj nie zwraca zera, porównanie zwraca albo true
albo false
.
Super sposób, nie znałem takiej zasady. Teraz moje kolejne funkcję skrócą się o wiele bardziej.
Pozdrawiam
Jeszcze jedno:
Tablicę ukierunkowaną na textboxy ( pewnie są to generyki ) nadal zastosować tylko funkcję poprawić na ładniejszą.
bool pusty = String.IsNullOrEmpty(namee.Text);
Dobrze, już wiemy jak. Teraz pytanie czy jedna funkcja dla jednego textbox-a czy lista textboxów i sprawdzanie grupowe? Wydaje mi się, że to pierwsze, bo może i jest mała redundancja, ale chyba ładniej wygląda funkcja jednolinijkowa niż for().
W sumie miałeś rację, poradziłem sobie, ale szczerze? Nie podoba mi się moje rozwiązanie :-) Czuje, że można to zrobić schludniej..
Utworzyłem zmienną
bool NotIsEmptyTextBox;
Funkcję, która nadaję odpowiednią wartość logiczną tej zmiennej:
public void CheckTextBoxs(List<TextBox> textboxs)
{
for (int i = 0; i < textboxs.Count; i++)
if (textboxs[i].TextLength > 0) NotIsEmptyTextBoxs = true;
else
{
NotIsEmptyTextBoxs = false;
break;
}
}
Możecie zaproponować schludniejsze rozwiązanie?
Weź się człowieku za naukę programowania, a nie za spamowanie na forum ciągłymi problemami.
Kurs / książka do ręki i jechane.
public bool CheckTextBoxes(IEnumerable<TextBox> textboxes)
{
return !textboxes.Any(tb=>String.IsNullOrEmpty(tb.Text));
}