- btz: Internet fejlesztés országosan!
- Luck Dragon: Asszociációs játék. :)
- gban: Ingyen kellene, de tegnapra
- droidic: Időutazás floppyval: A 486-os visszavág PCem-men
- sziku69: Fűzzük össze a szavakat :)
- LordAthis: Ismét egy "Idióta" A.I. Projekt, hogy meglovagolja az aktuális trendeket...
- eBay-es kütyük kis pénzért
- D1Rect: Nagy "hülyétkapokazapróktól" topik
- Klaus Duran: Youtube AI szinkron
- votlage71: Kábel menedzsment
Hirdetés
Új hozzászólás Aktív témák
-
Lortech
addikt
Azért hogy aztán vissza kelljen alakítani.
_seth_:
N int típusú, TextLength is, mivel egy szám / érték, ezért ToStringezni nem kell.
Propertyket (TextLength) felesleges eltárolni külön ilyen esetben, csak egyszer hívod.. Írd be nyugodtan a forba. De ha nagyon biztos programot akarsz írni, akkor az egész textBox1 stringet tárold el egy segédváltozóban, és annak írd be a ciklusba a hosszát.
str = char.Parse(...) ??? Mit akarsz ezen parsolni? Eleve a típus.parse típust ad vissza, itt char-t (egy darabot!), tehát ha minden áron le akarod parsolni charra, akkor kéne egy ciklus, ahol végigmész a string indexein egyenként, figyelve arra, hogy ne ilyen nulla hosszúságú a string, mert akkor kivétel.. De nem tudom elképzelni, hogy mi értelme van ennek.
A db változót nem árt nullázni, mert a második számolásnál már nem fog helyes értéket adni, mert az előző eredménye benne van, és azt növeled.
Ez a ciklus.. Minek mész végig 0..N, ha fel sem használod az indexet, és minden iterációnál ugyanazt csinálod?
Az i változót ne így deklaráld, mert könnyen hibákhoz és félreértésekhez vezethet.. Egy segédváltozó ritkán jó, ha az osztály mezője, tartozzanak az adott scopehoz, ott legyenek deklarálva.
pl itt: for (int i=0;...) {} Ez vonatkozik az str-re, N-re is, és vsz a db-re, mert csak egy ellenőrzésre kell vagy tájékoztatásra kell, nem az osztályhoz tartozik szorosan, és nem kell megőrizni sem. Ez nem pascal.
Ha csak a kisbetűs magánhangzókat akarod beleírni a switchbe, akkor célszerű a stringet ToLower-rel csupa kisbetűssé alakítani, hogy nagybetűs magánhangzókra is menjen, pl: string str = textBox1.Text.ToLower();
A ''számol'' gombra kattintás eseménykezelője kb így nézne ki sztem, ha a te kódodból indulok ki:
private void button1_Click(object sender, EventArgs e)
{
db = 0;
string str = textBox1.Text.ToLower();
for (int j = 0; j < str.Length; j++)
{
switch (str[j])
{
case 'a': db++; break;
case 'e': db++; break;
case 'i': db++; break;
case 'o': db++; break;
case 'u': db++; break;
}
}
MessageBox.Show(db.ToString());
} -
_seth_
veterán
Passz... csak próbálkoztunk mert nem akar működni a tanár meg csak ködösít
Jah: órán kaptam egy feladatot, színválasztó ablak behozása, label textjének form címébe másolása stb, 40 percig szórakoztam vele mert nem akarta a szöveget átmásolni, szólok a tanárnak: ''Én kész vagyok de a feladat nem.'' Tanár megnézi, 5, mert elvileg jól írtam be a másolás részt mégsem akarta átmásolni közben msn-en írogattam a többieknek, hogy mit, hogyan lehet megcsinálni, előző nap gyrososnál tartottam külön órát (én... hehe...notival persze) és akik ott voltak velem 4-esnél nem kaptak rosszabbat!
Új hozzászólás Aktív témák
● ha kódot szúrsz be, használd a PROGRAMKÓD formázási funkciót!
- OLED TV topic
- btz: Internet fejlesztés országosan!
- ASZTALI GÉP / ALKATRÉSZ beárazás
- Witcher topik
- Könyvajánló
- Audi, Cupra, Seat, Skoda, Volkswagen topik
- PROHARDVER! feedback: bugok, problémák, ötletek
- Parkside szerszám kibeszélő
- Luck Dragon: Asszociációs játék. :)
- Bambu Lab 3D nyomtatók
- További aktív témák...
- Samsung Galaxy A12 64GB Kártyafüggetlen 1 év Garanciával
- Villámgyors teljesítmény és 10 év nyugalom!
- MacBook Pro 16" M3 Pro 36GB 512GB Gar. SPACE BLACK
- Oppo Reno7 5G 256GB, Kártyafüggetlen, 1 Év Garanciával
- Nvidia Quadro P400/ P600/ P620/ P1000/ T400/ T600/ T1000 - Low profile (LP) + RTX A2000 6/12Gb
Állásajánlatok
Cég: FOTC
Város: Budapest