Hey guys. The assignment was handed in on Wednesday at like 10pm (around the time I posted it here). I got full credit for the assignment. Nevertheless I made a few of the changes you guys suggested - need all the practice I can get. Here are the results.
I struggled with the array that Xhad suggested as he wrote it but used this version that I think works okay.
I added a simple while statement so if you enter anything that isn't a number it re-asks. I think thats a good practice and good suggestion.
I haven't learned anything about argc and argv so they remain out of the program for now. Of course, now I have to learn what they mean since they came up in this post.
Shoe Lace makes the suggestion of giving them an exit path rather than entering a number. I agree that may be a better program but I think that because the nature of the program was academic, following the request of the prof makes more sense.
Code:
#include <cstdlib>
#include <iostream>
using namespace std;
void getInput (int *piUserInput);
void getNumbers(int[], int);
int findSmallestInput(int [], int);
int moreElegantSort (int[], int);
int main()
{
int iSmallest = 0, iAlternateSmallest = 0;
int *piUserInput = new int(0);
int* numberArray = 0;
// Create a function for getting user input
getInput(piUserInput);
int end = *piUserInput;
numberArray = new int[*piUserInput];
//create array that gets that amount of numbers
getNumbers(numberArray, end);
// exterior loop runs to swap every number since the largest
// is in the last slot in the Array.
iSmallest = findSmallestInput(numberArray, end);
cout << "The smallest number entered was " << iSmallest << "." << endl;
iAlternateSmallest = moreElegantSort (numberArray, end);
cout << "An alternate answer says the smallest number is " << iAlternateSmallest << "." << endl;
delete piUserInput;
delete []numberArray;
system("PAUSE");
return EXIT_SUCCESS;
}
void getInput(int *piUserInput)
{
cout << "How many numbers would you like to enter: ";
while (!(cin >> *piUserInput)){
cin.clear();
cin.ignore(256,'\n');
cout << "How many numbers would you like to enter: ";
}//end while
}
void getNumbers(int numberArray[], int end)
{
int n=0;
for(int i = 0; i < end; i++)
{
cout << "Please enter #" << i+1 << ": ";
cin >> n;
numberArray[i] = n;
}
}
int findSmallestInput(int numberArray[], int end)
{
int temp = 0;
for (int j = end -1; j>0; j--)
{
// swap numbers
for (int i = 0; i < end; i++)
{
if (numberArray[i] > numberArray[i+1])//compare and swap
{
temp = numberArray[i+1];
numberArray[i+1] = numberArray[i];
numberArray[i] = temp;
}// end compare and swap if
}// closes for loop that swaps
end--;
}// closes for loop that counts down
return numberArray[0];
}
//Here I'm passing by value instead of using "end"
//which makes it a touch more readable.
int moreElegantSort (int numberArray[], int iSize)
{
int min = 0;
// this sets min equal to whatever is in element 0
min = numberArray[0];
// this will loop as many times as the user inputs
for (int i = 0; i < iSize; i++)
{
// this as long as min(element 0) is greater than
// the element held in [i] thats fine, but if it
// isn't, then min is no longer element zero but is
// now element [i] the loop continues and return min
if (min > numberArray[i]) min = numberArray[i];
}
return min;
}
This week I'm creating objects for the first time. Once my homework is complete I'll post it again. I appreciate the feedback as it helps me learn and practice.
