Conversation
ala-n
left a comment
ala-n
left a comment
There was a problem hiding this comment.
Я конечно могу много еще к чему придраться, но в основном это может быть out of practice ну или текущих заданий)
Вообще нравится как пишешь код уже.
В основном все читается и структурировано.
Я еще до конца не досмотрел, но в любом случае ощущения крайне положительные ;) спасибо!
PS: мержни плиз текущий PullRequest ибо много изменениий уже.
| break; | ||
| } | ||
|
|
||
| fs.readFile(filePath, function(error, content) { |
There was a problem hiding this comment.
Ой малайца!
Первый кто на 10 претендует за 7 лабу )
|
|
||
| var extname = path.extname(filePath); | ||
| var contentType = 'text/html'; | ||
| switch (extname) { |
| }, | ||
|
|
||
| editPost: function (id, post) { | ||
| if (post.description && post.description.length > 200) { |
There was a problem hiding this comment.
А обязан ли по апи для 4 задания здесь быть заполненый пост. Вообще говоря нет. Но да ладно.
| } | ||
|
|
||
| let getTemp = ApplicationModel.getPhotoPost(id); | ||
| if (post.description) { |
There was a problem hiding this comment.
А вот тут потенциальная can't read 'description' of null :)
| }, | ||
|
|
||
| fillMapHash: function () { | ||
| for (let i = 0; i < ApplicationModel.photoPosts.length; i++) { |
| } | ||
| }, | ||
|
|
||
| fillInformation: function(){ |
There was a problem hiding this comment.
readFromLocalStorage или просто load (все равно временно)
Сейчас название не отражает действительное действие.
| }(); | ||
|
|
||
| let EventsModule = function () { | ||
| //по частям |
There was a problem hiding this comment.
??
Ешки, как много кэшнутых значений. Есть подозрение что черезчур много...
| } | ||
| if(selectAdd.selectedIndex !== 1){ | ||
| if(arrayHashtags.indexOf(selectAdd.options[selectAdd.selectedIndex].value) === -1) { | ||
| /*саша я тебя люблю :3 */ |
|
Так код конечно хороший но в на клиенте мало что работает, надо оживить хотябы основные функции. |
2 participants
No description provided.