Keresés

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

  • ArchElf

    addikt

    válasz Neil Watts #2836 üzenetére

    Nem félreérthető volt, hanem rossz.
    A te megoldásodban minden páros számra kiírta a textboxba (felülirva az előző szöveget), hogy az adott szám páros.

    1. private void buttonOKrb_Click(object sender, EventArgs e)
    2. {
    3. if (radioButtonParatlan.Checked)
    4. {
    5. for (int i = 0; i < veletlentar.Length; i++)
    6. {
    7. if (veletlentar[i] % 2 == 0)
    8. {
    9. labelParatlParos.Text = veletlentar[i] + " db páratlan számot sorsolt ki a program";
    10. }
    11. }
    12. }
    13. listBox_eredmenyKi.Items.Clear();
    14. }

    Ha kellene egy kis manuális debug:
    3. radioButtonParatlan.Checked -> feltételezhetjük, hogy páratlan számokíat keresel
    7. if (veletlentar % 2 == 0) -> itt viszont a párosakra ugrik be az if ágra (kettövel osztva nincs eredmény az páros. Ami neked kellett volna az a != 0.
    9.c labelParatlParos.Text = veletlentar(i) + " db páratlan számot sorsolt ki a program"; -> itt az aktuális elemé értékét írja ki - még csak nem is a sorszámát, vagy a számosságot (ezt javította Athlon64+).

    +1 nem értem a fixációt a tömbökkel; rendes gyűjteményekkel (Collection), sokkal könnyebben/értelmesebben lehet dolgozni, mert flexibilisek és egy csomó már előre elkészített funkciót is biztosít.

    AE

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