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

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

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

Оптимизация кода: создание "универсального" метода действия в контроллере

Есть следующая иерархия классов public abstract class OperationItemViewModel { public int Id {get;set;} public int OperationId {get;set;} public int OperationItemTypeId {get; protected ...
Bald's user avatar
  • 6,509
0 голосов
3 ответа
126 показов

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

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

Как оптимизировать процесс выборки информации о скаченых файлах? [закрыт]

Как можно ускорить процесс выборки информации о скачены файлах? На данный момент информация о скаченных файлах лежит в MySQL. Вот как это выглядит сейчас: $stmt = $this->mysqli->stmt_init(); $...
devnull's user avatar
  • 1,219
1 голос
1 ответ
88 показов

Как написать "безопасный" метод для сохранения файла?

Есть следующий код: public AttachedFile SaveAttachment(NewAttachment attachment) { var tmpFile = Path.GetTempFileName(); try { using (var f = File.Open(tmpFile, FileMode.Append)) ...
Bald's user avatar
  • 6,509
0 голосов
0 ответов
73 показа

Оцените реализацию жизни Конвея

Доброго дня всем. Оцените, пожалуйста, качество реализации и общего стиля кода (за исключением того, что все слито в один файл) игры "жизнь" Конвея. Пишу не так давно, поэтому важно мнение знающих ...
Bob Balkonsky's user avatar
2 голоса
1 ответ
165 показов

Как использовать ActionFilterAttribute для проверки доступности действия?

В asp.net-mvc проекте для проверки доступности того или иного метода в gui я использую фильтры действия, пример такого фильтра ниже: public class DisallowChangingTheApprovedOperationsAttribute : ...
Bald's user avatar
  • 6,509
5 голосов
2 ответа
946 показов

Оптимизация кода C#

Имеется код C# Он полностью выполняет свои требования но проблема в том что dataGridView сильно тормозит private void dataGridView1_RowPrePaint(object sender, DataGridViewRowPrePaintEventArgs e) { ...
Павел's user avatar
2 голоса
2 ответа
2k показов

Поиск НОДа с помощью bash

Изучаю bash. Хочу вывести наименьший общий делитель на экран, но никак не получается. Не могли вы мы указать на мои ошибки ? #!/bin/bash result=1 nod_func () { local n=$1 local m=$2 if ...
faoxis's user avatar
  • 7,748
-1 голос
2 ответа
175 показов

Плохой Javascript код [закрыт]

Выполнял тестовое задание. Из комментариев: Надо сделать игру. Поле 4х4 квадратика. У квадратов состояние по умолчанию "закрыто". Клик по квадрату переключает состояние на "открыто". Надо ...
Roman Pawliw's user avatar
7 голосов
2 ответа
2k показов

Как правильно "разделять" приложение на слои?

В своих приложениях я использую подход разделение функционала на слои, слои я стараюсь группировать на основании функциональности Data Access Object Data Access Layer Business Logic Layer GUI Для ...
Bald's user avatar
  • 6,509
3 голоса
3 ответа
115 показов

Как сделать данный код более компактным?

def rcvdata(cmd,size): global s; s.settimeout(1) try: sendCommand(cmd);data = s.recv(size) except: try: sendCommand(cmd);data = s.recv(size) except:...
edward's user avatar
  • 31
1 голос
3 ответа
547 показов

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

public static String getCommaSeparated(List<Long> list) { String s = ""; for (Long v : list) { if (s.length() > 0) s += ", "; s += v; } return s; }
Александр Дерменжи's user avatar
0 голосов
1 ответ
232 показа

Оформление кода. Вынос в классы

using System; using System.Collections.Generic; using System.IO; using System.Linq; using System.Runtime.Serialization.Formatters.Binary; using System.Xml.Serialization; namespace Database { ...
Антон Евсеев's user avatar
4 голоса
2 ответа
9k показов

Покритикуйте код игры Камень-Ножницы-Бумага

Помогите начинающему в Python - пример кода игры Камень-ножницы-Бумага. Хотел бы избавиться от global аргументов и вообще какие могут быть советы по оптимизации. С ООП еще толком не знаком. Еще не ...
user211564's user avatar
0 голосов
3 ответа
136 показов

Правильность кода начинающего

С точки зрения правил и стандарта, правилен ли мой код? class Program { static void Main(string[] args) { Console.WriteLine("Enter please x: "); int x = int.Parse(Console....
Piston's user avatar
  • 225
1 голос
1 ответ
2k показов

Быстрая сортировка и многопоточность C++

Решил реализовать быструю сортировку(Quick Sort) с использованием многопоточности C++(thread) и портфеля задач. Метод портфеля задач заключается в поддержании очереди из задач. Каждый свободный ...
rekrut's user avatar
  • 467
1 голос
0 ответов
64 показа

Перенести текст из таблицы [закрыт]

Добрый день, как перенести текст из таблицы в другое окно View Controller. Не могу понять, что не так сделал, на строке nextViewController.labelSpecieText = (cell?.textLabel?.text!...
D.Dante's user avatar
  • 313
1 голос
1 ответ
133 показа

Оптимизация запроса MySQL

Таблица CREATE TABLE `aircrafts_in_parsed_info` ( `id` int(11) NOT NULL AUTO_INCREMENT, `status` enum('success','error') DEFAULT NULL, `type` enum('roundtrip','oneway') NOT NULL, `date` ...
klobastov's user avatar
  • 173
-3 голоса
3 ответа
293 показа

Помогите сделать код читабельным

У меня есть такой код if (position!=1&&position!=2){ viewPager.setCurrentItem(position,false); drawerLayout.closeDrawers(); } if (position==1){ Toast.makeText(Home.this, "Акции ...
Andro's user avatar
  • 3,471
1 голос
0 ответов
92 показа

крестики-нолики неудачно спроектированы

сделал на backbone простейшую версию крестиков-ноликов: http://fh79272k.bget.ru/files/works/tic_tac_toe/1/ https://github.com/zlodiak/ttt_1 но есть сомнения по поводу выбранной архитектуры(если ...
cyklop77's user avatar
  • 1,791
0 голосов
0 ответов
259 показов

Как использовать шаблон в AngularJS для динамически создаваемого элемента (контекстного меню)?

Нужно сделать контекстное меню в виде директивы AngularJS. Я пока что сделал так: <span class="myelem" dd-ctx-menu="menuobj">Тестовый элемент</span> Где menuobj - это объект со списком ...
RussCoder's user avatar
  • 3,481
2 голоса
1 ответ
296 показов

C# Task async верно ли я его использую?

У меня есть 5 операций, которые должны выполняться одновременно. Я пробую использовать Task async. Я сделал такой тестовый пример. Результат меня устраивает, но правильно ли я использую эту ...
Leonard Bertone's user avatar
0 голосов
2 ответа
76 показов

JavaScript. Объявление неизменяемого метода для обьекта

Пытаюсь реализовать обьект в котором может быть произвольное количество свойств и функция-метод которая получает входящий параметр строку и проверяет есть ли свойство с таким именем. Свойства и метод ...
torokhkun's user avatar
  • 3,575
1 голос
1 ответ
3k показов

Поиск ближайшего значения

Написал код для поиска ближайшего значения к заданному, мне сказали что плохо, а собственно почему так и не ответили, может тут кто скажет. static int nearest(int n, int...args) { int nearest = 0;...
DuckImMud's user avatar
  • 169
1 голос
2 ответа
4k показов

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

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

Передача значений в функцию

В процедуре объявляются переменные tD1, tbetta, и ничем в дальнейшем не инициализируются, но код работает. Вопрос - почему он работает? Procedure MakeDO; var te,tD1,tbetta:real; t,ti:integer; ...
Klaod's user avatar
  • 187
0 голосов
0 ответов
206 показов

Помогите пожалуйста оптимизировать парсер, всё работает но очень долго..)

Доброй ночи )) Помогите пожалуйста с парсером, он загружается более 60 секунд, беда, может как то можно другими запросами, что то поменять, вот сам скрипт парсера: <?php set_time_limit (90); ?> ...
Константин's user avatar
6 голосов
1 ответ
469 показов

Организация процесса код ревью для небольшой команды с помощью Upsource

У JetBrains есть клевая система код ревью Upsource. В моей команде 3 человека. Хочу включить в наш рабочий процесс прохождение код ревью как обязательное требования вливания кода в ветку девелоп. ...
Andrey Kolomensky's user avatar
2 голоса
1 ответ
494 показа

Понимание strategy pattern

Всем привет. Получил критику своего кода касательно того, что применяется процедурный стиль написания. Решил уходить от этого. В частности, сейчас разбираю замену if else statement на strategy ...
JohnKiedis's user avatar
4 голоса
1 ответ
184 показа

Чем плох %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", ...
Qwertiy's user avatar
  • 124k
1 голос
1 ответ
196 показов

Таймер обратного отcчета

Делаю таймер обратного отсчета, таймер должен запускаться 1-го числа месяца, отсчет на 3 дня. По истечению 3 дней — запускаться снова. Каждого 1-го числа нового месяца должен сбрасываться. (...
Aleksandr's user avatar
  • 238
2 голоса
1 ответ
72 показа

инспекция кода : реализация бинарного поиска

public int BS(int a){ int first = 0; int last = q.length-1; int point = q.length/2; if (q.length != 0 && a >= q[0] && a <= q[q.length-1]){ while (a != ...
Пустота's user avatar
1 голос
1 ответ
4k показов

Реализация задачи Страуструпа «Быки и коровы»

Есть такая задачка у Бьёрна Страуструпа, «Быки и коровы». Суть её в следующем. компьютер генерирует четыре случайных числа и записывает их в контейнер. Пользователь должен угадать какое число загадал ...
Albert Akimov's user avatar
4 голоса
1 ответ
371 показ

Конкатенация строк в PHP [закрыт]

Какая запись выглядит более эстетично, если цель - красивый код? Такая $user = $this->userName . implode("-", $this->birthDate); Такая $user = $this->userName; $user .= implode("-", $this-...
qtm's user avatar
  • 357
0 голосов
1 ответ
668 показов

Запрос к API VK

//Запрос к VK, токен уже есть в переменной $tok $wall = file_get_contents("https://api.vk.com/method/users.get?user_ids=".implode(',', $users)."&v=5.40&access_token=$tok"); $wall = json_decode(...
PHPcoder's user avatar
  • 117
-4 голоса
1 ответ
114 показов

правильно ли я реализовал бинарный поиск?

public int BS(int a){ int first = 0; int last = q.length-1; int point = q.length/2; if (q.length != 0 && a >= q[0] && a <= q[q.length-1]){ while (a != ...
Пустота's user avatar
3 голоса
1 ответ
147 показов

Эксклюзивная очередь: недостатки моего решения

В книге The Little Book of Semaphores автор рассматривает задачу, которая называется Exclusive queue: нужно написать многопоточную программу без активного ожидания, использующую семафоры, которая ...
Im ieee's user avatar
  • 1,251
0 голосов
1 ответ
329 показов

создать функцию добавления элемента на страницу

Необходимо создать функцию добавления элементов на страницу для такого кода: // var wrapper = document.createElement('div'); // wrapper.classList.add('wrapper'); // var parentWrapper = document....
Volod's user avatar
  • 73
0 голосов
0 ответов
43 показа

Вопрос по оптимизации куска кода

Как и вообще возможно ли оптимизировать данный участок кода? string[] str = { solution, answer }; richTextBox1.Lines = str; richTextBox1.Select(0, str[0].Length); richTextBox1.SelectionColor = ...
Матвей Котцов's user avatar
-2 голоса
1 ответ
63 показа

Помогите, пожалуйста, доделать код

Наизобретал, по моему мнению, полную чушь, но это единственное до чего додумался. Проблема в том, что это работает, но решение ужасно каменное, хотелось бы завести что-то типа счетчика или условие, ...
Artem Suprunchuk's user avatar
1 голос
1 ответ
658 показов

Spring MVC отправка письма в другом потоке

Добрый день! Есть задача, с веба отправлять на почту письма с вложениями. Проблема в том, что вложения могут быть около 20мб., и если юзер жмет кнопку отправить то он должен ждать пару мин, пока файл ...
m_br0_89's user avatar
2 голоса
1 ответ
2k показов

BigInteger Простые числа

Увлёкся недавно простыми числами и мне понадобилась программка для проверки числа на простоту. Проверяю огромное количество чисел и работаю за пределами long. Первый раз работаю с BigInteger. Поэтому ...
Metaphysics's user avatar
3 голоса
1 ответ
86 показов

Переставить первый и третий байт в EAX, а четвертый обнулить

Пример: из 0xAABBCCDD получить 0x00DDCCBB Решил задачу так: MOV EAX, [EAX]; //инициализация MOV ECX, EAX; SHR ECX, 16 MOV DH, $00; MOV DL, AL; SHL EDX, 16; MOV DH, AH; MOV DL, CL; MOV EAX, ...
Vasek's user avatar
  • 1,267
0 голосов
2 ответа
266 показов

Улучшить качество кода php

Решил поучится php, делаю свою первую полноценную программу. Но слышал что много встроенных if это неправильно. Но я не понимаю как подругому сделать? Подскажите. Вот код программы: <?php ...
Sdafs Fasafs's user avatar
5 голосов
1 ответ
71 показ

Надо ли объединять двойной ключ ради эффективности?

Есть табличка, в которой primary key - это пара 32-битных ключей id1, id2. Соответственно, все ссылки по foreign key на эту таблицу тоже идут по этой паре. С точки зрения исходной модели у каждого ...
Натрий Хлорий's user avatar
3 голоса
1 ответ
318 показов

Knockout.js показать и скрыть сообщение

Задача: показать скрыть сообщение c каким-то результатом. Решение: реализовал через custom bindings, получилось довольно много кода. function MyViewModel() { var self = this; self....
user avatar
5 голосов
2 ответа
948 показов

Передача значения из View в Model и последующий вызов метода - как?

Следуя паттерну MVVM имеем View, ViewModel и некие классы для Model, в частности ExcelImporter для импорта и парсинга экселевского файла. Во вьюхе есть поле ввода адреса файла. Биндится к ...
eexaxa's user avatar
  • 73
5 голосов
1 ответ
661 показ

Java шаблон Observer и многопоточность

Мне необходимо создать приложение, которое будет получать данные по сети и передавать их другим классам для обработки. Я решила воспользоваться для этого шаблоном observer, но столкнулась с проблемой. ...
Gvenihvivar's user avatar
0 голосов
0 ответов
112 показов

Оптимизация цикла JavaScript

Предисловие: Когда я писал этот код я не особо задумывался о оптимизации и писал "Что бы работало :)" Вопрос: И тут возник вопрос, а можно ли как то оптимизировать этот цикл? Описание: Данный ...
ProstoJohn's user avatar
1 голос
1 ответ
611 показов

Рефакторинг кода Javascript

Привет, учусь писать элегантный код javascript на простых примерах. Чтобы вы изменили в этом коде, чтобы он, по-вашему, стал лучше? Спасибо. (function () { "use strict"; ...
user avatar