Sort Routine Fails to Sort Final Sequence in Array

Posted on 2011-03-15
Last Modified: 2012-05-11
So far, so good... HOWEVER, I want this code to print the array in descending order, based on the second column... it does that, but leaves the last (highest) pair of numbers at the bottom of the list!  What can be done to fix this?
#include <iostream>
#include <stdlib.h>
#include <vector>
#include <algorithm>

using namespace std;
const int FINISH = 2;
const int START = 5;
void getData(int arr[][FINISH]);
void displayData(int arr[][FINISH]);
bool insertionSort(const vector<int>& inner1, const vector<int>& inner2); 

int i = 0;
int j = 0;
int main()
      int timeArray[START][FINISH];


        vector<vector<int> > vect;

        for (int i = 0; i < START - 1; ++i) 
                vector<int> inner(timeArray[i], timeArray[i] + 2);
        sort(vect.begin(), vect.end(), insertionSort);

        for (int i = 0; i < START - 1; ++i) 
                timeArray[i][0] = vect[i][0];
                timeArray[i][1] = vect[i][1];

      cout << endl << endl << endl;

      return 0;

// used to prompt user to fill array with values
void getData(int arr[][FINISH])
      cout << "Enter start/finish times." << endl
        << "One at a time when prompted: " << endl;

         cout << endl;

   for(int i = 0; i <= START - 1; i++)
	   cout << "Enter start time #" << i + 1 << ": ";
      cin >> arr[i][j];

	  for(int j = 0; j <= 0; j++)
			cout << "Enter finish time #" << i + 1 << ": ";
		  cin >> arr[i][j + 1];

      cout << endl;
}// end getData()
// used to display array contents
void displayData(int arr[][FINISH])
	cout << endl << endl;
      for (int i = 0; i < START; i++)
            for (int j = 0; j < FINISH; j++)
                  cout << arr[i][j] << " ";
            cout << endl;
}// end displayData()
bool insertionSort(const vector<int>& inner1, const vector<int>& inner2) 
        return inner1[1] >= inner2[1];

Open in new window

I'm SO close...
Question by:Member_2_4213139
Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people just like you are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions
  • 2
LVL 53

Expert Comment

ID: 35141007
I notice that some of your loops go from 0 to (START - 1), and others go from 0 to (START - 2).
LVL 37

Accepted Solution

TommySzalapski earned 500 total points
ID: 35141035
I'm not too sure how you have this working, but shouldn't
for (int i = 0; i < START - 1; ++i)
be this?
for (int i = 0; i < START; ++i)

You are not copying the last item which is why it's still at the end.
You should notice it shows up twice.

Author Closing Comment

ID: 35141204
I didn't notice ANY "START - 2" ... but changing the START -1 to START did it!  THANK YOU!
LVL 53

Expert Comment

ID: 35141230
>> I didn't notice ANY "START - 2"

If you have a loop condition that says :

        i < START - 1

it means it'll go up to (START - 2) and no further.

Featured Post

On Demand Webinar - Networking for the Cloud Era

This webinar discusses:
-Common barriers companies experience when moving to the cloud
-How SD-WAN changes the way we look at networks
-Best practices customers should employ moving forward with cloud migration
-What happens behind the scenes of SteelConnect’s one-click button

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Suggested Solutions

Title # Comments Views Activity
ejb entity bean example 2 57
Outlook 13 102
Using Third Party DLL with Access VBA 14 66
Data Analysis 7 59
Displaying an arrayList in a listView using the default adapter is rarely the best solution. To get full control of your display data, and to be able to refresh it after editing, requires the use of a custom adapter.
Computer science students often experience many of the same frustrations when going through their engineering courses. This article presents seven tips I found useful when completing a bachelors and masters degree in computing which I believe may he…
The goal of the tutorial is to teach the user how to use functions in C++. The video will cover how to define functions, how to call functions and how to create functions prototypes. Microsoft Visual C++ 2010 Express will be used as a text editor an…
The viewer will be introduced to the member functions push_back and pop_back of the vector class. The video will teach the difference between the two as well as how to use each one along with its functionality.

752 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question