Not check a checkbox on condition

Question

This solution is currently working as intented but is there a "prettier" way to do this? :)

if (EditCheckLog()) {
    var chk = $(this).prop('checked');
    chk = !chk;
    $(this).prop('checked', chk);
    return;
}

Yes, this.checked =! this.checked. But this question is suittable for stackoverflow, it should be on CodeReview. — Karl-André Gagnon
– Karl-André Gagnon, Commented Feb 12, 2015 at 20:56
I'm voting to close this question as off-topic because this belongs to Code review — Huangism
– Huangism, Commented Feb 12, 2015 at 20:57
@Huangism Was searching the close reason... don't understand why "other stack network" doesn't have code review.... — Karl-André Gagnon
– Karl-André Gagnon, Commented Feb 12, 2015 at 20:58
@Karl-AndréGagnon strange indeed, perhaps this should be brought up in Meta — Huangism
– Huangism, Commented Feb 12, 2015 at 21:01

Josh Crozier · Accepted Answer · 2015-02-12 20:55:58Z

3

You don't need to use jQuery. Here is a simple approach:

if (EditCheckLog()) {
    this.checked = !this.checked;
    return;
}

answered Feb 12, 2015 at 20:55

Josh Crozier

242k56 gold badges401 silver badges316 bronze badges

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

1 Answer 1