Keresés

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

  • 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?

  • martonx

    veterán

    válasz Neil Watts #2767 üzenetére

    Sima debuggal seperc alatt ki tudnád vizsgálni, hogy mi is a baj. Hagy ne mi debugoljunk már fejben helyetted. Másrészt, ha már tanácsért fordultál ide, miért sima array-el szivatod magad? Hahó 2012 van. Javaslom valami fejlettebb collection használatát, amellyel a sorbarendezés egy szimpla parancs.

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