Имам проблем със извеждането на проценти.БТова ми е кода http://pastebin.com/iTutXyRB, а това условието на задачата : "Дадени са n цели числа в интервала [1…1000]. От тях някакъв процент p1 са под 200, друг процент p2 са от 200 до 399, друг процент p3 са от 400 до 599, друг процент p4 са от 600 до 799 и останалите p5 процента са от 800 нагоре. Да се напише програма, която изчислява и отпечатва процентите p1, p2, p3, p4 и p5."
Разгледай хубаво в примера как се пресмята.
Изхода ти трябва да изглежда така:
Като е хубаво този var на P-тата да го направиш на double
Console.WriteLine("{0:F2}%" , ( p1 / n ) * 100);
Console.WriteLine("{0:F2}%" , ( p2 / n ) * 100);
Console.WriteLine("{0:F2}%" , ( p3 / n ) * 100);
Console.WriteLine("{0:F2}%" , ( p4 / n ) * 100);
Console.WriteLine("{0:F2}%" , ( p5 / n ) * 100);
За да получиш в проценти частта от числа в дадена колона на хистограмата:
p = (p / n) * 100;
Можеш да редактираш редовете, където извеждаш на конзолата:
Console.WriteLine("Pi = " + (100 * (decimal)pi / n)); // i = 1, 2, 3, 4, 5
Като гледам кода ти броиш бройката на числата, а не процента на едните, процента на другите в съотношение към общата бройка... т.е.: p = (p / n) * 100
В самото условие е показано как се изчисляват в проценти, не е необходимо да ги правиш нови променливи, просто напиши логиката на изчислението в реда за принтиране.
Console.WriteLine("P1 = {0:F2}%", (p1 /n) * 100);
Един вариант е броя срещания да го разделиш на броя n цели числа. Получаваш 0.хх, което като го умножиш по 100 получаваш процентите. След това си го форматирай, както е указано в условието. Например ако искаш да изписва знака "%" и да показва до втория десетичен знак използвай например:
Console.WriteLine("P1 = {0:f2}%", (((double)p1/n)*100));
Тук съм кастнал към double, тъй като входът ти приема int-ове, а целочисленото деление не е подходящо в случая.
Имаш грешка в проверките! Пропускаш 399, 599 и 799!
Накрая при печатането на %-те печаташ бройката на числата, а не %-та от общата бройка. Общата бройка n е 100%, търсената бройка p е x% -> n/100 = p/x -> x = (p / n)*100 т.е. вместо p1, p2, ..., p5 трябва да отпечаташ "(p1 / (decimal)n) * 100", .... Тъй като p1 и n са цели за да е делението с десетично число като резултат трябва едното число да го направиш десетично!
Ето друго решение https://pastebin.com/mrarAeQ9
Благодаря, аз не слагах скоби и програмата първо е извършвало умножението.