You are viewing our Forum Archives. To view or take place in current topics click here.
#11. Posted:
ProJimmyRustler
  • E3 2016
Status: Offline
Joined: Jul 14, 201410Year Member
Posts: 1,720
Reputation Power: 71
Status: Offline
Joined: Jul 14, 201410Year Member
Posts: 1,720
Reputation Power: 71
xXBeastlyCodingXx wrote
Botch wrote
ProJimmyRustler wrote So if I'm understanding your post correctly, you will have to have a new option for each prestige? Why. There is a much more efficient method that works with all stats.

[ Register or Signin to view external links. ]

This allows users to apply input through textboxes rather then have an option for everything.

I agree with AnimeHypedUp. I don't know what you mean by efficient, Convert.ToInt32() and int.Parse() both turn a string into its equivalent as a 32 bit integer. There is no reason to use both on the same textbox.


But The Outstand of using both and Making it Ungodly Long is less Crashing And Bugs
Exactly. It's like doing anything, there is a million ways to do one thing.
#12. Posted:
vSmithy
  • TTG Natural
Status: Offline
Joined: Jan 30, 201014Year Member
Posts: 935
Reputation Power: 45
Status: Offline
Joined: Jan 30, 201014Year Member
Posts: 935
Reputation Power: 45
ProJimmyRustler wrote There is a much more efficient method that works with all stats.


That's hardly efficient. You could shorten that down to a couple of lines, lol.
#13. Posted:
ProJimmyRustler
  • Christmas!
Status: Offline
Joined: Jul 14, 201410Year Member
Posts: 1,720
Reputation Power: 71
Status: Offline
Joined: Jul 14, 201410Year Member
Posts: 1,720
Reputation Power: 71
vSmithy wrote
ProJimmyRustler wrote There is a much more efficient method that works with all stats.


That's hardly efficient. You could shorten that down to a couple of lines, lol.
If you would have read the other posts you would have seen that your other "Efficient" ways cause errors and crashing.
[ Register or Signin to view external links. ]
#14. Posted:
vSmithy
  • TTG Natural
Status: Offline
Joined: Jan 30, 201014Year Member
Posts: 935
Reputation Power: 45
Status: Offline
Joined: Jan 30, 201014Year Member
Posts: 935
Reputation Power: 45
ProJimmyRustler wrote
vSmithy wrote
ProJimmyRustler wrote There is a much more efficient method that works with all stats.


That's hardly efficient. You could shorten that down to a couple of lines, lol.
If you would have read the other posts you would have seen that your other "Efficient" ways cause errors and crashing.
[ Register or Signin to view external links. ]


There's no reason why that would happen. Just make a method which converts the contents of the textbox to an integer, then pokes that integer to the offset given. Cleans the code right up, and will shorten it to maybe 4 or 5 lines if that.
#15. Posted:
UNSC
  • Gold Member
Status: Offline
Joined: Oct 19, 201311Year Member
Posts: 1,180
Reputation Power: 101
Status: Offline
Joined: Oct 19, 201311Year Member
Posts: 1,180
Reputation Power: 101
vSmithy wrote
ProJimmyRustler wrote
vSmithy wrote
ProJimmyRustler wrote There is a much more efficient method that works with all stats.


That's hardly efficient. You could shorten that down to a couple of lines, lol.
If you would have read the other posts you would have seen that your other "Efficient" ways cause errors and crashing.
[ Register or Signin to view external links. ]


There's no reason why that would happen. Just make a method which converts the contents of the textbox to an integer, then pokes that integer to the offset given. Cleans the code right up, and will shorten it to maybe 4 or 5 lines if that.


Why fix something that does not need to be fixed?

As long as the long code works I don't see a problem... haha
#16. Posted:
vSmithy
  • TTG Natural
Status: Offline
Joined: Jan 30, 201014Year Member
Posts: 935
Reputation Power: 45
Status: Offline
Joined: Jan 30, 201014Year Member
Posts: 935
Reputation Power: 45
UNSC wrote
Why fix something that does not need to be fixed?

As long as the long code works I don't see a problem... haha


Of course, but it's bad practice (it's better to get in to the habit of creating methods to call where lots of controls are doing the same process) and it's always nice to have neater code so it's easier to fix up/update in the future.

Untidy code can be a real nightmare.
#17. Posted:
ISIS
  • Resident Elite
Status: Offline
Joined: Sep 11, 201410Year Member
Posts: 268
Reputation Power: 14
Status: Offline
Joined: Sep 11, 201410Year Member
Posts: 268
Reputation Power: 14
UNSC wrote
vSmithy wrote
ProJimmyRustler wrote
vSmithy wrote
ProJimmyRustler wrote There is a much more efficient method that works with all stats.


That's hardly efficient. You could shorten that down to a couple of lines, lol.
If you would have read the other posts you would have seen that your other "Efficient" ways cause errors and crashing.
[ Register or Signin to view external links. ]


There's no reason why that would happen. Just make a method which converts the contents of the textbox to an integer, then pokes that integer to the offset given. Cleans the code right up, and will shorten it to maybe 4 or 5 lines if that.


Why fix something that does not need to be fixed?

As long as the long code works I don't see a problem... haha
This is a common issue with people. Just because it works doesn't mean that its being done/coded right.
#18. Posted:
ProJimmyRustler
  • Summer 2018
Status: Offline
Joined: Jul 14, 201410Year Member
Posts: 1,720
Reputation Power: 71
Status: Offline
Joined: Jul 14, 201410Year Member
Posts: 1,720
Reputation Power: 71
AnimeHypedUp wrote
UNSC wrote
vSmithy wrote
ProJimmyRustler wrote
vSmithy wrote
ProJimmyRustler wrote There is a much more efficient method that works with all stats.


That's hardly efficient. You could shorten that down to a couple of lines, lol.
If you would have read the other posts you would have seen that your other "Efficient" ways cause errors and crashing.
[ Register or Signin to view external links. ]


There's no reason why that would happen. Just make a method which converts the contents of the textbox to an integer, then pokes that integer to the offset given. Cleans the code right up, and will shorten it to maybe 4 or 5 lines if that.


Why fix something that does not need to be fixed?

As long as the long code works I don't see a problem... haha
This is a common issue with people. Just because it works doesn't mean that its being done/coded right.
There are many solutions to one problem. Just because its not yours doesn't make it wrong. Its a XRPC Tool for Christ sake. %90 of the people using them wouldn't be able to tell you what the code does in the first place.
#19. Posted:
xChillest
  • Christmas!
Status: Offline
Joined: Jul 30, 201212Year Member
Posts: 2,396
Reputation Power: 123
Status: Offline
Joined: Jul 30, 201212Year Member
Posts: 2,396
Reputation Power: 123
This whole thread made me laugh lmao


@Town_, This has been released for months. Just because you "found" it doesn't mean you released it.

@PJR, The code could still be shortened just a little bit, and using integers in the first place would be a lot easier.

@Anime, am heckur

@xXBeastlyCodingXx, Actually doesn't have anything to do with stability. I could shorten it a lot and still not ever have a single issue.
#20. Posted:
ISIS
  • Resident Elite
Status: Offline
Joined: Sep 11, 201410Year Member
Posts: 268
Reputation Power: 14
Status: Offline
Joined: Sep 11, 201410Year Member
Posts: 268
Reputation Power: 14
xChillest wrote This whole thread made me laugh lmao


@Town_, This has been released for months. Just because you "found" it doesn't mean you released it.

@PJR, The code could still be shortened just a little bit, and using integers in the first place would be a lot easier.

@Anime, am heckur

@xXBeastlyCodingXx, Actually doesn't have anything to do with stability. I could shorten it a lot and still not ever have a single issue.
Listen chillest










am hekur
Jump to:
You are viewing our Forum Archives. To view or take place in current topics click here.