I'm not sure if this belongs here, but anyway.

There are many ways to code. Some better than others. For example, a recent project of mine has to communicate and interact across the Workspace with other scripts constantly. The problem? I had two options. One would be to use multiple scripts for the same amount of work, creating more lag. My choice? I used coroutines, consecutive functions, and 'shared' to store data. My choice was much less laggy and kept the Workspace looking usable. But on to improving your scripts.
-------------------------------------------------------------------------------------------------------


1. Instead of while true do wait(), I always use while wait() do. This saves you a decent amount of space eventually if your script has many while-do statements.

2. Pretend we were trying to insert a message.
When using Instance.new, never do this except for readablity:

Code:

obj = Instance.new("Message")
obj.Parent = Workspace
obj.Text = "Test"

A way to condense this into a small amount is to take advantage of an arguement some people ignore.

Code:

Instance.new("Message",Workspace).Text = "Test"

3. Take advantage of coroutines to prevent wasted script space. For example, instead of having one script checking for newly inserted parts, and another changing the time, we can combine them

Code:

coroutine.resume(coroutine.create(function ()
while wait() do
--time changer
end
end))
function function1(a)
if a.className == "Part" then
--blah
end end
game.Workspace.ChildAdded:connect(function1)

4. Remove whitespace and comments. The script has to still read whitespace and comments, so it results in wasted space and script time. If you use these as organization, remove them once your done.

5. Local vairables are called much faster than non-local. If you only need a variable for a small amount of code, just make it local to that function.

Code:

d = "hi"
function hi()
print(d)
end
--instead of that ^ do this
function hi()
local d = "hi"
print(d)
end

don't forget local variables, they're called much faster than global!

I'll add it in, thanks!

hey! i was the one who said that first! jk lol. no rly. i was.

I know, but I forgot.

i already knew about local variables being called faster than normal variables before you told us.

The only reason I don't use them is because it broke one of my more lengthy scripts and I somehow can't fix it now...

oh

I fixed it finally! My brain was too numbed to fix it last night.

lawl

XD

I would disagree with your comment whitewash though. What happens if someone else wants to edit your Free model script and no comments!! Major pain. Also, how much time is actually taken up reading these comments? 0.001 of a second? Plus, the Roblox admins sure didn't think comments were bad, their scripts are full of them! If the source code for the website has 'em everywhere! Overall, doing so would violate one of the many rules for good, strong, structured programming. Make it simple and be able to come back to it several months later and be able to quickly figure out how to fix it, instead of having to figure out how the script works all over again!

I was tought programming from a colledge aproach, even though I'm not even close to colledge. All the resources I read made it very clear that there had to be comments in any good code that was meant to be usable by anyone other than you.

Very true. I am probably going to start using comments again, I lost track of some of the things I made so I have to re-figure them out.

I think lua would precompile the syntax and would remove things like comments before it is run, so I doubt it would make a difference...

That, I don't know. I heard somewhere it makes it faster, but it doesn't make it easier to return to later.

lol my admin commands has a line with so many whitespaces that it reachs the limit for a line :P (I was trying to hide some code where people don't look)

Obfuscate the parts you don't want touched. Or the whole thing.

meh, I'm gonna make it free source to edit code, but I'm gonna add a delayed virus into it so it's a trial version that works for 1 week after inserting it and then it removes itself and the map! mwhahahaha, never trust my huge open source stuff >:D

Ctrl+A, find where the blue space looks suspiciously long, scroll to the end, delete.

dang X_X but most noobs wont even open the script apart from inserting their name into it >:D

That is true. But you want to hit as many people as possible.

yea, I'll just encrypt every thing except for the admin lists. (in the encrypted part of the code if will insert my name >:)

Obfuscate, don't encrypt. Encryption can be reversed and stolen, obfuscate can be only slightly reversed and is not worth the effort to steal.

wait, what the difference???? O_O