帮我优化下面的代码,我不喜欢下面代码的风格<!DOCTYPE html> <html> <head> <title>Login</title> <script src="https://cdn.bootcdn.net/ajax/libs/jquery/3.6.0/jquery.min.js"></script> <style> body { background-color: #f2f2f2; } form { display: flex; flex-direction: column; align-items: center; margin-top: 50px; background-color: #ffffff; padding: 50px; border-radius: 10px; box-shadow: 0 4px 8px 0 rgba(0, 0, 0, 0.2), 0 6px 20px 0 rgba(0, 0, 0, 0.19); } label { font-size: 18px; margin-bottom: 5px; } input[type=text], input[type=password] { padding: 10px; border-radius: 5px; border: none; margin-bottom: 20px; width: 300px; } input[type=button] { background-color: #4CAF50; color: white; border: none; border-radius: 5px; padding: 10px 20px; cursor: pointer; font-size: 18px; margin-bottom: 20px; width: 300px; } input[type=button]:hover { background-color: #3e8e41; } h1 { text-align: center; margin-top: 50px; font-size: 36px; color: #4CAF50; } </style> </head> <body> <h1>Login</h1> <form> <label for="username">Username:</label> <input type="text" id="username" name="username"> <label for="password">Password:</label> <input type="password" id="password" name="password"> <input type="button" id="submit" value="Submit"> </form> <script> $(document).ready(function() { $("#submit").click(function() { var username = $("#username").val(); var password = $("#password").val(); $.ajax({ url: "/login", type: "POST", data: { username: username, password: password }, success: function(response) { if (response == "success") { window.location.href = "/dashboard"; } else { alert("Invalid username or password."); } }, error: function() { alert("Error occurred."); } }); }); }); </script> </body> </html>
时间: 2024-01-04 12:03:11 浏览: 103
以下是我对代码进行的一些优化建议:
1. 使用外部 CSS 文件:将 CSS 样式移动到一个外部文件中,这样可以使 HTML 文件更加简洁,易于维护。
2. 将 JavaScript 移动到外部文件:同样地,将 JavaScript 代码移动到一个外部文件中,这样可以使 HTML 文件更加简洁,易于维护。
3. 将 JavaScript 代码封装到函数中:将 JavaScript 代码封装到一个函数中,可以使代码更加模块化,易于维护。
4. 使用严格模式:在 JavaScript 代码的开头加上 'use strict';,可以使代码更加规范,减少错误。
5. 使用箭头函数:使用箭头函数可以使代码更加简洁,易于阅读。
下面是优化后的代码:
HTML 文件:
<!DOCTYPE html>
<html>
<head>
<title>Login</title>
<link rel="stylesheet" href="style.css">
</head>
<body>
<h1>Login</h1>
<form>
<label for="username">Username:</label>
<input type="text" id="username" name="username">
<label for="password">Password:</label>
<input type="password" id="password" name="password">
<input type="button" id="submit" value="Submit">
</form>
<script src="script.js"></script>
</body>
</html>
CSS 文件:
body {
background-color: #f2f2f2;
}
form {
display: flex;
flex-direction: column;
align-items: center;
margin-top: 50px;
background-color: #ffffff;
padding: 50px;
border-radius: 10px;
box-shadow: 0 4px 8px 0 rgba(0, 0, 0, 0.2), 0 6px 20px 0 rgba(0, 0, 0, 0.19);
}
label {
font-size: 18px;
margin-bottom: 5px;
}
input[type=text], input[type=password] {
padding: 10px;
border-radius: 5px;
border: none;
margin-bottom: 20px;
width: 300px;
}
input[type=button] {
background-color: #4CAF50;
color: white;
border: none;
border-radius: 5px;
padding: 10px 20px;
cursor: pointer;
font-size: 18px;
margin-bottom: 20px;
width: 300px;
}
input[type=button]:hover {
background-color: #3e8e41;
}
h1 {
text-align: center;
margin-top: 50px;
font-size: 36px;
color: #4CAF50;
}
JavaScript 文件:
'use strict';
const submitForm = () => {
const username = $("#username").val();
const password = $("#password").val();
$.ajax({
url: "/login",
type: "POST",
data: { username, password },
success: (response) => {
if (response === "success") {
window.location.href = "/dashboard";
} else {
alert("Invalid username or password.");
}
},
error: () => {
alert("Error occurred.");
}
});
};
$(document).ready(() => {
$("#submit").click(submitForm);
});
阅读全文