The OpenNET Project / Index page

[ новости /+++ | форум | теги | ]



Вариант для распечатки  
Пред. тема | След. тема 
Форум Разговоры, обсуждение новостей
Режим отображения отдельной подветви беседы [ Отслеживать ]

Оглавление

SmartAdBlock - новый легковесный блокировщик рекламы для Chr..., opennews (?), 09-Мрт-19, (0) [смотреть все]

Сообщения [Сортировка по времени | RSS]


31. "SmartAdBlock - новый легковесный блокировщик рекламы для Chr..."  +1 +/
Сообщение от НяшМяш (ok), 10-Мрт-19, 01:08 
Почему я не стал ставить это, просто посмотрев исходники:
1. Форматирование табуляцией в 8 строк; одна функция может быть отформатирована вообще без отступов, следующая - с отступами в 2 символа, а остальной файл по 8 символов
2. Нет пробелов там, где нормальные люди их ставят (понравилась лепнина вида if(str.length>0){...}else{...})
3. Изменяют прототипы встроенных классов (String.prototype.replaceAll). Считается bad practice даже в мире вебмакак.
4. Почему-то массивы, в которые по ключу добавляют значения (да, я знаю, что массив это тот же обьект, но всё же) - suf=[];suf["com.ac"]=""; это норма. Зачем нужен именно массив, если это нормально (и красиво) складывается в обьекте? Кстати, а что это вообще за список suffix.js?
5. Лежит минифицированный jquery и я не нашёл чтобы он использовался (операции с сетью и ДОМом, которые я обнаружил, делаются вручную). Однако он вписан в манифест и в нём могут заминифицировать хоть майнера.
6. В конце концов в 2019 можно код и на ES6 написать для удобства (но это уже чисто придирка)

В итоге имеем, что:
1. Код оформлен так, что ревьюить даже при желании мало кому захочется (упрт отступы, имена переменных и т.п.)
2. Проект новый (14 дней), но озаботились оформлением именного юзера (видать, чтобы авторы не спалились). Пока все коммиты только от этого юзера. На доверии работают пацаны.

Однако, это всего-лишь мой внутренний аудитор включил параноика и я буду очень рад ошибиться и мы в будущем получим быстрый блокировщик рекламы.

Ответить | Правка | Наверх | Cообщить модератору

33. "SmartAdBlock - новый легковесный блокировщик рекламы для Chr..."  +10 +/
Сообщение от smartadblockemail (?), 10-Мрт-19, 01:36 
Hi - dev again - replying with the help of google translate.
to reply to few points:
-We never used github before and our plan was not to release it on github but 2 months after the project was out everyone was asking for github so we released it. That's why the username is new and the formatting is not particularly designed for a collaborative coding.
-jquery is used in contentscript
-suffix is used to pull the root of an url (for example the root of google.co.uk is not co.uk but google.co.uk) used later to match an url with the blocking rules.
-String.prototype.replaceAll is a classic I think it is even a javascript open issue. Anyway extensions run on separate namespaces so has no impact at all on the rest of the browser activity.

Thanks for taking the time to have a look.

Ответить | Правка | Наверх | Cообщить модератору

Архив | Удалить

Рекомендовать для помещения в FAQ | Индекс форумов | Темы | Пред. тема | След. тема




Партнёры:
PostgresPro
Inferno Solutions
Hosting by Hoster.ru
Хостинг:

Закладки на сайте
Проследить за страницей
Created 1996-2024 by Maxim Chirkov
Добавить, Поддержать, Вебмастеру