Sign in to follow this  
Followers 0
Teldin

What am I doing wrong?

6 posts in this topic

Func _Clear()

$countlines = _FileCountLines("test.txt")

If $countlines > 25 then

For $k = 25 to $countlines

GUICtrlSetData($output, "File has " & $countlines & " lines. Erasing line number " & $k)

_FileWriteToLine("test.txt", $k, "", 1)

Next

Else

GUICtrlSetData($output, "CLEARING DONE")

EndIf

EndFunc

I have to run it 6-7 times to erase all the lines (up to 25). It seems to only erase half the number of lines each time although it looks right in my $output...

Why?

Share this post


Link to post
Share on other sites



It looks to me like its supposed to erase all lines PAST 25. Also putting it in The AutoIt tags and telling us more completely what its supposed to do will help alot to.


[center][/center]Working on the next big thing.Currently Playing: Halo 4, League of LegendsXBL GT: iRememberYhslaw

Share this post


Link to post
Share on other sites

#3 ·  Posted (edited)

Why?

That is because when you delete record 26 then record 27 becomes 26 and will not be deleted.

Could be solved by doing:

For $k = $countlines to 25 step -1

Edited by JdeB

Visit the SciTE4AutoIt3 Download page for the latest versions  - Beta files                                How to post scriptsource        Forum Rules
 
Live for the present,
Dream of the future,
Learn from the past.
  :)

Share this post


Link to post
Share on other sites

Teldin

Hi, try like this:

Func _Clear()
   $countlines = _FileCountLines("test.txt")
   If $countlines > 25 then 
      For $k = $countlines to 25 Step -1
          GUICtrlSetData($output, "File has " & $countlines & " lines. Erasing line number " & $k)
          _FileWriteToLine("test.txt", $k, "", 1)
      Next
   Else
      GUICtrlSetData($output, "CLEARING DONE")
   EndIf
EndFunc

 

Spoiler

Using OS: Win 7 Professional, Using AutoIt Ver(s): 3.3.6.1 / 3.3.8.1

AutoIt_Rus_Community.png AutoIt Russian Community

My Work...

Spoiler

AutoIt_Icon_small.pngProjects: ATT - Application Translate Tool {new}| BlockIt - Block files & folders {new}| SIP - Selected Image Preview {new}| SISCABMAN - SciTE Abbreviations Manager {new}| AutoIt Path Switcher | AutoIt Menu for Opera! | YouTube Download Center! | Desktop Icons Restorator | Math Tasks | KeyBoard & Mouse Cleaner | CaptureIt - Capture Images Utility | CheckFileSize Program

AutoIt_Icon_small.pngUDFs: OnAutoItErrorRegister - Handle AutoIt critical errors {new}| AutoIt Syntax Highlight {new}| Opera Library! | Winamp Library | GetFolderToMenu | Custom_InputBox()! | _FileRun UDF | _CheckInput() UDF | _GUIInputSetOnlyNumbers() UDF | _FileGetValidName() UDF | _GUICtrlCreateRadioCBox UDF | _GuiCreateGrid() | _PathSplitByRegExp() | _GUICtrlListView_MoveItems - UDF | GUICtrlSetOnHover_UDF! | _ControlTab UDF! | _MouseSetOnEvent() UDF! | _ProcessListEx - UDF | GUICtrl_SetResizing - UDF! | Mod. for _IniString UDFs | _StringStripChars UDF | _ColorIsDarkShade UDF | _ColorConvertValue UDF | _GUICtrlTab_CoverBackground | CUI_App_UDF | _IncludeScripts UDF | _AutoIt3ExecuteCode | _DragList UDF | Mod. for _ListView_Progress | _ListView_SysLink | _GenerateRandomNumbers | _BlockInputEx | _IsPressedEx | OnAutoItExit Handler | _GUICtrlCreateTFLabel UDF | WinControlSetEvent UDF | Mod. for _DirGetSizeEx UDF
 
AutoIt_Icon_small.pngExamples: 
ScreenSaver Demo - Matrix included | Gui Drag Without pause the script | _WinAttach()! | Turn Off/On Monitor | ComboBox Handler Example | Mod. for "Thinking Box" | Cool "About" Box | TasksBar Imitation Demo

Like the Projects/UDFs/Examples? Please rate the topic (up-right corner of the post header: Rating AutoIt_Rating.gif)

* === My topics === *

==================================================
My_Userbar.gif
==================================================

 

 

 

AutoIt is simple, subtle, elegant. © AutoIt Team

Share this post


Link to post
Share on other sites

#5 ·  Posted (edited)

That solved it, thanks guys!

So "For $k = $countlines to 25 Step -1" says to erase line 25 until the number of $countlines has been reached?

Edited by Teldin

Share this post


Link to post
Share on other sites

That solved it, thanks guys!

So "For $k = $countlines to 25 Step -1" says to erase line 25 until the number of $countlines has been reached?

not exactly, but close. What it's doing is working backwards (hence the step -1). You start at the last line (in this case $countlines) and count down until $k is less than 25

Share this post


Link to post
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
Sign in to follow this  
Followers 0