Вопросы с меткой [инспекция-кода]

Вопросы по этой метке должны содержать только работающий код. Если у вас вопрос "почему не работает" — не ставьте эту метку. Обязательно опишите реализуемую функциональность и назначение кода. Используйте стандартный для языка стиль кодирования — имена функций и переменных, отступы, пробелы и прочее. Подробная инструкция есть в полном описании.

Фильтрация
Сортировка
Метки
13 голосов
1 ответ
2k показов

Работа с Cookie в HttpClient для парсинга с авторизацией

Мне всегда было интересно, как заставить HttpClient работать с куками подобно браузеру, затем сохранять их при выходе из приложения, и продолжить использовать после повторного запуска. И вот, наконец, ...
  • 43.4k
62 голоса
5 ответов
14k показов

Нужен async/await или не нужен?

Изучаю асинхронное программирование и вижу следующий метод async Task Produce(ITargetBlock<string> queue, int howmuch) { Random r = new Random(); while (howmuch-- > 0) { ...
  • 637
20 голосов
2 ответа
842 показа

Массовые асинхронные вызовы с ограничением на количество параллельных без семафора

Вполне простая задача - есть N упорядоченных запросов, которые надо выполнить асинхронно с ограничением на количество одновременно выполняемых, и потом достать их из них так же N упорядоченных ответов....
  • 43.4k
10 голосов
4 ответа
2k показов

Почему не стоит пользоваться функцией std::pow для целых чисел?

Мне не раз приходилось встречать рекомендации по типу "не пользуйтесь функцией std::pow". В связи с чем многие дают эту рекомендацию? void MyPow(int& a, int n) { int c = a; for (...
  • 717
3 голоса
1 ответ
516 показов

IoC в MVVM для чайников

Решил наконец-то научиться делать интерфейс приложений расширяемый на несколько View/ViewModel в рамках MVVM. Наткнулся на шаблон Inversion of Control + Depency Injection, информации по нему много, ...
  • 43.4k
1 голос
1 ответ
3k показов

Рисование треугольников в консоли

Изучаю шарп, хотелось бы получить критику и советы как лучше сделать! Три класса Program: Мейн класс вызывает классы Rectangle and Triangle. Rectangle: Содержит метод получение площади по двум ...
5 голосов
5 ответов
15k показов

Проверка на верное количество скобок [ { ( ) } ]

Задача следующая, написать код, который проверяет, верно ли расставлены скобки в выражении т.е. ("(5+5)/[4+4]*{2*2}" - True, "(3+[2*3)]" - False . Код я написал, честно говоря, ...
  • 403
2 голоса
0 ответов
237 показов

Обобщенный конвертер EAP => TAP

У меня есть объект - клиент, который подключается к серверу посредством сокет соединения. Это означает, что, когда мне что то надо получить с сервера, я отправляю ему пакет запроса, а он, когда ему ...
  • 31.7k
2 голоса
2 ответа
182 показа

Конвертирование памяти для видеокарты

private static ManagementObjectSearcher VideoControl = new ManagementObjectSearcher("root\\CIMV2", "SELECT * FROM Win32_VideoController"); ManagementObjectCollection GetVideo = VideoControl.Get(); ...
  • 1,979
1 голос
1 ответ
1k показов

Как улучшить вложенные циклы в Python

Доброго дня. С питоном дружу совсем недавно. И ввязался в парсинг сайта. У сайта есть таблица несколько колонок и несколько строк. Нужно обойти все строки и колонки и вывести их как надо. И не ...
  • 27
1 голос
2 ответа
2k показов

Оцените класс для работы с MySql

Недавно написал класс для работы с MySql. Прошу посмотреть, есть ли в этом классе дырки (sql injection). Каждая строка подписана для понимания кода. <?php Class bd{ private $mysqli; ...
  • 1,903
7 голосов
1 ответ
210 показов

Метод `All()` для пустой коллекции возвращает `true`: баг или фича?

У меня есть класс Item public class Item { public int ParentId {get;set;} public int TypeId {get;set;} public int? AttachedFileId {get;set;} } Мне необходимо для каждого родителя ...
  • 6,447
5 голосов
1 ответ
170 показов

Кодревью: модульные тесты для простых доменных объектов

И снова я с темой инспекции кода, почитав книги по модульному тестированию. Проект .NetFramework 4.6.1, фреймворк тестирования - nUnit. У меня есть два простых класса в доменной модели: Order и ...
  • 28.4k
5 голосов
1 ответ
219 показов

Как избежать дублирования кода в сервисном слое?

public interface IOperationService { IEnumerable<OperationItem> GetItems(int operationId); } public class OperationService: IOperationService { public IEnumerable<...
  • 6,447
3 голоса
2 ответа
170 показов

Почему плохо перебирать файлы/папки через ls? (Iterating over ls output is fragile. Use globs.)

Есть код, который должен работать в bash и zsh. Фрагмент кода перечисляет все директории, сортируя их по времени последнего изменения. Эту задачу он решает вполне успешно. Перечисляемые директории ...
  • 33.4k
2 голоса
2 ответа
95 показов

Можно ли упростить этот скрипт?

Есть скрипт, выводит простые числа: $simple = array(); for ($iii = 2; $iii <= 20; $iii++) { $flag = true; for ($jjj = 2; $jjj <= $iii/2; $jjj++) { if( $iii % $jjj == 0) ...
  • 695
1 голос
2 ответа
3k показов

Перебор результатов .querySelectorAll()

Создаю выпадающее многоуровневое меню. Проблема в том, что функция не срабатывает на тех элементах, которые идут после элемента, неудовлетворяющего условие if. Приведенный код - упрощенный аналог ...
  • 5,532
1 голос
1 ответ
236 показов

Код выполняется дважды (помогите найти баг)

var Ajax = { send: function(object) { var xhr = new XMLHttpRequest(); xhr.open(object.method, object.url, true); xhr.send(object.data); xhr.onreadystatechange = ...
0 голосов
3 ответа
114 показов

Как оптимизировать функцию экранирующую специальные символы в строке?

function addSlashes(str) { str = str.replace(/\[/g, '\\['); str = str.replace(/\\/g, '\\\\'); str = str.replace(/\^/g, '\\^'); str = str.replace(/\$/g, '\\$'); str = str.replace(/\|...
user avatar
0 голосов
0 ответов
2k показов

JavaScript Кнопка "наверх". Инспекция-кода

Изучаю JavaScript, решил сделать кнопку "наверх". Написал данный код: 'use strict'; /* begin begin Back to Top button */ (function() { function trackScroll() { var scrolled = window....
0 голосов
2 ответа
221 показ

Проинспектировать код калькулятора процентной ставки с учетом капитализации с проверками ввода

Я только начал изучать C# Это простой калькулятор процентной ставки с учетом капитализации. По сути это была задача. которую я немного усложнил для большей практики, поэтому добавил проверки на ввод "...
9 голосов
5 ответов
483 показа

Оптимизировать метод обсчета поля игры 2048

Сижу ломаю голову, как избавиться от повторяющегося кода. 2048 игру же знаете, пишу свой вариант реализации. Есть массив тайлов, каждый тайл представляет из себя const cell = { row : 1, col : 1, ...
  • 43.4k
8 голосов
2 ответа
264 показа

Опасно ли разворачивать foreach через using as

В C# цикл foreach разворачивается в нечто такое: Container container = new Container(); Enumerator enumerator = container.GetEnumerator(); try { while (enumerator.MoveNext()) { var ...
  • 120k
7 голосов
2 ответа
271 показ

Адекватный ли код я пишу?

Ребята, я тут практикуюсь с Promise в js. И решил написать функцию tree. Собирает все папки и файлы в каталоге, с неизвестным количеством вложений. Работает правильно, но я смотрю на этот код и не ...
6 голосов
1 ответ
2k показов

Лучший способ для смены букв "А" и "Б" местами

Есть функция, которая принимает аргумент в виде строки, состоящей из "А", "а", "Б" и "б". Каждая "Б" должна быть заменена на "А", каждая &...
6 голосов
1 ответ
355 показов

Конечный автомат

Набросал пример исходя из описания алгоритма. Хочу услышать замечания, что стоило бы изменить в этом коде: // https://en.wikipedia.org/wiki/Finite-state_machine // https://en.wikipedia.org/wiki/...
6 голосов
4 ответа
6k показов

Инспекция: нахождение корней квадратного уравнения на C++

Помогите, как можно улучшить программу по нахождению корней квадратного уравнения. #include<iostream> #include<cmath> using namespace std; double dis(double a, double b, double c);//...
6 голосов
1 ответ
863 показа

svg separator with border

Изучаю SVG. В HTML-верстке, бывают порой такие секции, как у меня в примере, я решил сделать несколько. Мой код(лучше смотреть на всю страницу): Первый вариант: .blob { padding: 190px 0 ...
5 голосов
1 ответ
549 показов

Моя реализация долго выполняющейся задачи с прогрессом и ее слабые места

Ситуация: имеется приложение asp.net mvc. Есть кнопка по нажатии на которую запускается процесс генерации некоего отчета. Отчет может генерироваться довольно долго например несколько минут. Необходимо ...
  • 1,378
5 голосов
1 ответ
246 показов

Первый опыт в node.js Можно ли не переделывая с нуля получить сносный результат?

Мой первый опыт программирования, если не считать студенческих лабораторных на Delphi(и те были очень давно). Цель: "приложение" которое на первой странице выдает кликабельный список из имен юзеров(...
4 голоса
1 ответ
111 показов

Инспекция кода: генетический поиск

Здравствуйте! Сейчас пытаюсь искать алгоритмы Маркова с помощью генетических алгоритмов. Есть такой вот код: #include <iostream> #include <vector> #include <utility> #include <...
  • 1,506
4 голоса
1 ответ
178 показов

Чем плох %n в printf?

http://ideone.com/ffjDAv #include <cstdio> char *names = "Windows\0System\0Config\0"; int main() { int l, r; for (char *name=names; *name; name+=r-l+1) if (printf("Folder: %n%s%n\n", ...
  • 120k
3 голоса
0 ответов
82 показа

Что можно улучшить в коде?

С помощью FFT определены амплитуда, частота и начальная фаза 512 гармоник. Код преобразования аудио файла с помощью FFT, Program.cs: using NAudio.Wave; using System; using System.IO; namespace fft { ...
  • 157
3 голоса
3 ответа
2k показов

Сделал простые табы на js, хотелось бы услышать критику

let tabs = document.getElementsByClassName('tab'); let sections = document.getElementsByClassName('section'); for(let i =0; i<tabs.length; i++){ tabs[i].onclick = tabclick } function ...
3 голоса
2 ответа
165 показов

Элемент управления для визуализации обратного отсчёта с анимацией дуги окружности

Я реализовал элемент управления для визуализации обратного отсчёта с анимацией дуги, который выглядит вот так: инспекция-кода Код рабочий, но требуется обратная по поводу качества реализации данного ...
3 голоса
1 ответ
169 показов

C#. Windows Service. Подтекает память

Подскажите, пожалуйста, в каком месте я допустил ошибку, что служба постепенно поджирает память Задача службы при старте загружать построчно адреса из файла Services.txt в List<Uri> и через ...
  • 3,576
3 голоса
1 ответ
92 показа

php, перфекционизм

Есть бэкенд на PHP, обслуживающий разные запросы и возвращающие json. Точка входа(упрощенно): try { switch ($method) { case 'login': $call = new Login($data); ...
  • 13.8k
3 голоса
2 ответа
472 показа

Ревью Архитектуры приложения

Есть решение разбитое на следующие слои: DAO, DAL, Services Gui где, DAO - здесь хранятся классы описывающие доменнные модели; DAL - Generic Repository и его реализация; Services - здесь у меня ...
  • 6,447
2 голоса
3 ответа
6k показов

Фильтрация объекта по значениям из массива

Есть объект, который надо отфильтровать по значениям из массива; свойства из массива гарантированно есть в объекте. Реализовал это таким образом: const teams = { 'dynamo': { points: ...
2 голоса
2 ответа
149 показов

Как избежать повторения кода в наследуемых классах?

Eсть базовый класс, от него наследуются другие, эти другие классы реализует в себе новые сущности, как избежать повторения одного и того же кода, при частичном повторении некоторых сущностей в ...
  • 339
1 голос
3 ответа
517 показов

Проверка на палиндром

Делал задание на проверку палиндром строка или нет. В интернет смотрел и код у всех очень большой и не очень понятен мне. Я сделал сам и получилось как-то очень мало кода. Может я сделал что-то не то? ...
  • 379
1 голос
0 ответов
113 показов

Насколько код соответствует стандартам ООП?

Только начинаю писать ООП, до этого писал только процедурно. Прошу пробежаться глазком по коду и оценить насколько код соответствует стандартам ООП. Код слайдера. https://jsbin.com/vocapebehe/1/edit?...
1 голос
5 ответов
206 показов

Возможна ли жизнь без goto?

@Artem здесь опубликовал следующее сообщение: static void SimplifySign(ref int numerator, ref int denominator) { if (numerator < 0 && denominator < 0) // (-)/(-) -> (+)/(+) ...
1 голос
0 ответов
580 показов

Связь между слоями и объектами (Object model, DAO, DTO, Services)

Код: https://github.com/Dermenji/WeatherOOP/tree/master/src/com/sirma Цель: создание модели объектов, слои (DAO, DTO) сервисы. Прошу о инспекции кода - советы по организации и структуре. Связь между ...
1 голос
1 ответ
325 показов

Критика кода. MVVM, WPF, C#

Решил для себя разобраться с MVVM, не совсем уверен правильно ли я организовал связь между Model и ViewModel. Хотел бы услышать ваше мнение по поводу реализации этого паттерна, а также претензии к ...
  • 308
1 голос
0 ответов
60 показов

Что можно улучшить в методе randomPointsInPolygon()?

Есть метод которому передаёшь float step(влияет на количество точек (1 * step / 10000), чем больше тем больше точек), List drawcord(набор точек которые определяют площадь полигона) public static ...
1 голос
1 ответ
156 показов

Можно ли ускорить выполнение curl, если сервер посылает Connection close?

Мне нужно рекурсивно загружать страницы с JSON, пока next_page не будет NULL. Окей, это сделано, страницы грузятся, но, 50 страниц, прогружаются где-то за 20-30 секунд, как я понимаю, это потому что ...
user avatar
1 голос
1 ответ
82 показа

Правильно ли я реализовал перезагрузку операции "()"?

Создать несколько объектов (например, a и b) разработанного класса. Класс – вектор (одномерный массив). Реализовать для объектов данного класса перегрузку операции () (a(i)=b(j)). class vector { ...
  • 850
1 голос
0 ответов
185 показов

Как создать скорректировать оптимизировать аккордеон?

Создал дословный сценарий: Назначить класс родителям, чтобы добавить оформление знаком Провести проверку на открытые вкладки и закрыть их а) Назначить функции открытия закрытия по клику и смены ...
  • 527
1 голос
1 ответ
124 показа

Нужна помощь в корректировке дизайна приложения крестики нолики

Я сделал игру крестики нолики но как выяснилось когда я начал писать тесты что протестировать через юнит тестами ее просто не возможно. Помогите поправить так что-бы соблюдались принципы ООП. Игра: ...
  • 5,237