Jump to content

Unable to have button write line to a file


Recommended Posts

Hello,

I'm trying to make an interface that writes the info typed into a text box to a log file when clicking a button. Clearing the text box on submit would be an added bonus but not required. Here is the code I'm working with.

#include <Date.au3>
#include <GuiConstantsEx.au3>
#include <StaticConstants.au3>
#include <TreeViewConstants.au3>

$tCur = _Date_Time_GetLocalTime()
$sDateTime = _Date_Time_SystemTimeToDateTimeStr($tCur)

; Open log file
$results = "Results.txt"
$file = FileOpen(@ScriptDir & "\" & $results, 1)
; Check if file opened for writing OK
If $file = -1 Then
    MsgBox(0, "Error", "Unable to open file.")
    Exit
 EndIf

GUICreate("Write", 350, 200)
GUISetBkColor(0xFFFFFF)

$text = GUICtrlCreateInput("", 42, 27, 200, 20)

; MENU
$File = GUICtrlCreateMenu("File")
$Exit = GUICtrlCreateMenuItem("Exit", $File)
$Help = GUICtrlCreateMenu("Help")
$About = GUICtrlCreateMenuItem("About", $Help)

; BUTTONS
$nextb = GUICtrlCreateButton("Next", 36, 115, 75, 30)

GUISetState(@SW_SHOW)

    While 1
    $guimsg = GUIGetMsg()
        Select
            Case $guimsg = $Exit
               Exit
            Case $guimsg = $GUI_EVENT_CLOSE
               Exit
             Case $guimsg = $nextb
               Call ("one")
            Case $guimsg = $About
              MsgBox(64, "About", "Version 1.0")
        EndSelect
     WEnd

Func one()
   $nstep = GUICtrlRead($text)
   FileWriteLine($file, $sDateTime & @TAB & $nstep & @TAB & "Next")
   Return
EndFunc
Edited by Melba23
Added code tags
Link to comment
Share on other sites

You reused the $file variable name for the menu item, so there's no file to write to when you call the one function. Give them unique names.

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 Gude
How 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

  • Moderators

majaeger,

Welcome to the AutoIt forum. :)

When you post code please use Code tags - see here how to do it. Then you get a scrolling box and syntax colouring as you can see above now I have added the tags. ;)

M23

Public_Domain.png.2d871819fcb9957cf44f4514551a2935.png 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 columns
ChooseFileFolder ---- Single and multiple selections from specified path treeview listing
Date_Time_Convert -- Easily convert date/time formats, including the language used
ExtMsgBox --------- A highly customisable replacement for MsgBox
GUIExtender -------- Extend and retract multiple sections within a GUI
GUIFrame ---------- Subdivide GUIs into many adjustable frames
GUIListViewEx ------- Insert, delete, move, drag, sort, edit and colour ListView items
GUITreeViewEx ------ Check/clear parent and child checkboxes in a TreeView
Marquee ----------- Scrolling tickertape GUIs
NoFocusLines ------- Remove the dotted focus lines from buttons, sliders, radios and checkboxes
Notify ------------- Small notifications on the edge of the display
Scrollbars ----------Automatically sized scrollbars with a single command
StringSize ---------- Automatically size controls to fit text
Toast -------------- Small GUIs which pop out of the notification area

 

Link to comment
Share on other sites

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 account

Sign in

Already have an account? Sign in here.

Sign In Now
 Share

  • Recently Browsing   0 members

    • No registered users viewing this page.
×
×
  • Create New...