Перейти к содержимому
Asior

Робот-рыболов

Рекомендуемые сообщения

Немного почитал код, и в голову пришло несколько советов.

--[[ opencomputers рыболов create serafim

"by serafim" надо. Иначе выходит, что это рыболов серафима создал, а не наоборот.

print("только роботы могут использовать эту программу")
os.sleep(5)
computer.shutdown()

А зачем тут шутдаун? При любой ошибке игроку дается 5 сек. чтобы её прочесть, а потом комп/робот вырубается. На мой взгляд лучше просто без всяких слипов выйти из программы. Например через os.exit().

slot = i_c.getStackInInternalSlot(13)

Таки всякие внутренние переменные тоже стоит объявлять как local.

print("ошибка !")
print("нет сигнала от датчика")

Для таких комбинаций команд, которые часто повторяются вместе, удобно завести функцию. Типа log(message) или error(message).

_ = event.pull(60,'redstone_changed')

В случае, когда не важно вообще, что там функция вернёт, можно просто вызвать её, как обычную функцию. event.pull(60, 'redstone_changed'). Однако, имеет смысл подстраховаться, и проверить, что произошёл именно эвент. Если мне не изменяет память, есть возможность того, что функция просто вернёт nil вместо эвента (даже если его не было на самом деле).

lovka()
drop()

И наконец, сочетание английского и транслита в одной программе терзает глаз.

:)

Поделиться сообщением


Ссылка на сообщение
Поделиться на других сайтах

увы английский у меня хромает

оптимизация вещь тонкая, щас допилим

Поделиться сообщением


Ссылка на сообщение
Поделиться на других сайтах
create serafim

Столько раз видал как люди подписывают свои работы через by, что create меня просто убил :smile3:

 

 

 

оптимизация вещь тонкая

Вот вот, не о медальках думать надо, а о оптимизации  :smile9:  И когда опыта наберёшься, будешь изначально писать как надо, а не исправлять позже.

Изменено пользователем qwertyMAN

Поделиться сообщением


Ссылка на сообщение
Поделиться на других сайтах
В 18.08.2016 в 19:55, qwertyMAN сказал:
И когда опыта наберёшься, будешь изначально писать как надо, а не исправлять позже.

Тут не поспориш,  потому и раздел для начинающих

 

Totoro

Поправил, спасибо за замечания

Изменено пользователем serafim

Поделиться сообщением


Ссылка на сообщение
Поделиться на других сайтах

Небольшое обновление:  http://pastebin.com/RPLWqKTZ  или  pastebin get RPLWqKTZ r

 

добавил статистику, и в случае ошибки можно продолжить ловить нажав любую кнопку

Скрытый текст

wfQrZNB.png

также добавил список мусора, теперь робот будет его выкидывать 

 

афк рыбалка из кучи роботов

Изменено пользователем serafim

Поделиться сообщением


Ссылка на сообщение
Поделиться на других сайтах

@Asior используемый хостинг картинок сломался, можешь залить картинки для этой программы куда-нить типо гитхаба? Чтоб 100% не просрочились ссылки

Поделиться сообщением


Ссылка на сообщение
Поделиться на других сайтах

Перезалью если его востановят.

Поделиться сообщением


Ссылка на сообщение
Поделиться на других сайтах
В 29.05.2021 в 20:24, Asior сказал:

Перезалью если его востановят.

восстановили.

Поделиться сообщением


Ссылка на сообщение
Поделиться на других сайтах

Присоединяйтесь к обсуждению

Вы можете написать сейчас и зарегистрироваться позже. Если у вас есть аккаунт, авторизуйтесь, чтобы опубликовать от имени своего аккаунта.

Гость
Ответить в тему...

×   Вы вставили отформатированное содержимое.   Удалить форматирование

  Разрешено использовать не более 75 эмодзи.

×   Ваша ссылка была автоматически встроена.   Отобразить как ссылку

×   Ваш предыдущий контент был восстановлен.   Очистить редактор

×   Вы не можете вставлять изображения напрямую. Загружайте или вставляйте изображения по ссылке.


×
×
  • Создать...