Ошибка variable used without being declared

I think it is best to declare variables to avoid the error arising.
 

#RequireAdmin
Opt("MustDeclareVars", 1)
Opt("TrayAutoPause", 0)
#Region
#AutoIt3Wrapper_Icon=\xxxAdministrationAutoItIconsOffice 2013.ico
#AutoIt3Wrapper_Outfile=\xxxAdministrationAutoItInstallationsscripteOffice 2016 x86.exe
#AutoIt3Wrapper_Res_Fileversion=1.0.0.9
#AutoIt3Wrapper_Res_Fileversion_AutoIncrement=y
#AutoIt3Wrapper_Res_Language=1031
#EndRegion 
;#AutoIt3Wrapper_Au3Check_Parameters=-d -w 1 -w 2 -w 3 -w- 4 -w 5 -w 6 -w 7
;
; Programm:                 Office 2016 x86
; Datum:                        18.11.2015
; AutoIt3 Version:  3.3.14.1
;

;#include <file.au3>
;#include <array.au3>
;#include <EventLog.au3>

Global Const $STR_REGEXPARRAYMATCH = 1
Global Const $tagRECT = "struct;long Left;long Top;long Right;long Bottom;endstruct"
Global Const $KF_EXTENDED = 0x0100
Global Const $KF_ALTDOWN = 0x2000
Global Const $KF_UP = 0x8000
Global $__g_sSourceName_Event

Global $ScriptName = "Office 2016 x86"
Global $ProgrammPfad = "ProgrammeInstallationOffice2016x86"
Global $ProgrammDatei = "setup.exe"
Global $Parameter = " /adminfile Office2016.MSP"
Global $ConfigPfad = "AdministrationAutoIt"
Global $ConfigFile = "config.ini"
;Global $config,$SplitPath,$SplitServer,$aArray,$counter,$standort,$server,$string,$pos,$CheckRegistry,$success,$Setup
Local $hEventLog, $aData[4] = [3, 1, 2, 3]
Local $config,$SplitPath,$SplitServer
If (FileExists(@ScriptDir & "" & $ConfigFile)) Then
    $config = @ScriptDir & "" & $ConfigFile
Else
    Dim $SplitDrive, $SplitDir, $SplitFileName, $SplitExtension
    $SplitPath = _PathSplit(@ScriptFullPath, $SplitDrive, $SplitDir, $SplitFileName, $SplitExtension)
    $SplitServer = $SplitPath[1]
    $config = $SplitServer & $ConfigPfad & $ConfigFile
EndIf
Local $aArray,$counter,$standort,$server
If (FileExists($config)) Then
    $aArray = IniReadSection($config, "IP")
    $counter = $aArray[0][0]
    For $i = 1 To $counter
        If (StringInStr(@IPAddress1, $aArray[$i][1])) Or (StringInStr(@IPAddress2, $aArray[$i][1])) Then
            $standort = $aArray[$i][0]
            $server = IniRead($config, "Server", $standort, "Fehler")
        EndIf
    Next
    $server = "\" & $server & ""
Else
    MsgBox(16, $ScriptName, "Die Konfigurationsdatei " & $config & " wurde nicht gefunden.")
    $hEventLog = _EventLog__Open("", $ScriptName)
    _EventLog__Report($hEventLog, 1, 0, 2, "Administrator", "Die Konfigurationsdatei wurde nicht gefunden", $aData)
    _EventLog__Close($hEventLog)
    Exit (1)
EndIf
TrayTip($ScriptName & " - Installation", $ScriptName & " wird installiert. Bitte warten...", 2000)
Local $string = RegRead("HKEY_LOCAL_MACHINESOFTWAREWow6432NodeMicrosoftWindowsCurrentVersionUninstall{7CDE44B1-0759-483A-924C-CB65DF98F5F8}", "DisplayName")
Local $pos = StringInStr($string, "Clientkomponenten")
Local $CheckRegistry=False
If ($pos = 1) Then
    $CheckRegistry = True
Else
    $CheckRegistry = False
EndIf
If ($CheckRegistry = True) Then
    MsgBox(64, $ScriptName, "Clientkomponenten für Outlook entdeck, wird nun deinstalliert", 3)
    RunWait('msiexec /qb- /x {7CDE44B1-0759-483A-924C-CB65DF98F5F8} /norestart')
    MsgBox(64, $ScriptName, "Clientkomponenten für Outlook wurden deinstalliert", 3)
EndIf
$string = RegRead("HKEY_LOCAL_MACHINESOFTWAREWow6432NodeMicrosoftWindowsCurrentVersionUninstall{90140000-0011-0000-0000-0000000FF1CE}", "DisplayName")
$pos = StringInStr($string, "Microsoft Office")
If ($pos = 1) Then
    $CheckRegistry = True
Else
    $CheckRegistry = False
EndIf
If ($CheckRegistry = True) Then
    MsgBox(64, $ScriptName, "Office 2010 Installation entdeck, wird nun deinstalliert", 3)
    RunWait($server & "ProgrammeInstallationOffice2010SP1x86setup.exe /uninstall ProPlus /config \xxxProgrammeInstallationOffice2010SP1x86ProPlus.WWuninstall.xml")
    MsgBox(64, $ScriptName, "Office 2010 wurde deinstalliert", 3)
EndIf
$string = RegRead("HKEY_LOCAL_MACHINESOFTWAREWow6432NodeMicrosoftWindowsCurrentVersionUninstall{90150000-0011-0000-0000-0000000FF1CE}", "DisplayName")
$pos = StringInStr($string, "Microsoft Office")
If ($pos = 1) Then
    $CheckRegistry = True
Else
    $CheckRegistry = False
EndIf
Local $success
If ($CheckRegistry = True) Then
    MsgBox(64, $ScriptName, "Office 2013 Installation entdeck, wird nun deinstalliert", 3)
    $success = RunWait($server & "ProgrammeInstallationOffice2013SP1x86setup.exe /uninstall ProPlus /config \xxxProgrammeInstallationOffice2013SP1x86ProPlus.WWuninstall.xml")
    If ($success <> 0) Then
        MsgBox(64, $ScriptName, "Office 2013 wurde erfolgreich deinstalliert", 3)
    Else
        MsgBox(64, $ScriptName, "Office 2013 wurde nicht deinstalliert.", 3)
        Exit (1)
    EndIf
EndIf
Local $Setup = $server & $ProgrammPfad & $ProgrammDatei
If FileExists($Setup) Then
    $success = RunWait($Setup & $Parameter)
    If ($success <> 0) Then
        MsgBox(16, $ScriptName, "Die Installation von Office 2016 ist fehlgeschalgen.")
        $hEventLog = _EventLog__Open("", $ScriptName)
        _EventLog__Report($hEventLog, 1, 0, 2, "Administrator", "Die Installation von Office 2016 ist fehlgeschalgen. Fehlercode: " & $success, $aData)
        _EventLog__Close($hEventLog)
        Exit (1)
    Else
        MsgBox(16, $ScriptName, "Die Installation von Office 2016 war erfolgreich")
        $hEventLog = _EventLog__Open("", $ScriptName)
        _EventLog__Report($hEventLog, 0, 0, 2, "Administrator", "Die Installation von Office 2016 war erfolgreich", $aData)
        _EventLog__Close($hEventLog)
    EndIf
Else
    MsgBox(16, $ScriptName, "Die Datei " & $Setup & " wurde nicht gefunden.")
    $hEventLog = _EventLog__Open("", $ScriptName)
    _EventLog__Report($hEventLog, 1, 0, 2, "Administrator", "Die Installationsdatei wurde nicht gefunden", $aData)
    _EventLog__Close($hEventLog)
    Exit (1)
EndIf
TrayTip($ScriptName & " installiert", "Die Installation wurde abgeschlossen", 10)
Sleep(2000)
Exit (0)


Func _EventLog__Close($hEventLog)
    Local $aResult = DllCall("advapi32.dll", "bool", "CloseEventLog", "handle", $hEventLog)
    If @error Then Return SetError(@error, @extended, False)
    Return $aResult[0] <> 0
EndFunc   ;==>_EventLog__Close
Func _EventLog__Open($sServerName, $sSourceName)
    $__g_sSourceName_Event = $sSourceName
    Local $aResult = DllCall("advapi32.dll", "handle", "OpenEventLogW", "wstr", $sServerName, "wstr", $sSourceName)
    If @error Then Return SetError(@error, @extended, 0)
    Return $aResult[0]
EndFunc   ;==>_EventLog__Open
Func _EventLog__Report($hEventLog, $iType, $iCategory, $iEventID, $sUserName, $sDesc, $aData)
    Local $tSID = 0
    If $sUserName <> "" Then
        $tSID = _Security__GetAccountSid($sUserName)
    EndIf
    Local $iData = $aData[0]
    Local $tData = DllStructCreate("byte[" & $iData & "]")
    Local $iDesc = StringLen($sDesc) + 1
    Local $tDesc = DllStructCreate("wchar[" & $iDesc & "]")
    Local $tPtr = DllStructCreate("ptr")
    DllStructSetData($tPtr, 1, DllStructGetPtr($tDesc))
    DllStructSetData($tDesc, 1, $sDesc)
    For $iI = 1 To $iData
        DllStructSetData($tData, 1, $aData[$iI], $iI)
    Next
    Local $aResult = DllCall("advapi32.dll", "bool", "ReportEventW", "handle", $hEventLog, "word", $iType, "word", $iCategory, "dword", $iEventID, "struct*", $tSID, "word", 1, "dword", $iData, "struct*", $tPtr, "struct*", $tData)
    If @error Then Return SetError(@error, @extended, False)
    Return $aResult[0] <> 0
EndFunc   ;==>_EventLog__Report
Func _PathSplit($sFilePath, ByRef $sDrive, ByRef $sDir, ByRef $sFileName, ByRef $sExtension)
    Local $aArray = StringRegExp($sFilePath, "^h*((?:\\?\)*(\\[^?/\]+|[A-Za-z]:)?(.*[/\]h*)?((?:[^./\]|(?(?=.[^/\]*.).))*)?([^/\]*))$", $STR_REGEXPARRAYMATCH)
    If @error Then
        ReDim $aArray[5]
        $aArray[0] = $sFilePath
    EndIf
    $sDrive = $aArray[1]
    If StringLeft($aArray[2], 1) == "/" Then
        $sDir = StringRegExpReplace($aArray[2], "h*[/\]+h*", "/")
    Else
        $sDir = StringRegExpReplace($aArray[2], "h*[/\]+h*", "\")
    EndIf
    $aArray[2] = $sDir
    $sFileName = $aArray[3]
    $sExtension = $aArray[4]
    Return $aArray
EndFunc   ;==>_PathSplit
Func _Security__GetAccountSid($sAccount, $sSystem = "")
    Local $aAcct = _Security__LookupAccountName($sAccount, $sSystem)
    If @error Then Return SetError(@error, @extended, 0)
    If IsArray($aAcct) Then Return _Security__StringSidToSid($aAcct[0])
    Return ''
EndFunc   ;==>_Security__GetAccountSid
Func _Security__GetLengthSid($pSID)
    If Not _Security__IsValidSid($pSID) Then Return SetError(@error + 10, @extended, 0)
    Local $aCall = DllCall("advapi32.dll", "dword", "GetLengthSid", "struct*", $pSID)
    If @error Then Return SetError(@error, @extended, 0)
    Return $aCall[0]
EndFunc   ;==>_Security__GetLengthSid
Func _Security__IsValidSid($pSID)
    Local $aCall = DllCall("advapi32.dll", "bool", "IsValidSid", "struct*", $pSID)
    If @error Then Return SetError(@error, @extended, False)
    Return Not ($aCall[0] = 0)
EndFunc   ;==>_Security__IsValidSid
Func _Security__LookupAccountName($sAccount, $sSystem = "")
    Local $tData = DllStructCreate("byte SID[256]")
    Local $aCall = DllCall("advapi32.dll", "bool", "LookupAccountNameW", "wstr", $sSystem, "wstr", $sAccount, "struct*", $tData, "dword*", DllStructGetSize($tData), "wstr", "", "dword*", DllStructGetSize($tData), "int*", 0)
    If @error Or Not $aCall[0] Then Return SetError(@error, @extended, 0)
    Local $aAcct[3]
    $aAcct[0] = _Security__SidToStringSid(DllStructGetPtr($tData, "SID"))
    $aAcct[1] = $aCall[5]
    $aAcct[2] = $aCall[7]
    Return $aAcct
EndFunc   ;==>_Security__LookupAccountName
Func _Security__SidToStringSid($pSID)
    If Not _Security__IsValidSid($pSID) Then Return SetError(@error + 10, 0, "")
    Local $aCall = DllCall("advapi32.dll", "bool", "ConvertSidToStringSidW", "struct*", $pSID, "ptr*", 0)
    If @error Or Not $aCall[0] Then Return SetError(@error, @extended, "")
    Local $pStringSid = $aCall[2]
    Local $aLen = DllCall("kernel32.dll", "int", "lstrlenW", "struct*", $pStringSid)
    Local $sSID = DllStructGetData(DllStructCreate("wchar Text[" & $aLen[0] + 1 & "]", $pStringSid), "Text")
    DllCall("kernel32.dll", "handle", "LocalFree", "handle", $pStringSid)
    Return $sSID
EndFunc   ;==>_Security__SidToStringSid
Func _Security__StringSidToSid($sSID)
    Local $aCall = DllCall("advapi32.dll", "bool", "ConvertStringSidToSidW", "wstr", $sSID, "ptr*", 0)
    If @error Or Not $aCall[0] Then Return SetError(@error, @extended, 0)
    Local $pSID = $aCall[2]
    Local $tBuffer = DllStructCreate("byte Data[" & _Security__GetLengthSid($pSID) & "]", $pSID)
    Local $tSID = DllStructCreate("byte Data[" & DllStructGetSize($tBuffer) & "]")
    DllStructSetData($tSID, "Data", DllStructGetData($tBuffer, "Data"))
    DllCall("kernel32.dll", "handle", "LocalFree", "handle", $pSID)
    Return $tSID
EndFunc   ;==>_Security__StringSidToSid

Edited November 19, 2015 by Trong

grammar

• #1

понятно что это переменная вызывается но не обьявленна.
так вот как узнать какая именно?
ну окошко выскакивает а какую искать?
хорошо если код не большой то можно искать, а если пару тыс строк то уже и затруднительно.
я то чисто случайно нашел но это чисто случайно.
а осмысленно можно узнать какая именно?

подскажите пожалуйста.

• #2

D:Projectvalue.au3 (1) : ==> Variable used without being declared.:
ConsoleWrite($Value)
ConsoleWrite(^ ERROR

И, что тут непонятного? :blink:

Добавлено:

Сообщение автоматически объединено: 7 Дек 2011

agikon [?]

Когда программист пишет пару тысяч строк, такая ошибка как объявление переменной у него не должна вызывать затруднения. 😉

• #3

Garrett
хоть убей не понял куда это писать и куда потом смотреть в случае ошибки?
$Value это уже известная переменная.
а если переменных много, то тогда каждую переменную писать в ConsoleWrite ?

а я ж говорю окошко выскакивает и там ничего конкретно не написано.
кстати ошибка может появлятся не при запуске скрипта, а во время работы когда идёт считывание ошибочной переменной.

• #4

agikon
В примере выше, красным помечено:

D:Projectvalue.au3 (1) : ==> Variable used without being declared.:

номер строки, где обнаружена ошибка

имя переменной

указатель на позицию в строке
Смотреть всё это в консоли SciTE.

agikon [?]

а если переменных много, то тогда каждую переменную писать в ConsoleWrite ?

Нет, это только пример.

Скриншот, пожалуйста, в студию.

• #5

Garrett [?]

Скриншот, пожалуйста, в студию.

ТС говорит наверное про уже скомпилированный скрипт.

• #6

ТС говорит наверное про уже скомпилированный скрипт.

Даже там пишут номер строки, где обнаружена ошибка.

• #7

единственно что понял. что эт онадо нажать F5 и смотреть в консоли скайти, это я понял.
но что всё остальное непонятно.

Garrett
если в скрипте например 20 переменных то что тогда?
каждую писать

ConsoleWrite($1)
ConsoleWrite($2)
ConsoleWrite($3)

…
и что такое value.au3 ? это инклуд который ошибки показывает? у меня в папке инклудов такого нет. где его скачать можно подскажите пожалуйста.

Zaramot:Даже там пишут номер строки, где обнаружена ошибка.
да пишет только такой строки нет вообще в скрипте.

kaster

Мой Аватар, он лучший самый

• #8

Zaramot [?]

Даже там пишут номер строки, где обнаружена ошибка.

только там нумерация друга, если подключены библиотеки.

• #9

по пунктам пожалуйста можете написать что куда вписывать чтобы показывало какая переменная ошибку выдаёт?

• #10

agikon [?]

по пунктам пожалуйста можете написать что куда вписывать чтобы показывало какая переменная ошибку выдаёт?

Загрузите свой скрипт в SciTE и нажмите Ctrl+F5
В консоли всё увидите.

• #11

а как загрузить в SciTE?
у меня при компиляции компилирует скайти, это не одно и то же?
под «загрузить в скайти» что имеете в виду? как загрузить в скайти?

а вообще нажимаю Ctrl+F5 и в консоли несколько строк появляется и всё. сам скрипт не запускается.

• #12

agikon [?]

под «загрузить в скайти» что имеете в виду?

Открыть документ, с расширением AU3 (скрипт) в SciTE

• #13

ну у меня все скрипты открываются в скайти, нажимаю Ctrl+F5 и в консоли несколько строк появляется и всё. сам скрипт не запускается.

вы же поймите скрипт запускается с некими параметрами которые считываются с ини-файла, а если я запускаю через скайти то наверно файл инициализации не обрабатывается.

• #14

agikon [?]

ну у меня все скрипты открываются в скайти, нажимаю Ctrl+F5 и в консоли несколько строк появляется и всё. сам скрипт не запускается.

Он и не должен запускаться. Запускается проверка синтаксиса, где вы и можете посмотреть, какие ошибки допущены в написание кода.

• #15

так при компиляции варнингов то и нет.
ошибка появляется когда скрипт работает и когда код с ошибочной переменной обрататывается, только тогда скрипт ошибку выдаёт.

вот я ж и говорю как в окне с ошибкой «Variable used without being declared» понять какая переменная вызвала эту ошибку?
если код то указывается, но например в скрипте 1000 строк, то код например 1624.
вот в чём вся суть вопроса.

• #16

agikon [?]

так при компиляции варнингов то и нет.

А кто говорит, что они есть? :blink:

• #17

agikon [?]

вы же поймите скрипт запускается с некими параметрами которые считываются с ини-файла

скрипт не скомпилированный тоже читает из ini-файла. Ищи ошибку тестируя нескомпилированный скрипт.

• #18

ошибку я не могу найти потому что не указывает на строку ошибки.
оно то пишет что проблема в переменной но какой хз.

• #19

agikon
Выложите ваш скрипт.

• #20

agikon
Или скрин сообщения ошибки.

Обновлено: 29.01.2023

Современные компиляторы обладают огромным количеством диагностик. И удивительно, что очень малая их часть включена по умолчанию.

Огромное количество претензий, которые предъявляют к языку C++ в этих ваших интернетах, — про сложность, небезопасность, стрельбу по ногам и т.п., — относятся как раз к тем случаям, когда люди просто не знают о том, что можно решить эти проблемы лёгким движением пальцев по клавиатуре.

Давайте же исправим эту вопиющую несправедливость, и прольём свет истины на возможности компилятора по предотвращению ошибок.

Нужно больше предупреждений

Но и это ещё не всё. Есть несколько флагов, которые почему-то не входят ни в один из «аргегаторов», но крайне важны и полезны.

-Wctor-dtor-privacy

Предупреждает о том, что класс выглядит неиспользуемым, потому что конструкторы и деструкторы закрыты, а друзей и открытых статических функций-членов у него нет.

Аналогично, сообщает, что у класса есть закрытые функции-члены, а открытых нет ни одной.

-Wnon-virtual-dtor

Предупреждает о том, что у класса есть виртуальные функции-члены, но деструктор при этом не виртуальный. Очень сложно представить себе такой класс. Вероятнее всего, это ошибка.

-Wold-style-cast

Предупреждает о приведении типов в стиле языка C. В плюсах есть прекрасные и ужасные static_cast , dynamic_cast , reinterpret_cast и const_cast , которые более локальны и более описательны. Сишный способ слишком сильный и — о, ужас, — небезопасный. Лучше его не использовать вообще.

-Woverloaded-virtual

Предупреждает о попытке в классе-наследнике перегрузить виртуальную функцию базового класса:

-Wsign-promo

Крайне полезный флаг. Предупреждает о неочевидном выборе перегруженной функции:

Вероятнее всего, хотели-таки позвать вторую перегрузку, а не первую. А если всё-таки первую, то будьте любезны сказать об этом явно.

-Wduplicated-branches

Предупреждает о том, что ветви if и else одинаковы:

Условный оператор ?: также под прицелом:

Для меня абсолютная загадка, почему этот флаг не включён не то, что в -Wall , а вообще по умолчанию.

-Wduplicated-cond

Предупреждает об одинаковых условиях в цепочках if-else-if :

-Wfloat-equal

Предупреждает о проверке на равенство между двумя числами с плавающей точкой. Скорее всего, это ошибка, и сравнение нужно проводить с заданной точностью.

Если же требуется именно сравнить на равенство (такое редко, но бывает), то можно использовать std::equal_to , который под предупреждение не попадает.

-Wshadow=compatible-local

Полезная опция, которая не даёт перекрыть локальную переменную другой локальной переменной при условии, что они имеют совместимые типы.

-Wcast-qual

Предупреждает о преобразовании указателя, при котором сбрасываются квалификаторы. Например, чтобы случайно не потерять const .

-Wconversion

Очень, очень, очень важный флаг. Он предупреждает об огромном количестве неявных сужающих (то есть потенциально приводящих к потере информации) преобразований, которые могут быть следствием ошибки программиста. Например:

Если вы раньше никогда не включали этот флаг, то будет интересно.

-Wzero-as-null-pointer-constant

Предупреждает об использовании целочисленного нуля вместо nullptr .

-Wextra-semi

Флаг для педантов. Сообщает о лишней точке с запятой после определения функции-члена.

-Wsign-conversion

Как и -Wconversion помогает предотвратить большое количество неявных преобразований, которые запросто могут быть ошибками:

-Wlogical-op

Предупреждает о подозрительных логических выражениях. Например, когда вместо побитового «И» поставили логическое «И», или логическое выражение имеет одинаковые операнды:

-Wextra

«Агрегатор» дополнительных предупреждений. Включает много интересного, чего нет в -Wall (как и в случае с -Wall , рассмотрены будут не все возможности).

-Wempty-body

Предупреждает о пустом теле условных выражений или цикла do-while . Чаще всего это говорит об опечатке, меняющей логику программы:

-Wimplicit-fallthrough

Предупреждает о «проваливании» в операторе switch :

Компилятор предполагает, что программист забыл break , и case 2 не должен проваливаться:

В C++17 для обозначения явного намерения появился специальный атрибут — fallthrough :

-Wmissing-field-initializers

Предупреждает о том, что отдельные члены структуры не были проинициализированы. Скорее всего это просто забыли сделать:

-Wredundant-move

Предупреждает о ненужном вызове std::move в случаях, когда компилятор сам сделает всё, что нужно:

-Wtype-limits

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

-Wshift-negative-value

Предупреждает об операциях сдвига для отрицательных значений. Отрицательными могут быть только знаковые числа, а для них это некорректно:

-Wunused-parameter

Предупреждает о неиспользуемом параметре функции. Возможно, про него просто забыли, и в этом случае функция может работать некорректно.

В C++17 для явного выражения намерения существует атрибут maybe_unused :

-Wunused-but-set-parameter

Предупреждает о том, что в параметр функции было записано значение, но после этого он ни разу не использовался. Возможно, про него снова забыли:

Ода компилятору

Компилятор – лучший друг плюсовика. Компилятор — это не просто транслятор формального человекочитаемого языка в машинные коды. Компилятор — лучший помощник в написании программ.

Важная (и не единственная) помощь, которую оказывает компилятор — поиск ошибок. И я говорю не об опечатках, несовпадении типов и прочих синтаксических ошибках. Я говорю об огромном наборе ошибок, которые можно выловить с помощью механизма предупреждений.

Часто встречаю мнение о том, что предупреждений слишком много, они дают ложноположительные результаты, мешают работать, замыливают глаз, отвлекают от «настоящих» ошибок и т.п. Такое действительно бывает, но это большая редкость.

Большинство предупреждений — это не «бзик» компилятора, который можно просто проигнорировать. Предупреждение — это потенциальная ошибка. Предупреждение — это сигнал от компилятора о том, что написано одно, а требуется, возможно, что-то совершенно иное.

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

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

Сразу хочу оговориться, что далее речь пойдёт исключительно о языке C++ и компиляторе GCC (впрочем, подавляющая часть информации актуальна и для компилятора Clang). Информацию о других компиляторах и языках придётся искать в соответствующих справочниках.

-Wall — это «агрегатор» базовых предупреждений. В языке C++ он включает в себя длинный перечень предупреждений, каждое из которых будет рассмотрено отдельно (на самом деле, рассмотрены будут не все, а только те, которые непосредственно помогают выявлять ошибки).

Несмотря на название, этот флаг включает далеко не все предупреждения, которые умеет обнаруживать компилятор.

-Waddress

Предупреждает о странной работе с адресами. Например, об использовании адреса функции в условном выражении. Такое может произойти, если забыть поставить скобки после имени функции:

Также этот флаг может спасти от типичной ошибки новичка — сравнения строкового литерала с адресом. Очевидно, программист хотел сравнить строки, но в результате сравнил два указателя:

-Warray-bounds=1

Предупреждает о выходе за пределы массивов. Используется только вместе с -O2 .

-Wbool-compare

Предупреждает о сравнении булева выражения с целым числом, которое нельзя трактовать как булево:

-Wbool-operation

Предупреждает о подозрительных операциях с булевыми выражениями. Например, о побитовом отрицании:

Что касается инкрементов и декрементов булевых переменных, то в C++17 это просто ошибки, безо всяких предупреждений.

-Wcatch-value

Предупреждает о обработчиках исключений, которые принимают полиморфные объекты по значению:

Есть и более сильные версии предупреждения: -Wcatch-value=n (см. справку к компилятору).

-Wchar-subscripts

Предупреждает об обращении к массиву по индексу, тип которого char . А ведь char является знаковым на многих машинах:

-Wcomment

Предупреждает о наличии последовательности, начинающей новый комментарий ( /* ), внутри многострочного комментария, либо о разрыве строки при помощи обратного слеша внутри однострочного комментария.

-Wint-in-bool-context

Предупреждает о подозрительном использовании целых чисел там, где ожидаются булевы выражения, например, в условных выражениях:

Другой пример — операции побитового сдвига в булевом контексте. Вполне вероятно, что здесь произошла опечатка, и имелся в виду не сдвиг, а сравнение:

А также сообщает о любых видах умножения в булевом контексте.

-Winit-self

Предупреждает об инициализации переменных самими сабями. Используется только вместе с флагом -Wuninitialized .

-Wlogical-not-parentheses

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

Используется для того, чтобы отлавливать подозрительные конструкции вроде следующей:

Традиционный способ сообщить компилятору, что так и было задумано — поставить скобки, о чём и сообщает компилятор.

-Wmaybe-uninitialized

Предупреждает о том, что существует возможность использования непроинициализированной переменной.

В данном конкретном случае решается с помощью конструкции default :

-Wmemset-elt-size

Предупреждает о подозрительных вызовах функции memset , когда первый аргумент — это массив, а третий аргумент — количество элементов в массиве, но не количество байт, занимаемой этим массивом в памяти.

-Wmemset-transposed-args

Предупреждает о том, что пользователь, вероятно, перепутал порядок аргументов в функции memset :

-Wmisleading-indentation

Предупреждает о том, что отступы в коде не отражают структуру этого кода. Особенно это актуально для конструкций if , else , while и for . Пример:

-Wmissing-attributes

Предупреждает о ситуации, когда специализация шаблона объявлена не с тем же списком атрибутов, что и оригинальный шаблон.

-Wmultistatement-macros

Предупреждает о макросах, состоящих из нескольких инструкций, и используемых в выражениях if , else , while и for . В такой ситуации под действие выражений попадает только первая инструкция макроса, и это, вероятно, ошибка:

-Wnonnull

Предупреждает о передаче нулевого указателя в функцию, аргументы которой помечены атрибутом nonnull .

-Wnonnull-compare

Предупреждает о сравнении с нулём аргумента функции, помеченного атрибутом nonnull .

-Wparentheses

Типичный случай — опечатались, и вместо равенства написали присвоение:

Компилятор, естественно, сомневается:

Либо исправляем код, либо убеждаем компилятор в том, что мы хотели именно присвоение:

-Wpessimizing-move

Иногда явная попытка переноса может ухудшить производительность. Пример:

-Wreorder

Предупреждает о том, что порядок инициализации членов класса не соответствует порядку их объявления. Поскольку компилятор может переупорядочить инициализацию этих членов, результат может быть неочевидным.

-Wreturn-type

Предупреждает о том, что из функции не вернули заявленный результат:

-Wsequence-point

Сообщает о подозрительных операциях относительно точек следования. Любимый пример (никогда так не делайте):

-Wsign-compare

Одно из важнейших предупреждений. Сообщает о сравнении знаковых и беззнаковых чисел, которое может произвести некорректный результат из-за неявных преобразований. К примеру, отрицательное знаковое число неявно приводится к беззнаковому и внезапно становится положительным:

-Wsizeof-pointer-div

Предупреждает о подозрительном делении друг на друга двух результатов выражения sizeof , когда размер указателя делится на размер объекта. Обычно это бывает, когда пытаются вычислить размер массива, но вместо массива по ошибке берут указатель:

-Wsizeof-pointer-memaccess

Предупреждает о подозрительных параметрах, передаваемых в строковые функции и функции для работы с памятью ( str. , mem. и т.п.), и использующих оператор sizeof . Например:

-Wstrict-aliasing

Каламбур типизации (strict aliasing) — это отдельная большая тема для разговора. Предлагаю читателю найти литературу по этой теме самостоятельно.

В общем, это тоже крайне полезное предупреждение.

-Wswitch

Предупреждает о том, что не все элементы перечисления задействованы в конструкции switch :

-Wtautological-compare

Предупреждает о бессмысленном сравнении переменной с самой собой:

Кроме того, сообщает о сравнениях при участии битовых операций, которые имеют всегда один и тот же результат (всегда истинно или всегда ложно):

-Wtrigraphs

Предупреждает о наличии триграфов, которые могут изменить смысл программы. Не сообщается о триграфах в теле комментария, за исключением случаев, когда триграф трактуется как перевод строки.

-Wuninitialized

Предупреждает об использовании переменных и членов класса, которые не были проинициализированы:

-Wunused-function

Предупреждает о том, что статическая функция объявлена, но не определена, либо о том, что статическая функция, не помеченная как inline , не используется.

-Wunused-variable

Предупреждает о том, что переменная не используется.

Для того, чтобы помочь компилятору понять, что так и задумывалось, можно использовать конструкцию static_cast<void>(. ) :

Заключение

Резюмируя, для компилятора GCC (Clang кое-что из этого не умеет, к сожалению) я рекомендую включать следующий минимальный набор флагов, по необходимости дополняя его более сложными диагностиками.

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

1)То что автор не я можно догадаться.(Понимаю здесь работают не гадалки, но всё же)
Скажи что тебе даст воспроизведение ошибки, и как ты это получишь.
и я сделаю всё сам, и дам тебе инфу нужную.
Всю черновую работу можете поручить мне, главное скажите что делать.

2)А почему кнопка паузы не работает не известно?
3)Автор какой-то немец( я только изменил его немного) и контактов свои не оставил.

AZJIO

Меценат

biko3
Пишет что переменные не декларированы, попробуй декларировать.

И попробуй использовать скрипт не скомпилированным. Тогда даст точную строку ошибки.

-Wpedantic

-Wall и -Wextra — это не всё, на что способен компилятор.

В дополнение к ним существует флаг -Wpedantic (он же -pedantic ), который проверяет соответствие кода стандарту ISO C++, сообщает об использовании запрещённых расширений, о наличии лишних точек с запятой, нехватке переноса строки в конце файла и прочих полезных штуках.

Содержание

-Werror

С этого, вообще говоря, стоило бы начать. Данный флаг делает все предупреждения ошибками. Код не скомпилируется при наличии хотя бы одного предупреждения.

Без этого флага всё остальное имеет мало смысла. Но если понять и принять мысль о том, что предупреждение — это что-то подозрительное, и их быть не должно, то именно этот флаг и позволит поддерживать код в чистоте.

В дополнение к -Werror существует флаг -pedantic-errors , который не эквивалентен комбинации -Wpedantic -Werror .

[FIXED] Error: Variable used without being declared.

6 posts • Page 1 of 1
Bailx Posts: 6 Joined: Mon Jul 10, 2006 1:22 am

[FIXED] Error: Variable used without being declared.

Post by Bailx » Mon Jul 10, 2006 1:33 am

Was hoping someone might be able to help me out with this. I am running into an error, and the very first one i’ve ever had with rvm integrator i might add, anyways. i must note that the only differences between this time and the last time is last time i used 2.0.7, this time it’s 2.0.7a, and all the update packs are the same, except i added these two:

otherwise it’s identical.

here is a screenshot

i was just about to try it another time, without the two new addons, but still with 2.0.7a. and I thought maybe i’d post and see if anyone had seen this before (couldn’t find anything searching)

Bailx Posts: 6 Joined: Mon Jul 10, 2006 1:22 am

Post by Bailx » Mon Jul 10, 2006 2:09 am

huh.. well this is odd.

i did that procedure twice (above) and it failed twice exactly the same way.

then i did the 2.0.7a update seperatly — worked fine

then i did Multi_Siginet_MultiTheme-AddonsPack_1.7.cab & Kels_Brushed_Inspirat_v1.cab

that worked, then i did the rest.

odd that they didn’t all work together, but worked seperatly.

also another thats odd. the first two times i just pressed ctrl + a to select all of the addons, but for some reason (and you can see in the log) Kels_Brushed_Inspirat_v1.cab doesn’t show up in the popup screen (when you choose your *.cab files) *.* in the box, and the file shows up.

The variable ‘znak’ is being used without being initialized.

Переменная ‘znak’ используется без инициализации.

Инициализация при объявлении выглядит так: char znak = 0; , а не как у вас char znak; — это объявление без инициализации.

Нельзя обращаться к переменной, пока в нее не присвоили какое либо значение.

Скорее всего это не ошибка, а предупреждение, т.к. обращаться к такой переменной можно, нельзя что-то предполагать о содержимом этой переменной и производить на основании этого какие-то дальнейшие действия.
Чаще всего такая ситуация — это ошибка программиста, как в данном случае. Но бывают и исключения.

Читайте также:

  

• Bulletstorm системные требования

  

• Как убрать иконку в лиге легенд

  

• Программа для замены моделей в гта 5

  

• Ошибка загрузки сохраненных данных dark souls 2

  

• Atlas copco ошибки компрессора

Текст ошибки Перевод текста Unable to open the script file. Не удается открыть файл сценария. Badly formatted «Func» statement. Неправильный формат оператора «Func» Unable to parse line. Не удалось разобрать строку. Missing right bracket ‘)’ in expression. Отсутствует правая скобка ‘)’ в выражении. Missing operator in expression. Отсутствует оператор в выражении. Unbalanced brackets in expression. Незакрытые скобки в выражении. Error in expression. Ошибка в выражении Error parsing function call. Ошибка разбора (парсинга, синтаксиса) вызываемой функции Incorrect number of parameters in function call. Неверное количество параметров при вызове функции «ReDim» used without an array variable. «ReDim» используется с переменной не являющейся массивом Illegal text at the end of statement (one statement per line). Недопустимый текст в конце оператора (один оператор в строке) «If» statement has no matching «EndIf» statement. Оператор «If» не имеет сопровождающий его оператор «EndIf» «Else» statement with no matching «If» statement. Оператор «Else» не имеет сопровождающий его оператор «If» «EndIf» statement with no matching «If» statement. Оператор «EndIf» не имеет сопровождающий его оператор «If» Too many «Else» statements for matching «If» statement. Слишком много операторов «Else» для одного оператора «If» «While» statement has no matching «Wend» statement. Оператор «While» не имеет сопровождающий его оператор «Wend» «Wend» statement with no matching «While» statement. Оператор «Wend» не имеет сопровождающий его оператор «While» Variable used without being declared. Переменная используется без первоначального объявления Array variable has incorrect number of subscripts or subscript dimension range exceeded. Переменная массива имеет неверное количество индексов или индекс превышает размер массива. Array variable subscript badly formatted. Неправильный формат индекса в переменной массива. Subscript used with non-Array variable. Индекс используется с переменной не являющейся массивом Too many subscripts used for an array. Слишком много индексов для массива. Пример Dim $a[1] = [[1]] Missing subscript dimensions in «Dim» statement. Отсутствуют индексы измерений в операторе «Dim». Пример Dim $a[1] = 1 No variable given for «Dim», «Local», «Global» or «Const» statement. Отсутствует объявление переменной в операторах «Dim», «Local», «Global» или «Const» Expected a «=» operator in assignment statement. Ожидается оператор «=» в конструкции присваивания. Invalid keyword at the start of this line. Недопустимое ключевое слово в начале этой линии Array maximum size exceeded. Превышен максимальный размер массива «Func» statement has no matching «EndFunc». Оператор «Func» не имеет сопровождающий его оператор «EndFunc» Duplicate function name. Дубликат имени функции (Одна и та же функция встречается в скрипте дважды) Unknown function name. Неизвестное имя функции (Вызов отсутствующей функции) Unknown macro. Неизвестное имя макроса (опечатка или устаревший макро) Unable to execute the external program. Не удается выполнить внешнюю программу. Unknown option or bad parameter specified. (?) Неизвестный параметр или плохо указанного параметра. Unable to load the internet libraries. (?) Не удается загрузить библиотеки Интернет Unable to open file, the maximum number of open files has been exceeded. Не удается открыть файл, превышено максимальное количество открытых файлов. Invalid file handle used. Используется неверный дескриптор файла Invalid file filter given. (?) Неверно задан файловый фильтр Expected a variable in user function call. Ожидается переменная при вызове пользовательской функции. Пример Func _FuncName(ByRef) «Do» statement has no matching «Until» statement. Оператор «Do» не имеет сопровождающий его оператор «Until» «Until» statement with no matching «Do» statement. Оператор «Until» не имеет сопровождающий его оператор «Do» «For» statement is badly formatted. Оператор «For» имеет неправильный формат «Next» statement with no matching «For» statement. Оператор «Next» не имеет сопровождающий его оператор «For» «ExitLoop/ContinueLoop» statements only valid from inside a For/Do/While loop. Операторы «ExitLoop/ContinueLoop» допускаются только внутри циклов For/Do/While. «For» statement has no matching «Next» statement. Оператор «For» не имеет сопровождающий его оператор «Next» «Case» statement with no matching «Select»or «Switch» statement. Оператор «Case» не имеет сопровождающий его оператор «Select» или «Switch» «EndSelect» statement with no matching «Select» statement. Оператор «EndSelect» не имеет сопровождающий его оператор «Select» Recursion level has been exceeded — AutoIt will quit to prevent stack overflow. Уровень рекурсии был превышен, AutoIt завершает работу, чтобы предотвратить переполнение стека Unable to access RunAs API. Не удается получить доступ RunAs API String missing closing quote. Строка не содержит закрывающую кавычку Unterminated string. Незавершенная строка Badly formated variable or macro. Неправильный формат переменной или макро This keyword cannot be used after a «Then» keyword. Это ключевое слово не может быть использована после ключевого слова «Then» «Select» statement is missing «EndSelect» or «Case» statement. Оператор «Select» не имеет сопровождающий его оператор «EndSelect» или «Case» «If» statements must have a «Then» keyword. Оператор «If» должен иметь ключевое слово «Then» Cannot assign values to constants. Невозможно присвоить значения константе. Cannot make existing variables into constants. Невозможно сделать существующие переменные в константы Object referenced outside a «With» statement. Объект ссылается за пределами оператора «With». Nested «With» statements are not allowed. Вложенные операторы «With» не допускается Variable must be of type «Object». Переменная должна быть типом «Object» The requested action with this object has failed. Запрашиваемое действие с этим объектом не удалось Variable appears more than once in function declaration. (?) Переменная появляется более одного раза в объявлении функции ReDim array can not be initialized in this manner. ReDim массива не может быть выполнен таким способом. Пример ReDim $arr1[1] = [1] An array variable can not be used in this manner. Переменная массива не может быть использована таким образом. Can not redeclare a constant. Невозможно декларировать константу повторно Can not redeclare a parameter inside a user function. Невозможно объявить переменную, переданную как параметр внутри пользовательской функции. Can pass constants by reference only to parameters with «Const» keyword. Можно передавать константы как ссылки только на параметры с ключевым словом «Const». Например _FuncName(ByRef $w) Can not initialize a variable with itself. текст Incorrect way to use this parameter. Неправильный способ использования этого параметра «EndSwitch» statement with no matching «Switch» statement. Оператор «EndSwitch» не имеет сопровождающий его оператор «Switch» «Switch» statement is missing «EndSwitch» or «Case» statement. Оператор «Switch» не имеет сопровождающий его оператор «EndSwitch» или «Case» «ContinueCase» statement with no matching «Select»or «Switch» statement. Оператор «ContinueCase» не имеет сопровождающий его оператор «Select» или «Switch» Assert Failed! Утверждение неудачно! AutoIt has encountered a fatal crash as a result of:r Unable to execute DLLCall. AutoIt столкнулся с фатальным крахом в результате осуществления: не удается выполнить DLLCall Obsolete function/parameter. Устаревшая функция / параметр Invalid Exitcode (reserved for AutoIt internal use). Недопустимый Exitcode (зарезервировано для внутреннего использования AutoIt). Cannot parse #include Не удается выполнить разбор #include Error opening the file Ошибка при открытии файла (при несуществующем #include-файле)

I am still in the process of learning how to script my own GUIs. Koda has been a big help, I have been messing around with it, learning little bits at a time. However, I have encountered an error that I can not seem to get around. The current GUI I am working on is a simple tool to change bot settings for a game. This gui should have 5 buttons for difficulty setting, and a list of check boxes for the number of bots.

BUT, when I select a difficulty level, bot number, and hit save I get this error:

'Variable used without being declared'  

The error is occuring on this line (botSAVE button at the end):

FileWriteLine($file2, "aiSettings.setMaxNBots " & $botnum)    

I thought the button functions ‘Func numClick()’ set this variable based on the button that was pressed, but obviously I have overlooked something. Hopefully it is an easy fix. Does anybody see what might be causing this error? I have posted this question on autoitcsript forums but they seem to shy away from game automation scripts so I didn’t really get much help over there.

#include <ButtonConstants.au3>
#include <GUIConstantsEx.au3>
#include <StaticConstants.au3>
#include <TabConstants.au3>
#include <WindowsConstants.au3>
Opt("GUIOnEventMode", 1)
#Region ### START Koda GUI section ### Form=c:usersadmindesktopform1.kxf
$Form1 = GUICreate("Battlefield 2 Bot-Tool", 418, 499, 759, 83)
GUISetOnEvent($GUI_EVENT_CLOSE, "Form1Close")
GUISetOnEvent($GUI_EVENT_MINIMIZE, "Form1Minimize")
GUISetOnEvent($GUI_EVENT_MAXIMIZE, "Form1Maximize")
GUISetOnEvent($GUI_EVENT_RESTORE, "Form1Restore")
$Pic1 = GUICtrlCreatePic("C:UsersadminDesktopbflogo.jpg", 0, 0, 417, 233)
$Settings = GUICtrlCreateTab(8, 240, 401, 249)
GUICtrlSetOnEvent(-1, "SettingsChange")
$Bot = GUICtrlCreateTabItem("Bots")
$botSAVE = GUICtrlCreateButton("Save these changes", 220, 433, 169, 33)
GUICtrlSetOnEvent(-1, "botSAVEClick")
$botDEFAULT = GUICtrlCreateButton("Restore default settings", 28, 433, 169, 33)
GUICtrlSetOnEvent(-1, "botDEFAULTClick")
$botskill = GUICtrlCreateGroup(" Bot skill level", 28, 265, 169, 145)
$skill1 = GUICtrlCreateRadio("  RECRUIT", 44, 289, 113, 17)
GUICtrlSetTip(-1, "Easy opposition, a damn turkey shoot")
GUICtrlSetOnEvent(-1, "skill1Click")
$skill2 = GUICtrlCreateRadio("  TRAINED", 44, 313, 113, 17)
GUICtrlSetTip(-1, "Default setting, opponents are deadly at close range")
GUICtrlSetOnEvent(-1, "skill2Click")
$skill3 = GUICtrlCreateRadio("  HARDENED", 44, 337, 113, 17)
GUICtrlSetTip(-1, "Enemies are a force to be reckoned with")
GUICtrlSetOnEvent(-1, "skill3Click")
$skill4 = GUICtrlCreateRadio("  VETERAN", 44, 361, 113, 17)
GUICtrlSetTip(-1, "Enemies are experienced, and dangerous")
GUICtrlSetOnEvent(-1, "skill4Click")
$skill5 = GUICtrlCreateRadio("  ELITE", 44, 385, 113, 17)
GUICtrlSetTip(-1, "Crack shot opponents will strategize against you. You will not survive")
GUICtrlSetOnEvent(-1, "skill5Click")
GUICtrlCreateGroup("", -99, -99, 1, 1)
$botnumber = GUICtrlCreateGroup(" Number of bots ", 220, 265, 169, 145)
$num1 = GUICtrlCreateRadio("2", 236, 289, 25, 17)
GUICtrlSetOnEvent(-1, "num1Click")
$num2 = GUICtrlCreateRadio("4", 236, 313, 25, 17)
GUICtrlSetOnEvent(-1, "num2Click")
$num3 = GUICtrlCreateRadio("6", 236, 337, 25, 17)
GUICtrlSetOnEvent(-1, "num3Click")
$num4 = GUICtrlCreateRadio("8", 236, 361, 25, 17)
GUICtrlSetOnEvent(-1, "num4Click")
$num5 = GUICtrlCreateRadio("10", 236, 385, 33, 17)
GUICtrlSetOnEvent(-1, "num5Click")
$num6 = GUICtrlCreateRadio("12", 284, 289, 33, 17)
GUICtrlSetOnEvent(-1, "num6Click")
$num7 = GUICtrlCreateRadio("14", 284, 313, 33, 17)
GUICtrlSetOnEvent(-1, "num7Click")
$num8 = GUICtrlCreateRadio("16", 284, 337, 33, 17)
GUICtrlSetOnEvent(-1, "num8Click")
$num9 = GUICtrlCreateRadio("18", 284, 361, 33, 17)
GUICtrlSetOnEvent(-1, "num9Click")
$num10 = GUICtrlCreateRadio("20", 284, 385, 33, 17)
GUICtrlSetOnEvent(-1, "num10Click")
$num11 = GUICtrlCreateRadio("24", 340, 289, 33, 17)
GUICtrlSetOnEvent(-1, "num11Click")
$num12 = GUICtrlCreateRadio("28", 340, 313, 33, 17)
GUICtrlSetOnEvent(-1, "num12Click")
$num13 = GUICtrlCreateRadio("32", 340, 337, 33, 17)
GUICtrlSetOnEvent(-1, "num13Click")
$num14 = GUICtrlCreateRadio("36", 340, 361, 33, 17)
GUICtrlSetOnEvent(-1, "num14Click")
$num15 = GUICtrlCreateRadio("40", 340, 385, 33, 17)
GUICtrlSetOnEvent(-1, "num15Click")
GUICtrlCreateGroup("", -99, -99, 1, 1)
$Server = GUICtrlCreateTabItem("Server")
$serverDEFAULT = GUICtrlCreateButton("Restore default settings", 28, 433, 169, 33)
$serverSAVE = GUICtrlCreateButton("Save these changes", 220, 433, 169, 33)
GUICtrlCreateTabItem("")
GUISetState(@SW_SHOW)
Opt("MustDeclareVars",1)
#EndRegion ### END Koda GUI section ###

While 1
    Sleep(100)
WEnd


Func Form1Close()
  If @GUI_WINHANDLE = $Form1 Then
    Exit
  EndIf
EndFunc

Func Form1Maximize()
EndFunc

Func Form1Minimize()
EndFunc

Func Form1Restore()
EndFunc

Func num1Click()
$botnum = "2"
EndFunc

Func num2Click()
   $botnum = "4"
EndFunc

Func num3Click()
   $botnum = "6"
EndFunc

Func num4Click()
   $botnum = "8"
EndFunc

Func num5Click()
   $botnum = "10"
EndFunc

Func num6Click()
   $botnum = "12"
EndFunc

Func num7Click()
   $botnum = "14"
EndFunc

Func num8Click()
   $botnum = "16"
EndFunc

Func num9Click()
   $botnum = "18"
EndFunc

Func num10Click()
   $botnum = "20"
EndFunc

Func num11Click()
   $botnum = "24"
EndFunc

Func num12Click()
   $botnum = "28"
EndFunc

Func num13Click()
   $botnum = "32"
EndFunc

Func num14Click()
   $botnum = "36"
EndFunc

Func num15Click()
   $botnum = "40"
EndFunc

Func SettingsChange()
EndFunc

Func skill1Click()
   $skillset = "0.1"
EndFunc

Func skill2Click()
   $skillset = "0.3"
EndFunc

Func skill3Click()
   $skillset = "0.6"
EndFunc

Func skill4Click()
   $skillset = "0.8"
EndFunc

Func skill5Click()
   $skillset = "1.0"
EndFunc

Func botDEFAULTClick()
EndFunc

Func botSAVEClick()
   Local $file = FileOpen("AIDefault.ai", 1)
If $file = -1 Then
    MsgBox(0, "Protected file", "Please ensure that the file 'AIDefault.ai' is not set to Read Only.")
    exit
    EndIf
FileWriteLine($file, "aiSettings.setNSides 2 ")
FileWriteLine($file, "aiSettings.setAutoSpawnBots 1 ")
FileWriteLine($file, "aiSettings.setMaxNBots 64 ")
FileWriteLine($file, "aiSettings.maxBotsIncludeHumans 1")
FileWriteLine($file, "aiSettings.setBotSkill 0.4  ")
FileWriteLine($file, "run BotNames.ai ")
FileWriteLine($file, "aiSettings.setInformationGridDimension 32")
FileWriteLine($file, "run AIPathFinding.ai")
FileWriteLine($file, "run AIBotChanger.ai ")
FileClose($file)

$file2 = FileOpen("AIBotChanger.ai", 1)
If $file2 = -1 Then
    MsgBox(0, "Protected file", "Please ensure that the file 'AIBotChanger.ai' is not set to Read Only.")
    Exit
EndIf
FileWriteLine($file2, "aiSettings.overrideMenuSettings 1")
FileWriteLine($file2, "aiSettings.setMaxNBots " & $botnum)
FileWriteLine($file2, "aiSettings.setBotSkill " & $skillset)
FileWriteLine($file2, "aiSettings.maxBotsIncludeHumans 0")
FileClose($file2)
EndFunc  

-If anybody sees what I have done wrong, OR has a link to a topic that covers declaring variables with a button click, Help is always appreciated. The only topics I have found are incredibly vague, or differ slightly from what I am look for