Ocena kodu v2 - PHP I JS

pytanie zadane 17 stycznia 2018 w Nasze projekty przez Gambr Dyskutant (7,530 p.)

Cześć. Już raz prosiłem o ocenę kodu gry. Dziwna nazwa domeny, wiem, ale to pozostałość po konkursie historycznym i teraz trzymam tam różne male rzeczy. Dostałem trochę cennych wskazówek i na ich podstawie przebudowałem kod. Czy obecna wersja jest już w porządku? Na stronie znajdziecie link do GitHuba.

Dzięki za oceny

2 odpowiedzi

odpowiedź 17 stycznia 2018 przez marcin99b Szeryf (82,180 p.)

if(lastKeyPressed)
	 {
		 if(activeLetter.length==0)
		 {
			wonScenario();
		 }

		else if(zdanie.charAt(activeLetter[0])==lastKeyPressed)
		 {
			zdanie = zdanie.substr(1,zdanie.length-1);
			zdanieDIV.html(zdanie);
			activeLetter.shift();
			updateScore(true);
		 }
		 else
		 {
			updateScore(false);
		 }
	}
	else return false;

To duże ciało if, lepiej wyglądałoby tak:

if(!lastKeyPressed)
  return false; //Return kończy działanie metody

if(activeLetter.length==0)
{
	wonScenario();
}
else if(zdanie.charAt(activeLetter[0])==lastKeyPressed)
{
  zdanie = zdanie.substr(1,zdanie.length-1);
  zdanieDIV.html(zdanie);
  activeLetter.shift();
  updateScore(true);
}
else
{
	updateScore(false);
}

Na 100% da sie to jeszcze bardziej poprawić, ale nie chce mi sie kombinować

komentarz 17 stycznia 2018 przez Gambr Dyskutant (7,530 p.)

Spoko to ja pokombimuję.

A w porównaniu z tym pytaniem z wczoraj, na które odpowiedziałeś, to jest chyba lepiej?

1	komentarz 17 stycznia 2018 przez marcin99b Szeryf (82,180 p.) Nie zagłębiałem sie za bardzo w kod Ale na pierwszy rzut oka, widać poprawę

odpowiedź 17 stycznia 2018 przez Jedras Maniak (54,860 p.)

Nie analizowałem linia po linii, ale głównie mi się rzuciło w oczy:

mysqli_real_escape_string($connection,$_SESSION['nick'])));

a później używasz PDO.

$_SESSION['dangerous']="<span style='color:red; font-size:15px;'>Nick should contain only alphanumeric characters";

Pomieszanie z poplątaniem. Najlepiej byłoby zainteresować się MVC, a jeśli nie to chociaż dodawać klasy, a nie generować inline CSS za pomocą PHP. W wielu miejscach masz to samo później.