started modulation to improve routing

Author: Sameer-Goutam06

notes.txt

@@ -1,32 +1,2 @@
-Design Issues:
-
-MongoDB- 3 collections Users,Careers,Assessments
-User Collection- collects info of users along with passwords and checks to enabal if an account is pre-existing
-Careers Collection- collects and sorts data on basis of certain keys
-Assessments Collection- Questions to be given
-
-Get method pages:
-error.ejs
-home.ejs
-Careers.ejs
-assessment.ejs
-login.ejs
-register.ejs
-Career.ejs
-
-Post method pages:
-login.ejs
-register.ejs
-assessment.ejs
-
-Data analysis for Assessments for every user required
-
-Registration page to Login
-Login Page to Profile
-
-Data needed the most for career pages
-1. Career name
-2. Career description
-3. Career requirements
-4. Career salary
-5. Career growth rate
+Design Modifications:
+Need to seperate profile and career routing in index.js

routes/careerRoutes.js

@@ -5,19 +5,29 @@ const path= require('path');
 const port=8080;
 
 //we need to create a folder nameed public to serve static files such as html,css,ejs..
-app.use(express.static(path.join(__dirname,"/public/CSS")));
-app.use(express.static(path.join(__dirname,"/public/JS")));
+app.use(express.static(path.join(__dirname,"../public/CSS")));
+app.use(express.static(path.join(__dirname,"../public/JS")));
 
 //using of ejs which is already installed in express folder hence we need to use it using app.set()
 //view => templates
 app.set("view engine", "ejs");
 
 //setting path to run server globally and to avoid views loading from root directory when it is located in a leaf directory
-app.set("views",path.join(__dirname,"/views"));
+app.set("views",path.join(__dirname,"../views"));
 
 //to get the data from the post method
+//app.use(bodyParser.urlencoded({ extended: true }));
+
 app.use(express.urlencoded({extended:true}));
 
+//importing routes 
+// const careerRoutes = require('./routes/careerRoutes');
+// const profileRoutes = require('./routes/profileRoutes');
+
+// // Use routes
+// app.use('/careers', careerRoutes);
+// app.use('/profile', profileRoutes);
+
 //importing mongoose
 const mongoose = require('mongoose');
 main()
@@ -64,7 +74,7 @@ app.listen(port,()=>{
 //home route
 app.get("/",(req,res)=>
 {
-    res.render("home.ejs",{port});
+    res.render("home.ejs",{logged:false,uname:undefined});
 });
 
 //careers route
@@ -73,7 +83,7 @@ app.get("/careers",(req,res)=>{
 });
 
 //Careers route
-app.get("/c/:industry",(req,res)=>{
+app.get("/industry/:industry",(req,res)=>{
     Career.find({cindustry:req.params.industry})
     .then((careers)=>
     {
@@ -83,11 +93,11 @@ app.get("/c/:industry",(req,res)=>{
 });
 
 // individual career route
-app.get("/ca/:career",(req,res)=>{
-    Career.findOne({cname:req.params.career})
+app.get("/career/:career",(req,res)=>{
+    Career.find({cname:req.params.career})
     .then((results)=>
     {
-        res.render("career.ejs",{career:results[0],cin:results[0].cindustry.join(", ")});
+        res.render("career.ejs",{data:results[0]});
     })
     .catch((err)=>
     {
@@ -103,37 +113,67 @@ app.get("/login",(req,res)=>{
 //Login Route post request to get data and check if user exists
 app.post("/login",(req,res)=>
 {
-    let username,password=req.body;
+    let {username,password}=req.body;
     console.log(username,password);
-    let check=User.findOne({cname:username})
-    .then((res)=>
+    User.findOne({cname:username})
+    .then((user)=>
     {
-        if (res.password!==password)
+        if(!user)
         {
-            res.render("error.ejs",{data:"Passwords doesnt match"});
+            return res.render("error.ejs",{data:"User doesn't exist"});
         }
-        else
+        if (user.password!==password)
         {
-            res.redirect("home.ejs");
+            return res.render("error.ejs",{data:"Passwords doesnt match"});
         }
+        res.render("home.ejs",{logged:true,uname:username});
     })
-    .catch((err)=>{res.render("error.ejs",{data:"User doesn't exist"})});
+    .catch((err)=>
+    {
+        res.render("error.ejs",{data:err});
+    });
 });
 
 //registration route
 app.get("/register",(req,res)=>{
-    res.render("register.ejs",{port});
+    res.render("register.ejs",{logged:false});
 });
 
 //registration route to check if user exists before
 app.post("/register",(req,res)=>
 {
-    let {username,email,password,confirmPassword}=req.body;
-    if (password !== confirmPassword) {
+    let {username,mail,pass,confirmPass}=req.body;
+    if (pass !== confirmPass) {
         return res.render('error.ejs', { data: 'Passwords do not match' });
     }
-    else
-    {
+    User.findOne({name:username})
+    .then((user)=>{
+        if(user)
+        {
+            return res.render("error.ejs",{data:"User already exists"});
+        }
+        const newUser = new User({ name: username, email: mail, password: pass });
+        return newUser.save();
+    }).then(()=>{
+        res.render("login.ejs",{logged:false})
+    })
+    .catch((err)=>{
+        console.log(err);
+    });
+});
 
-    }
+//profile route
+app.get("/profile/:user",(req,res)=>{
+    let uname=req.params.user;
+    User.findOne({name:uname})
+    .then((results)=>{
+        res.render("profile.ejs",{user:results});
+    })
+    .catch((err)=>{
+        res.render("error.ejs",{data:err});
+    });
+});
+
+app.get("/logout",(req,res)=>{
+    res.render("home.ejs",{logged:false,uname:undefined});
 });

index.js renamed to routes/index.js

@@ -0,0 +1,20 @@
+const mongoose = require('mongoose');
+main()
+    .then(() => {
+        console.log("Career Schema connected");
+    })
+    .catch((err) => {
+        console.log(err);
+    });
+async function main(){
+    await mongoose.connect('mongodb://127.0.0.1:27017/RTSR');
+}
+//creating schema for career collection
+const careerSchema=new mongoose.Schema({
+    cname:String,
+    cdesc:String,
+    crequirements:String,
+    csalary:Number,
+    cgrowthrate:Number,
+    cindustry:Array
+});

routes/models/Career.js

@@ -0,0 +1,20 @@
+const mongoose = require('mongoose');
+main()
+    .then(() => {
+        console.log("User schema connected");
+    })
+    .catch((err) => {
+        console.log(err);
+    });
+async function main(){
+    await mongoose.connect('mongodb://127.0.0.1:27017/RTSR');
+}
+//creating schema for user collection
+const userSchema=new mongoose.Schema
+(
+    {
+        name:String,
+        email:String,
+        password:String
+    }
+);

routes/models/User.js

@@ -3,7 +3,7 @@
 <head>
   <meta charset="UTF-8">
   <meta name="viewport" content="width=device-width, initial-scale=1.0">
-  <title>CareerXplorer:<%= career.cname %></title>
+  <title>CareerXplorer: <%= data.cname %></title>
   <link rel="preconnect" href="https://fonts.googleapis.com">
   <link rel="preconnect" href="https://fonts.gstatic.com" crossorigin>
   <link href="https://fonts.googleapis.com/css2?family=Raleway:wght@400;500;700&family=Roboto:wght@400;500;700&display=swap" rel="stylesheet">
@@ -92,27 +92,27 @@
             <span class="C">Career</span><span class="X">Xplorer</span>
         </div>
         <div class="title">
-            <h1><%=career.cname%></h1>
+            <h1><%=data.cname%></h1>
         </div>
     </div>
     <div class="content">
         <div class="section requirements">
             <h2>Requirements</h2>
-            <h4><%= career.crequirements %></h4>
+            <h4><%= data.crequirements %></h4>
         </div>
         <div class="section industries">
             <h2>Industries</h2>
-            <h4><%= cin %></h4>
+            <h4><%= data.cindustry.join(", ") %></h4>
         </div>
         <div class="section income">
             <h2>Salary</h2>
-            <h4>$<%= career.csalary %> per year</h4>
+            <h4>$<%= data.csalary %> per year</h4>
             <h2>Growth Rate</h2>
-            <h4><%= career.cgrowthrate %></h4>
+            <h4><%= data.cgrowthrate %></h4>
         </div>
         <div class="section desc">
             <h2>Career Description</h2>
-            <p><%= career.cdesc %></p>
+            <p><%= data.cdesc %></p>
         </div>
     </div>
   </div>