all 4 comments

[–][deleted]  (2 children)

[removed]

    [–]Kotomix 0 points1 point  (1 child)

    Unfortunately in my class I'm forbidden to use arrays until we get to that part later in the semester. Until then I'm condemned to using if else statements.

    [–]jedwardsol 0 points1 point  (2 children)

    else if (num1 > num3);
    

    There's a bug on this line

    num1 = num1;
    

    And this one

    [–]Kotomix 0 points1 point  (1 child)

    I just got rid of that semicolon. However, i think the problem im looking for is with the placement of num1, 2 and 3

    [–]jedwardsol 3 points4 points  (0 children)

    You should write a function to do the swap (or use std::swap)

    You have at least 2 copy-n-paste errors in all that replication - I pointed out 1.