Wait() and notifyall() problem in servlet.

Question

Can anyone help me with this.....
I am creating an application which has one gateway inside which
handles for logger, dbmodule etc are made and passed to biz logic.
This biz logic using logger obj made in gateway, does the logging.
Now i hv to do serial implementation of logging i.e I have
a static member which is of type Vector, which will hold
a list of messages and associated parameters. The write() method will
create a formatted message with all values inserted, all prefixes and
suffixes added, etc. Then it will append the message to the Vector of
messages. And there will be a separate thread, which will be started
the
first time the first Logger instance is created. This thread will keep
picking up messages from this Vector in FIFO order and writing them
out.Mind you, I don't mean i need to use a single shared Logger
instance.
Separate Logger instances must be used to hold separate values for
sessionID, username, etc. Only the internal logging I/O stream handle
should be shared.
For this what i hv created logger class as follows:
logger {
SyslogAppender SA;
logger() {
new SA;
ThreadGroup TG = new ThreadGroup("string");
Logging target = new Logging();
Thread DT = new Thread(TG, target, "daemon");
DT.setDaemon(true);
DT.start();
class write {
write() {
//initialisation;
void writetovector() {
addtovector
notifyall;
class logging implements runnable {
public void run() {
while(vector is not empty){
log
synchronised(this){
wait();
Now in the servlet, in init(), logger is called with this constructor
and therefore there is only one instance of syslog appender to log in
syslog.
In service i just call the function writetovector through a method in
logger.
However when i run the implemented version of this, logging does not
take place.
I feel my implementation of wait and notifyall is not correct. Or is
there any other problem?
thanx in advance

Answer 1

There are many problems with your code. It looks like you retyped it, and it now contains many syntax errors. Two problems I can see, though, are that you do not call notifyAll from synchronized code, and your call to wait is not in a loop.