Advice: Learn to indent properly your code, it show its structure and it helps reading and understanding. It also helps spotting structures mistakes.
Random number = new Random();
int maxValue = 10;
int total = 0;
int firstNumber = number.Next(1, maxValue);
int secondNumber = number.Next(1, maxValue);
if (firstNumber < secondNumber)
{
secondNumber = number.Next(1, firstNumber);
do {
secondNumber++;
}
while (firstNumber % secondNumber != 0);
}
else
{
do {
secondNumber++;
}
while (firstNumber % secondNumber != 0);
}
total = firstNumber / secondNumber;
Indentation style - Wikipedia[
^]
Best C++ Formatter and Beautifier[
^]
Online C/C++ Formatter, Indenter and Beautifier – Techie Delight[
^]
Professional programmer's editors have this feature and others ones such as parenthesis matching and syntax highlighting.
Notepad++ Home[
^]
ultraedit[
^]
Enabling Open Innovation & Collaboration | The Eclipse Foundation[
^]
-----
do {
secondNumber++;
}
while (firstNumber % secondNumber != 0);
This loop start from the principle that
secondNumber
is not an integer divisor of
firstnumber
, which is weird.
How do you handle firstNumber=1 ?
It is easier to do it the other way arround:
int total = number.Next(1, maxValue);
int secondNumber = number.Next(1, maxValue);
int firstNumber = total * secondNumber;