SkellySoul Posted February 4, 2011 Share Posted February 4, 2011 Hello I am making a small little game and ran into a funny problem that I can't think of solving As the progress bar drains it changes color but the problem is it always sets it to green because once the Progress < 75 it will always be lower and set it to green. If GUICtrlRead($Progress) = 0 Then _Game_Over() ElseIf GUICtrlRead($Progress) < 75 Then GUICtrlSetColor($Progress, 0x00FF00) ElseIf GUICtrlRead($Progress) < 60 Then GUICtrlSetColor($Progress, 0xFFFF00) ElseIf GUICtrlRead($Male_Progress) < 30 Then GUICtrlSetColor($Progress, 0xFF0000) EndIf Link to comment Share on other sites More sharing options...
BrewManNH Posted February 4, 2011 Share Posted February 4, 2011 You need to reverse your comparisons, put the lowest number first and work your way up to 75. That way if it matches one the lower comparisons, the higher ones won't be checked. If I posted any code, assume that code was written using the latest release version unless stated otherwise. Also, if it doesn't work on XP I can't help with that because I don't have access to XP, and I'm not going to.Give a programmer the correct code and he can do his work for a day. Teach a programmer to debug and he can do his work for a lifetime - by Chirag GudeHow to ask questions the smart way! I hereby grant any person the right to use any code I post, that I am the original author of, on the autoitscript.com forums, unless I've specifically stated otherwise in the code or the thread post. If you do use my code all I ask, as a courtesy, is to make note of where you got it from. Back up and restore Windows user files _Array.au3 - Modified array functions that include support for 2D arrays. - ColorChooser - An add-on for SciTE that pops up a color dialog so you can select and paste a color code into a script. - Customizable Splashscreen GUI w/Progress Bar - Create a custom "splash screen" GUI with a progress bar and custom label. - _FileGetProperty - Retrieve the properties of a file - SciTE Toolbar - A toolbar demo for use with the SciTE editor - GUIRegisterMsg demo - Demo script to show how to use the Windows messages to interact with controls and your GUI. - Latin Square password generator Link to comment Share on other sites More sharing options...
kaotkbliss Posted February 4, 2011 Share Posted February 4, 2011 Or you could use ranges instead, like: ElseIf GUICtrlRead($Progress) < 75 And GUICtrlRead($Progress) > 59 Then GUICtrlSetColor($Progress, 0x00FF00) 010101000110100001101001011100110010000001101001011100110010000 001101101011110010010000001110011011010010110011100100001 My Android cat and mouse gamehttps://play.google.com/store/apps/details?id=com.KaosVisions.WhiskersNSqueek We're gonna need another Timmy! Link to comment Share on other sites More sharing options...
Moderators Melba23 Posted February 4, 2011 Moderators Share Posted February 4, 2011 SkellySoul,As you have discovered, GUICtrlSetColor does not work if the Progress has a theme from the OS. So you have 2 choices:- 1. Use the the 3 available $PBST colours (green, yellow, red).- 2. Remove the theme from the progress bar and use GUICtrlSetColor - but this also changes the appearance of the progress. Anyway, you choose: expandcollapse popup#include <GUIConstantsEx.au3> #include <ProgressConstants.au3> Const $PBST_NORMAL = 0x0001 Const $PBST_ERROR = 0x0002 Const $PBST_PAUSED = 0x0003 $hGUI = GUICreate("Progress Test", 500, 130) $hProg_Theme = GUICtrlCreateProgress(10, 10, 480, 30) GUICtrlSetData(-1, 100) $hProg_NoTheme = GUICtrlCreateProgress(10, 50, 480, 30) GUICtrlSetData(-1, 100) ; Remove the theme DllCall("UxTheme.dll", "int", "SetWindowTheme", "hwnd", GUICtrlGetHandle($hProg_NoTheme), "wstr", 0, "wstr", 0) GUICtrlSetColor(-1, 0x00FF00) $hProgressButton = GUICtrlCreateButton("Progress", 10, 90, 80, 30) GUISetState() While 1 Switch GUIGetMsg() Case $GUI_EVENT_CLOSE Exit Case $hProgressButton ; GUICtrlSendMsg($hProg_Theme, $PBM_SETSTATE, $PBST_NORMAL, 0) ; Just to show the green message For $i = 100 To 1 Step -1 GUICtrlSetData($hProg_Theme, $i) GUICtrlSetData($hProg_NoTheme, $i) Sleep(50) Switch $i Case 60 GUICtrlSendMsg($hProg_Theme, $PBM_SETSTATE, $PBST_PAUSED, 0) GUICtrlSetColor($hProg_NoTheme, 0xFFFF00) Case 30 GUICtrlSendMsg($hProg_Theme, $PBM_SETSTATE, $PBST_ERROR, 0) GUICtrlSetColor($hProg_NoTheme, 0xFF0000) EndSwitch Next ; For some reason Vista requries a double whammy to set it to zero! GUICtrlSetData($hProg_Theme, 0) GUICtrlSetData($hProg_Theme, 0) GUICtrlSetData($hProg_NoTheme, 0) GUICtrlSetData($hProg_NoTheme, 0) Sleep(50) EndSwitch WEndHappy? M23 Any of my own code posted anywhere on the forum is available for use by others without any restriction of any kind Open spoiler to see my UDFs: Spoiler ArrayMultiColSort ---- Sort arrays on multiple columnsChooseFileFolder ---- Single and multiple selections from specified path treeview listingDate_Time_Convert -- Easily convert date/time formats, including the language usedExtMsgBox --------- A highly customisable replacement for MsgBoxGUIExtender -------- Extend and retract multiple sections within a GUIGUIFrame ---------- Subdivide GUIs into many adjustable framesGUIListViewEx ------- Insert, delete, move, drag, sort, edit and colour ListView itemsGUITreeViewEx ------ Check/clear parent and child checkboxes in a TreeViewMarquee ----------- Scrolling tickertape GUIsNoFocusLines ------- Remove the dotted focus lines from buttons, sliders, radios and checkboxesNotify ------------- Small notifications on the edge of the displayScrollbars ----------Automatically sized scrollbars with a single commandStringSize ---------- Automatically size controls to fit textToast -------------- Small GUIs which pop out of the notification area Link to comment Share on other sites More sharing options...
guinness Posted February 4, 2011 Share Posted February 4, 2011 Nice example Melba23! Have been using for a while, so this is a nice addition to my collection. UDF List: _AdapterConnections() • _AlwaysRun() • _AppMon() • _AppMonEx() • _ArrayFilter/_ArrayReduce • _BinaryBin() • _CheckMsgBox() • _CmdLineRaw() • _ContextMenu() • _ConvertLHWebColor()/_ConvertSHWebColor() • _DesktopDimensions() • _DisplayPassword() • _DotNet_Load()/_DotNet_Unload() • _Fibonacci() • _FileCompare() • _FileCompareContents() • _FileNameByHandle() • _FilePrefix/SRE() • _FindInFile() • _GetBackgroundColor()/_SetBackgroundColor() • _GetConrolID() • _GetCtrlClass() • _GetDirectoryFormat() • _GetDriveMediaType() • _GetFilename()/_GetFilenameExt() • _GetHardwareID() • _GetIP() • _GetIP_Country() • _GetOSLanguage() • _GetSavedSource() • _GetStringSize() • _GetSystemPaths() • _GetURLImage() • _GIFImage() • _GoogleWeather() • _GUICtrlCreateGroup() • _GUICtrlListBox_CreateArray() • _GUICtrlListView_CreateArray() • _GUICtrlListView_SaveCSV() • _GUICtrlListView_SaveHTML() • _GUICtrlListView_SaveTxt() • _GUICtrlListView_SaveXML() • _GUICtrlMenu_Recent() • _GUICtrlMenu_SetItemImage() • _GUICtrlTreeView_CreateArray() • _GUIDisable() • _GUIImageList_SetIconFromHandle() • _GUIRegisterMsg() • _GUISetIcon() • _Icon_Clear()/_Icon_Set() • _IdleTime() • _InetGet() • _InetGetGUI() • _InetGetProgress() • _IPDetails() • _IsFileOlder() • _IsGUID() • _IsHex() • _IsPalindrome() • _IsRegKey() • _IsStringRegExp() • _IsSystemDrive() • _IsUPX() • _IsValidType() • _IsWebColor() • _Language() • _Log() • _MicrosoftInternetConnectivity() • _MSDNDataType() • _PathFull/GetRelative/Split() • _PathSplitEx() • _PrintFromArray() • _ProgressSetMarquee() • _ReDim() • _RockPaperScissors()/_RockPaperScissorsLizardSpock() • _ScrollingCredits • _SelfDelete() • _SelfRename() • _SelfUpdate() • _SendTo() • _ShellAll() • _ShellFile() • _ShellFolder() • _SingletonHWID() • _SingletonPID() • _Startup() • _StringCompact() • _StringIsValid() • _StringRegExpMetaCharacters() • _StringReplaceWholeWord() • _StringStripChars() • _Temperature() • _TrialPeriod() • _UKToUSDate()/_USToUKDate() • _WinAPI_Create_CTL_CODE() • _WinAPI_CreateGUID() • _WMIDateStringToDate()/_DateToWMIDateString() • Au3 script parsing • AutoIt Search • AutoIt3 Portable • AutoIt3WrapperToPragma • AutoItWinGetTitle()/AutoItWinSetTitle() • Coding • DirToHTML5 • FileInstallr • FileReadLastChars() • GeoIP database • GUI - Only Close Button • GUI Examples • GUICtrlDeleteImage() • GUICtrlGetBkColor() • GUICtrlGetStyle() • GUIEvents • GUIGetBkColor() • Int_Parse() & Int_TryParse() • IsISBN() • LockFile() • Mapping CtrlIDs • OOP in AutoIt • ParseHeadersToSciTE() • PasswordValid • PasteBin • Posts Per Day • PreExpand • Protect Globals • Queue() • Resource Update • ResourcesEx • SciTE Jump • Settings INI • SHELLHOOK • Shunting-Yard • Signature Creator • Stack() • Stopwatch() • StringAddLF()/StringStripLF() • StringEOLToCRLF() • VSCROLL • WM_COPYDATA • More Examples... Updated: 22/04/2018 Link to comment Share on other sites More sharing options...
SkellySoul Posted February 4, 2011 Author Share Posted February 4, 2011 Thanks very much I should of have caught that but still to early to think properly Link to comment Share on other sites More sharing options...
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now