Блог

Я пытаюсь вычислить процент успешных заданий в целом. Код:

 int total = valid   invalid;
int percent = (valid / total) * 100;
if (percent == 0)
{
  MessageBox.Show(Convert.ToString(total)   ":"   Convert.ToString(valid));
  break;
}
  

Если все задания выполнены успешно, процент равен 100%.
Если одна работа плохая, процент равен 0, и я получаю сообщение с:
9:8

8/9 * 100 = 88,888, а не 0.

 int percent = Convert.ToInt32((valid / total) * 100);
  

результата не дает. Пожалуйста, помогите мне.
Я русский, поэтому прошу прощения за плохой английский.

Я предполагаю, что оба valid и invalid определяются как int s . В этом случае результатом всех вычислений также будет int s .

Вам либо нужно привести одно из значений к типу с плавающей запятой в операции, либо, и это, вероятно, было бы проще, определить одно из значений как тип с плавающей запятой для начала:

 double total = valid   invalid;
int percent = (valid / total) * 100;
  

ваши действительные и итоговые переменные имеют значение int, что означает, что ваше действительное / итоговое деление будет усечено до целого числа, которое в вашей ситуации всегда будет равно 0.

Если бы вы изменили одну из переменных на double, это позволило бы выполнить разделение так, как вы ожидаете.

 int percent = Convert.ToInt32(((double)valid / total) * 100);
  

Теперь это приведет к тому, что ваше предложение if больше не будет работать должным образом, поэтому вам придется изменить это, чтобы, вероятно, проверить, что оно не 100

поэтому вместо

 if (percent == 0)
  

вы хотели бы сделать что-то вроде

 if (percent != 100)
  

вместо

Вам просто нужно много работать с приведением, чтобы вы не выполняли целочисленное деление и не искали результат с плавающей запятой, например:

     double percent = ((double)valid / total) * 100;
  

Ваша проблема в том, что вы работаете с целым числом (целое число), но для правильной работы деления вы должны использовать рациональное число.

Когда вы выполняете 8/9, у вас есть 0, потому что восемь меньше девяти, поэтому результат будет между 1 и 0. Для представления числа от 1 до нуля вы должны использовать тип с плавающей запятой.

 float total = valid   invalid;
  

Теперь разделение выглядит следующим образом:

  8 / 9.0 
  

но это еще не конец, если вы присвоите число с плавающей запятой целому числу, у вас будет предупреждение компилятора.

 float result = (valid / total) * 100;
  

Сделайте так, чтобы ваша строка была:

 int percent = (int)(((float)valid / (float)total) * 100f);
  

Преобразуйте ваши целые числа в числа с плавающей запятой перед выполнением вычислений.

 int percent = Math.Round(((float)valid / (float)total) * 100);
  

Вам нужно использовать какую-то математику с плавающей запятой, а не целые числа.

 int total = valid   invalid;
int percent = ((decimal)valid / (decimal)total) * 100m;
  

Проблема в том, что результатом (valid / total) является int (целое число), как valid и total есть ints , поэтому результат деления будет усечен до целого числа.

Решение состоит в том, чтобы заставить единицу быть a float , double или decimal :

 int percent = (int)Convert.ToInt32(((float)valid / total) * 100);
  

одним из ваших факторов должно быть значение с плавающей запятой, иначе в результате вы получите целое число.

 int total = valid   invalid;

int percent = (int)((valid / (float)total) * 100);

if (percent == 0)
{
    MessageBox.Show(Convert.ToString(total)   ":"   Convert.ToString(valid));
    break;
} 
  

Вы должны использовать float вместо int :

 float percent = ((float)valid/(float)total) * 100.00f;  
  

Редактировать:
Да, я забыл привести vallid и total переместить.

Попробуйте:

 float total = valid   invalid;
float percent = (valid / total) * 100;
if (percent == 0)
{
  MessageBox.Show(Convert.ToString(total)   ":"   Convert.ToString(valid));
  break;
}
  

Редактировать:

Процент также изменен на плавающий.