Написал свой первый мини скрипт, но он не работает скажите что не так?

Question

[Fakes]

require "lib.moonloader"
local keys = require "vkeys"
 function main()
 if not isSampLoaded() or not isSampfuncsLoaded()
   then return
   end
    while not isSampAvailable()
do wait()
end
sampAddChatMessage('CarHelp активен', 0x800080)
while true do
wait(0)
   if isKeyJustPressed(VK_L)
   then 
   sampSendChat('/lock')
   end
   if isKeyJustPressed(VK_K)
   then 
   sampSendChat('/key')
end

Comments

[shurshur]

'end' expected (to close 'while' at line 11) near

Ну вот с этого надо начать. function main, к слову, тоже не закрыта end.

И кто так отступы-то делает??? Читать невозможно.

Далее, функция main нигде не вызывается. Если это скрипт именно для запуска, а не для вызова по имени функции из какого-то стороннего решения, то он и не будет работать - ведь ничего в нём не делается, кроме загрузки двух модулей и определения неиспользуемой функции main

Answer 1

[dollar]

Главное, что не так - не соблюдены отступы.
Если соблюдать, то все незакрытые end будут чётче видны.

Comments

[Fakes]

Как правильно отступать?
Подскажите пожалуйста

[shurshur]

Fakes, как-то так:

require "lib.moonloader"

local keys = require "vkeys"

function main()

  if not isSampLoaded() or not isSampfuncsLoaded()
    then return
  end

  while not isSampAvailable()
  do
    wait()
  end

  sampAddChatMessage('CarHelp активен', 0x800080)

  while true do
    wait(0)

    if isKeyJustPressed(VK_L)
    then 
      sampSendChat('/lock')
    elseif isKeyJustPressed(VK_K)
    then
      sampSendChat('/key')
    end

  end

end

Каждый end находится на том же уровне, что и конструкция, которую он закрывает. Например, while ... end, function ... end, итд итп. До кучи я ещё и второй if на проверку нажатия кнопки завернул в elseif - наверное это имелось в виду? И добавил пустых строк для того, чтобы код не выглядел как выглядел бы художественный текст без разбиения на абзацы.

[dollar]

Строгого стандарта нет, но есть общепринятый, а вам поможет хоть какой-нибудь стиль, даже свой собственный.

Вон shurshur предложил вариант. Хотя я бы поменьше строк пропускал (люблю когда всё на экране помещается). Но это дело вкуса. И еще then не переносится обычно в Lua.

Пример 1

require "lib.moonloader"

local keys = require "vkeys"
function main()
  if not isSampLoaded() or not isSampfuncsLoaded() then
    return
  end
  while not isSampAvailable() do
    wait()
  end

  sampAddChatMessage('CarHelp активен', 0x800080)
  while true do
    wait(0)
    if isKeyJustPressed(VK_L) then 
      sampSendChat('/lock')
    end
    if isKeyJustPressed(VK_K) then 
      sampSendChat('/key')
    end
  end
end

Пример 2

if (условние1) then
  (код1)
elseif (условие2) then
  (код2)
else
 (код3)
end

[Fakes]

shurshur, Спасибо огромное ,буду пытаться что-то делать

[Fakes]

dollar, и вам спасибо большое

[shurshur]

dollar, я на lua писал только скрипты для Verlihub - и то уже довольно давно. Нормального опыта в lua у меня никогда не было и синтаксис его я помню весьма приблизительно. Впрочем, этот язык тем и хорош, что он настолько прост, что его не надо во всех нюансах помнить, как питон с его мутабельностью объектов и многими другими неожиданностями.

Answer 2

[Дима Фотт]

Очень очень очень плохое оформление кода.

require "lib.moonloader"
local keys = require "vkeys"
function main()
   repeat wait(0) until isSampAvailable()
   sampAddChatMessage('CarHelp активен', 0x800080)
   while true do
      wait(0)
      if isKeyJustPressed(VK_L) then sampSendChat('/lock') end
      if isKeyJustPressed(VK_K) then sampSendChat('/key') end
   end
end