Size: a a a

Angular.js (1.x) — русскоговорящее сообщество

2021 July 20

Я

Ярослав in Angular.js (1.x) — русскоговорящее сообщество
вот сам код
источник

Я

Ярослав in Angular.js (1.x) — русскоговорящее сообщество
хз правда почему там контент тайп джейсон, хотя на самом деле возвращает массив байт
источник

R

Remite in Angular.js (1.x) — русскоговорящее сообщество
Сохрани как джейсон и почитай шо прелетело, может там ошибка какая-то вернулась вместо файла :)
источник

Я

Ярослав in Angular.js (1.x) — русскоговорящее сообщество
всё норм, массив байт
источник

R

Remite in Angular.js (1.x) — русскоговорящее сообщество
а конвертни в строку, шо скажет
источник

Я

Ярослав in Angular.js (1.x) — русскоговорящее сообщество
что именно в строку конвертнуть, массив байт?
источник

Я

Ярослав in Angular.js (1.x) — русскоговорящее сообщество
всё, порешал
источник

Я

Ярослав in Angular.js (1.x) — русскоговорящее сообщество
сделал так
источник

C☭

Chadwick ☭ in Angular.js (1.x) — русскоговорящее сообщество
источник
2021 July 21

Я

Ярослав in Angular.js (1.x) — русскоговорящее сообщество
А сделайте плс ревью))
источник

Я

Ярослав in Angular.js (1.x) — русскоговорящее сообщество
нижняя часть кода, проверка если в массиве уже есть файл и если он .csv то мы очищаем массив и пушим туда новый
источник

Я

Ярослав in Angular.js (1.x) — русскоговорящее сообщество
вот мне кажется, не слишком ли там замудрено сделано ил норм
источник

C☭

Chadwick ☭ in Angular.js (1.x) — русскоговорящее сообщество
Есть моменты
источник

C☭

Chadwick ☭ in Angular.js (1.x) — русскоговорящее сообщество
Нужно кофе сначала
источник

C☭

Chadwick ☭ in Angular.js (1.x) — русскоговорящее сообщество
- если есть badFile то return, зачем дальше идти?
- замени file.Name на file.name
- если godfile это один объект, то зачем ты его клониш еще в file?
- xxx.include('.csv')  не безопасна. лучше юзай regex для теста того что файл имеет точное совпадение,, в твоем же случае файл "badfile.csv.exe" будет спокойно запушен в базу
- newCsvFile:bool вообще не несет в себе смысловой нагрузки. замени на "isCsvFile"
- existigCsvFile это лист,  а значит должно быть "existigCsvFiles"
- длинный IF стоит вынести в переменную и назвать ее осмысленно, чтобы не вчитываясь в экспрежн было ясно в чем суть ифа
- почему в конце идет очищение массива и добавление файла?
источник

Я

Ярослав in Angular.js (1.x) — русскоговорящее сообщество
- existigCsvFile это лист,  а значит должно быть  - почему лист? я это делаю в любом случае если хотя бы 1 файл уже подобный есть и мы всегда получаем 1 файл
источник

Я

Ярослав in Angular.js (1.x) — русскоговорящее сообщество
- почему в конце идет очищение массива и добавление файла?

если уже есть 1 цсв файл, то мы его заменяем
источник

Я

Ярослав in Angular.js (1.x) — русскоговорящее сообщество
а чтобы не париться искать по индексу и менять ,просто очищаем массив
источник

Я

Ярослав in Angular.js (1.x) — русскоговорящее сообщество
так как изначально там 2 файла может быть 1 цсв и 1 тхт, если цсв поменяли, то тхт тоже нужно менять
источник

Я

Ярослав in Angular.js (1.x) — русскоговорящее сообщество
а так очищаем массив и тхт файл сам так же очищается
источник