Batch bestand voor verplaatsen van bestanden

[SwooX]

Mijn vraag:
Ik ben bezig om een batch bestand te maken voor het verplaatsen van een bestand. Dit lukt mij echter kan ik niet correct een logfile aanmaken. De logfile wordt aangemaakt maar ik krijg duizenden regels van 'Bestand is verplaatst'. Dat is dan ook mijn ingestelde regel voor als het gelukt is.

Relevante software en hardware die ik gebruik:
Kladblok

Wat ik al gevonden of geprobeerd heb:
IF EXIST text.txt (
GOTO :Succeed )
ELSE (
GOTO :Failed
)

:Succeed
Echo Bestand is verplaatst.
test.bat >> C:\Users\--\Desktop\Log\Log.txt

:Failed
Echo Bestand is niet verplaatst.
test.bat >> C:\Users\--\Desktop\Log\Log.txt

Heb de username eruit geplukt voor de privacy.

[ElCondor]

Ik zie de Move opdracht helemaal niet...
Zou je het niet in PowerShell schrijven? Want daar heb je wat meer controle over wat en waar je wilt loggen.

[Mar.tin]

[verwijderd]

[ Voor 107% gewijzigd door Mar.tin op 28-07-2022 12:29 ]

[SwooX]

ElCondor schreef op vrijdag 10 februari 2017 @ 13:33:
Ik zie de Move opdracht helemaal niet...
Zou je het niet in PowerShell schrijven? Want daar heb je wat meer controle over wat en waar je wilt loggen.

De move heb ik eruit gehaald, die werkt wel. Wil echter alleen toepassen dat er een log-file komt.

Wat heb je zelf al gevonden? Volgens mij zijn er met een klein beetje googlewerk honderden, zo niet duizenden mensen te vinden die iets soortgelijks willen..

Hoeveel bestanden probeer je te verplaatsen?

Ik gebruik voor logging in .batjes meestal iets in de strekking van
code:
1
%DATE% %TIME% %COMPUTERNAME% %USERNAME% - %variabele voor bestandsnaam% Copy Success! >> log.txt

Aanvullend is een probleem dat je nadat je de succeed regel schrijft, je doorgaat naar :Failed.

Heb zelfs pagina 2 bezocht op Google

Enige waar ik wat wijzer van werd is dus wat ik heb geplaatst net.

[ Voor 45% gewijzigd door SwooX op 10-02-2017 13:36 ]

[naitsoezn]

Roep je nou de hele tijd recursief het batch-bestand zelf aan bij :Succeed of :Failed?

[SwooX]

naitsoezn schreef op vrijdag 10 februari 2017 @ 13:38:
Roep je nou de hele tijd recursief het batch-bestand zelf aan bij :Succeed of :Failed?

Betreft programmeren ben ik echt een rookie, dus heb echt geen idee wat je bedoelt.

Bedoel je dat van de output? met >>

[Dido]

naitsoezn schreef op vrijdag 10 februari 2017 @ 13:38:
Roep je nou de hele tijd recursief het batch-bestand zelf aan bij :Succeed of :Failed?

Zo te zien wel. Hij lijkt te denken dat hij daarmee alleen maar de output wegschrijft, maar hij roept inderdaad opnieuw de batch-file aan, met dezelfde output. Zo gek is het dus niet dat hij oneindig blijft doorgaan

[naitsoezn]

SwooX schreef op vrijdag 10 februari 2017 @ 13:40:
[...]

Betreft programmeren ben ik echt een rookie, dus heb echt geen idee wat je bedoelt.

Bedoel je dat van de output? met >>

Als je batch-bestand test.bat heet, dan roep je met "test.bat >> C:\Users\--\Desktop\Log\Log.txt" telkens zichzelf aan, terwijl je de output naar Log.txt stuurt. Oneindig.

[Rone]

Met je huidige manier wordt na een 'succeed' alsnog 'failed' uitgevoerd, want daar lus je niet omheen. Verder kun je een echo direct aan een log-file toevoegen, in plaats van de batch-file zelf aan te roepen. Dit werkt al beter:

Batchfile:

IF EXIST text.txt GOTO Succeed

Echo Bestand is niet verplaatst.>> Log.txt
GOTO End

:Succeed
Echo Bestand is verplaatst.>> Log.txt

:End

[robertpNL]

Netter:

code:

move /Y A.txt B.txt
if ERRORLEVEL 1 Echo Niet gelukt >> Log.txt
if ERRORLEVEL 0 Echo Gelukt >> Log.txt

[SwooX]

Rone schreef op vrijdag 10 februari 2017 @ 13:43:
Met je huidige manier wordt na een 'succeed' alsnog 'failed' uitgevoerd, want daar lus je niet omheen. Verder kun je een echo direct aan een log-file toevoegen, in plaats van de batch-file zelf aan te roepen. Dit werkt al beter:

Batchfile:

IF EXIST text.txt GOTO Succeed

Echo Bestand is niet verplaatst.>> Log.txt
GOTO End

:Succeed
Echo Bestand is verplaatst.>> Log.txt

:End

Deze doet het qua aantal regels (1) wel! Alleen nu geeft die aan dat het bestand niet is verplaatst terwijl dit wel het geval is.

[Mar.tin]

[verwijderd]

[ Voor 95% gewijzigd door Mar.tin op 28-07-2022 12:29 ]

[SwooX]

Mar.tin schreef op vrijdag 10 februari 2017 @ 13:49:
[...]


Zet eens een pause achter je code, staat er niet ergens toegang geweigerd?

In andere woorden, heb je wel rechten om uit te lezen of het bestand bestaat, ik merk zulke vage problematiek nog wel eens op hier als we een legacy procesje verhuizen

1 file(s) moved..

Press any key to continue.

[Rone]

SwooX schreef op vrijdag 10 februari 2017 @ 13:47:
[...]

Deze doet het qua aantal regels (1) wel! Alleen nu geeft die aan dat het bestand niet is verplaatst terwijl dit wel het geval is.

Dat is in principe onmogelijk. Als het bestand dat genoemd wordt bij IF EXIST bestaat zou alleen :Succeed moeten worden uitgevoerd. Heb je overal de juiste bestandsnamen en folders vermeld?

[SwooX]

Rone schreef op vrijdag 10 februari 2017 @ 13:54:
[...]

Dat is in principe onmogelijk. Als het bestand dat genoemd wordt bij IF EXIST bestaat zou alleen :Succeed moeten worden uitgevoerd. Heb je overal de juiste bestandsnamen en folders vermeld?

@ECHO OFF
move "C:\Users\--\Desktop\Test\text.txt" C:\Users\--\Desktop\Test2\text.txt

IF EXIST text.txt GOTO Succeed

Echo Bestand is niet verplaatst.>> C:\Users\--\Desktop\Log\Log.txt
GOTO End

:Succeed
Echo Bestand is verplaatst.>> C:\Users\--\Desktop\Log\Log.txt

:End

Ik zie geen probleem of ik kijk er stomweg overheen.

[Rone]

IF EXIST text.txt GOTO Succeed

Ja, dus vermeld daar eens het volledige pad naar text.txt, dus C:\Users\--\Desktop\Test2\text.txt.

Je geeft nu het move-commando eerst, dus dat vindt gewoon plaats. Daarna check je of text.txt betaat in het pad van het batch-bestand, wat níet het geval is, dus krijg je de verkeerde log-output.

[ Voor 41% gewijzigd door Rone op 10-02-2017 14:00 ]

[anboni]

Eerste stap in leren programmeren is herkennen wanneer je zelf iets moet gaan schrijven en wanneer je een bestaande tool kunt toepassen. Bestanden kopieren of verplaatsen, inclusief logging, kun je met robocopy doen. Vanaf Windows 7 volgens mij standaard beschikbaar.

code:

robocopy <bron> <doel> /mov /log+:log.txt

[Mar.tin]

[verwijderd]

[ Voor 94% gewijzigd door Mar.tin op 28-07-2022 12:29 ]

[SwooX]

Mar.tin schreef op vrijdag 10 februari 2017 @ 13:57:
[...]


Je checkt nu vanuit de locatie van het .bat bestand of test.txt in die map bestaat ipv op het pad waar hij heengeschreven is

Dit deed het hem! De IF exist gebeurd dus na het moven, snap ik nu ook wel aangezien die elke regel als een commando oppikt.

Iets met helden en cape.

Kan ik dit ook toepassen met een tweede bestand binnen de map?

Moet ik dan gewoon nogmaals de IF Exist kopieren en plakken maar dan met de naam van het 2e bestand?

[ Voor 16% gewijzigd door SwooX op 10-02-2017 14:02 ]

[Squ1zZy]

PowerShell:

<#

.SYNOPSIS
  This script will move a file if it exists

.DESCRIPTION

  This script will move a given file if it exists and logs the filename if it's being moved

.PARAMETER

  None

.INPUTS

  None

.OUTPUTS

  Log file stored in C:\Windows\Temp\

.NOTES

  Version: 0.1
  Author: SwooX
  Creation Date: 02/10/2017
  Purpose/Change: v0.1 - Initial script development

.EXAMPLE

  .\<scriptName>.ps1

#>

#------------------------------------------[Initialisations]---------------------------------------

# Set Error Action to Silently Continue
# $ErrorActionPreference = "SilentlyContinue"

#-------------------------------------------[Declarations]-----------------------------------------

# Script Name and Version
$scriptName = "SwooX Move Files Script"
$scriptVersion = "0.1"

# Log File Info
$date = Get-Date -format MM-dd-yyyy.HH_mm_ss
$logPath = "C:\Windows\Temp"
$logName = "$scriptName.$date.log"
$logFile = Join-Path -Path $logPath -ChildPath $logName

# Variables that can be set if needed
$fileName = "C:\SwooX.txt"

#--------------------------------------------[Functions]-------------------------------------------

Function SwooX-moveFiles {
  Param()

  Begin {
    Add-Content $logFile "Moving files..."
  }

  Process {
    Try {
      If (Test-Path $fileName) {
        Add-Content $logFile "Moving file $fileName..."
      }
      Else {
        Add-Content $logFile "File $fileName doesn't exist..."
      }
    }

    Catch {
      Write-Host "Failed - Moving files..." -ForegroundColor Red
      Add-Content $logFile $_.Exception
    }
  }

  End {
    If($?) {
      Add-Content $logFile "Moving files completed successfully..."
      Add-Content $logFile " "
    }
  }
}

#--------------------------------------------[Execution]-------------------------------------------

Add-Content $logFile "$scriptName version $scriptVersion started at $(Get-Date) by $env:UserName"
Add-Content $logFile " "

SwooX-moveFiles

Add-Content $logFile "$scriptName version $scriptVersion ended at $(Get-Date) by $env:UserName"

[Will_M]

Kun je niet beter een Do While loop maken?