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

• #4264 Peter Kiss őstag trisztan94 #4260

  Új Válasz 2013-04-24 17:54:03 #4264

  Ú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

  

  Peter Kiss

  őstag

  válasz trisztan94 #4260 üzenetére

  Milyen ciklusok?

  private void b_osszertek_Click(object sender, EventArgs e)
  {
  var ossz = 0;
  for (var i = 0; i < 32; i++)
  {
  ossz = ossz + (ar[i] * raktaron[i]);
  }
  l_megjelenites.Text = "A raktárkészlet összértéke: " + ossz.ToString() + " Ft";
  }
  
  private void b_kifogyott_Click(object sender, EventArgs e)
  {
  var kifogyott = raktaron.Any(x => x == 0);
  //nem használod semmire
  }
  
  private void b_adottAr_Click(object sender, EventArgs e)
  {
  var adottAr = Convert.ToInt32(tb_adottAr.Text);
  var szamlalo = ar.Count(x => x < adottAr);
  
  l_megjelenites.Text = szamlalo + "-féle könyv kapható " + adottAr + "Ft alatt.";
  }

  Az elsővel nem lehet mit kezdeni, mert rosszul van valami tervezve (a két különálló tömb nem világos nekem).

• #4263 dabadab titán trisztan94 #4260

  Új Válasz 2013-04-24 17:53:47 #4263

  Ú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

  

  dabadab

  titán

  válasz trisztan94 #4260 üzenetére

  Jo ez igy, legalabbis a metodusokra osztas.
  Ami nagyon nem jo, az egyreszt a bedrotozott konstansok, amiket raadasul szammal irtal ki mindenhol (kepzels el, hogy kiderul, hogy ezentul mondjuk 50 akarmirol kell infot tarolni, baromi maceras lesz mindenhol atirni), masreszt meg ha mar ugyis egy dologhoz tartozik az ar meg a raktaron tulajdonsag, akkor azokat tarolhatnad egy strukturaban es abbol csinalhatnal tombot, ahelyett, hogy ket tombod lenne. Ez mar csak azert is jo lenne, mert ezutan a for helyett hasznalhatnad a foreachet, ami minden bizonnyal olvashatobba tenne a kodot, ehelyett:

  for (int i = 0; i < 32; i++)
  {
  ossz = ossz + (ar[i] * raktaron[i]);
  }

  lenne az, hogy

  foreach(dologType dolog in dolgok)
  {
  ossz += dolog.ar * dolog.raktaron;
  }

• #4261 sztanozs veterán trisztan94 #4260

  Új Válasz 2013-04-24 17:50:06 #4261

  Ú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

  

  sztanozs

  veterán

  válasz trisztan94 #4260 üzenetére

  Mivel a for ciklusok mindig mást csinálnak így nem tudsz túl sokat nyerni a dologgal.
  Ráadásul a második metódusod szerintem rossz. Ha bármelyik könyvből 0 vanna, akkor a végén azt írja ki, hogy kifogyott (bármi kifogyott-e a raktáron) - de azt nem fogod tudni, hgy melyik fogyott ki. Persze jó volna tudni, mi a cél... Mondjuk nem is csinál a végén semmit a metódus, csak számolgat és a végén kilép.

  Másik dolog - használd a string.Format-ot öszetett stringek készítésére:
  l_megjelenites.Text = string.Format("A raktárkészlet összértéke: {0} Ft", ossz);

  l_megjelenites.Text = string.Format("{0}-féle könyv kapható {1} Ft alatt.", szamlalo, adottAr);

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