Java högsta poäng :S?

System.out.println(score); måste du ha efter måsvingen på din for-loop

Hint, int pos gör inte mkt nytta;)
Även nåt med din loop^^

nrOfPpl = players.length?

int pos = i; har ingen mening

int highscore = 0;
for(int i = 0; i < nrOfPpl; i++)
{
      if(players[i].getScore() > highscore)
          highscore = players[i].getScore();
}
System.out.println(highscore);

Bör fungera ekvivalent? (har inte möjlighet att testa)

int highscore = 0;
for(int i = 0; i < nrOfPpl; i++)
{
      if(players[i].getScore() > highscore)
          highscore = players[i].getScore();
}
System.out.println(highscore);

Den där lösningen ser korrekt ut!

int highscore = 0;
for(int i = 0; i < nrOfPpl; i++)
{
      if(players[i].getScore() > highscore)
          highscore = players[i].getScore();
}
System.out.println(highscore);

Skulle dock säga att det är bättre att mellan lagra resultatet från getScore() än att anropa metoden två gånger, då du eventuellt aldrig vet hur prestanda krävande den metoden är.

Med tanke på hur komplext programmet i sig verkar vara antog jag att det är en funktion som kör

int getScore() {
     return score;
}

Men du har en vettig poäng!

Eftersom folk vill skriva kod så vill jag också vara med! :((

{
   System.out.println("Highscore: "+getHighscore(players, 0, 0));
}
	
private static int getHighscore(Player[] players,int i, int score) {
    return players[i].getScore() > score ? i < players.length-1 ? getHighscore(players,i+1,players[i].getScore()) : players[i].getScore() : i < players.length-1 ? getHighscore(players,i+1,score) : score;
}

Rekursiv ternary oneliner!

“To iterate is human, to recurse divine.”
(L. Peter Deutsch)

Kan vara så att det används senare, i kod som inte visats.

{
   System.out.println("Highscore: "+getHighscore(players, 0, 0));
}
	
private static int getHighscore(Player[] players,int i, int score) {
    return players[i].getScore() > score ? i < players.length-1 ? getHighscore(players,i+1,players[i].getScore()) : players[i].getScore() : i < players.length-1 ? getHighscore(players,i+1,score) : score;
}

Kan vara den mest oläsbara koden jag sett på länge. Straffslag på den!

Oklart hur tävlingen om att göra bra kod blev till en tävling att göra oläsbar och odebugbar kod ^^

Den kan inte användas senare då den är deklarerad lokalt i if-satsen -.-

Behöver ju inte debugga sånt som alltid funkar!;) Sen var det nog inte mening att den skulle vara läslig Borde säga sig självt ju
Bara erkänna att den e vacker på sitt egna lilla sätt^^
Hade inte commitat den sådär på jobbet om jag säger så

Där får jag väl ge mig. Missade att det var en deklaration också

int hs = 0;
for(int i = 0; i < players.length; i++)
  if(players[i].getScore() > hs)
    hs = players[i].getScore();

System.out.println(highscore);

Jag gör en variant av den här och använder en av Javas inbygda metoder som eventuellt kan bli optimerad av den virtuella maskinen.

int hs = 0;
for(int i = 0; i < players.length; i++)
  hs=Math.max(hs,players[i].getScore());

System.out.println(hs);

//C

PS. Personligen tycker jag att man ALLTID ska ange måsvingar runt vilkorsuttryck men behöll formatet för att vara enhetlig med övriga här.

Jag tycker Gnejs rekursiva exempel var väl värt att lägga upp. När vi vet vad det åstadkommer på förhand så är det utbildande att titta på för att lära sig hur språket kan fungera på andra sätt och hur problemet kan lösas på ett annat sätt (rekursivt). Och jag menar inte att man ska "lära sig" det sättet och skriva produktionskod. Men elegant var det (om än javasyntaxen gör det lite väl jobbigt).
Jag använder ofta trinära villkorsoperatorn när det blir elegantare. Exempel:

return result != null ? result : 0;

Kontra:

if (result != null) {
        return result;
} else {
        return 0;
}

vilket kan kortas ner till lite nättare (men jag gillar trinära bättre):

if (result != null)
        return result;
return 0;
}

Tummen upp för att alltid göra måsvingar, är en mycket bra vana.

Finns ingen egentlig anledning till att lösa just denna uppgiften rekursivt bortsett från just utbildningssyftet. Däremot finns det en hel del fall där rekursion kan vara ypperligt värdefullt för att lösa en uppgift effektivt, även om det kan vara svårt att bemästra rekursion. Men kan alltid känna sig lite extra stolt när man löst något knepigt med hjälp av rekursion. Bra att känna till hur som helst!

Har man inte sett ternary uttryck så kan det tyckas vara svårare att förstå men när man väl är van så kan en välskriven och indenterad/uppdelad ternary vara lättare/snabbare att förstå/följa än en hop med if /if else eller dyl.

Det som är det vackra med programmering, varje problem går att lösa på så många olika sätt

Känns däremot som att tråden glidit ifrån ursprungsfrågan men det finns ändå en del nyttigheter skrivna här, gör säkert gött för något som orkar läsa!:P

int hs = 0;
for(int i = 0; i < players.length; i++)
  if(players[i].getScore() > hs)
    hs = players[i].getScore();

System.out.println(highscore);

Usch, förstår inte folk som kodar java. PHP IS THE SHIT!

Hur kan du säga något sånt? PHP är nog det mest förvirrande och okonsistenta språk jag sett.

Jag kan bara PHP Java ser så jäkla svårt ut jämfört med PHP

return result != null ? result : 0;

if (result != null) {
        return result;
} else {
        return 0;
}

if (result != null)
        return result;
return 0;
}

Har inget emot ternary, använder det dagligen på jobbet. Det som skar i mina ögon var den långa harangen av ternary

Säger inte att denna är bättre än iteration, men det är ännu en rekursiv lösning:

public static int maxScore(Player[] players) {
    return maxScore(players, 0, players.length);
}

public static int maxScore(Player[] players, int begin, int end) {
    int numPlayers = end - begin;
    if (numPlayers < 2) {
        // If only one player is left, return its score.
        return players[begin].getScore();
    } else {
        // Split the array in two parts and return the greatest of their maximum values.
        return Math.max(
        	maxScore(players, begin, end - (numPlayers / 2)),
        	maxScore(players, end - (numPlayers / 2), end)
        );
    }
}

Var medvetet att lägga den på en rad så

return players[i].getScore() > score ?//Highscore?
	i < players.length-1 ?//Highscore - Last player?
		getHighscore(players,i+1,players[i].getScore())//Highscore, not last
		:	players[i].getScore() ://Highscore, last
	i < players.length-1 ?//Not highscore - Last player?
		getHighscore(players,i+1,score)//Not highscore, not last
		: 	score;//Not highscore, last

Lite mer läsvänlig version^^
Hade indenterat kommentarerna men forumet vill inte behålla dem på ett vettigt sätt så-.-

Mer rekursion <3 det är kärlek

Sen kan du ju försöka skriva en desktop-applikation eller server-mjukvara i PHP.

Var språk har sina fördelar och nackdelar samt tillämpningsområden.

Så varsågod och ta på dig dumstruten och ställ dig i skamvrån, vanilla.