1. vend dig til at bruge "using namespace std;" det er dårlig programmeringspraksis, istedet skal du så skrive std:: foran de funtioner der er includeret i dette namespace som fx. std::cout og std::cin. Først vil det synes som spild af tid, men man vender sig hurtigt til det.
2. Du initialiserer "J" 3 gange, det er noget skidt. Hvis du blot undlader den første initialisering, skulle der ikke være flere problemer i det.
3. Du har et array af størrelsen 11, altså skal der i din første (og anden og tredje) "for" løkke stå 11 istedet for 10. alternativt kan du skrive <= istedet for <.
4. if(myArray[j] <
// dette giver ingen mening.
5. I det hele taget tror jeg du er lidt forviret med hensyn til hvordan tingene gøres.
6. her er hvordan jeg ville gøre:
#include <iostream>
int main() {
// Her initialiserer vi vores variabler:
int myArray[] = {1,2,3,4,5,6,7,8,9,10,11}; // array med værdier af størrelsen int.
int small = myArray[0]; // small og big initialiseres med en værdi fra arrayet så vi har noget at teste op imod.
int big = myArray[0];
for(int j = 1; j < 11; j++) { // læg mærke til at vi starter fra 1, ikke 0.
if(myArray[j] < small) { // så tester vi og ser og nogle af de andre tal i arrayet er mindre en den vi har tildelt.
small = myArray[j];
}
}
for(int j = 1; j < 11; j++) { // samme pincip som ovenover
if(myArray[j] > big) {
big = myArray[j];
}
}
// og så printer vi på livet løs:
std::cout << "Printing List: ";
for(int j = 0; j < 11; j++) {
std::cout << myArray[j] << " ";
}
std::cout << std::endl << "Printing Smallest Number: ";
std::cout << small << std::endl;
std::cout << "Printing Biggest Number: ";
std::cout << big << std::endl;
std::cin.get(); // Dette er blog for at undgå at consollen lukker af sig selv, du har muligvis ikke brug for dette.
}Held og lykke.