Comments (6)
$("#privateMsgModal .modal-title").html("Private chat with " + this.id);
$(".modal-title").attr("id", this.id);
$("#privateMsgModal").modal({ backdrop: "static" });
Во-первых, первую строку стоит переписать с использованием find
:
$("#privateMsgModal").find(".modal-title").html("Private chat with " + this.id);
Во-вторых, $("#privateMsgModal")
стоит сохранить в локальную переменную, чтобы в третьей строке снова не искать нужный элемент.
from nodachat.
У тебя во многих местах используется подобного рода код:
$("<div/>").addClass("media-body").append($mediaDiv);
Было бы неплохо создать helper-функцию, которая будет принимать аргументы (тег, класс, т.д.) и возвращать элемент. Тем самым получится уменьшить объемы jQuery кода и сделать весь код более читабельным.
from nodachat.
$("#btnSendMessage").click(function () {
var message = $("#inputMessage").val();
$("#inputMessage").val("");
message = validateInput(message);
if (message) {
var data = { from: userEmail, to: "all", message: message };
sendMessage(data);
}
});
$("#inputMessage")
стоит сохранить в локальную переменную, чтобы потом снова не искать нужный элемент.
from nodachat.
$("#btnSendPrivateMessagee").click(function () {
var message = $("#inputPrivateMessage").val();
$("#inputPrivateMessage").val("");
message = validateInput(message);
if (message) {
var to = $(".modal-title").attr("id");
var data = { from: userEmail, to: to, message: message };
sendMessage(data);
}
});
То же самое относится к $("#inputPrivateMessage")
.
from nodachat.
Можно также сделать одну функцию-обработчик для последних двух случаев, но это не особо важно.
from nodachat.
from nodachat.
Related Issues (20)
- Bower: хранение и подключение пакетов
- Node JS: именование роутов (в routes.js)
- Node JS: передача app в sockets.js
- Node JS: улучшение createParticipantsMessage функции
- Socket IO: именование событий
- HTML: написание inline-стилей является плохой практикой
- JavaScript: весь клиентский код в chat.js файле
- JavaScript: jQuery цепочки
- CSS: замечания HOT 3
- Node JS: отсутствуют сессии
- Node JS: отсутствует 'use strict'
- Веб-приложение: замечания
- Node JS: кавычки
- Node JS: socket.io
- Node JS: дублирование кода в app.js
- Node JS: стоит использовать везде debug вместо console.log HOT 1
- Node JS: лишний код в dataAccess.js
- Node JS: routes и controllers реализованы в одном файле routes.js
- Node JS: структура папок
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from nodachat.