| Yoko http://forum.yoko.com.ua/ |
|
| Мой первый скрипт - на музыку :) http://forum.yoko.com.ua/viewtopic.php?f=20&t=4227 |
Page 1 of 1 |
| Author: | Tergos [ 2005-06-14 14:15:18 ] |
| Post subject: | Мой первый скрипт - на музыку :) |
Что скажете по общему стилю написания? Где можно было сделать лучше и как? Code: sub Music() |
|
| Author: | Grin [ 2005-06-14 15:19:47 ] |
| Post subject: | |
Пообщему стилю?;) если бы этот скрипт работал то было бы наверно ничего;) UO.FindType('instrument[i]') - фантастика UO.Sayu(',warmode 1') - класс until UO.InJournal('have hidden') or ('hide here') - фантастика While not UO.Hidden()==1 - не рационально UO.Useobject('instrumentcurrent') - из области фантастики;) Как думаешь сколько ошибок тебе сразу выдаст парсер?;) в общем тепер ьпо пунктам.... UO.FindType('instrument[i]') ну и что тут? а тут унас зарание созданый тип объекта в вкладке ObjectType или просто строчка не понятного содержания а нам нужен элемент массива... по этому нах кавычки UO.Useobject('instrumentcurrent') тоже самое, утебя переменая instrumentcurrent объявлина поэтому выдергивай ее содержимое нах кавычки UO.Sayu(',warmode 1') uo.warmode(1) until UO.InJournal('have hidden') or ('hide here') вот обрати внимание на вторую часть ('hide here') как ты это понимаешь? Самое страное что я понимаю эо просто как выражение... ясно дело что ты там хотел написчать UO.InJournal('hide here') Но это уже не модно и все уже пишут UO.InJournal('have hidden|hide here') While not UO.Hidden()==1 зачем писат ьтакую логику? не проще просто While not UO.Hidden()? |
|
| Author: | Tergos [ 2005-06-14 15:36:18 ] |
| Post subject: | |
Grin wrote: UO.FindType('instrument[i]') ну и что тут? а тут унас зарание созданый тип объекта в вкладке ObjectType или просто строчка не понятного содержания а нам нужен элемент массива... по этому нах кавычки Не знал что он нормально всопримет просто имя переменной. Grin wrote: UO.Sayu(',warmode 1') uo.warmode(1) хмм. т.е. команды вызываемые через запятую можно использовать в скрипте напрямую. Гуд. Grin wrote: until UO.InJournal('have hidden') or ('hide here') вот обрати внимание на вторую часть ('hide here') как ты это понимаешь? Самое страное что я понимаю эо просто как выражение... ясно дело что ты там хотел написчать UO.InJournal('hide here') Но это уже не модно и все уже пишут UO.InJournal('have hidden|hide here') Нигде не видел пометки что такое выражение воспринимается нормально. Вот за это биг сенкс. Ну и разумеется имелось в виду UO.InJournal('hide here') Grin wrote: While not UO.Hidden()==1
зачем писать такую логику? не проще просто While not UO.Hidden()? т.е. опять же он нормально воспримет такое условие. Не был уверен в этом. В целом - Вери Биг Сенкс за разъяснения. |
|
| Author: | casper46 [ 2005-06-14 15:48:44 ] |
| Post subject: | |
Пацаны завик так парится на счет какогото мусикалшипа на те скриптик сразу качается антисентемент музыка и ПИСмайкинг Code: Var Animal1='0x003856CD' Писал не я но скрипт овигеный короче до ГМа качается за 36часов!!! [Убью ну вставляйте теги читать невозможно. I'm] |
|
| Author: | Tergos [ 2005-06-15 07:04:59 ] |
| Post subject: | |
Я не люблю пользоваться чужими готовыми кодами. Как справочником - да, а для дела - сам напишу. Вот что получилось после плейтеста: Code: sub Music() |
|
| Author: | Edred [ 2005-06-15 14:28:01 ] |
| Post subject: | |
Grin wrote: UO.Useobject('instrumentcurrent') тоже самое, утебя переменая instrumentcurrent объявлина поэтому выдергивай ее содержимое нах кавычки Кавычки здесь конечно стоят зря, но на настоящую ошибку ты сам внимание не обратил. Дело в том, что instrumentcurrent - это тип объекта, поэтому правильно надо было UO.Usetype(instrumentcurrent) Quote: UO.Sayu(',warmode 1') uo.warmode(1) Оба написания работоспособны, на скорость не влияет. Здесь никакой ошибки нету. Quote: Но это уже не модно и все уже пишут UO.InJournal('have hidden|hide here') Первый раз слышу про такую моду. Прежде чем советовать - поинтересовался бы у человека, может он на старой длл-ке сидит. Я вот например сижу на 312.19 и большинству советую того же. Так что на фиг такую моду. Quote: While not UO.Hidden()==1
зачем писат ьтакую логику? не проще просто While not UO.Hidden()? Опять же, на работоспособность не влияет. |
|
| Author: | Tergos [ 2005-06-15 15:44:39 ] |
| Post subject: | |
Дллька новая, посему работает и с '|'. А насчет того. что не ошибки - меня это и интересовало. оптимизация кода |
|
| Author: | -=SweD=- [ 2005-06-25 12:18:59 ] |
| Post subject: | |
casper46 wrote: Пацаны завик так парится на счет какогото мусикалшипа на те скриптик сразу качается антисентемент музыка и ПИСмайкинг Ну и как мне разобраться в твоём скрипте?Code: Var Animal1='0x003856CD' Code: UO.LDblClick(357,164)Вот Это зачем например ? |
|
| Author: | Boris [ 2005-06-25 13:36:23 ] |
| Post subject: | |
еще массив задан неверно Code: DIM[4] так надо, у тебя всего 4 эллемента а не 5 |
|
| Page 1 of 1 | All times are UTC+02:00 |
| Powered by phpBB® Forum Software © phpBB Limited https://www.phpbb.com/ |
|