![]() |
[Share] Useful Codes - Printable Version +- Sinisterly (https://sinister.ly) +-- Forum: Coding (https://sinister.ly/Forum-Coding) +--- Forum: Visual Basic & .NET Framework (https://sinister.ly/Forum-Visual-Basic-NET-Framework) +--- Thread: [Share] Useful Codes (/Thread-Share-Useful-Codes) |
[Share] Useful Codes - shizelkid - 02-26-2013 Please note that i didnt make this, it was made by CodeBreaker But still use full. (08-18-2010, 12:09 PM)CodeBreaker Wrote: RE: [Share] Useful Codes - cxS - 02-27-2013 By headlines: 1. "Open Something" - Don't use shell, the Process class will work just fine still for doing this. 2. "Delete a file" Code: If System.IO.File.Exists(FileToDelete) = True Then You don't need "= True" in there. That's just an extra unnecessary step in evaluating the boolean expression. 3. [color=red]"Kill a process" - This code assumes that there is a process running by the name of taskmgr.exe, at least one, and so if there is no running processes by that name, you'll get an exception for referencing index (0). Secondly, the name, you want to input without the ".exe". 4. "Delete a registry key" - Microsoft.Win32 has functionality to interact with the registry as a note here. In your other Registry "snippet" you are using Microsoft.Win32, while here you are using My.Computer from Microsoft.VisualBasic. 5. "Text To Speech" - SpeechSynthesizer is 100 times better than using old SAPI here. This is more the VB6 way of doing it... And it promotes poor code for how you create the object. 6. "Fade in" Code: For FadeIn = 0.0 To 1.1 Step 0.1 Why 1.1? Opacity is from 0.0 to 1.0. I wouldn't do it this way though either. 7. "Fade out" - Same thing as Fade in, you may want to go to 0... Although why start with Integers here and have to divide by 100 each time? In "Fade in" Double was used. 8. "Gmail bomber" - SmtpClient implements the IDisposable interface, and no Dispose() method is being called here... You could also use the Using statement. 9. "Info Grabber" - The Environment class would work here too for a few of these, and I would recommend it as well. 10. "Case Example" - You've got implicit type casting going on in this bit of code, and Rnd() should be replaced with the functionality from the Random class in my opinion. Int() is an old VB6 carry over as well... 11. "Login Example" - Should be noted here that this should not be used for a very secure application. 12. "Xfire Add Friender and status changer (Just an example)" - Could do this with webrequests unless for some reason the elements on the page have to be loaded as well for a visual? 13. "Screen Capture" - Should be disposing of a few resources here... 14. "Add Your Application To Startup" - Not very pretty code here. Bad use of Try Catch. Performance loss is to be mentioned here as a consideration for what the way this code is written. 15. "HotKeyChat Spammer" - GetAsyncKeyState is not being used in a suitable manner here. It is not event based, so the reason for why you need a timer here that calls it consistently is a sign of poor application design. 16. "CD-Rom Prank" - There are better ways to do this as well through P/Invoke Win32 functions. This is the worst way of doing it, and probably something more suited if programming in VB6. 17. "Shutdown Manager" - I'm seeing Shell() again... The Process class from System.Diagnostics should be used instead. *Note: I skipped over some. This was just a running post of what I seen as I skimmed through some of the examples. RE: [Share] Useful Codes - shizelkid - 02-27-2013 Well thank you, again it was just a share. The way it sounds, you know basic well. How about you make one so that way it is easier and better!? RE: [Share] Useful Codes - cxS - 02-27-2013 (02-27-2013, 08:35 PM)shizelkid Wrote: Well thank you, again it was just a share. The way it sounds, you know basic well. How about you make one so that way it is easier and better!? Make a tutorial? I'm not much of a tutorial writer, mostly because I just don't like writing them. I can revise the code though. RE: [Share] Useful Codes - shizelkid - 02-28-2013 That would be lovely. Thank you. RE: [Share] Useful Codes - cxS - 02-28-2013 "Open Something": Code: Process.Start("notepad.exe") Code: Process.Start("calc.exe") "Kill A Process": Code: Dim procs As Process() = Process.GetProcessesByName("taskmgr") "Text To Speech": Code: Private R As New Random() Note: - Imports System.Speech - Imports System.Collections.ObjectModel - Imports System.Speech.Synthesis ' Need to add reference manually "Fade out": *Same thing as Fade in but with Double's. "Gmail Bomber": Code: Using MyMailMessage As New MailMessage(TextBox1.Text, TextBox3.Text) "Info Grabber": There's various ways to do this without using My.Computer (and especially if you're using C#, and you don't feel like adding a reference to the Microsoft.VisualBasic dll). There's Win32 API's that can be P/Invoke'd to retrieve the information for some of these. "Screen Capture": A Bitmap object is disposable. There is a few other minor things that I would change here, but other than that it is good enough. "HotKeyChat Spammer": A keyboard hook would be much more efficient... And less resource intensive. "CD-Rom Prank": You can P/Invoke mciGetErrorString and mciSendString to achieve this. "Shutdown Manager": Same thing... Code: Process.Start("shutdown.exe", "-s") RE: [Share] Useful Codes - BreShiE - 02-28-2013 This thread teaches some very bad coding habits, as cxS has already pointed out. For example, he's telling you to use WebBrowsers instead of WebClients. It's a nice thread for the noobies who just Copypasta their code, but for someone who would like to learn, this thread is awful, haha. RE: [Share] Useful Codes - cxS - 02-28-2013 (02-28-2013, 09:10 AM)BreShiE Wrote: This thread teaches some very bad coding habits, as cxS has already pointed out. For example, he's telling you to use WebBrowsers instead of WebClients. It's a nice thread for the noobies who just Copypasta their code, but for someone who would like to learn, this thread is awful, haha. WebBrowser isn't horrible, even though it uses an old version of IE. The problem is the way people use it. For doing things on the web that don't require the visual. And for people that add a WebBrowser to use it with Visible property set to false, that's even worse. Why load all the elements and parse them out on a webpage if you don't need to take an HTTP request that far? RE: [Share] Useful Codes - BreShiE - 02-28-2013 (02-28-2013, 09:18 AM)cxS Wrote:(02-28-2013, 09:10 AM)BreShiE Wrote: This thread teaches some very bad coding habits, as cxS has already pointed out. For example, he's telling you to use WebBrowsers instead of WebClients. It's a nice thread for the noobies who just Copypasta their code, but for someone who would like to learn, this thread is awful, haha. Well, yeah that's what I meant. If you need a GUI to go along with what you're doing, then it's fine, but if you only need 1 certain string from the request, then use WebRequests. There's no point loading a whole control when you're not going to use it for its purpose. It's a waste of resources and time. |