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

• #2733 stevve veterán Peter Kiss #2731

  Új Válasz 2012-01-08 11:00:48 #2733

  Ú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

  

  stevve

  veterán

  válasz Peter Kiss #2731 üzenetére

  Az if szerintem nem a legjobb TryParse mellé. így nincs hibakezelés.
  Egy lehetséges refaktorálás: link

  Kérdés, hogy kell-e üríteni a listát minden gombnyomáskor. Illetve én a randomnak egy saját metódust csinálnék, úgy még szebb.

  core2:
  Nem igazán értem ezt a karakteres dolgot. Mit kellene ott csinálni? De amúgy a program célját se értem. Mi a feladat?

  Másfelől 1-2 tanács:
  - alulvonás helyett inkább egybe írd a neveket és CamelCasing-et használj.
  - olyan nevet ne használj, hogy "textBox1_adatbe". Ha a kontrol nevét is bele akarod írni, akkor inkább inputText vagy bevitelTextBox vagy ilyenek.
  - "namespace _1_vektor" - n ilyet biztosan ne használj. Nagyon csúnya. a Namespace legyen konkrét és globális - valami jól eltalált név.
  - "int[] egesz = new int[] { 1, 2, 3, 4, 5, 6, 7, 8 };" - ez felesleges, az egész számokat a rendszer ismeri és a 10 az nem két karakter, hanem egy szám. hol a 9?
  - ha már nagyon a tömbök felé mész, akkor az a sok tömb kipakolható a metódusból egy-egy field-be is:
  private int[] _egesz = new int[] { 1, 2, 3, 4, 5, 6, 7, 8 };, de kár vesződni vele, mert ez úgy, ahogy van felesleges. Van egy mindig 9 elemű tömböd és annak veszed a hosszát...

• #2732 Neil Watts veterán Peter Kiss #2731

  Új Válasz 2012-01-08 10:52:57 #2732

  Ú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

  

  Neil Watts

  veterán

  válasz Peter Kiss #2731 üzenetére

  Az ok, ez így működik is, csak éppen a tömbös problémát kerüli meg...

  Nekem a tömb nem kell másra, minthogy van egy textboxom (egyik kommentemben benne van a kép) abba beleírók egy X elemszámot (pl. 10) és kiírja a listboxba az X elemszámnyi véletlenszámot/karaktert.

  Ez ezzel a kóddal működik is / tömb nélkül nem.
  Karakterre sajnos nem megy.

  Üdv. core2

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