Diskuze: Chyba v základech

gondorek619

Člen

gondorek619:13.1.2015 22:55

Zdravím, asi si budete myslet že jsem úplný trotl, ale vůbec netuším, co zde může být za chybu.
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace ConsoleApplication1
{
class Program
{
static void Main(string[] args)
{
int cislo = int.Parse(Console.ReadLine());
int soucet = int.Parse(Console.ReadLine());
int prumer = int.Parse(Console.ReadLine());
for (int i = 0; i < 100; i++)
soucet = soucet + cislo;
if ((soucet < 100) && (soucet > 0))
{
Console.WriteLine("Gratuluji, zadal jsi spravne");
Console.ReadLine();
}
else
{
Console.WriteLine("Je mi lito, zadal jsi spatne cislo.");
Console.ReadKey();
}
}
}
}
V programování jsem celkem nový, takže bych byl rád za nějaké takové "jednodušší" vysvětlení. Předem díky
PS: A dokázal by mi někdo říci, jak vypočítat průměr čísel které jsou teda menší než 100 a větší než 0?

Odpovědět

13.1.2015 22:55

Jan Vargovský

Tvůrce

Jan Vargovský:13.1.2015 23:17

"co zde může být za chybu"

Hodilo by se nám říct, co ten program má dělat.

Nahoru Odpovědět

13.1.2015 23:17

gondorek619

Člen

gondorek619:13.1.2015 23:20

"Je dána posloupnost 50 celých čísel. Spočítejte hodnotu aritmetického průměru těch čísel posloupnosti, která jsou větší než nula a zároveň menší než sto." Tohle bylo zadání.

Nahoru Odpovědět

13.1.2015 23:20

Jan Vargovský

Tvůrce

Jan Vargovský:13.1.2015 23:44

var cisla = new List<int>();
do
    cisla.Add(int.Parse(Console.ReadLine()));
while (cisla.Count < 50);
Console.WriteLine(cisla.Where(c => c > 0 && c < 100).Average());

Ne dobře, abych ti nějak pomohl. Zadáváš 50 čísel, ale v cyklu máš 100. Máš přičítat čísla, které splňují podmínku > 0 a zároveň < 100. K těm podmínkám na konci programu, tipuji, že si je chtěl mít uvnitř cyklu a "gratulovat" mu po každém číslu, které splňuje podmínku.

Klidně si ty čísla nemusíš ukládat, jen udělej něco takového:

int suma, pocetCisel;
for (int i = 0; i < 50; i++)
{
 int cislo = int.Parse(Console.Readline());
 if ((cislo < 100) && (cislo > 0))
 { suma += cislo; pocetCisel++;
 }
}

// Vypis suma/pocetCisel

Editováno 13.1.2015 23:46

Nahoru Odpovědět

13.1.2015 23:44

gondorek619

Člen

gondorek619:18.1.2015 15:20

Díky za pomoc.
Bohužel jsem se setkal s další nepříjemností
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace ConsoleApplication1
{
class Program
{
static void Main(string[] args)
{
int pocet = 0;
int prumer = 0;
int patnact = 0;
int plat = 0;
int reditel = 0;
int rozdil = 0;
int nejnizsi = 0;
int soucet = 0;
{
do
{
Console.WriteLine("Zadej plat v tisících");
plat = int.Parse(Console.ReadLine());
}
while (plat < 25);
Console.WriteLine("prumer je {0}", prumer);
Console.WriteLine("rediteluv plat je {0}", reditel);
Console.WriteLine("rozdíl nejvyššího platu je {0}", rozdil);
Console.WriteLine("počet zaměstnanců je {0}", pocet);
Console.ReadKey();
}
prumer = soucet / pocet;
soucet = soucet + pocet;
reditel = reditel - prumer;
rozdil = reditel - nejnizsi;
pocet++;
if (plat > 15)
patnact++;
}
}
}
Tento program by měl vykonat následující: a) počet zaměstnanců podniku

průměrný plat všech zaměstnanců podniku
o kolik je ředitelův plat vyšší než průměrný plat všech ostatních zaměstnanců
rozdíl mezi nejvyšším (tj. ředitelovým) a nejnižším platem
počet zaměstnanců s platem vyšším než 15 000,- Kč.

PS: Jediný ředitel může mít plat vyšší než 25 000.
PS2: Celý program jsem udělal tak že 25 000 = 25
PS3: Zkoušel jsem to i před závorkami před do a za while
PS4: Tyto hodnoty jsem zkusil dát i pod do, ale bezúspěšně.
prumer = soucet / pocet;
soucet = soucet + pocet;
reditel = reditel - prumer;
rozdil = reditel - nejnizsi;
pocet++;
Předem všem děkuji za odpověd

Editováno 18.1.2015 15:22

Nahoru Odpovědět

18.1.2015 15:20

Jan Vargovský

Tvůrce

Jan Vargovský:18.1.2015 16:37

Vlož kód ještě jednou do tagu code (viz tlačítko u příspěvku). Napiš co přesně ti nefunguje. Doporučuju si nastudovat jak se používají metody, patlat to do mainu je už trošku nepřehledné, když máš vyfiltrovat x informací o nějakých datech.

Nahoru Odpovědět

18.1.2015 16:37

gondorek619

Člen

gondorek619:18.1.2015 16:40

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace ConsoleApplication1
{
class Program
{
static void Main(string[] args)
{
int pocet = 0;
int prumer = 0;
int patnact = 0;
int plat = 0;
int reditel = 0;
int rozdil = 0;
int nejnizsi = 0;
int soucet = 0;
{
do
{
Console.WriteLine("Zadej plat v tisících");
plat = int.Parse(Console.ReadLine());
}
while (plat < 25);
Console.WriteLine("prumer je {0}", prumer);
Console.WriteLine("rediteluv plat je {0}", reditel);
Console.WriteLine("rozdíl nejvyššího platu je {0}", rozdil);
Console.WriteLine("počet zaměstnanců je {0}", pocet);
Console.ReadKey();
}
prumer = soucet / pocet;
soucet = soucet + pocet;
reditel = reditel - prumer;
rozdil = reditel - nejnizsi;
pocet++;
if (plat > 15)
patnact++;
}
}
}

Vypíše mi to u všech věcí které potřebuji zjistit (průměr etc.) že to vychází 0, at zadám hodnoty jaké chci. Před do ta zavorka byt nema, ale v tom chyba není. To už jsem davno smazal

Editováno 18.1.2015 16:41

Nahoru Odpovědět

18.1.2015 16:40

Jan Vargovský

Tvůrce

Jan Vargovský:18.1.2015 17:24

Tvůj kód vypisuje nuly, protože od té doby co deklaruješ a inicializuješ všechny své proměnné, tak s nimi nic neděláš.

Neřekl si co přesně má ten program dělat - zadáváš platy zaměstnanců, ale kolik jich zadáváš? Čím se má ten načítající cyklus ukončit? Teď to máš tak, že poslední plat je asi ředitele, ale nevím, jestli to tak chceš.

Zatím ti můžu poradit jen s tím jak upravit kód:

int pocet = 0;
int prumer = 0;
int patnact = 0;
int plat = 0;
int reditel = 0;
int rozdil = 0;
int nejnizsi = 0;
int soucet = 0;

jde napsat takto

int pocet, prumer, patnact, plat, reditel, rozdil, nejnizsi, soucet;

Díky toho, že je to hodnotový typ, tak to je potom všechno s defaultní hodnotou 0.

Nahoru Odpovědět

18.1.2015 17:24

gondorek619

Člen

gondorek619:18.1.2015 17:34

Tento program by měl vykonat následující: a) vypsat počet zaměstnanců podniku

vypsat průměrný plat všech zaměstnanců podniku
vypsat o kolik je ředitelův plat vyšší než průměrný plat všech ostatních zaměstnanců
rozdíl mezi nejvyšším (tj. ředitelovým) a nejnižším platem
počet zaměstnanců s platem vyšším než 15 000,- Kč.

Ukončit se má platem nad 25, tj. plat ředitele
právě že nevím kolik jich zadám, tak od toho tam je ten while

Editováno 18.1.2015 17:35

Nahoru Odpovědět

18.1.2015 17:34

Jan Vargovský

Tvůrce

Jan Vargovský:18.1.2015 17:42

int pocet = 0;
int prumer = 0;
int patnact = 0;
int plat = 0;
int reditel = 0;
int nejnizsi = int.MaxValue;
int soucet = 0;
Console.WriteLine("Zadej platy v tisících");
do
{
    plat = int.Parse(Console.ReadLine());
    soucet += plat;
    pocet++;
    if (plat > 15)
        patnact++;
    if (nejnizsi > plat)
        nejnizsi = plat;
}
while (plat < 25);
reditel = plat;

prumer = soucet / pocet;

Console.WriteLine(pocet); // a
Console.WriteLine(prumer); // b
Console.WriteLine(reditel - prumer); // c
Console.WriteLine(reditel - nejnizsi); // d
Console.WriteLine(patnact); // e

Bere to prumer i od toho ředitele, tak to si kdyžtak fixni...

Editováno 18.1.2015 17:46

Nahoru Odpovědět

18.1.2015 17:42

gondorek619

Člen

gondorek619:18.1.2015 17:53

Díky moc za pomoc :-)

Nahoru Odpovědět

18.1.2015 17:53

Jan Vargovský

Tvůrce

Jan Vargovský:18.1.2015 17:56

Btw, rada do budoucna. Raději přemýšlej a pak až se ptej. Udělat chybu, že něco vypíšu a pak až s tím něco dělám je dost zbrklá.

Nahoru Odpovědět

18.1.2015 17:56

gondorek619

Člen

gondorek619:18.1.2015 18:00

Já jsem to zkoušel fixovat skoro hodinu a půl, ale nemohl jsem přijít na to, proč to nejelo. Jestli se ještě můžu zeptat, proč je u int nejnizsi = int.MaxValue? To jsme ještě nebrali, tak mě to celkem zajímá. :-)

Nahoru Odpovědět

18.1.2015 18:00

Jan Vargovský

Tvůrce

Jan Vargovský:18.1.2015 18:05

Je to největší možná hodnota v intu. Aby se při prvním načtení načetl nějaký první plat, protože defaultně 0 - nic nižšího už být nemůže. Můžeš to klidně napsat takhle.

int pocet = 0;
 int prumer = 0;
 int patnact = 0;
 int plat = 0;
 int reditel = 0;
 int nejnizsi = 0;
 int soucet = 0;
 Console.WriteLine("Zadej platy v tisících");
 do
 {
     plat = int.Parse(Console.ReadLine());
     if (plat > 15)
         patnact++;
     if (pocet == 0 || nejnizsi > plat)
         nejnizsi = plat;
     soucet += plat;
     pocet++;
 }
 while (plat < 25);
 reditel = plat;

 prumer = soucet / pocet;

 Console.WriteLine(pocet); // a
 Console.WriteLine(prumer); // b
 Console.WriteLine(reditel - prumer); // c
 Console.WriteLine(reditel - nejnizsi); // d
 Console.WriteLine(patnact); // e

Nahoru Odpovědět

18.1.2015 18:05

Nejčastěji vyhledáváné

Diskuze: Chyba v základech - cykly