I've done my reading and understand what a Try/Catch block does and why it's important to use one. But I'm stuck on knowing when/where to use them. Any advice? I'll post a sample of my code below in hopes that someone has some time to make some recommendations for my example.
public AMPFileEntity(string filename) { transferFileList tfl = new transferFileList(); _AMPFlag = tfl.isAMPFile(filename); _requiresPGP = tfl.pgpRequired(filename); _filename = filename.ToUpper(); _fullSourcePathAndFilename = ConfigurationSettings.AppSettings.Get("sourcePath") + _filename; _fullDestinationPathAndFilename = ConfigurationSettings.AppSettings.Get("FTPStagePath") + _filename; _hasBeenPGPdPathAndFilename = ConfigurationSettings.AppSettings.Get("originalsWhichHaveBeenPGPdPath"); } public int processFile() { StringBuilder sb = new StringBuilder(); sb.AppendLine(" "); sb.AppendLine(" --------------------------------"); sb.AppendLine(" Filename: " + _filename); sb.AppendLine(" AMPFlag: " + _AMPFlag); sb.AppendLine(" Requires PGP: " + _requiresPGP); sb.AppendLine(" --------------------------------"); sb.AppendLine(" "); string str = sb.ToString(); UtilityLogger.LogToFile(str); if (_AMPFlag) { if (_requiresPGP == true) { encryptFile(); } else { UtilityLogger.LogToFile("This file does not require encryption. Moving file to FTPStage directory."); if (File.Exists(_fullDestinationPathAndFilename)) { UtilityLogger.LogToFile(_fullDestinationPathAndFilename + " alreadyexists. Archiving that file."); if (File.Exists(_fullDestinationPathAndFilename + "_archive")) { UtilityLogger.LogToFile(_fullDestinationPathAndFilename + "_archive already exists. Overwriting it."); File.Delete(_fullDestinationPathAndFilename + "_archive"); } File.Move(_fullDestinationPathAndFilename, _fullDestinationPathAndFilename + "_archive"); } File.Move(_fullSourcePathAndFilename, _fullDestinationPathAndFilename); } } else { UtilityLogger.LogToFile("This file is not an AMP transfer file. Skipping this file."); } return (0); } private int encryptFile() { UtilityLogger.LogToFile("This file requires encryption. Starting encryption process."); // first check for an existing PGPd file in the destination dir. if exists, archive it - otherwise this one won't save. it doesn't overwrite. string pgpdFilename = _fullDestinationPathAndFilename + ".PGP"; if(File.Exists(pgpdFilename)) { UtilityLogger.LogToFile(pgpdFilename + " already exists in the FTPStage directory. Archiving that file." ); if(File.Exists(pgpdFilename + "_archive")) { UtilityLogger.LogToFile(pgpdFilename + "_archive already exists. Overwriting it."); File.Delete(pgpdFilename + "_archive"); } File.Move(pgpdFilename, pgpdFilename + "_archive"); } Process pProc = new Process(); pProc.StartInfo.FileName = "pgp.exe"; string strParams = @"--encrypt " + _fullSourcePathAndFilename + " --recipient infinata --output " + _fullDestinationPathAndFilename + ".PGP"; UtilityLogger.LogToFile("Encrypting file. Params: " + strParams); pProc.StartInfo.Arguments = strParams; pProc.StartInfo.UseShellExecute = false; pProc.StartInfo.RedirectStandardOutput = true; pProc.Start(); pProc.WaitForExit(); //now that it's been PGPd, save the orig in 'hasBeenPGPd' dir UtilityLogger.LogToFile("PGP encryption complete. Moving original unencrypted file to " + _hasBeenPGPdPathAndFilename); if(File.Exists(_hasBeenPGPdPathAndFilename + _filename + "original_which_has_been_pgpd")) { UtilityLogger.LogToFile(_hasBeenPGPdPathAndFilename + _filename + "original_which_has_been_pgpd already exists. Overwriting it."); File.Delete(_hasBeenPGPdPathAndFilename + _filename + "original_which_has_been_pgpd"); } File.Move(_fullSourcePathAndFilename, _hasBeenPGPdPathAndFilename + _filename + "original_which_has_been_pgpd"); return (0); } }
}
It is not necessary to catch all exceptions. In Java there is two types of exceptions: checked and unchecked. The rule is simple a checked exception has to be handled by the caller while an unchecked exception can be handled either by not catching it, or by catching it.
A try / catch block is basically used to handle errors in JavaScript. You use this when you don't want an error in your script to break your code.
Try/catch blocks allow a program to handle an exception gracefully in the way the programmer wants them to. For example, try/catch blocks will let a program print an error message (rather than simply crash) if it can't find an input file. Try blocks are the first part of try/catch blocks.
The basic rule of thumb for catching exceptions is to catch exceptions if and only if you have a meaningful way of handling them.
Don't catch an exception if you're only going to log the exception and throw it up the stack. It serves no meaning and clutters code.
Do catch an exception when you are expecting a failure in a specific part of your code, and if you have a fallback for it.
Of course you always have the case of checked exceptions which require you to use try/catch blocks, in which case you have no other choice. Even with a checked exception, make sure you log properly and handle as cleanly as possible.
If you love us? You can donate to us via Paypal or buy me a coffee so we can maintain and grow! Thank you!
Donate Us With