The code can improve?
The code is as follows:
# include
# include
Using STD: : cin;
Using STD: : cout;
Using STD: : endl;
Using STD: : uniform_int_distribution;
Using STD: : default_random_engine;
P://* minimum * (p + 1) : * (p + 2) : maximum output random number
Int * p=new int [3].//to * p dynamically allocated memory
//a custom function line breaks
Void newLine ();
Bool checkIsOver ();
Void setNum (int * p);
Void getRandNum (int * p);
Int main () {
Bool IsOver=true;//used to judge whether the program end
Cout & lt; <"Note: random number range for closed interval" & lt;While (IsOver) {//IsOver judge whether to end
SetNum (p);
NewLine ();
GetRandNum (p);
NewLine ();
IsOver=checkIsOver ();//will return values to IsOver judgment operator
}
Cout & lt; <"Program has quit.. "& lt;The delete [] p;//release dynamic memory
return 0;
}
//line breaks
Void newLine () {
Cout & lt;}
//determine whether the program end
Bool checkIsOver () {
Char temp.
NewLine ();
Cout & lt; <"Whether or not to continue the input? (y, n, n) : ";
For (;; ) {
Cin & gt;> Temp.
If (temp=='y')
return true;
Else if (temp=='n')
return false;
The else
Cout & lt; <"The input data is wrong! Please input again: ";
}
}
//output random number according to the requirements of customers
Void getRandNum (int * p) {
The static uniform_int_distribution & lt; Int> U (* p, * (p + 1));//range for the closed interval are shown in brackets
The static default_random_engine e;
For (int I=1; I & lt;=* (p + 2); I++) {//I for the pseudo-random number
Cout & lt; If (I % 8==0)//every 8 number line breaks
Cout & lt;}
}
//record user input minimum and maximum
Void setNum (int * p) {
For (;; ) {
Cout & lt; <"Enter the minimum:";
Cin & gt;> * p;
Cout & lt; <"A maximum input:";
Cin & gt;> * (p + 1);
If (* p & lt;=* (p + 1))
break;
The else
Cout & lt; <"The input data is wrong! Please input again!"}
Cout & lt; <"Number of input and output random Numbers:";
Cin & gt;> * (p + 2);
}
CodePudding user response:
Parameter named trough point too much...Global variable p
IsOver said over to false, checkIsOver return values conflicts with the role of IsOver
The for loop is not starting from 0, you first line not only 7 digits?
Y/n support only lowercase