Loop som skriver ut det största talet.

Får damp. Varför fungerar inte denna kod? Jävla fi***eger

Därför att du har allt innanför while-loopen. Detta borde fungera bättre:


Glöm inte bort att main-funktionen måste returnera ett värde heller.

Vad får du för fel då?
Kan du förklara exakt vad som ska hända?

Är det så att du vill:
Att du knappar in ett tal som sparas i ett array? Därefter söker du igenom arrayet i en for loop och om ett array är större än det tidigare högsta angivna (max) så ska printf funktionen skriva ut vad talet är?


Som det ser ut nu måste du skriva in ett tal för varje gång while loopen körs, dvs 100 gånger. Du vill antagligen bara skriva in talet 1 gång, och inte 100 gånger. Detta kan du lösa genom att sätta detta FÖRE ii++; i while loopen.

if(ii==0)
{scanf("%d", &element[ii])};

dvs att köra scanf ENDAST om ii är lika med 0, alltså första gången while loopen körs, före ii++.


Men då antar jag att du bara sätter av att element[det intrycka värdet];
Det du kanske hellre vill göra är att element[ii] = det intryckta värdet.

Dvs att

Platsen ii i element har det intryckta värdet.

Exempel
Vi säger att vi trycker in talet 5 i scanf.

Det du säger är
element[5]; som praktiskt taget betyder element[5].

Men det jag tror att du vill säga är.
element[ii] = 5;
Då har platsen ii i element värdet fem.

ii är placeringen i hyllan, element är själva hyllan.

Har dock aldrig provat med bokstäver i arrays. Brukar alltid köra siffror.

Exempelvis
element[0] = 1;
element[1] = 5;

printf("Bajs %d", element[0]);
printf("/n korv %d", element[1]);
Det som skrivs är följande
Bajs 1
korv 5


Kan vara så att jag missförstått fullständigt men detta uppfattar jag som en skoluppgift om arrays och användning av olika print och scan funktioner.
Har inte satt mig in alls i huruvida while och for looperna fungerar tillsammans i din kod i och med att jag inte riktigt vet vad koden ska göra.

Har du uppgiftstexten så borde detta gå att få till med ett nafs.

Säg bara till så kan jag rota lite

Ja nu hade det ju kommit ett svar som var lite mer effektivt i sin förklaring. Så jag kan ju ha missat arraygrejen fullständigt och att det faktiskt funkar att köra koden nu. Men jag är spänd på en fortsättning!

Saken är den att programmet skall "löpande" läsa in heltal från terminalen och uppdatera det största talet efter varje inmatning.

Går du på amfetamin mannen? Programmet skall läsa in heltal löpande och uppdatera det största talet efter varje inmatning. pponmm:s lösning är inte giltig eftersom där matas alla element in först innan det största elementet skrivs ut, det största element skall skrivas ut efter varje inmatning.

Är det såhär du menar? c++

[PHP]#include <iostream>

using namespace std;

int main(){

const int SIZE = 10;

int element[SIZE], max = 0, min = 0;

for (int i = 0; i < SIZE; ++i){

cout << "\n Mata in ett tal: ";
cin >> element[++i];

if (element[i] > element[0]){
element[0] = element[i];
max = element[0];
cout << " \n Största elementet är nu: " << element[0];
}
}

cout << " \n Största elementet är: " << max ;

cin.get();

return 0;
}[/PHP]

Du kan ju också testa det med random

[PHP]#include <iostream>
#include <random>

using namespace std;

int main(){

random_device rd;
mt19937 mt(rd());
uniform_int_distribution<int> dist(0, 100);

const int SIZE = 10;

int element[SIZE], max = 0, min = 0;

for (int i = 0; i < SIZE; ++i){

//cout << "\n Mata in ett tal: ";
//cin >> element[++i];

element[++i] = dist(mt);

cout << "\nDraget nr: " << element[i];

if (element[i] > element[0]){
element[0] = element[i];
max = element[0];
cout << " \n Största elementet är nu: " << element[0];
}
}

cout << " \n Största elementet är: " << max ;

cin.get();

return 0;
}[/PHP]

I min tidigare kod, ändra till

[PHP]#include <iostream>
#include <random>

using namespace std;

int main(){

random_device rd;
mt19937 mt(rd());
uniform_int_distribution<int> dist(1, 100);

const int SIZE = 5;

int element[SIZE+1], max = 0, min = 0;

for (int i = 0; i < SIZE; ){

//cout << "\n Mata in ett tal: ";
//cin >> element[++i];

element[++i] = dist(mt);

cout << "\nDraget nr: " << element[i];

if (element[i] > element[0]){
element[0] = element[i];
max = element[0];
cout << " \n Största elementet är nu: " << element[0];
}
}

cout << " \n Största elementet är: " << max ;

cin.get();

return 0;
}[/PHP]

Annars går vi ju out of bounds, eftersom vi lägger största talet på plats 0

Googla bubble sortering

En sådan enkel sorteringsalgoritm ska man lita ut själv och inte googla. Kan man fundera ut bubblesort själv så har man lagt ut en bra bana för sin resterande karriär som programmerare. MEn är du en sådan som tar genvägar till allt?

Bubblesort är en algoritm, helt oberoende från platform,syntax, avancerade bibliotek, behållare etc

Detta behöver man inte googla.

Problemen ligger i raderna jag markerat
Eftersom du börjar med att incrementera ii innan du läser in första värdet så läggs det värdet på plats 1 i arrayen elements. Du läser sedan ut första max från index 0 i arrayen. Den platsen är helt oinitierad och kan innehålla precis vad som helst, dvs troligen ett tal större än vad du matar in.

Ett annat problem med att incrementera ii först är att du kommer läsa in ditt 100'e tal utanför din array eftersom ii då kommer vara 100, men arrayen sträcker sig bara mellan 0 och 99. När du gör det kan du korrumpera stacken och orsaka alla möjliga jobbiga crasher. I just det här fallet är det nog igen fara dock.

Så för att fixa programmet borde du flytta "ii++" till längst ner i while-loopen.

Då behöver du inte arrayen (om du inte planerar att göra andra saker med talen i senare övningar). Skippa den, spara en massa minne och bli av med den otrevliga maxgränsen på 100 tal.