Thread: Homework: Arrays & Functions - Print Two Arrays on Same Table After Sorting

Hello again!

Having trouble with homework involving (title). Here is the assignment:

Write a program that allows the user to enter 10 numbers from the keyboard. Sort the numbers using any sort routine you wish. The output from your program should be 2 columns of numbers. The left column should be the numbers in the order they were originally entered and the right column should be the sorted list. The columns should be labeled. You will need 2 arrays to accomplish this.


Use separate functions for input, sorting, and printing.

So, I have to use separate functions for each of these. I would think it would be easiest to do the input in Main() and then the sorting and printing in another function, but of course since you can't return arrays I am kind of stuck on how I return the new array after sorting. I thought about doing the sorting in Main(), but then I would need to still do the original arrays input in a function and would still run into the same problem.

Here is what I have so far:

Code:

//Cameron Taylor

#include <stdio.h>

#define MAXARRAY 10

int highLow(int[], int);
int print(int[], int[], int);

int main (){

    int unsorted[MAXARRAY], i, j, temp;
    int sorted, sortedArray[MAXARRAY], printArrays;

    for (i = 0; i < MAXARRAY; i++){
        printf("Enter 10 numbers to be sorted");
        scanf("%d", &unsorted[i]);
    }

    sorted = highLow(unsorted, MAXARRAY);
    sorted = sortedArray[MAXARRAY];

    printArrays = print(unsorted, temp, MAXARRAY);

    printf("end");

    return 0;
}

int highLow(int unsorted[], int sortA){

    int i, j, temp, sortedArray[MAXARRAY];
        for (i = 0; i < (sortA - 1); ++i){

            for (j = 0; j < sortA - 1 - i; ++j){
                if (unsorted[j] > unsorted[j+1]){
                    temp = unsorted[j+1];
                    unsorted[j+1] = unsorted[j];
                    unsorted[j] = sortedArray[j];
                    sortedArray[j] = temp;
                }
            }
        }
    return (temp);
}   

int print(int unsorted[], int sortedArray[], int sortB){

    int i, j;

        for (i = 0; i < sortB; i++){
            printf("%d %d", unsorted[i], sortedArray[i]);
        }

    return 0;

}

I know it seems simplistic right now, but I am just trying to get it to work first and then go back and beautify it up a bit.

Thanks!

It's not true that "you can't return arrays" from a function. You could dynamically allocate one (or return a static one!).

But you don't need that here. You just have to pass in both arrays, which become pointers to the first elements of the original arrays and can therefore not only be used to read but also to change the original values.

Try structuring your program like this:

Code:

void print(int src[], int dst[], int sz) {
    // ...
}

void mysort(int src[], int dst[], int sz) {
    // ...
}


int getInput(int a[]) {
    // ...
}

int main() {
    int src[SIZE], dst[SIZE], sz = 0;

    sz = getInput(src);
    mysort(src, dst, sz);
    print(src, dst, sz);

    return 0;
}

Ok, I hope this is sort of what you were looking for:

Code:

//Cameron Taylor

#include <stdio.h>

#define MAXARRAY 10

void print(int[], int[], int);
void highLow(int[], int[], int);

void print(int unsorted[], int sortedArray[], int sz){


    int i, j;

        for (i = 0; i < sz; i++){
            printf("%d %d", unsorted[i], sortedArray[i]);
        }

}

void highLow(int unsorted[], sortedArray[], int sz) {


    int i, j;

        for (i = 0; i < (sz - 1); ++i){
            for (j = 0; j < sz - 1 - i; ++j){
                if (unsorted[j] > unsorted[j+1]){
                    temp = unsorted[j+1];
                    unsorted[j+1] = unsorted[j];
                    unsorted[j] = sortedArray[j];
                    
                }
            }
        }
}   

int getInput(int a[]){

    int unsorted[MAXARRAY], i;

    for (i = 0; i < MAXARRAY; i++){
        printf("Enter 10 numbers to be sorted");
        scanf("%d", &unsorted[i]);
    }
}

int main(){
    int unsorted[MAXARRAY], sortedArray[MAXARRAY], sz = 0;

    sz = getInput(unsorted);
    highLow(unsorted, sortedArray, sz);
    print(unsorted, sortedArray, sz);

    return 0;
}

Getting this compiler error though:

commiedic@localhost Project13$ make Exercise1
cc Exercise1.c -o Exercise1
Exercise1.c:21:30: error: unknown type name ‘sortedArray’
make: *** [Exercise1] Error 1

Originally Posted by Cameron Taylor

Ok, I hope this is sort of what you were looking for:

Code:

//Cameron Taylor

#include <stdio.h>

#define MAXARRAY 10

void print(int[], int[], int);
void highLow(int[], int[], int);

void print(int unsorted[], int sortedArray[], int sz){


    int i, j;

        for (i = 0; i < sz; i++){
            printf("%d %d", unsorted[i], sortedArray[i]);
        }

}

void highLow(int unsorted[], sortedArray[], int sz) {


    int i, j, temp;

        for (i = 0; i < (sz - 1); ++i){
            for (j = 0; j < sz - 1 - i; ++j){
                if (unsorted[j] > unsorted[j+1]){
                    temp = unsorted[j+1];
                    unsorted[j+1] = unsorted[j];
                    unsorted[j] = temp;
                    
                }
            }
        }
}   

int getInput(int a[]){

    int unsorted[MAXARRAY], i;

    for (i = 0; i < MAXARRAY; i++){
        printf("Enter 10 numbers to be sorted");
        scanf("%d", &unsorted[i]);
    }
}

int main(){
    int unsorted[MAXARRAY], sortedArray[MAXARRAY], sz = 0;

    sz = getInput(unsorted);
    highLow(unsorted, sortedArray, sz);
    print(unsorted, sortedArray, sz);

    return 0;
}

Getting this compiler error though:

You never declare SortedArray's type in your function parameters.

lol forgot the "int"...

Code:

//Cameron Taylor

#include <stdio.h>

#define MAXARRAY 10

void print(int[], int[], int);
void highLow(int[], int[], int);

void print(int unsorted[], int sortedArray[], int sz){


    int i, j;

        for (i = 0; i < sz; i++){
            printf("%d %d", unsorted[i], sortedArray[i]);
        }

}

void highLow(int unsorted[], int sortedArray[], int sz) {


    int i, j, temp;

        for (i = 0; i < (sz - 1); ++i){
            for (j = 0; j < sz - 1 - i; ++j){
                if (unsorted[j] > unsorted[j+1]){
                    temp = unsorted[j+1];
                    unsorted[j+1] = unsorted[j];
                    unsorted[j] = sortedArray[j];
                    
                }
            }
        }
}   

int getInput(int a[]){

    int unsorted[MAXARRAY], i;

    for (i = 0; i < MAXARRAY; i++){
        printf("Enter 10 numbers to be sorted");
        scanf("%d", &unsorted[i]);
    }
}

int main(){
    int unsorted[MAXARRAY], sortedArray[MAXARRAY], sz = 0;

    sz = getInput(unsorted);
    highLow(unsorted, sortedArray, sz);
    print(unsorted, sortedArray, sz);

    return 0;
}

Well at least it is running :P

commiedic@localhost Project13$ make Exercise1
cc Exercise1.c -o Exercise1
commiedic@localhost Project13$ ./Exercise1
Enter 10 numbers to be sorted1
Enter 10 numbers to be sorted2
Enter 10 numbers to be sorted3
Enter 10 numbers to be sorted4
Enter 10 numbers to be sorted5
Enter 10 numbers to be sorted6
Enter 10 numbers to be sorted7
Enter 10 numbers to be sorted8
Enter 10 numbers to be sorted9
Enter 10 numbers to be sorted10
0 -1233512952
commiedic@localhost Project13$

Why this:

Code:

fprintf(stdout, "\t%d\t%d\n", unsorted[i], sortedArray[i]);

Why use fprintf and stdout instead of just a normal printf("%d %d", unsorted[i], sortedArray[i]);

Originally Posted by Cameron Taylor

Why this:

Code:

fprintf(stdout, "\t%d\t%d\n", unsorted[i], sortedArray[i]);

Why use fprintf and stdout instead of just a normal printf("%d %d", unsorted[i], sortedArray[i]);

Good question, it's mainly because printf() really calls fprintf() behind the scenes. In my opinion, you might as well do it for the program. Also, fprintf() gives you more control, whereas printf() doesn't so much. If you ever wanted to use stderr to print a error message when you redirected stdout to a file, you should use fprintf(). As for the \t ( tab ) and the \n ( newline ) escape characters, those just make it much easier to read the results.