Avatar
gondorek619
Člen
Avatar
gondorek619:

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.Collec­tions.Generic;
using System.Linq;
using System.Text;

namespace ConsoleApplication1
{
class Program
{
static void Main(string[] args)
{
int cislo = int.Parse(Con­sole.ReadLine());
int soucet = int.Parse(Con­sole.ReadLine());
int prumer = int.Parse(Con­sole.ReadLine());
for (int i = 0; i < 100; i++)
soucet = soucet + cislo;
if ((soucet < 100) && (soucet > 0))
{
Console.Write­Line("Gratulu­ji, zadal jsi spravne");
Console.ReadLine();
}
else
{
Console.Write­Line("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
Avatar
Jan Vargovský
Redaktor
Avatar
Odpovídá na gondorek619
Jan Vargovský:

"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
Avatar
gondorek619
Člen
Avatar
Odpovídá na Jan Vargovský
gondorek619:

"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
Avatar
Jan Vargovský
Redaktor
Avatar
Odpovídá na gondorek619
Jan Vargovský:
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
Avatar
gondorek619
Člen
Avatar
gondorek619:

Díky za pomoc. :)
Bohužel jsem se setkal s další nepříjemností
using System;
using System.Collec­tions.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.Write­Line("Zadej plat v tisících");
plat = int.Parse(Con­sole.ReadLine());
}
while (plat < 25);
Console.Write­Line("prumer je {0}", prumer);
Console.Write­Line("rediteluv plat je {0}", reditel);
Console.Write­Line("rozdíl nejvyššího platu je {0}", rozdil);
Console.Write­Line("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

  1. průměrný plat všech zaměstnanců podniku
  2. o kolik je ředitelův plat vyšší než průměrný plat všech ostatních zaměstnanců
  3. rozdíl mezi nejvyšším (tj. ředitelovým) a nejnižším platem
  4. 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
Avatar
Jan Vargovský
Redaktor
Avatar
Odpovídá na gondorek619
Jan Vargovský:

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
Avatar
gondorek619
Člen
Avatar
gondorek619:
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
Avatar
Jan Vargovský
Redaktor
Avatar
Odpovídá na gondorek619
Jan Vargovský:

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
Avatar
gondorek619
Člen
Avatar
gondorek619:

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

  1. vypsat průměrný plat všech zaměstnanců podniku
  2. vypsat o kolik je ředitelův plat vyšší než průměrný plat všech ostatních zaměstnanců
  3. rozdíl mezi nejvyšším (tj. ředitelovým) a nejnižším platem
  4. 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
Avatar
Jan Vargovský
Redaktor
Avatar
Odpovídá na gondorek619
Jan Vargovský:
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  +1 18.1.2015 17:42
Avatar
Jan Vargovský
Redaktor
Avatar
Odpovídá na gondorek619
Jan Vargovský:

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
Avatar
gondorek619
Člen
Avatar
gondorek619:

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
Avatar
Jan Vargovský
Redaktor
Avatar
Odpovídá na gondorek619
Jan Vargovský:

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
Děláme co je v našich silách, aby byly zdejší diskuze co nejkvalitnější. Proto do nich také mohou přispívat pouze registrovaní členové. Pro zapojení do diskuze se přihlas. Pokud ještě nemáš účet, zaregistruj se, je to zdarma.

Zobrazeno 14 zpráv z 14.