Мини графический редактор на canvas и vuex

#javascript #vuejs #инспекция_кода #vuex

Сделал небольшой редактор изображений для изучения возможностей vuex с состояниями. 



Никто не хочет в рамках инспекции кода высказать свои замечания к написанному?

Файл components/Main.vue








Файл store/index.js

import Vue from 'vue'
import Vuex from 'vuex'

Vue.use(Vuex)

const ModeEnum = Object.freeze({
  "PointerTool": "Pointer",

  "LineTool": "Line (click canvas to choose start point)",
  "LineToolStartPoint": "Line (click canvas to choose finish point)",

  "CircleTool": "Circle (click canvas to choose center)",
  "CircleToolCenterPoint": "Circle (choose radius)",

  "RectangleTool": "Rectangle (click canvas to set top left corner)",
  "RectangleToolTopLeftCorner": "Rectangle (click canvas to set bottom right corner)",
});

const state = {
  cursor: {
    X: 0,
    Y: 0,
  },
  temporaryPoint: {
    X: 0,
    Y: 0,
  },
  currentMode: ModeEnum.PointerTool,
}

const mutations = {
  coord(state, pos) {
    state.cursor.X = pos.X;
    state.cursor.Y = pos.Y;
  },

  setMode(state, mode) {
    state.currentMode = mode;
  },

  savePointCoordinates(state, pos) {
    state.temporaryPoint.X = pos.X;
    state.temporaryPoint.Y = pos.Y;
  },

  drawLine(state, pos) {
    var canvas = document.getElementById("cnv");
    var context = canvas.getContext("2d");

    context.beginPath();
    context.moveTo(state.temporaryPoint.X, state.temporaryPoint.Y);
    context.lineTo(pos.X, pos.Y);
    context.stroke();
  },

  drawCircle(state, pos) {
    var canvas = document.getElementById("cnv");
    var context = canvas.getContext("2d");

    context.beginPath();
    var radius = Math.sqrt((state.temporaryPoint.X - pos.X) * (state.temporaryPoint.X
- pos.X) + (state.temporaryPoint.Y - pos.Y) * (state.temporaryPoint.Y - pos.Y));
    context.arc(state.temporaryPoint.X, state.temporaryPoint.Y, radius, 0, Math.PI
* 2, false);
    context.strokeStyle = "red";
    context.stroke();
  },

  drawRectangle(state, pos) {
    var canvas = document.getElementById("cnv");
    var context = canvas.getContext("2d");

    context.beginPath();
    context.rect(state.temporaryPoint.X, state.temporaryPoint.Y, pos.X, pos.Y);
    context.stroke();
  },
}

const actions = {
  coord({
    commit
  }, event) {
    var canvas = document.getElementById("cnv");
    var rect = canvas.getBoundingClientRect();
    var pos = {
      X: event.clientX - rect.left,
      Y: event.clientY - rect.top,
    };

    commit('coord', pos);
  },

  setPointerMode: ({
    commit
  }) => commit('setMode', ModeEnum.PointerTool),

  setLineMode: ({
    commit
  }) => commit('setMode', ModeEnum.LineTool),

  setCircleMode: ({
    commit
  }) => commit('setMode', ModeEnum.CircleTool),

  setRectangleMode: ({
    commit
  }) => commit('setMode', ModeEnum.RectangleTool),

  canvas_left_click({
    commit
  }, event) {
    var canvas = document.getElementById("cnv");
    var rect = canvas.getBoundingClientRect();
    var pos = {
      X: event.clientX - rect.left,
      Y: event.clientY - rect.top,
    };

    switch (state.currentMode) {
      case ModeEnum.PointerTool:
        console.log('Current state is: ' + state.currentMode);
        break;

      case ModeEnum.LineTool:
        commit('savePointCoordinates', pos);
        commit('setMode', ModeEnum.LineToolStartPoint);
        break;

      case ModeEnum.LineToolStartPoint:
        commit('drawLine', pos);
        commit('setMode', ModeEnum.LineTool);
        break;

      case ModeEnum.CircleTool:
        commit('savePointCoordinates', pos);
        commit('setMode', ModeEnum.CircleToolCenterPoint);
        break;

      case ModeEnum.CircleToolCenterPoint:
        commit('drawCircle', pos);
        commit('setMode', ModeEnum.CircleTool);
        break;

      case ModeEnum.RectangleTool:
        commit('savePointCoordinates', pos);
        commit('setMode', ModeEnum.RectangleToolTopLeftCorner);
        break;

      case ModeEnum.RectangleToolTopLeftCorner:
        commit('drawRectangle', pos);
        commit('setMode', ModeEnum.RectangleTool);
        break;

      default:
        break;
    }
  },

  destroy() {
    var main = document.getElementById("cnv");
    main.ctx = main.getContext("2d");
    main.ctx.clearRect(0, 0, 600, 300);
  },
}

const getters = {}

export default new Vuex.Store({
  state,
  getters,
  actions,
  mutations
})


Мои собственные заметки:


мне не нравится, что объект canvas фактически не является состоянием. Фактически
приходящие команды рисования изменяют его состояние, верно? Тогда можно было бы вынести
процедуру получения контекста
нормально ли сохранять промежуточные клики (промежуточные переменные между событиями)
во временную переменную, возможно есть какой-то более изящный способ? 

    

Ответы

Ответ 1

Store предназначен только для хранения данных, которые нужны нескольким компонентам.
Все, что касается только лишь одного компонента, должно находиться в компонента. Т.е.,
все методы, которые касаются отрисовки, я бы перенес непосредственно в компонент. Context,
соответственно тоже может храниться в компоненте, и тогда не будет необходимости получать
его каждый раз.
Actions - предназначены для асинхронной работы с хранилищем. Те методы, которые используются
у Вас в actions этому не соответствуют. Поэтому я бы их тоже перенес в компонент. 

Хотя... при работе с canvas, огромные функции, которые управляют холстом я выношу
в отдельные файлы и просто импортирую их в компонент. 

Что касается моделуй в Store, то у меня это выглядит так:



файл modules/index.js

import person from './person';
import preview from './preview';
import grid from './grid';
import header from './header';
import bigData from './bigData';
import loading from './loading';

export default {
  person,
  preview,
  grid,
  header,
  bigData,
  loading,
};


файл index.js 

import Vue from 'vue';
import Vuex from 'vuex';
import modules from './modules';

Vue.use(Vuex);

const store = new Vuex.Store({
  modules,
});

export default store;


как пример, modules/loading.js Модуль отвечает за отображание лоадера при запросах
на сервер.

const loading = {
  state: {
    isLoading: false,
  },

  mutations: {
    SHOW_LOADING: state => {
      state.isLoading = true;
    },
    HIDE_LOADING: state => {
      state.isLoading = false;
    },
  },
};

export default loading;


Но Vue очень гибок, и все что я сказал - это только лишь моя практика. Каждый может
делать так, как ему будет удобно.

Как упростить условие

#python #python_3x #инспекция_кода

Есть "программа" в которой есть классы Viking, Wizzard Assasin, далее должны формироваться
две команды red_team и blue_team. В каждой команде может быть 3 участника dude, dude2,
dude3 задача такая рандомно назначить переменным dude/2/3 классы. 
Я реализовал это так:

chance = randint (1, 3)
If chance == 1:
    dude = Viking (name = 'Red Viking')
if chance == 2:
    dude = Wizzard (name = 'Red Wizzard')
if chance == 3:
    dude = Assasin  (name = 'Red Assasin')
red_taem.append (dude.name)

chance = randint (1, 3)
If chance == 1:
    dude2 = Viking (name = 'Red Viking')
if chance == 2:
    dude2 = Wizzard (name = 'Red Wizzard')
if chance == 3:
    dude2 = Assasin  (name = 'Red Assasin')
red_team.append (dude2.name)

chance = randint (1, 3)
If chance == 1:
    dude3 = Viking (name = 'Red Viking')
if chance == 2:
    dude3 = Wizzard (name = 'Red Wizzard')
if chance == 3:
    dude3 = Assasin  (name = 'Red Assasin')
red_team.append (dude3.name)


И вот этот вот ужас только для одной команды из трех участников! А если бы их было 20?

Собственно вопрос: как это вот это все упростить? Чтобы не писать 100 000 сторк кода.

П.С. Если предлагаете вариант решения пожалуйста объясните мне как ваш вариант будет
работать.

Классы: тут на самом деле та еще белеберда...

class Viking:
    def __init__(self,  name):
        self.name = name
        self.hp = 100
        self.energy  = 100
        self.damage = 30
        self.block = 15
        self.role = 'tank'

class Assasin:
    def __init__(self, name):
        self.name = name
        self.hp = 100
        self.energy  = 100
        self.damage = 30
        self.block = 15
        self.role = 'ass'

class Wizzard:
    def __init__(self, name):
        self.name = name
        self.hp = 100
        self.energy  = 100
        self.damage = 30
        self.block = 15
        self.role = 'tank'

    

Ответы

Ответ 1

Функция random.choices(population, weights=None, *, cum_weights=None, k=1) возвращает
список из k случайно выбранных элементов из population.

Пример:

from random import choices

def build_team(N, color):
    dudes = [Viking(name=f'{color} Viking'), 
             Wizzard(name=f'{color} Wizzard'), 
             Assasin(name=f'{color} Assasin')]
    return choices(dudes, k=N)

red_team = build_team(3, 'Red')
blue_team = build_team(3, 'Blue')




UPDATE:

чтобы при выводе объекта класса на печать получить имя (self.name) - определите метод
__repr__() для всех ваших классов:

class Viking:
    def __init__(self,  name):
        self.name = name
        self.hp = 100
        self.energy  = 100
        self.damage = 30
        self.block = 15
        self.role = 'tank'

    def __repr__(self):
        return self.name

v = Viking(name='Red Viking')
print(v)
# Red Viking

Переставить первый и третий байт в 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, EDX;


Можно ли улучшить этот код?

ОС: win32
    

Ответы

Ответ 1

MOV  EAX, [EAX]  ; EAX=0xAABBCCDD
XOR  EBX, EBX    ;                 BL=0x00
XCHG AH,  BL     ; EAX=0xAABB00DD  BL=0xCC
ROR  EAX, 16     ; EAX=0x00DDAABB
MOV  AH,  BL     ; EAX=0x00DDCCBB

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

#php #алгоритм #инспекция_кода

Есть бэкенд на PHP, обслуживающий разные запросы и возвращающие json.

Точка входа(упрощенно):

try {
    switch ($method) {
        case 'login':
            $call = new Login($data);
            break;
        case 'list':
            $call = new List($data);
            break;
        default:
            $call = new WrongMethod($data);
    }
    $out  = ['r' => $call(),'n' => (string)Error::OK,'e' => ''];
} catch (Error $e) {
    $out = [
        'r' => $call::defaultResult())),
        'n' => (string)$e->getCode(),
        'e' => $e->getMessage()
    ];
}
print_r(json_encode($out, JSON_UNESCAPED_UNICODE), false);


Уже увидели ошибку? ;)

Проблема: 
$call::defaultResult() - в блоке catch объект $call не определен.

Все классы пуляют Error extends Exception и реализуют интерфейс

interface Call {
    public function __construct(array $data);
    public function __invoke();
    public static function defaultResult();
}


defaultResult нужен потому, что _invoke может вернуть скаляр, обычный массив либо
ассоциативный массив и если при ошибке использовать значение какого-то одного типа,
то фронт споткнется на десериализации. То есть мне нужно гарантированно получить в
блоке catch такую же структуру поля 'r', как и в блоке try

Я вижу два варианта решения:


Вынести из конструктора все, что может бросить исключение. Такой подход не нравится
тем, что у меня в конструкторах исключительно проверка входных данных на валидность
и их приведение в случае необходимости. ИМХО этому функционалу самое место в конструкторе.
Перейти на тёмную сторону и сделать как-то так:

(Этот вариант мне вообще не нравится. Прям вызывает дискомфорт и потерю аппетита.)

function getMethodClass($method) {
    switch ($method) {
        case 'login':
            return "Login";
        case 'list':
            return "GetList";
        default:
            return "WrongMethod";
    }
}
$className = $callsNameSpace."\\" . getMethodClass($method);
try {
    $call = new $className($data);
...
} catch (Error $e) {
    $out = ['r' => call_user_func(array($className, 'defaultResult')),
...
}



Внимание, вопрос!

Может я не прав относительно своих фобий и первый(или второй) вариант вполне себе хорош?

Может я заработался и не вижу простого и очевидного решения? Такого чтоб и работало
нормально и с души не воротило?

UPD PHP 5.6
    

Ответы

Ответ 1

Вы забыли о еще одном очень-очень важном моменте. В первом вашем варианте конструкция:

$call::defaultResult()


невалидна (как минимум, идеологически). Фактически вы пытаетесь вызвать метод класса
у экземпляра, что лишено смысла.

С учетом этого, второй ваш вариант намного лучше. Хотя я немного причесал бы его
(в примере ниже использую PHP 5.5+):

$map = [
    'login' => Login::class,
    'list' => List::class,
];
$class_name = isset($map[$method]) ? $map[$method] : WrongMethod::class;

try {
    $call = new $class_name($data);
    // ...
} catch (Error $e) {
    $out = ['r' => $class_name::defaultResult()];
}


Замечание:

Для ошибок валидации входных данных в PHP традиционно используется \InvalidArgumentException
и его подклассы. А \Error - это специальный класс ошибок введенный в PHP7. Возможно
стоит немного изменить имя вашего класса исключений, чтобы повысить читаемость кода
и избежать двусмысленности после перехода на PHP7.

SystemInfo как улучшить код?

#c_sharp #инспекция_кода

Проблема в том что выходит слишком много foreach, как всё это можно сократить?

private static string pcCPU;
private static string pcGPU;
private static string pcLocalIP;
private static string pcExternalIP;
private static string pcAntivirus;

string savePath = @"C:\Sys.txt";
using (FileStream file = new FileStream(savePath, FileMode.Append))
{
    using (StreamWriter twf = new StreamWriter(file, Encoding.UTF8))
    {
        foreach (ManagementBaseObject avResult in avsearcher.Get())
        {
            pcAntivirus = "& " + (avResult.GetPropertyValue("displayName")).ToString();
            pcAntivirus = (pcAntivirus.StartsWith("& ")) ? 
                                pcAntivirus.Substring(1) : 
                                pcAntivirus;
            twf.WriteLine(" антивирус: " + pcAntivirus);
         }
         foreach (ManagementObject gr in gs.Get())
         {
             foreach (PropertyData pty in gr.Properties)
             {
                 if (pty.Name == "Description")
                 {
                     pcGPU += pty.Value.ToString();
                     twf.WriteLine("  Видеоадаптер: " + pcGPU);
                 }
             }                 
         }
         foreach (ManagementObject gob in sar.Get())
         {
             twf.WriteLine("  Модель компьютера: " + gob["Manufacturer"] + gob["Model"]);
         }
    }
}


P.S:
Я вот попробовал так сделать, но тут не получается:

1 - Выводит ошибку Invalid Class

2 - в twf нельзя записать WriteALLtext, можно только Write

StringBuilder sb = new StringBuilder();
foreach (ManagementBaseObject avResult in avsearcher.Get())
{
Sb.Append(" антивирус: ").Append(pcAntivirus).Append("\n");
}
foreach (ManagementObject gr in gs.Get())
{
foreach (PropertyData pty in gr.Properties)
{
if (pty.Name == "Description")
{
Sb.Append("  Видеоадаптер: ").Append(pcGPU).Append(pty.Value.ToString()).Append("\n");;
}
}
}
foreach (ManagementObject gob in sar.Get())
{
Sb.Append("  Модель компьютера: ").Append(gob["Manufacturer"]).Append(gob["Model"]).Append("\n");;
}
twf.Write(sb.ToString());


А вот ошибка: 

На просторах интернета нашёл интересный код: с использованием List

Информация о системе
    

Ответы

Ответ 1

Например, вы можете отделить получение данных от их вывода.

// зачем вам странный трюк с "& "?
var avNames = avsearcher.Get()
        .Cast()
        .Select(av => (string)av["displayName"])
        .ToList();

var graphicCardList = gs.Get()
        .Cast()
        .Select(mo => (string)mo["Description"])
        .ToList();

var computerInfoList = sar.Get()
        .Cast()
        .Select(gob => (string)gob["Manufacturer"] + gob["Model"])
        .ToList();


И теперь запись:

File.AppendAllLines(
        savePath,
        avNames.Select(s => $"  Антивирус: {s}"),
        Encoding.UTF8);

File.AppendAllLines(
        savePath,
        graphicCardList.Select(s => $"  Видеоадаптер: {s}"),
        Encoding.UTF8);

File.AppendAllLines(
        savePath,
        computerInfoList.Select(s => $"  Модель компьютера: {s}"),
        Encoding.UTF8);


Циклы не нужны вовсе.


Ответ 2

сократить вряд ли, а улучшить - разбить на методы отдельные логические участки.
Текст собирать в StringBuilder и по завершению запись в файл.

Invalid Class - платформа х86 стоит? смените на AnyCPU.
Не стоит все брать в using, он нужен для сохранения в файл, когда дойдет до сохранения,
там стрим и создать и записать.
В sb.ToString() как раз и будет весь текст, он write-ом и запишется

Как оптимизировать данный код

#javascript #инспекция_кода

Здравствуйте,
Есть 2 блока, хочется чтоб .workDataHeight блок брал высоту .work-block.
Написал следующее, подскажите пожалуйста как оптимизировать(ведь делается одно и тоже):  

    var workDataHeightRel = $('.workDataHeight').attr('rel');
    var workBlockRel = $('.work-block').attr('rel');

    if (workBlockRel = 1) {
        $('.workDataHeight[rel="1"]').height($('.work-block[rel="1"]').height());
    }
    if(workBlockRel = 2) {
        $('.workDataHeight[rel="2"]').height($('.work-block[rel="2"]').height());
    }
    if(workBlockRel = 3) {
        $('.workDataHeight[rel="3"]').height($('.work-block[rel="3"]').height());
    }
    if(workBlockRel = 4) {
        $('.workDataHeight[rel="4"]').height($('.work-block[rel="4"]').height());
    }
    if(workBlockRel = 5) {
        $('.workDataHeight[rel="5"]').height($('.work-block[rel="5"]').height());
    }
    if(workBlockRel = 6) {
        $('.workDataHeight[rel="6"]').height($('.work-block[rel="6"]').height());
    }  


Так только первый блок отрабатывает:  

$(".workDataHeight[rel ="+ workDataHeightRel +"]").height($(".work-block[rel ="+
workBlockRel +"]").height());


Вот весь пример с html, css и js: 



var workDataHeightRel = $('.workDataHeight').attr('rel');
var workBlockRel = $('.work-block').attr('rel');

if (workBlockRel = 1) {
  $('.workDataHeight[rel="1"]').height($('.work-block[rel="1"]').height());
}
if(workBlockRel = 2) {
  $('.workDataHeight[rel="2"]').height($('.work-block[rel="2"]').height());
}
if(workBlockRel = 3) {
  $('.workDataHeight[rel="3"]').height($('.work-block[rel="3"]').height());
}
if(workBlockRel = 4) {
  $('.workDataHeight[rel="4"]').height($('.work-block[rel="4"]').height());
}
if(workBlockRel = 5) {
  $('.workDataHeight[rel="5"]').height($('.work-block[rel="5"]').height());
}
if(workBlockRel = 6) {
  $('.workDataHeight[rel="6"]').height($('.work-block[rel="6"]').height());
}
#work{padding:50px;background-color:#eee}#work .work-block{margin-bottom:60px}#work
.work-block:last-child{margin-bottom:0px}#work .work-block .workDataHeight{padding:0px
!important}#work .work-block .work-data{height:100%;color:#34495e;position:relative}#work
.work-block .work-data::before{content:" ";position:absolute;top:46px;left:50%;bottom:46px;width:1px;margin-left:-1.5px;background-color:#34495e}#work
.work-block .work-data p{margin:0px;font-family:"RobotoBold",sans-serif}#work .work-block
.work-data .data-top{position:absolute;top:20px;width:100%}#work .work-block .work-data
.data-bottom{position:absolute;bottom:20px;width:100%}#work .work-block .work-text{height:100%;padding:20px
20px}#work .work-block .work-text h4{font-family:"RobotoBold",sans-serif}#work .work-block
.work-text a{color:#f15324}#work .work-block .work-text p{margin:10px 0 0 0}#work .work-block-inverse
.work-text{text-align:right}#brand{height:200px !important}






    

      

        
        


          

            

              

                
text
                
text
              
            
            

              

                
text
                text
                
text
              
            
          

          

            

              

                
text
                text
                
texttexttexttexttexttexttexttexttex
ttexttexttexttexttexttexttexttexttexttexttextt
xttexttexttexttexttexttexttexttexttexttexttexttexttext
xttexttexttexttexttexttexttexttexttexttexttexttexttext
              
            
            

              

                
text
                
text
              
            
          

          

            

              

                
text
                
text
              
            
            

              

                
text
                text
                
text
              
            
          

          

            

              

                
text
                text
                
text
              
            
            

              

                
text
                
text
              
            
          

          

            

              

                
text
                
text
              
            
            

              

                
text
                text
                
text
              
            
          

          

            

              

                
text
                text
                
texttexttexttexttexttexttexttexttex
ttexttexttexttexttexttexttexttexttexttexttextt
xttexttexttexttexttexttexttexttexttexttexttexttexttext
              
            
            

              

                
text
                
text
              
            
          

        
      
    
  




или jsfiddle
    

Ответы

Ответ 1

Достаточно одной таблэтки строчки в цикле

$('.workDataHeight').each(function(){
    $(this).height($(this).closest('.work-block').height());
});


closest - Ближайший подходящий предок. Т.е. Для каждого из выбранных элементов, closest()
будет искать ближайший подходящий элемент из числа следующих: сам выбранный элемент,
его родитель, его прародитель, и так далее, до начало дерева DOM.


Ответ 2

Циклом можно сделать



for(var i = 0; i < $('.work-block').length; i++){
  $('.workDataHeight[rel="'+i+'"]').height($('.work-block[rel="'+i+'"]').height());
};
#work{padding:50px;background-color:#eee}#work .work-block{margin-bottom:60px}#work
.work-block:last-child{margin-bottom:0px}#work .work-block .workDataHeight{padding:0px
!important}#work .work-block .work-data{height:100%;color:#34495e;position:relative}#work
.work-block .work-data::before{content:" ";position:absolute;top:46px;left:50%;bottom:46px;width:1px;margin-left:-1.5px;background-color:#34495e}#work
.work-block .work-data p{margin:0px;font-family:"RobotoBold",sans-serif}#work .work-block
.work-data .data-top{position:absolute;top:20px;width:100%}#work .work-block .work-data
.data-bottom{position:absolute;bottom:20px;width:100%}#work .work-block .work-text{height:100%;padding:20px
20px}#work .work-block .work-text h4{font-family:"RobotoBold",sans-serif}#work .work-block
.work-text a{color:#f15324}#work .work-block .work-text p{margin:10px 0 0 0}#work .work-block-inverse
.work-text{text-align:right}#brand{height:200px !important}






    

      

        
        


          

            

              

                
text
                
text
              
            
            

              

                
text
                text
                
text
              
            
          

          

            

              

                
text
                text
                
texttexttexttexttexttexttexttexttex
ttexttexttexttexttexttexttexttexttexttexttextt
xttexttexttexttexttexttexttexttexttexttexttexttexttext
xttexttexttexttexttexttexttexttexttexttexttexttexttext
              
            
            

              

                
text
                
text
              
            
          

          

            

              

                
text
                
text
              
            
            

              

                
text
                text
                
text
              
            
          

          

            

              

                
text
                text
                
text
              
            
            

              

                
text
                
text
              
            
          

          

            

              

                
text
                
text
              
            
            

              

                
text
                text
                
text
              
            
          

          

            

              

                
text
                text
                
texttexttexttexttexttexttexttexttex
ttexttexttexttexttexttexttexttexttexttexttextt
xttexttexttexttexttexttexttexttexttexttexttexttexttext
              
            
            

              

                
text
                
text
              
            
          

        
      
    
  

Как упростить два обработчика события?

#javascript #jquery #инспекция_кода

Мой код работает. Но думаю, что как-то не так написал. Можно ли такое сократить? 



$( "#clickbtn_recovery" ).click(function() {
  $( ".modal-backdrop.in" ).hide( 500 );
});
  $( "#clickbtn_recovery" ).click(function() {
  $( "#exampleModal_auth" ).hide( 500 );
});

кнопка нажать
Тут прячутся два разных элемента




HTML-код модального окна под спойлером ниже.



 
  

    

      

        

          
          
Авторизация
      
 
      
 
               

               

               
               
               
   
               

               
               
               
                     
...
   
                     
...
   
                     
...
                 
      
      
      

        
        
             
        
      
    
  
         
 
  

    

      

        

          
          
Востановление
      
 
      
 
               

               

               
               
               
   
                     
...
   
                     
...
   
                     
...
                 
      
      
      

        
             
        
      
    
  
         



    

Ответы

Ответ 1

Селекторы можно перечислять через запятую:

$( "#clickbtn_recovery" ).click(function() {
  $( ".modal-backdrop.in, #exampleModal_auth" ).hide( 500 );
});

Код ревью неофициального приложение для тостера? [закрыт]

#android #инспекция_кода

        
             
                
                    
                        
                            Закрыт. Этот вопрос не по теме. Ответы на него в данный
момент не принимаются.
                            
                        
                    
                
                            
                                
                
                        
                            
                        
                    
                        
                            Хотите улучшить этот вопрос? Переформулируйте вопрос,
чтобы он соответствовал тематике «Stack Overflow на русском».
                        
                        Закрыт 2 года назад.
                                                                                
           
                
        
Исправил все баги и недочеты что написали в прошлый раз, добавил нормальное отображение
текста и фотографии, теперь ссылка будет ссылкой, фотка фоткой и тд...

Добавил проект этот в github

И переписал его, чтобы он удовлетворял принципам MVP (Хотя не уверен т.к. model имеет
ссылку на Presenter)

Прошу подсказать с ревью кода
    

Ответы

Ответ 1

По одному классу CommentAnswerObject. Думаю в остальных так же.

Зачем тут два огромных конструктора, фактически дублирующих друг друга?

public CommentAnswerObject(String urlImage, String name, String dogName, String text,
String date, int numberUserComments, int numberLikeComents, boolean like, List
toggleObjects) {
    this.urlImage = urlImage;
    this.name = name;
    this.dogName = dogName;
    this.text = text;
    this.date = date;
    this.numberUserComments = numberUserComments;
    this.numberLikeComents = numberLikeComents;
    this.like = like;
    this.toggleObjects = toggleObjects;
}

public CommentAnswerObject(String urlImage, String name, String dogName, String text,
String date, int numberUserComments, int numberLikeComents, boolean like) {
    this.urlImage = urlImage;
    this.name = name;
    this.dogName = dogName;
    this.text = text;
    this.date = date;
    this.numberUserComments = numberUserComments;
    this.numberLikeComents = numberLikeComents;
    this.like = like;
}


Второй легко заменяется на 

public CommentAnswerObject(String urlImage, String name, String dogName, String text,
String date, int numberUserComments, int numberLikeComents, boolean like){
    this(urlImage, name, dogName, text, date, numberUserComments, numberLikeComments,
like, new ArrayList<>());
}


Такое огромное количество аргументов конструктора в 99% случаев говорит о том, что
класс неправильно спроектирован.

Действительно ли в этом объекте нужны все эти поля? Как он используется? Какова его
ЕДИНСТВЕННАЯ ОТВЕТСТВЕННОСТЬ?

Вот это экстрактится в класс CommentContent

this.urlImage = urlImage;
this.name = name;
this.dogName = dogName;    //имя собаки?
this.text = text;


Для чего используются эти два свойства?

this.numberUserComments = numberUserComments;
this.numberLikeComents = numberLikeComents;


По CardObject. Все то-же, но и еще: у вас мешанина методов. Один конструктор сверху,
другие разбросаны по тексту. Почитайте Роберта Мартина "Чистый код" раздел "Вертикальное
форматирование"

Класс HTTP. По феншую лучше Http. Статика - зло. В данном случае лучше применить
паттерн Singletone. Он тоже не сахар, но при умелой готовке вполне себе ничего.

ParsingPage - вообще жесть. Чистый Код в руки и читать до дыр. Методы адски перегружены
логикой и вызывают кровотечение из глаз при попытке понять, что происходит. Для начала
разбить монстров на кучу мелких методов, выполняющих атомарные операции и понять, что
половина из них дублирует друг друга. Вообще данный класс штуки на 4 разбивать надо,
на глаз если.

Например класс ParsingPage метод parsFullTag (закроем глаза на статику), первая итерация
"причесывания"

    private static NameAndTagFullInfoObject full;
    private static String html;

    public static NameAndTagFullInfoObject parsFullTag(String html) {
        full = new NameAndTagFullInfoObject();
        ParsingPage.html = html;

        setName();
        setUrlPhoto();
        if (html.contains("
"))
            setDopName();
        if (html.contains("
")) {
            setContribution();
        }
        setQuestion();
        setAnswer();
        setDecisions();
        setTextInto();

        return full;
    }

    private static void setName() {
        full.setName(html.split("