Твой норм, я считаю. Не считаются голоса - наши с простаком, ноунеймов, дизайнеров и некодеровДа бля, а меня то опять за что?
Шо вы такие душные.
Мой голос тоже не засчитывается? Хотя по факту в обе работы расписал.
Твой норм, я считаю. Не считаются голоса - наши с простаком, ноунеймов, дизайнеров и некодеровДа бля, а меня то опять за что?
Шо вы такие душные.
Мой голос тоже не засчитывается? Хотя по факту в обе работы расписал.
О каких массивах в одну строку ты говоришь? ПокажиА кто не понял мой код? Я вроде бы не говорил что твой, либо мой код - хуевый. Читаемость кода у тебя лучше, но в данный момент я оцениваю структуру. Я бы не сказал что у нас сильно отличается читаемость. Просто у тебя массивы в одну строку (можно сказать), а у меня плагин специальный, который автоматически все разворачивает. Тут я уже не при чем. А если что не понятно по моему коду, то скажи, потому что как по мне, читаемость кода 99%
Выразился не правильно. Массивы объектов у тебя структурированы по типу {item: item, id: id, rare: rare}, а у меня раскидано по строчкам, что занимает более 100-200 строк. Но это так делает плагин, с VisualО каких массивах в одну строку ты говоришь? Покажи
Тебе сделать ревью твоего кода?Выразился не правильно. Массивы объектов у тебя структурированы по типу {item: item, id: id, rare: rare}, а у меня раскидано по строчкам, что занимает более 100-200 строк. Но это так делает плагин, с Visual
Я выделил это потому что мне показалось это более читаемым, а не чтобы как-то предьявить за что-то.
Хмм, а что не логичного в 7-м варианте? Там же все логично и просто. На клиентку приходит JSON со значениями редкости. Тот же самый, что приходит и у мадары. Тут смысл то один и тот же. Просто мадара это сделал в числах, а я сделал это именно так. Может быть и было правильнее и проще сделать так, как сделал мадара. Но код как по мне, от этого не хужеУ обоих есть ошибки и косяки. Я конечно не сторонник Учихи и более по душе мне Простак, но тут честно голос присуждаю Учихе.
У простака есть нелогичные моменты, тот же самый 7 вариант, другому кодеру будет сложней его разобрать, есть еще косяки с версткой.
// Если хочется писать быдлокод, то пиши вот так
if (id === 1)
this.setState({ activeFist: true })
else if (id === 2)
this.setState({ activeFist: false })
// Или вот так
// Вообще непонятно, почему ты передаешь какие-то цифры, чтобы потом что-то включить/выключить
// И табы реализуются совсем по-другому
// А конкретно вот так
// Есть список табов и текущий index таба
this.state = {
tabs: ["Рыбы", "Морепродукты"],
tab: 0
}
// Функция смены таба
changeSection = (i) => {
if (this.state.tabs[i] != null)
this.setState({ tab: id })
}
// Такая форма записывается следующим образом
let rares = {
rare: "rare-block rare-rare-item",
epic: "rare-block epic-rare-item",
legendary: "rare-block legendary-rare-item"
}
// Далее получаем className
let itemClass = rares[item.rare]
<div className={itemClass} />
Это ты типа соединил мой код и простака? И от себя добавил дец ?)Парни, оцените мой говнокод.
Готов открыть окно от духоты.
GitHub - MrPeaceDuke/cef-fish
Contribute to MrPeaceDuke/cef-fish development by creating an account on GitHub.github.com
Сделал за пару часов.
Не, у вас по коду ничего не брал, данных входные только взял в каком-то репозитории.(Вроде у тебя)Это ты типа соединил мой код и простака? И от себя добавил дец ?)
Хм, на первом скрине табуляцию не правильно отобразил. Закрывающая скобка идёт не от super(props). И ещё раз говорю, у меня плагин, который табулирует все автоматически, чтобы это смотрелось - хорошо. Если ты не согласен с плагином, то увы, вопросы задавать нужно не мне.@Prostak почему твой код плох? Давай разберем
1. Табуляция - ты её совсем не соблюдаешь.
Посмотреть вложение 2437
Ни в твоих прошлых работах её не было, ни сейчас. Это очень важный момент. Другой кодер потеряет всякое желание в твоем коде разбираться, если там нет толковой табуляции.
Ресурс для чтения информации: https://learn.javascript.ru/coding-style
2. if/else statement
Посмотреть вложение 2438
JavaScript:// Если хочется писать быдлокод, то пиши вот так if (id === 1) this.setState({ activeFist: true }) else if (id === 2) this.setState({ activeFist: false }) // Или вот так // Вообще непонятно, почему ты передаешь какие-то цифры, чтобы потом что-то включить/выключить // И табы реализуются совсем по-другому // А конкретно вот так // Есть список табов и текущий index таба this.state = { tabs: ["Рыбы", "Морепродукты"], tab: 0 } // Функция смены таба changeSection = (i) => { if (this.state.tabs[i] != null) this.setState({ tab: id }) }
3. Нейминг
Посмотреть вложение 2439
Длинные названия не имеющие смысловой нагрузки.
У тебя везде либо item-block, либо filter-block.
Какую смысловую нагрузку несет слово block в названии класса?
Когда ты пишешь <div />, уже понятно, что это будет block
Еще увидел у тебя "take-this-item", как призыв к действую в названии класса. Такое тоже будет недопустимо в нейминге.
Название класса должно характеризовать элемент, а не обращаться к кодеру.
Это отличается от функций JS, где должен использоваться исключительно глагол - changeIndex, disableElement, enableElement и тд
4. Использование this.state в render
Посмотреть вложение 2441
Здесь могу посоветовать прочитать про pure render
5. SVG компонент как поле в объекте???? и одинаковый ID для разных элементов
Посмотреть вложение 2442
Если уж так хочется SVG, то делаем отдельный компонент, который будет рендерить SVG и только SVG. Если хочется ID использовать, то на крайний случай можно передать в props число и будет Layer_1, Layer_2 и тд
6. Возвращаемся к if/else statement
Посмотреть вложение 2443
JavaScript:// Такая форма записывается следующим образом let rares = { rare: "rare-block rare-rare-item", epic: "rare-block epic-rare-item", legendary: "rare-block legendary-rare-item" } // Далее получаем className let itemClass = rares[item.rare] <div className={itemClass} />
7. Напоследок "поясни за шмот"
@koltr Кто там кукарекал про то, что в моем коде нет компонентов?
Так вот тебе 2 компонента, которые являются не компонентами, а хер пойми чем.
Они одинаковы на 95%
Зачем было делать 2 функции, когда можно в одной обрабатывать логику
Посмотреть вложение 2445
Но бля, это плагин. Я сейчас умоюсь, кофе выпью и гифку сюда закину. То что плагин автоматически все подстраиваетСкрин про табуляцию криво сделал. Вот тебе ещё один скрин
Я же у тебя спросил в дискорде, готов ли ты выложить работу или нет. Зачем сейчас эти отмазки?Я уже писал что у меня были планы на 10 число, 11 числа я можно сказать был не очень свеж и сел за работу вечером.
Если ты сам не поправил после генерации кода плагином, то это говорит о том, что тебе похуй)Но бля, это плагин. Я сейчас умоюсь, кофе выпью и гифку сюда закину. То что плагин автоматически все подстраивает
Шо, какие отмазки? Я не отмазывался) я сказал о том что я юзал да одинаковых компонента, потому что не успевал. А переносить на последующие числа я не мог, у меня автошкола и заказы. Я не пытаюсь как-то показать что я был урезан временем. Все было в моих руках.Я же у тебя спросил в дискорде, готов ли ты выложить работу или нет. Зачем сейчас эти отмазки?
После каждого сохранения плагин автоматически все подстраивает дядь, если ты этого не понимаешь, то это говорит о том, что ты даун)Если ты сам не поправил после генерации кода плагином, то это говорит о том, что тебе похуй)
Оскорбления пошли. Ты не злись. Я возвращаю тебя обратно в академию. Попробуешь в следующем годуПосле каждого сохранения плагин автоматически все подстраивает дядь, если ты этого не понимаешь, то это говорит о том, что ты даун)
В плане оскорбления? Я не оскарблял тебя, а сказал по факту. Я до этого 3-4 раза написал про плагин, ты не знаешь как работают такие плагины? По твоей логике, как это должно происходить, если ты пишешь мне «Поправь и сохрани»? Типо ты думал что оно в реальном времени все меняется? Ну бред для дауна, согласисьОскорбления пошли. Ты не злись. Я возвращаю тебя обратно в академию. Попробуешь в следующем году
Какие плагины? Тебе впадлу писать код? Так не берись за это дело.В плане оскорбления? Я не оскарблял тебя, а сказал по факту. Я до этого 3-4 раза написал про плагин, ты не знаешь как работают такие плагины? По твоей логике, как это должно происходить, если ты пишешь мне «Поправь и сохрани»? Типо ты думал что оно в реальном времени все меняется? Ну бред для дауна, согласись
We use cookies and similar technologies for the following purposes:
Do you accept cookies and these technologies?
We use cookies and similar technologies for the following purposes:
Do you accept cookies and these technologies?