panicPlayer

Пользователь

Регистрация: 15.07.2019

Сообщения: 537

Рейтинг: -25

Нарушения: 1000

panicPlayer

Регистрация: 15.07.2019

Сообщения: 537

Рейтинг: -25

Нарушения: 1000

https://github.com/k4rj/dota/blob/master/src/main/resources/static/js/main.js




знакомлюсь с vue жи есть




шарющие, так нормально? или можно как-то отрефакторить?



saw_tooth

Пользователь

Регистрация: 20.08.2013

Сообщения: 5550

Рейтинг: 3287

saw_tooth

Регистрация: 20.08.2013

Сообщения: 5550

Рейтинг: 3287

panicPlayer сказал(а):


знакомлюсь с vue жи есть


Нажмите, чтобы раскрыть...


Не мешало бы еще стандартную библиотеку подучить.

panicPlayer

Пользователь

Регистрация: 15.07.2019

Сообщения: 537

Рейтинг: -25

Нарушения: 1000

panicPlayer

Регистрация: 15.07.2019

Сообщения: 537

Рейтинг: -25

Нарушения: 1000

saw_tooth сказал(а):


Не мешало бы еще стандартную библиотеку подучить.


Нажмите, чтобы раскрыть...


ну то время, а тут пока чисто костяк понюхать.


кайфую от рашан доков, аж не привычно)


сидишь как мойдодыра читаешь

B_Rabbit

Пользователь

Регистрация: 11.12.2017

Сообщения: 6795

Рейтинг: 4200

B_Rabbit

Регистрация: 11.12.2017

Сообщения: 6795

Рейтинг: 4200

img

и чуть чуть оффтопа


Цитата:


function getIndex(list, id) {
for (var i = 0; i < list.length; i++ ) {
if (list.id === id) {
return i;
}
}


return -1;
}


Нажмите, чтобы раскрыть...


lodash наше все. Избавит от написания велосипедов.

panicPlayer

Пользователь

Регистрация: 15.07.2019

Сообщения: 537

Рейтинг: -25

Нарушения: 1000

panicPlayer

Регистрация: 15.07.2019

Сообщения: 537

Рейтинг: -25

Нарушения: 1000

B_Rabbit сказал(а):


сразу переходи на однофайловые компоненты. Еще бы пулл данных подключить не мешало (Vuex). Ну и навигация без перезагрузки будет приятным плюсом (vue-routers). Вот тебеhttps://awesome-vue.js.org там есть куча плюшек полезных.




и чуть чуть оффтопа




lodash наше все. Избавит от написания велосипедов.


Нажмите, чтобы раскрыть...


хы, спс) чекну

deleted1313

Пользователь

Регистрация: 24.07.2019

Сообщения: 75

Рейтинг: 12

deleted1313

Регистрация: 24.07.2019

Сообщения: 75

Рейтинг: 12

Посмотри async функции и методы массива (map, filter, reduce, find и т.д.)

HiThere

Пользователь

Регистрация: 24.06.2016

Сообщения: 3475

Рейтинг: 2614

HiThere

Регистрация: 24.06.2016

Сообщения: 3475

Рейтинг: 2614

зачем искать индекс через for цикл когда для этого придумали indexOf ??


for цикл это вообще пережиток 90х, даже такое старье как java пытается активно это выпилить


Кста посмотрел на vue, 1 в 1 как реакт, зачем тогда его учить если есть тот же реакт который более востребован?

panicPlayer

Пользователь

Регистрация: 15.07.2019

Сообщения: 537

Рейтинг: -25

Нарушения: 1000

panicPlayer

Регистрация: 15.07.2019

Сообщения: 537

Рейтинг: -25

Нарушения: 1000

HiThere сказал(а):


зачем искать индекс через for цикл когда для этого придумали indexOf ??


for цикл это вообще пережиток 90х, даже такое старье как java пытается активно это выпилить


Кста посмотрел на vue, 1 в 1 как реакт, зачем тогда его учить если есть тот же реакт который более востребован?


Нажмите, чтобы раскрыть...


я хз, кто-то юзает реакт, кто-то вью, поэтому хороший боец должен быть готов ко всему )




вью как говорят более перспективный шоль.




почему не писать, если при fori > готовый цикл, какая разница?)




deleted1313 сказал(а):


Посмотри async функции и методы массива (map, filter, reduce, find и т.д.)


Нажмите, чтобы раскрыть...


это ты про шо?


про вью? или про стрим?

deleted1313

Пользователь

Регистрация: 24.07.2019

Сообщения: 75

Рейтинг: 12

deleted1313

Регистрация: 24.07.2019

Сообщения: 75

Рейтинг: 12

panicPlayer сказал(а):


я хз, кто-то юзает реакт, кто-то вью, поэтому хороший боец должен быть готов ко всему )




вью как говорят более перспективный шоль.




почему не писать, если при fori > готовый цикл, какая разница?)




это ты про шо?


про вью? или про стрим?


Нажмите, чтобы раскрыть...


Я про стандарт JS, посмотри на MDN async functions и Array.prototype.map, Array.prototype.filter, Array.prototype.reduce и тд

panicPlayer

Пользователь

Регистрация: 15.07.2019

Сообщения: 537

Рейтинг: -25

Нарушения: 1000

panicPlayer

Регистрация: 15.07.2019

Сообщения: 537

Рейтинг: -25

Нарушения: 1000

deleted1313 сказал(а):


Я про стандарт JS, посмотри на MDN async functions и Array.prototype.map, Array.prototype.filter, Array.prototype.reduce и тд


Нажмите, чтобы раскрыть...


а ну готовые итерации, ну то такое дело, я ж не про рефакторинг такого типа имел, а удаление там промежуточных, не нужных действий может, или там способы реализации есть быстрее и проще.


я люблю бить по клавишам, поэтому мне не лень)

deleted1313

Пользователь

Регистрация: 24.07.2019

Сообщения: 75

Рейтинг: 12

deleted1313

Регистрация: 24.07.2019

Сообщения: 75

Рейтинг: 12

panicPlayer сказал(а):


а ну готовые итерации, ну то такое дело, я ж не про рефакторинг такого типа имел, а удаление там промежуточных, не нужных действий может, или там способы реализации есть быстрее и проще.


я люблю бить по клавишам, поэтому мне не лень)


Нажмите, чтобы раскрыть...


promise.then заменяются на async, итерация for на методы массива, хотя бы это

panicPlayer

Пользователь

Регистрация: 15.07.2019

Сообщения: 537

Рейтинг: -25

Нарушения: 1000

panicPlayer

Регистрация: 15.07.2019

Сообщения: 537

Рейтинг: -25

Нарушения: 1000

deleted1313 сказал(а):


promise.then заменяются на async, итерация for на методы массива, хотя бы это


Нажмите, чтобы раскрыть...


окей




а чем отличаются вот эти v-click от@click ?

deleted1313

Пользователь

Регистрация: 24.07.2019

Сообщения: 75

Рейтинг: 12

deleted1313

Регистрация: 24.07.2019

Сообщения: 75

Рейтинг: 12

panicPlayer сказал(а):


окей




а чем отличаются вот эти v-click от@click ?


Нажмите, чтобы раскрыть...


Одно и то же, через@сокращенная запись

Just_a_fire

Пользователь

Регистрация: 09.03.2016

Сообщения: 3597

Рейтинг: 3090

Just_a_fire

Регистрация: 09.03.2016

Сообщения: 3597

Рейтинг: 3090

В props желательно указать типы свойств, а ещё можно значение по умолчанию, если требуется.


Для передачи свойств и методов компоненту-наследнику можно использовать provide.


Для темплейтов лучше использовать шаблонные строки(ES6, поддерживается всеми современными браузерами).

Laurarouge

Пользователь

Регистрация: 11.12.2016

Сообщения: 1608

Рейтинг: 816

Laurarouge

Регистрация: 11.12.2016

Сообщения: 1608

Рейтинг: 816

HiThere сказал(а):


зачем искать индекс через for цикл когда для этого придумали indexOf ??


for цикл это вообще пережиток 90х, даже такое старье как java пытается активно это выпилить


Кста посмотрел на vue, 1 в 1 как реакт, зачем тогда его учить если есть тот же реакт который более востребован?


Нажмите, чтобы раскрыть...


Потому что Реакт - помойка

Eustass

Пользователь

Регистрация: 09.02.2013

Сообщения: 1027

Рейтинг: 1044

Eustass

Регистрация: 09.02.2013

Сообщения: 1027

Рейтинг: 1044

img

не хочу дизморалить, но чет где-то на хабре недавно прочитал что vue парашненько обновили типа и что-то поменяли в худшую сторону




п.с. нашел - https://habr.com/ru/post/457250/


п.с. я не шарю, просто полистываю хабр и запомнил этот пост

3xc3xcwarior

Пользователь

Регистрация: 07.07.2019

Сообщения: 405

Рейтинг: -130

Нарушения: 1000

3xc3xcwarior

Регистрация: 07.07.2019

Сообщения: 405

Рейтинг: -130

Нарушения: 1000

img

B_Rabbit сказал(а):


lodash наше все. Избавит от написания велосипедов.


Нажмите, чтобы раскрыть...


Cтандартная библиотека ваша все, не юзайте ничего кроме нее, а еще не смотрите как она реализована и не волнуйтесь по тому поводу можно ли сделать быстрее
https://github.com/electronicarts/EASTL

B_Rabbit

Пользователь

Регистрация: 11.12.2017

Сообщения: 6795

Рейтинг: 4200

B_Rabbit

Регистрация: 11.12.2017

Сообщения: 6795

Рейтинг: 4200

img

3xc3xcwarior сказал(а):


Cтандартная библиотека ваша все, не юзайте ничего кроме нее, а еще не смотрите как она реализована и не волнуйтесь по тому поводу можно ли сделать быстрее
https://github.com/electronicarts/EASTL


Нажмите, чтобы раскрыть...


а при чем тут плюсы?



panicPlayer

Пользователь

Регистрация: 15.07.2019

Сообщения: 537

Рейтинг: -25

Нарушения: 1000

panicPlayer

Регистрация: 15.07.2019

Сообщения: 537

Рейтинг: -25

Нарушения: 1000

Eustass сказал(а):


не хочу дизморалить, но чет где-то на хабре недавно прочитал что vue парашненько обновили типа и что-то поменяли в худшую сторону




п.с. нашел - https://habr.com/ru/post/457250/


п.с. я не шарю, просто полистываю хабр и запомнил этот пост


Нажмите, чтобы раскрыть...


видимо я за 30 сек прочитал больше статьи чем ты)


он пишет, что доволен изменениями, а так пофиг, я чисто для себя

3xc3xcwarior

Пользователь

Регистрация: 07.07.2019

Сообщения: 405

Рейтинг: -130

Нарушения: 1000

3xc3xcwarior

Регистрация: 07.07.2019

Сообщения: 405

Рейтинг: -130

Нарушения: 1000

img

B_Rabbit сказал(а):


а при чем тут плюсы?




Нажмите, чтобы раскрыть...


При том что там что на плюсах стандартная библиотека написана явно лучше чем в js
и даже она не идеальная, потому я дал ссылку на либу которая делает тоже что и стандартная,
только быстрее.
Вывод из этого можно сделать такой, что не стоит слепо юзать функции из стандарта
И тем более навязывать кому то их исопльзование. ТЕМ БОЛЕЕ чел учится, это наоборот
хорошо что он сам пишет велосипеды
Я уверен, что никто, не ты ни все кто тут ответил ни разу даже не заглядывали как этот indexOF работает
и даже не знаете где смотреть.

АнтиНищун

Пользователь

Регистрация: 08.05.2013

Сообщения: 3279

Рейтинг: 1003

АнтиНищун

Регистрация: 08.05.2013

Сообщения: 3279

Рейтинг: 1003

Not Found

panicPlayer

Пользователь

Регистрация: 15.07.2019

Сообщения: 537

Рейтинг: -25

Нарушения: 1000

panicPlayer

Регистрация: 15.07.2019

Сообщения: 537

Рейтинг: -25

Нарушения: 1000

panicPlayer сказал(а):


я дурак сори)


забыл вынести


Нажмите, чтобы раскрыть...


вынес проверяй

АнтиНищун

Пользователь

Регистрация: 08.05.2013

Сообщения: 3279

Рейтинг: 1003

АнтиНищун

Регистрация: 08.05.2013

Сообщения: 3279

Рейтинг: 1003

Not Found

panicPlayer

Пользователь

Регистрация: 15.07.2019

Сообщения: 537

Рейтинг: -25

Нарушения: 1000

panicPlayer

Регистрация: 15.07.2019

Сообщения: 537

Рейтинг: -25

Нарушения: 1000

spellmaster сказал(а):


https://github.com/k4rj/dota/commit/fa7950dd71950cdb269a196ccc00af472f1b2b8f, проверил, тебе rebase нужен ( погугли )


Нажмите, чтобы раскрыть...


а так я перегенерил код, тот устаревший бтв

deleted1313

Пользователь

Регистрация: 24.07.2019

Сообщения: 75

Рейтинг: 12

deleted1313

Регистрация: 24.07.2019

Сообщения: 75

Рейтинг: 12

panicPlayer сказал(а):


а так я перегенерил код, тот устаревший бтв


Нажмите, чтобы раскрыть...


Переменные еще объявляй через const вместо var