SRL Archive - OSR Help

kk, so I have a string setup to tell the script what procedures to run. It runs fairly well (not flawless) but when it does fail it decides to skip the mainloop to the next one

whatarewedoing := 'Leather';

^^ Declareplayers string

if whatarewedoing = 'Leather' then
  begin
  mainloopleather;
  end;
 if whatarewedoing = 'Jewelry' then
  begin
  mainloopjewelry;
  end;
 if whatarewedoing = 'Cannonball' then
  begin
  mainloopsmithing;
  end;                                  

For some reason, if it fails making leather at any moment (due to lag or failure to find dtm) it skips over to the Jewelry mainloop.

any advice?

Can you post more? Lag has nothing to do with what you posted. Also use a case:

case lowercase(whatarewedoing) of
    'leather' : mainloopleather;
    'jewelry' : mainloopjewelry;
    'cannonball' : mainloopsmithing;
  end;

// WORKS PERFECTLY
program new;

const whatarewedoing = 'Leather';

begin
if whatarewedoing = 'Leather' then
  begin
  writeln('Doing: '+ whatarewedoing);
  end;
 if whatarewedoing = 'Jewelry' then
  begin
  writeln('Doing: '+ whatarewedoing);
  end;
 if whatarewedoing = 'Cannonball' then
  begin
  writeln('Doing: '+ whatarewedoing);
  end;
end.

// BUT I WOULD USE THIS
program new;

const whatarewedoing = 'Leather';

begin
  case whatarewedoing of
    'Leather': begin
                 writeln('Doing: '+ whatarewedoing);
               end;
    'Jewelry': begin
                 writeln('Doing: '+ whatarewedoing);
               end;
    'Cannonball': begin
                 writeln('Doing: '+ whatarewedoing);
               end;
  end;
end.

From: Awkwardsaw
Can you post more? Lag has nothing to do with what you posted. Also use a case:

case lowercase(whatarewedoing) of
    'leather' : mainloopleather;
    'jewelry' : mainloopjewelry;
    'cannonball' : mainloopsmithing;
  end;

I'm saying a lag in one of the procedures within the loops. If it fails a procedure it forgets its making leather and does jewelry instead. Yeah I know I havn't added lowercase yet, I have like 100 different items to add so thatll be in the end

From: Frement

// WORKS PERFECTLY
program new;

const whatarewedoing = 'Leather';

begin
if whatarewedoing = 'Leather' then
  begin
  writeln('Doing: '+ whatarewedoing);
  end;
 if whatarewedoing = 'Jewelry' then
  begin
  writeln('Doing: '+ whatarewedoing);
  end;
 if whatarewedoing = 'Cannonball' then
  begin
  writeln('Doing: '+ whatarewedoing);
  end;
end.

// BUT I WOULD USE THIS
program new;

const whatarewedoing = 'Leather';

begin
  case whatarewedoing of
    'Leather': begin
                 writeln('Doing: '+ whatarewedoing);
               end;
    'Jewelry': begin
                 writeln('Doing: '+ whatarewedoing);
               end;
    'Cannonball': begin
                 writeln('Doing: '+ whatarewedoing);
               end;
  end;
end.

I'll give it a go, but somethings not right here 0.o

You're forgetting to repeat it…?

Thanks guys, I guess it was the banking that was messing it up. It wouldn't detect that I was Crafting leather. It works great now. Thanks anyway guys!

Also, add a LowerCase() around the string, since people might not type the string correctly.

From: Kyle Undefined
Also, add a LowerCase() around the string, since people might not type the string correctly.

Also, try reading the thread before you post <3.

From: punkd
I'm saying a lag in one of the procedures within the loops. If it fails a procedure it forgets its making leather and does jewelry instead. Yeah I know I havn't added lowercase yet, I have like 100 different items to add so thatll be in the end

From: Sex
Also, try reading the thread before you post <3.

Give him a break! He's a busy guy! =P

Haha what he said ^ :p