Why this JavaScript code doesn't work?

Question

1.00/5 (1 vote)

See more:

I tried to create a very very simple implementation of jQuery.
please check out following code.Why it doesn't work?
Try code at here.
HTML:

HTML

<p id="a">Hello</p>

JavaScript:

JavaScript

function $(selector)
{  
    var x=new aQuery(selector);
    return x;
}

function aQuery(id)
{
    this.objId=id;
    function html(newh)
    {
        document.getElementById(this.objId).innerHTML=newh;
    }
}



$("a").html("Hello aQuery");

Posted 21-Aug-14 5:34am

AmiR1999

Updated 21-Aug-14 5:39am

Thomas Daniels

v3

Add a Solution

Comments

OriginalGriff 21-Aug-14 11:39am

No.
I'm not going off to a random site to "try something that doesn't work". I have no idea what I will find at the other end!

If you want us to help you, then:
1) Tell us about the problem. What should it do? What does it do? How do you make it do that?
2) Show us the relevant code fragment - just the bit we need, not the whole thing - so we can see it easily.

Help us to help you!
Use the "Improve question" widget to edit your question and provide better information.

AmiR1999 21-Aug-14 11:42am

I edited the question.Why are you so strict and angrY my friend? :D

OriginalGriff 21-Aug-14 11:53am

That's not strict or angry: it's just explaining that you need to think about the people you are asking for help, and make it as easy for them as possible.

AmiR1999 22-Aug-14 7:55am

I'm sorry,thank you.

OriginalGriff 22-Aug-14 8:05am

That's OK! :laugh:

AmiR1999 21-Aug-14 11:40am

Thank you @ProgramFOX for the update :D

Thomas Daniels 21-Aug-14 11:44am

You're welcome!

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Thomas Daniels · Accepted Answer · 2014-08-21T05:43:00

Solution 1

You should create the html function inside aQuery like this:

JavaScript

this.html = function(newh)
{
    document.getElementById(this.objId).innerHTML=newh;
}

Posted 21-Aug-14 5:43am

Thomas Daniels

Comments

AmiR1999 21-Aug-14 11:45am

Thank you. Perfect!

Thomas Daniels 21-Aug-14 11:48am

You're welcome!