Új hozzászólás Aktív témák

• #2770 amargo addikt Neil Watts #2767

  Új Válasz 2012-01-28 22:46:08 #2770

  Új hozzászólás

  Összes hozzászólása itt Válaszok az összes hozzászólására itt Válaszok erre a hozzászólásra

  Privát üzenet küldése

  

  amargo

  addikt

  válasz Neil Watts #2767 üzenetére

  Az egész blokk érdekes, de most tekintsünk el, mert egy része így még működő képes lehet. De hozzáteszem, hemzseg az egész kód a hibáktól!

  char KezdoVektor = Convert.ToChar(textBoxKezdoBetuBeVektor.Text);
  
  if (textBoxKezdoBetuBeVektor.Text != "")
  {
  for (int i = 0; i < nevBe.Length; i++)
  {
  if (KezdoVektor == nevBe[db][0])
  {
  listBoxKezdobetuListazKi.Items.Add(nevBe[db]);
  }
  }
  }
  }

  Ellenben azért, amit ArchElf írt, azrt értelmezted is? Mert minek oda ciklus, ha mindig csak az aktuális db ellenőrződ? Semmi értelme 50x ugyanazt leellenőrizned. De a kutya is itt van elásva!

  Nézd meg a felette lévő kódodat:
  textBoxNevBeVektor.Clear();
  
  if (textBoxNevBeVektor.Text != "")
  {
  nevBe[db] = textBoxNevBeVektor.Text;
  db++;
  }

  Tehát egyből törlöd a mező tartamát, majd nem adsz semmit semmihez.
  Ezek után nézd meg az első általam írt kódodat:
  nevBe.Length //Ez helyes, 50 elemű.
  if (KezdoVektor == nevBe[db][0]) //Itt pedig 0. elemre hivatkozol, majd annak szeretnéd elkérni az első karakterét. De ez nem fog menni, mert az Null -> meg is kapod helyesen az üzeneted.

  Nem hiába írtam az elején és ne támadásnak vedd, de egyszerűen az egész kód hemzseg a hibáktól és hiányosságoktól. Ez tényleg csak figyelmetlenség, de mivel szemmel nem tudok olyan jól összehasonlítani - számomra a változónevek is feleslegesen hosszúak - így csak azon akadt meg a szemem, hogy ott minek akarsz törölni?

Új hozzászólás Aktív témák