onclick HTML not working

Question

<html>
    <head>
        <title>DisciplinasMatriculadas</title>
        <script type="text/javascript" src="index.js"></script>
    </head>
    <BODY style="font-family: sans-serif">

        <fieldset>
            <legend>Matricula de aluno em disciplina</legend>
            <form action="/myform" method="GET">
            ID Aluno: <input type="text" name="aluno" id="aluno">
            <br/>
            ID Disciplina: <input type="text" name="disc" id="disc">
            <br/>
            </form>
        <input type="button" value="Add" onclick="addDisciplina();alert('Done.');">
        </fieldset>
    </body>
</html>

Hi, I'm trying to make this onclick function work (newbie here), but I can't figure out what is the problem. Could you guys help me, please?

PS: The index.js works fine if I run it by CMD.

Thanks!

PS2: I've edited like you guys said, but still didn't work. Here's the js:

addDisciplina = function() {

var mysql = require('mysql');
var express = require('express'); 
var app = express(); 
app.get('/myform', function(req, res){ 
    var aluno = req.query.aluno;
    var disc = req.query.disc; 
}); 

var connection = mysql.createConnection({
 host: "localhost",
 user: "root",
 password: "161616",
 database: "matricula"
});

var post  = {idDisciplinasMatriculada: aluno*3+disc*2, idDisciplina: disc, idAluno: aluno};

connection.query('INSERT INTO DisciplinasMatriculadas SET ?', post, function(err, result) {});

}

The index.js is NodeJs code, it will not run by frontend side, what you need to do is to create a route(RESTFUL API), and make a ajax call or http request. — Joey Etamity
– Joey Etamity, Commented Jul 15, 2016 at 12:44

Djaouad · Accepted Answer · 2016-07-15 11:36:07Z

1

you're closing </select> while there are no <select>.

answered Jul 15, 2016 at 11:36

Djaouad

22.8k4 gold badges37 silver badges57 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Joey Etamity · Accepted Answer · 2016-07-15 11:42:54Z

0

<HTML>
<HEAD>
    <TITLE>DisciplinasMatriculadas</TITLE>
    <SCRIPT language="javascript" src="index.js"></SCRIPT>
</HEAD>
<BODY style="font-family: sans-serif">

    <fieldset>
        <legend>Matricula de aluno em disciplina</legend>
        <form action="/myform" method="GET"><select>
        ID Aluno: <input type="text" name="aluno" id="aluno"></select>
        <br/>
        <select>
    ID Disciplina: <input type="text" name="disc" id="disc"></select>
    <br/>
    </form>
<input type="button" value="Add" onclick="addDisciplina();alert('Done.');">
</fieldset>

Correct the Html and try again :)

answered Jul 15, 2016 at 11:42

Joey Etamity

8664 silver badges9 bronze badges

1 Comment

Minhoo Over a year ago

I've removed the selects, they were useless, but still not working :(

Luke · Accepted Answer · 2016-07-15 12:10:53Z

0

You have some errors in HTML file. First of all, you don't need </select> at all if you don't have <select> tag opened. The <select> should be a parent of a child element <option>, for example:

<select>
  <option value="volvo">Volvo</option>
  <option value="saab">Saab</option>
  <option value="mercedes">Mercedes</option>
  <option value="audi">Audi</option>
</select>

You can use multiple onclick events, just separate them with a semicolon (;). Use <script type="text/javascript" src="index.js"></script> instead of language="javascript", you don't need it and don't forget using a lowercase symbols, it's the best practise. This is jsfiddle for you.

answered Jul 15, 2016 at 12:10

Luke

351 silver badge6 bronze badges

1 Comment

Minhoo Over a year ago

Done, but still nothing :(

Collectives™ on Stack Overflow

onclick HTML not working

3 Answers 3

Comments

1 Comment

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related