) (замечтался)]]>) (замечтался)]]>
Выход из окна заданий лучше сделать:
top.frames[3].location = "http://" + external.SCity.Text + ".combats.com/dungeon.pl?" + Math.random();
как минимум ради наглядности
А для однократной проверки нужно внести переменную ещё одну будет и её в условия подабавлять + продумать, когда изменит своё состояние на другое, мол пройдена проверка. А возврат к начальному значению можно задать во время начала похода по самому данжу.
Да и прописывать было лучше else if, чем просто if
оно же по-кругу повторяться тогда будет со скоростью, указанной в настройках бота.
Т.е. проверилось одно условие, через, допустим, 4000мс второе условие...
if(IsMapVisible())
{
....................................
}
else
{
этот код
if(s.indexOf("Создать группу") > 0)
{
external.SetStatusLabel("Создаю заявку");
PutKanalZayavka();
}
else if(s.indexOf("Начать") > 0)
{
external.SetStatusLabel("Начинаю поход");
StartKanalZayavka();
}
}
if(s.indexOf("Задания") > 0)
{
external.SetStatusLabel("Беру задание");
top.frames[3].location = "http://" + external.SCity.Text + ".combats.com/dungeon.pl?ql=" + Math.random();
return;
}
if(s.indexOf("Завершить задание") > 0)
{
external.SetStatusLabel("Завершаю задание");
top.frames[3].document.all.finishquest.click();
return;
}
if(s.indexOf("Задания отсюда недоступны") > 0)
{
external.SetStatusLabel("Заданий нет");
top.frames[3].location = "http://" + external.SCity.Text + ".combats.com/" + dungName;
return;