Анализ на най-лошия сорс код в света

Ако перифразираме на обикновен език, ще получим приблизително следното: ‘Има ли потребител с име X и парола Y?. Ако има, то да се върне true‘.

А сега да погледнем ето този фрагмент от кода:

if ("true" === "true") {
  return false;
}

Съвсем безсмислено. Защо тази функция да не връща просто false? За какво й е условие, което винаги е истина?

А сега нека да анализираме следния блок:

$('#login').click(function() {
  var username = $("#username").val();
  var password = $("#password").val();

  var authenticated = authenticateUser(username, password);

  if (authenticated === true) {
    $.cookie('loggedin', 'yes', { expires: 1 });
  } else if (authenticated === false) {
    $("error_message").show(LogInFailed);
  }
});

Фрагментът на този код, в който се осъществява работата със значенията с използването на jQuery, изглежда нормално. Проблемът е в организацията на работата с бисквитките loggedin.

Вече стана дума за това, че дори и да нямам акаунт мога просто да отворя конзолата на Chrome и чрез командата $.cookie('loggedin', 'yes', { expires: 1 });, да се окажа логант в сайта за един ден.

А по какъв начин тази уеб страница може да разбере, кой именно е автентифициран потребител? Може би тя си извежда нещо собствено, предназначено само за тези, които успешно са преминали проверката за име и парола, без да бъдат извеждани някакви лични данни? Това няма как да разберем.

Проблемите с форматирането на кода

Форматирането е навярно най-малкият и незначителен проблем на този код. Но той съвсем ясно показва, че който се е занимавал с този код, отнякъде е копирал неговите отделни фрагменти.

Ето един пример за използването на двойни кавички при форматирането на редовете:

var username = $("#username").val();
var password = $("#password").val();

А в друго място се използват единични кавички:

$.cookie('loggedin', 'yes', { expires: 1 });

Това може и да изглежда, че не е особено важно, но ни показва, че кодерът е копирал отнякъде фрагменти сорс код, най-вероятно от StackOverflow и при това дори не го е преписал – директно Copy и Paste, като по този начин е копирал и стила на оригинала. Това е дреболия, но показва, че програмистът не се грижи особено много, за да разбере как всъщност работи този код. Този кодер иска да постигне само едно – кодът някак си да работи и толкова.

Нека да изкажа своята гледна точка. Лично аз всеки ден търся в Google по нещо, което е свързано с моя сорс код. Но на мен ми изглежда много по-важно да разбера как правилно да работя с текстовите бисквитки, а не просто без мисъл в главата да копирам и поставя късче чужд код. И какво ще стане, ако програмата по някаква причина спре да работи? По какъв начин един програмист, който не разбира какво става в неговия код, ще открие грешката?

Изводи

Сигурен съм, че този фрагмент от сорс код е някакъв фейк. Така например, именно тук видях пример за синхронно изпълнение на SQL заявка:

var accounts = apiService.sql(
  "SELECT * FROM users"
);

Обикновено, тези задачи се решават ето така:

var accounts = apiService.sql("SELECT * FROM users", (err, res) => {
  console.log(err); // some error
  console.log(res); // query result
});

А може да се направи и ето така:

var accounts = await apiService.sql(
  "SELECT * FROM users"
);

Дори и методът apiService.sql да връща резултатите в синхронен режим (в което аз се съмнявам), то ще трябва да се направи обръщение кум базата данни, за да се изпълни заявката/запитване, и резултатът да се подаде в точката на извикване. Не е трудно да се досетим, че тази операции не могат да се извършват синхронно

Но дори и това да е някой реален код, сигурен съм, че е написан от някой съвсем начинаещ кодер, джуниър. Да си призная, когато бях програмист, първите две-три седмици също пишех подобен код (моля да ме извините :D).

Не, това не е грешка на някой младши програмист.

Нека да си представим, че това е съвсем истински код, който работи някъде. Някой младши кодер е положил немалки усилия, за да накара това чудо да заработи. Но този разработчик трябва да научи как правилно се работи с SQL заявките, с cookie файловете и с всичко останало. От тази гледна точна, това може да се счете за нормално.

Но шефът на този кодер трябва да контролира неговата работа, да му покаже недостатъците, да го накара да разбере грешките си и да се поучи от тях. По този начин този твърде лош код няма да попадне в крайната продукция.

Със сигурност зная, че има компании, които не са особено загрижени за качеството на кода, който отива в крайната продукция. Кодът решава ли поставената задача? Ако това е така – без излишни въпроси, той се добавя в крайния продукт. Кодът е написан от младши кодер и не е проверен от опитен програмист? Какво от това? Слагайте го в крайната продукция и да продаваме. Трябва да се спазват сроковете.

Да, в живота има известна доза огорчения.

Допълнение от 8 август 2020 година

Когато пишех този материал, предполагах, че кодът, който анализирах е фейк. Но след като обсъдихме нещата в Reddit, на мен ми изпратиха ето този линк. Оказа се, че въпросният код се използва в някаква компютърна система с 1500 потребители. Явно не съм бил прав. Това си е съвсем реален код, който се използва ежедневно.

А вие срещали ли сте фрагменти с наистина и откровено лош код, пълен с всякакви проблеми?