Errata

Now reads:

"Placing a plus sign (+) before your string of query term(s)
matches only those records containing all of the terms you
enter (Boolean AND).

This is incorrect. If a user enters something like this into the search engine:

+javascript books

the search engine code will not isolate a record with javascript and books;
instead it will find all records with either "javascript" or "books." The code
provided by the book and downloadable does not do a correct AND Boolean search.

if (searchType == (SEARCHANY | SEARCHALL)) { requireAll(searchArray); }

MUST BE

if (searchType == (SEARCHALL)) { requireAll(searchArray); }

var refineAllString = allString.substring(0,allString.indexOf('|HTTP'));

MUST BE

var refineAllString = allString.substring(1,allString.indexOf('|HTTP'));

In the book is the example of records.js incorrect. It starts with the url.
The source ends with the url.

If there is a search with + the script won't work because the + isn't omitted
form the compare

After line 22
20 function validate(entry) {
21 if (entry.charAt(0) == "+") {
22 entry = entry.substring(1,entry.length);
23 document.forms[0].query.value = entry; //NEW!!!!
24 searchType = SEARCHALL;
25 }

In the inner "for" loop, wouldn't it be more efficient to simply "break" out
of the loop at this point, as the Boolean AND fails on the first occurance?
The way it is written, it will continue to compare the tokenized elements of
the search string to the database record, rather than just move on to the next
record.

187 <input type=hidden name="standin" value="">

The example that builds out the web safe color palette gives "AA" as one of
the two-digit components for hex codes. "AA" is incorrect -- "00" is omitted
in its place.

The code is bulky. The if block is exactly the same as the else block, except
for a difference in the indices. This should be reduced to a single for loop -
the indices should be sorted out by ternary operator or if/else statement.

1] Currently only the database is searched. this requires a lot of manual
entry to make thorough searching available. A utility to generate urls.js is
needed. This program will walk through a folder, adding records for each file
(maybe user can limit valid file extentions) and data from that file. A
simple way is to add only the meta contents area ie. stuff sent to commercial
search engines. A bigger project is to condense entire page, limiting to
keywords etc. I am going to do this in a regular programming language but
interested if this can be done in javascript...

2] Another utility for manual or automagically produced url.js is a verifier
to make sure it is correct syntax before use ..... ie paired symbols, enough
delims, required fields not null , commas on all but last entry.

If JavaScript capabilities are turned off in browser, there is no signal to
user that search will not work! Is there a method of checking browser config
and issuing warning!

You have used variable adImages in line one and variable ads in second line.

Line 16 of the example closes the array with }
This is incorrect and will not work. You need to use ); after the final question.

This is correct in the downloadable zip of the same file.

I really don't think that the technique described actually works.

1) If you remove the location.replace() in the cleanSlate() method then
you'll see that setting the SRC attribute in the frameset to self.dummy1
does nothing.

2) If you set the value of SRC to something else [like "about:blank"] and
change cleanSlate() to return some other value [rather than evaluating
dummy1 & 2], the code still seems to work.

It looks to me like you can't "cheat the SRC".

The formula used to calculate the number of combinations is wrong. The number
of possible combinations C of K questions from a question pool with N
questions (N>K) is given by:

C = N!/(N-K)!

where n! = 1 x 2 x 3 x ... x (n-1) x n. E.g., if you have to pick 5 questions
from a pool of 10, there are 10 possible picks for the first question, 9 for
the second, 8 for the third, 7 for the fourth and 6 for the fifth: C = 6 x 7 x
8 x 9 x 10 = 10!/(10-5)! = 10!/5! = 30240. If N=K, the formula simplifies to
C=N! as 0!=1 by definition. Hence, the section needs a re-write.

What about extra maths tying down the CPU?

At the bottom of pate 154, you have a declaration for expiry. However, in the
function SetCookie on page 155, you appear to be using expires as the
variable/object name (expires.toGMTString()). The simplest approach would
appear to be to change

expires.toGMTString()

to

expiry.toGMTString

in the definition of SetCookie.

(refer also other unconfirmed error report on same item)

It seems to me that the 'expiry' variable defined on p. 154 should have been
used as default expiration of the cookie, as per the discussion in the
subsequent text. The program as written uses the input parameter for expiry,
or "". This could be achieved, with something like:

...
((expires) ? "; expires=" + expires.toGMTString() :
"; expires=" + expiry.toGMTString() ) +

allowing user to override the default.

Checkboxes aren't supposed to have a third element here - that's what you say
later on and from what I can tell that's how the code expects it. Format
newNames() ?

calculation display the wrong info

If you just change a quantity and forget to click the change bag button, the
display shows automatically all the new calculations, etc. Nevertheless, if
you do the check out (not following the warning), since the bag was not
updated correctly you get everything OK except the quantitites. This could
generate many problems.

A simple solution is to include in the Check out button the change bag so you
will always change bag before the check out really proceeds.

For example:

<INPUT TYPE=BUTTON VALUE="Check Out" onClick="parent.frames[1].changeBag(this.
form, true); parent.frames[1].checkOut(this.form);">

U forgot to include the name of the file where Action form parameter refers to.
Pointer to <server name> <cgi dir> bag.cgi is located in manager.html file. It is
concluded from the reading that pointer is in bag.cgi file

The first line defines "Number" object instead of defining "Navigator" object.

the Method indexOf is incorrectly printed as "indexof".