Check out Makruk (Thai Chess), our featured variant for March, 2025.

Enter Your Reply

The Comment You're Replying To
H. G. Muller wrote on Sun, Apr 27 01:53 AM EDT in reply to Fergus Duniho from Sat Apr 26 05:25 PM:

In JavaScript any undeclared variable you assign to will automatically get global scope. I left some variables intentionally undeclared in the script, to make it resistant against appearing multiple times on the page. The initialization can only be run once, as it removes all the Diagram definitions and replaces them by the actual images. But global declarations of a variable would also be seen in the second and later loading, and then destroy the value the initial loading and initialization set these to. When the array that holds all the collected original Diagram definitions was erased that way, none of the Diagrams of the page would work anymore.

This problem occurred when loading is so slow that some later loadings of the script occur after initialization (which is done after some delay) has already been done. I would have expected that using an 'onload' handler for calling the Init() routine, rather than performing it after some timeout, would solve that. But I could never get this to work. It is also a pain that for multiple loadings the last loaded script would prevail; I would prefer the first-loaded script to prevail. In general that would be the one that is in the article, and it is a bad thing that anyone could post a Comment to the page that could wreck it, by posting a Diagram that uses some obsolete version of the script.

I suppose this could be done by wrapping the entire script in a function, so that none of the declarations (also those of functions) become local to that wrapper. The wrapper frunction could then assign the functions that need to be global to undeclared global variables to provide access to those for the event handlers, and take measures that would make it return immediately when called a second time. Like

function script() { // wrapper for the entire script
  if(initDone) return;
  initDone = 1;
  F = function (args) { .... }; // declare functions and make them global
  G = function (args) { .... };
  Init = function (args) { .... };
  setTimeout('Init()', 500); // run Init after 500 msec
}
script(); // call it

But this would still offer no protection against older versions of the script that explicitly declare F and G as globals ("function F(args) {...}"), and are loaded later. Unless I would change the name of all functions that need to be accessed globally.

Declaring something as global in script embedded in the article should be harmless, as this will only be seen by the browser's JavaScript engine once.


Edit Form

Comment on the page Xiangqi: Chinese Chess

Conduct Guidelines
This is a Chess variants website, not a general forum.
Please limit your comments to Chess variants or the operation of this site.
Keep this website a safe space for Chess variant hobbyists of all stripes.
Because we want people to feel comfortable here no matter what their political or religious beliefs might be, we ask you to avoid discussing politics, religion, or other controversial subjects here. No matter how passionately you feel about any of these subjects, just take it someplace else.
Avoid Inflammatory Comments
If you are feeling anger, keep it to yourself until you calm down. Avoid insulting, blaming, or attacking someone you are angry with. Focus criticisms on ideas rather than people, and understand that criticisms of your ideas are not personal attacks and do not justify an inflammatory response.
Quick Markdown Guide

By default, new comments may be entered as Markdown, simple markup syntax designed to be readable and not look like markup. Comments stored as Markdown will be converted to HTML by Parsedown before displaying them. This follows the Github Flavored Markdown Spec with support for Markdown Extra. For a good overview of Markdown in general, check out the Markdown Guide. Here is a quick comparison of some commonly used Markdown with the rendered result:

Top level header: <H1>

Block quote

Second paragraph in block quote

First Paragraph of response. Italics, bold, and bold italics.

Second Paragraph after blank line. Here is some HTML code mixed in with the Markdown, and here is the same <U>HTML code</U> enclosed by backticks.

Secondary Header: <H2>

  • Unordered list item
  • Second unordered list item
  • New unordered list
    • Nested list item

Third Level header <H3>

  1. An ordered list item.
  2. A second ordered list item with the same number.
  3. A third ordered list item.
Here is some preformatted text.
  This line begins with some indentation.
    This begins with even more indentation.
And this line has no indentation.

Alt text for a graphic image

A definition list
A list of terms, each with one or more definitions following it.
An HTML construct using the tags <DL>, <DT> and <DD>.
A term
Its definition after a colon.
A second definition.
A third definition.
Another term following a blank line
The definition of that term.