Java-Gaming.org    
Featured games (78)
games approved by the League of Dukes
Games in Showcase (429)
Games in Android Showcase (89)
games submitted by our members
Games in WIP (467)
games currently in development
News: Read the Java Gaming Resources, or peek at the official Java tutorials
 
    Home     Help   Search   Login   Register   
Pages: [1]
  ignore  |  Print  
  what is faster?  (Read 1626 times)
0 Members and 1 Guest are viewing this topic.
Offline Rakiayn

Senior Newbie





« Posted 2013-11-21 23:27:37 »

I have a class that I use to do certain calculations.
it has some methods thatare called by other classes to do stuff like pick a random letter.
now my question is how can I make this as fast as posibble.

lets call this object "calc"

should I create a local instance of this "calc" object every time I want to use one of its methods?

or should I create this "calc" once in every object that uses calc and then use that link to call a method?
Offline nerb
« Reply #1 - Posted 2013-11-21 23:49:32 »

should I create a local instance of this "calc" object every time I want to use one of its methods?

No, I wouldn't. Otherwise you're going to end up with lots of 'calc' objects being created, abandoned and GC'd. It's generally a good idea to avoid creating new objects where possible; particularly if you are doing it in frequently called methods or loops.

or should I create this "calc" once in every object that uses calc and then use that link to call a method?

Possibly. But assuming 'calc' makes calculations, you could just make them static methods. Then you can call Calc.whatever() without instantiating a Calc object.
Online RobinB

JGO Knight


Medals: 37
Projects: 1
Exp: 3 years


Spacegame in progress


« Reply #2 - Posted 2013-11-21 23:50:09 »

Well what seems more logical?
The answer is kind of obvious i think.

*For this problem static is very much suited.
Games published by our own members! Check 'em out!
Legends of Yore - The Casual Retro Roguelike
Offline Rakiayn

Senior Newbie





« Reply #3 - Posted 2013-11-21 23:58:13 »

at the moment I have every object that uses "calc" create one instance of it and use that object.
is it worth it to change this to the static method?

here is the code of the class btw:
1  
2  
3  
4  
5  
6  
7  
8  
9  
10  
11  
12  
13  
14  
15  
16  
17  
18  
19  
20  
21  
22  
23  
24  
25  
26  
27  
28  
29  
30  
31  
32  
33  
34  
35  
36  
37  
38  
39  
40  
41  
42  
43  
44  
45  
46  
47  
48  
49  
50  
51  
52  
53  
54  
55  
56  
57  
58  
59  
60  
61  
62  
63  
64  
65  
66  
67  
68  
69  
70  
71  
72  
73  
74  
75  
76  
77  
78  
79  
80  
81  
82  
83  
84  
85  
86  
87  
88  
89  
90  
91  
92  
93  
94  
95  
96  
97  
98  
99  
100  
101  
102  
103  
104  
105  
106  
107  
108  
109  
110  
111  
112  
113  
114  
115  
116  
117  
118  
119  
120  
121  
122  
123  
124  
125  
126  
127  
128  
129  
130  
131  
132  
133  
134  
135  
136  
137  
138  
139  
140  
141  
142  
143  
144  
145  
146  
147  
148  
149  
150  
151  
152  
153  
154  
155  
156  
157  
158  
159  
160  
161  
162  
163  
164  
165  
166  
167  
168  
169  
170  
171  
172  
173  
174  
175  
176  
177  
178  
179  
180  
181  
182  
183  
184  
185  
186  
187  
188  
189  
190  
191  
192  
193  
194  
195  
196  
197  
198  
199  
200  
201  
202  
203  
204  
205  
206  
207  
208  
209  
210  
211  
212  
213  
214  
215  
216  
217  
218  
219  
220  
221  
222  
223  
224  
225  
226  
227  
228  
229  
230  
231  
232  
233  
234  
235  
236  
237  
238  
239  
240  
241  
242  
243  
244  
245  
246  
247  
248  
249  
250  
251  
252  
253  
254  
255  
256  
257  
258  
259  
260  
261  
262  
263  
264  
265  
266  
267  
268  
269  
270  
271  
272  
273  
274  
275  
276  
277  
278  
279  
280  
281  
282  
283  
284  
285  
286  
287  
288  
289  
290  
291  
292  
293  
294  
295  
296  
297  
298  
299  
300  
301  
302  
303  
304  
305  
306  
307  
308  
309  
310  
311  
312  
313  
314  
315  
316  
317  
318  
319  
320  
321  
322  
323  
324  
325  
326  
327  
328  
329  
330  
331  
332  
333  
334  
335  
336  
337  
338  
339  
340  
341  
342  
343  
344  
345  
346  
347  
348  
349  
350  
351  
352  
353  
354  
355  
356  
357  
358  
359  
360  
361  
362  
363  
364  
365  
366  
367  
368  
369  
370  
371  
372  
373  
374  
375  
376  
377  
378  
379  
380  
381  
382  
383  
384  
385  
386  
387  
388  
389  
390  
391  
392  
393  
394  
395  
396  
397  
398  
399  
400  
401  
402  
403  
404  
405  
406  
407  
408  
409  
410  
411  
412  
413  
414  
415  
416  
417  
418  
419  
420  
421  
422  
423  
424  
425  
426  
427  
428  
429  
430  
431  
432  
433  
434  
435  
436  
437  
438  
439  
440  
441  
442  
443  
444  
445  
446  
447  
448  
449  
450  
451  
452  
453  
454  
455  
456  
457  
458  
459  
460  
461  
462  
463  
464  
465  
466  
467  
468  
469  
470  
471  
472  
473  
474  
475  
476  
477  
478  
479  
480  
481  
482  
483  
484  
485  
486  
487  
488  
489  
490  
491  
492  
493  
494  
495  
496  
497  
498  
499  
500  
501  
502  
503  
504  
505  
506  
507  
508  
509  
510  
511  
512  
513  
514  
515  
516  
517  
518  
519  
520  
521  
522  
523  
524  
525  
526  
527  
528  
529  
530  
531  
532  
533  
534  
535  
536  
537  
538  
539  
540  
541  
542  
543  
544  
545  
546  
547  
548  
549  
550  
551  
552  
553  
554  
555  
556  
557  
558  
559  
560  
561  
562  
563  
564  
565  
566  
567  
568  
569  
570  
571  
572  
573  
574  
575  
576  
577  
578  
579  
580  
581  
582  
583  
584  
585  
586  
587  
588  
589  
590  
591  
592  
593  
594  
595  
596  
597  
598  
599  
600  
601  
602  
603  
604  
605  
606  
607  
608  
609  
610  
611  
612  
613  
614  
615  
616  
617  
618  
619  
620  
621  
622  
623  
624  
625  
626  
627  
628  
629  
630  
631  
632  
633  
634  
635  
636  
637  
638  
639  
public class Physics extends ROGameObject
{
    public Random r = new Random();  
   
   
    @Override
    public void update()
    {
       
    }

    //check if two objcts collide
   public boolean checkCollisions(ROGameObject go1, ROGameObject go2)
    {
        Rectangle r1 = new Rectangle((int)go1.getx(),(int)go1.gety(),(int)go1.getxsize(),(int)go1.getysize());
        Rectangle r2 = new Rectangle((int)go2.getx(),(int)go2.gety(),(int)go2.getxsize(),(int)go2.getysize());
       
        return r1.intersects(r2);
    }  
   
    public int removenegative(int value)
    {
        if (value < 0)
        {
            return (value*-1);
        }
        return value;
             
    }
   
   
   
    //function checks if a value is between minimum and maximum value
   public float floatbetweenfunction(float minvalue, float midvalue, float maxvalue)
    {      
        if (minvalue < midvalue && midvalue < maxvalue)
        {
            return (midvalue);
        }
        if (midvalue <= minvalue )          
        {    
            return (minvalue);
        }
        if (midvalue >= maxvalue )          
        {    
            return (maxvalue);
        }
       
        return (midvalue);
    }
   
    //function checks if a value is between minimum and maximum value
   public int intbetweenfunction(int minvalue, int midvalue, int maxvalue)
    {      
        if (minvalue < midvalue && midvalue < maxvalue)
        {
            return (midvalue);
        }
        if (midvalue <= minvalue )          
        {    
            return (minvalue);
        }
        if (midvalue >= maxvalue )          
        {    
            return (maxvalue);
        }
       
        return (midvalue);
    }    
   
    //returns a random number between minvalue and maxvalue
   public int RandomInt(int minvalue, int maxvalue)
    {          
        return (minvalue + (int)(Math.random() * ((maxvalue - minvalue) + 1)));  
    }
   
    public String Randombit()
    {
        float rand = r.nextFloat();
        if (rand < 0.5)
        {
            return("0");
        }
        else
        {
            return("1");
        }
    }
    public String Randomswitch()
    {
        float rand = r.nextFloat();
        if (rand < 0.5)
        {
            return("!");
        }
        else
        {
            return("?");
        }
    }    
   
   
    public char flipbit(char bit)
    {

        if (bit == '1')
        {
            return('0');
        }
        else if (bit == '0')
        {
            return('1');
        }
        return (bit);
    }    
    public String flipbitstring(String newstring)
    {
     
        String string1 = "";
        String string2 = "";
        String string3 = "";
       
        int newrand = RandomInt(0,newstring.length()-1);
        for (int i = 0; i < newstring.length(); i++)
        {
            if (i < newrand)
            {
                string1+= newstring.charAt(i);
            }
            else if (i == newrand)
            {
                string2 += flipbit(newstring.charAt(i));
            }
            else if (i > newrand)
            {
                string3+= newstring.charAt(i);
            }
        }
        return(string1+string2+string3);
    }
   
    //returns the biggest value of two values
   public int MaxInt(int value1, int value2)
    {    
        if (value1 > value2)  
        {
            return value1;
        }
        else
            return value2;
    }    
   
    public static double PytagorasInt(int value1, int value2)
    {    
        double nvalue1 = (double)value1;
        double nvalue2 = (double)value2;        
        double int1 = (nvalue1*nvalue1) + (nvalue2*nvalue2);
       
        return Math.sqrt(int1);
   
    }  
   
    public int returnarray(int currentarray,int newarray)
    {
        return (intbetweenfunction(0,currentarray+newarray,game.amountofgrids-1));          
    }
   
    public int inversedaxis(int position)
    {
        return (game.subgridsize - position - 1);
    }    
   
   
    public int plusormin()
    {
        float newchance = r.nextFloat();
        if (newchance <= 0.5)
        {
            return (-1);
        }
        return(1);    
    }
   

    public boolean Randombool()
    {  
        return Math.random() < 0.5;
    }    
 
    public char inttoletter(int newint)
    {    
        if (newint == 0)    {   return('A'); }
        else if (newint == 1)    {   return('B'); }
        else if (newint == 2)    {   return('C'); }        
        else if (newint == 3)    {   return('D'); }    
        else if (newint == 4)    {   return('E'); }        
        else if (newint == 5)    {   return('F'); }            
        else if (newint == 6)    {   return('G'); }        
        else if (newint == 7)    {   return('H'); }    
        else if (newint == 8)    {   return('I'); }        
        else if (newint == 9)    {   return('J'); }          
        else if (newint == 10)    {   return('K'); }        
        else if (newint == 11)    {   return('L'); }    
        else if (newint == 12)    {   return('M'); }        
        else if (newint == 13)    {   return('N'); }            
        else if (newint == 14)    {   return('O'); }        
        else if (newint == 15)    {   return('P'); }    
        else if (newint == 16)    {   return('Q'); }        
        else if (newint == 17)    {   return('R'); }          
        else if (newint == 18)    {   return('S'); }        
        else if (newint == 19)    {   return('T'); }    
        else if (newint == 20)    {   return('U'); }        
        else if (newint == 21)    {   return('V'); }            
        else if (newint == 22)    {   return('W'); }        
        else if (newint == 23)    {   return('X'); }    
        else if (newint == 24)    {   return('Y'); }        
        else if (newint == 25)    {   return('Z'); }
       
        else if (newint == 26)    {   return('a'); }    
        else if (newint == 27)    {   return('b'); }      
        else if (newint == 28)    {   return('c'); }            
        else if (newint == 29)    {   return('d'); }            
        else if (newint == 30)    {   return('e'); }            
        else if (newint == 31)    {   return('f'); }            
        else if (newint == 32)    {   return('g'); }  
        else if (newint == 33)    {   return('h'); }  
       
        return('A');
       
    }
    public float lettertofloat(char letter)
    {
        if (letter=='A')    {   return(0f/33f);  }
        else if (letter=='B')    {   return(1f/33f);  }    
        else if (letter=='C')    {   return(2f/33f);  }  
        else if (letter=='D')    {   return(3f/33f);  }
        else if (letter=='E')    {   return(4f/33f);  }
        else if (letter=='F')    {   return(5f/33f);  }
        else if (letter=='G')    {   return(6f/33f);  }    
        else if (letter=='H')    {   return(7f/33f);  }    
        else if (letter=='I')    {   return(8f/33f);  }  
        else if (letter=='J')    {   return(9f/33f);  }
        else if (letter=='K')    {   return(10f/33f);  }
        else if (letter=='L')    {   return(11f/33f);  }
        else if (letter=='M')    {   return(12f/33f);  }      
        else if (letter=='N')    {   return(13f/33f);  }
        else if (letter=='O')    {   return(14f/33f);  }
        else if (letter=='P')    {   return(15f/33f);  }        
        else if (letter=='Q')    {   return(16f/33f);  }
        else if (letter=='R')    {   return(17f/33f);  }
        else if (letter=='S')    {   return(18f/33f);  }      
        else if (letter=='T')    {   return(19f/33f);  }
        else if (letter=='U')    {   return(20f/33f);  }
        else if (letter=='V')    {   return(21f/33f);  }              
        else if (letter=='W')    {   return(22f/33f);  }        
        else if (letter=='X')    {   return(23f/33f);  }
        else if (letter=='Y')    {   return(24f/33f);  }
        else if (letter=='Z')    {   return(25f/33f);  }  
       
        else if (letter=='a')    {   return(26f/33f);  }    
        else if (letter=='b')    {   return(27f/33f);  }            
        else if (letter=='c')    {   return(28f/33f);  }            
        else if (letter=='d')    {   return(29f/33f);  }            
        else if (letter=='e')    {   return(30f/33f);  }  
        else if (letter=='f')    {   return(31f/33f);  }            
        else if (letter=='g')    {   return(32f/33f);  }      
        else if (letter=='h')    {   return(33f/33f);  }    
       
        return(0);                
    }    
   
    //number between -1 and 1
   public float lettertofloat2(char letter,float tresholdfactor)
    {
        if (letter=='A')    {   return(tresholdfactor*(-1f+(0f/33f)));  }
        else if (letter=='B')    {   return(tresholdfactor*(-1f+(2f/33f)));  }    
        else if (letter=='C')    {   return(tresholdfactor*(-1f+(4f/33f)));  }  
        else if (letter=='D')    {   return(tresholdfactor*(-1f+(6f/33f)));  }
        else if (letter=='E')    {   return(tresholdfactor*(-1f+(8f/33f)));  }
        else if (letter=='F')    {   return(tresholdfactor*(-1f+(10f/33f)));  }
        else if (letter=='G')    {   return(tresholdfactor*(-1f+(12f/33f)));  }    
        else if (letter=='H')    {   return(tresholdfactor*(-1f+(14f/33f)));  }    
        else if (letter=='I')    {   return(tresholdfactor*(-1f+(16f/33f)));  }  
        else if (letter=='J')    {   return(tresholdfactor*(-1f+(18f/33f)));  }
        else if (letter=='K')    {   return(tresholdfactor*(-1f+(20f/33f)));  }
        else if (letter=='L')    {   return(tresholdfactor*(-1f+(22f/33f)));  }
        else if (letter=='M')    {   return(tresholdfactor*(-1f+(24f/33f)));  }      
        else if (letter=='N')    {   return(tresholdfactor*(-1f+(26f/33f)));  }
        else if (letter=='O')    {   return(tresholdfactor*(-1f+(28f/33f)));  }
        else if (letter=='P')    {   return(tresholdfactor*(-1f+(30f/33f)));  }        
        else if (letter=='Q')    {   return(tresholdfactor*(-1f+(32f/33f)));  }
        else if (letter=='R')    {   return(tresholdfactor*(-1f+(34f/33f)));  }
        else if (letter=='S')    {   return(tresholdfactor*(-1f+(36f/33f)));  }      
        else if (letter=='T')    {   return(tresholdfactor*(-1f+(38f/33f)));  }
        else if (letter=='U')    {   return(tresholdfactor*(-1f+(40f/33f)));  }
        else if (letter=='V')    {   return(tresholdfactor*(-1f+(42f/33f)));  }              
        else if (letter=='W')    {   return(tresholdfactor*(-1f+(44f/33f)));  }        
        else if (letter=='X')    {   return(tresholdfactor*(-1f+(46f/33f)));  }
        else if (letter=='Y')    {   return(tresholdfactor*(-1f+(48f/33f)));  }
        else if (letter=='Z')    {   return(tresholdfactor*(-1f+(50f/33f)));  }    
       
        else if (letter=='a')    {   return(tresholdfactor*(-1f+(52f/33f)));  }            
        else if (letter=='b')    {   return(tresholdfactor*(-1f+(54f/33f)));  }  
        else if (letter=='c')    {   return(tresholdfactor*(-1f+(56f/33f)));  }  
        else if (letter=='d')    {   return(tresholdfactor*(-1f+(58f/33f)));  }  
        else if (letter=='e')    {   return(tresholdfactor*(-1f+(60f/33f)));  }
        else if (letter=='f')    {   return(tresholdfactor*(-1f+(62f/33f)));  }
        else if (letter=='g')    {   return(tresholdfactor*(-1f+(64f/33f)));  }          
        else if (letter=='h')    {   return(tresholdfactor*(-1f+(66f/33f)));  }    
       
        return(0);                
    }        
   
    public int lettertoint(char letter)
    {
        if (letter=='A')    {   return(0);  }
        else if (letter=='B')    {   return(1);  }    
        else if (letter=='C')    {   return(2);  }  
        else if (letter=='D')    {   return(3);  }
        else if (letter=='E')    {   return(4);  }
        else if (letter=='F')    {   return(5);  }
        else if (letter=='G')    {   return(6);  }    
        else if (letter=='H')    {   return(7);  }    
        else if (letter=='I')    {   return(8);  }  
        else if (letter=='J')    {   return(9);  }
        else if (letter=='K')    {   return(10);  }
        else if (letter=='L')    {   return(11);  }
        else if (letter=='M')    {   return(12);  }      
        else if (letter=='N')    {   return(13);  }
        else if (letter=='O')    {   return(14);  }
        else if (letter=='P')    {   return(15);  }        
        else if (letter=='Q')    {   return(16);  }
        else if (letter=='R')    {   return(17);  }
        else if (letter=='S')    {   return(18);  }      
        else if (letter=='T')    {   return(19);  }
        else if (letter=='U')    {   return(20);  }
        else if (letter=='V')    {   return(21);  }              
        else if (letter=='W')    {   return(22);  }        
        else if (letter=='X')    {   return(23);  }
        else if (letter=='Y')    {   return(24);  }
        else if (letter=='Z')    {   return(25);  }  
       
        else if (letter=='a')    {   return(26);  }          
        else if (letter=='b')    {   return(27);  }    
        else if (letter=='c')    {   return(28);  }          
        else if (letter=='d')    {   return(29);  }    
        else if (letter=='e')    {   return(30);  }          
        else if (letter=='f')    {   return(31);  }            
        else if (letter=='g')    {   return(32);  }    
        else if (letter=='h')    {   return(33);  }    
       
        return(0);                
    }    
   
   
    public char letterup(char letter)
    {
        if (letter=='A')    {   return('B');  }
        else if (letter=='B')    {   return('C');  }    
        else if (letter=='C')    {   return('D');  }    
        else if (letter=='D')    {   return('E');  }  
        else if (letter=='E')    {   return('F');  }  
        else if (letter=='F')    {   return('G');  }  
        else if (letter=='G')    {   return('H');  }    
        else if (letter=='H')    {   return('I');  }  
        else if (letter=='I')    {   return('J');  }  
        else if (letter=='J')    {   return('K');  }  
        else if (letter=='K')    {   return('L');  }    
        else if (letter=='L')    {   return('M');  }  
        else if (letter=='M')    {   return('N');  }          
        else if (letter=='N')    {   return('O');  }  
        else if (letter=='O')    {   return('P');  }  
        else if (letter=='P')    {   return('Q');  }      
        else if (letter=='Q')    {   return('R');  }  
        else if (letter=='R')    {   return('S');  }    
        else if (letter=='S')    {   return('T');  }          
        else if (letter=='T')    {   return('U');  }  
        else if (letter=='U')    {   return('V');  }    
        else if (letter=='V')    {   return('W');  }                
        else if (letter=='W')    {   return('X');  }            
        else if (letter=='X')    {   return('Y');  }  
        else if (letter=='Y')    {   return('Z');  }  
        else if (letter=='Z')    {   return('a');  }  
       
        else if (letter=='a')    {   return('b');  }  
        else if (letter=='b')    {   return('c');  }          
        else if (letter=='c')    {   return('d');  }      
        else if (letter=='d')    {   return('e');  }    
        else if (letter=='e')    {   return('f');  }    
        else if (letter=='f')    {   return('g');  }  
        else if (letter=='g')    {   return('h');  }      
        else if (letter=='h')    {   return('h');  }  
       
        else
        {
            return(letter);
        }                  
    }
   
    public char letterdown(char letter)
    {
        if (letter=='A')    {   return('A');  }
        else if (letter=='B')    {   return('A');  }    
        else if (letter=='C')    {   return('B');  }    
        else if (letter=='D')    {   return('C');  }  
        else if (letter=='E')    {   return('D');  }  
        else if (letter=='F')    {   return('E');  }  
        else if (letter=='G')    {   return('F');  }    
        else if (letter=='H')    {   return('G');  }  
        else if (letter=='I')    {   return('H');  }  
        else if (letter=='J')    {   return('I');  }  
        else if (letter=='K')    {   return('J');  }    
        else if (letter=='L')    {   return('K');  }  
        else if (letter=='M')    {   return('L');  }          
        else if (letter=='N')    {   return('M');  }  
        else if (letter=='O')    {   return('N');  }  
        else if (letter=='P')    {   return('O');  }      
        else if (letter=='Q')    {   return('P');  }  
        else if (letter=='R')    {   return('Q');  }    
        else if (letter=='S')    {   return('R');  }          
        else if (letter=='T')    {   return('S');  }  
        else if (letter=='U')    {   return('T');  }    
        else if (letter=='V')    {   return('U');  }                
        else if (letter=='W')    {   return('V');  }            
        else if (letter=='X')    {   return('W');  }  
        else if (letter=='Y')    {   return('X');  }  
        else if (letter=='Z')    {   return('Y');  }
       
        else if (letter=='a')    {   return('Z');  }  
        else if (letter=='b')    {   return('a');  }          
        else if (letter=='c')    {   return('b');  }      
        else if (letter=='d')    {   return('c');  }    
        else if (letter=='e')    {   return('d');  }    
        else if (letter=='f')    {   return('e');  }  
        else if (letter=='g')    {   return('f');  }      
        else if (letter=='h')    {   return('g');  }          
        else
        {
            return(letter);
        }    
    }
   
    public char letterrandom(char letter)
    {
        Boolean letterisletter=false;
       
        int rand = RandomInt(0, 33);
       
             if (letter=='A')    {   letterisletter=true;  }
        else if (letter=='B')    {   letterisletter=true;  }    
        else if (letter=='C')    {   letterisletter=true;  }    
        else if (letter=='D')    {   letterisletter=true;  }  
        else if (letter=='E')    {   letterisletter=true;  }  
        else if (letter=='F')    {   letterisletter=true;  }  
        else if (letter=='G')    {   letterisletter=true;  }    
        else if (letter=='H')    {   letterisletter=true;  }  
        else if (letter=='I')    {   letterisletter=true;  }  
        else if (letter=='J')    {   letterisletter=true;  }  
        else if (letter=='K')    {   letterisletter=true;  }    
        else if (letter=='L')    {   letterisletter=true;  }  
        else if (letter=='M')    {   letterisletter=true;  }          
        else if (letter=='N')    {   letterisletter=true;  }  
        else if (letter=='O')    {   letterisletter=true;  }  
        else if (letter=='P')    {   letterisletter=true;  }      
        else if (letter=='Q')    {   letterisletter=true;  }  
        else if (letter=='R')    {   letterisletter=true;  }    
        else if (letter=='S')    {   letterisletter=true;  }          
        else if (letter=='T')    {   letterisletter=true;  }  
        else if (letter=='U')    {   letterisletter=true;  }    
        else if (letter=='V')    {   letterisletter=true;  }                
        else if (letter=='W')    {   letterisletter=true;  }            
        else if (letter=='X')    {   letterisletter=true;  }  
        else if (letter=='Y')    {   letterisletter=true;  }  
        else if (letter=='Z')    {   letterisletter=true;  }
       
        else if (letter=='a')    {   letterisletter=true;  }  
        else if (letter=='b')    {   letterisletter=true;  }          
        else if (letter=='c')    {   letterisletter=true;  }      
        else if (letter=='d')    {   letterisletter=true;  }    
        else if (letter=='e')    {   letterisletter=true;  }    
        else if (letter=='f')    {   letterisletter=true;  }  
        else if (letter=='g')    {   letterisletter=true;  }    
        else if (letter=='h')    {   letterisletter=true;  }  
             
        if (letterisletter==true)    
        {
            if (rand==0)    {return('A');   }
            if (rand==1)    {return('B');   }                
            if (rand==2)    {return('C');   }  
            if (rand==3)    {return('D');   }        
            if (rand==4)    {return('E');   }        
            if (rand==5)    {return('F');   }        
            if (rand==6)    {return('G');   }              
            if (rand==7)    {return('H');   }          
            if (rand==8)    {return('I');   }          
            if (rand==9)    {return('J');   }          
            if (rand==10)    {return('K');   }          
            if (rand==11)    {return('L');   }          
            if (rand==12)    {return('M');   }  
            if (rand==13)    {return('N');   }          
            if (rand==14)    {return('O');   }          
            if (rand==15)    {return('P');   }          
            if (rand==16)    {return('Q');   }          
            if (rand==17)    {return('R');   }  
            if (rand==18)    {return('S');   }          
            if (rand==19)    {return('T');   }          
            if (rand==20)    {return('U');   }
            if (rand==21)    {return('V');   }  
            if (rand==22)    {return('W');   }  
            if (rand==23)    {return('X');   }          
            if (rand==24)    {return('Y');   }  
            if (rand==25)    {return('Z');   }  

            if (rand==26)    {return('a');   }  
            if (rand==27)    {return('b');   }          
            if (rand==28)    {return('c');   }          
            if (rand==29)    {return('d');   }          
            if (rand==30)    {return('e');   }          
            if (rand==31)    {return('f');   }  
            if (rand==32)    {return('g');   }    
            if (rand==33)    {return('h');   }      
        }    
        return(letter);
    }    
   
    public char letterinvert(char letter)
    {
        if (letter=='A')    {   return('h');  }
        else if (letter=='B')    {   return('g');  }    
        else if (letter=='C')    {   return('f');  }    
        else if (letter=='D')    {   return('e');  }  
        else if (letter=='E')    {   return('d');  }  
        else if (letter=='F')    {   return('c');  }  
        else if (letter=='G')    {   return('b');  }    
        else if (letter=='H')    {   return('a');  }  
        else if (letter=='I')    {   return('Z');  }  
        else if (letter=='J')    {   return('Y');  }  
        else if (letter=='K')    {   return('X');  }    
        else if (letter=='L')    {   return('W');  }  
        else if (letter=='M')    {   return('V');  }          
        else if (letter=='N')    {   return('U');  }  
        else if (letter=='O')    {   return('T');  }  
        else if (letter=='P')    {   return('S');  }      
        else if (letter=='Q')    {   return('R');  }  
        else if (letter=='R')    {   return('Q');  }    
        else if (letter=='S')    {   return('P');  }          
        else if (letter=='T')    {   return('O');  }  
        else if (letter=='U')    {   return('N');  }    
        else if (letter=='V')    {   return('M');  }                
        else if (letter=='W')    {   return('L');  }            
        else if (letter=='X')    {   return('K');  }  
        else if (letter=='Y')    {   return('J');  }  
        else if (letter=='Z')    {   return('I');  }  
       
        else if (letter=='a')    {   return('H');  }  
        else if (letter=='b')    {   return('G');  }          
        else if (letter=='c')    {   return('F');  }      
        else if (letter=='d')    {   return('E');  }    
        else if (letter=='e')    {   return('D');  }    
        else if (letter=='f')    {   return('C');  }    
        else if (letter=='g')    {   return('B');  }    
        else if (letter=='h')    {   return('A');  }            
        else
        {
            return(letter);
        }                  
    }    
   
    public String inverseletter(String prestring,int lettertoinverse)
    {
        String newstring = "";
   
        for (int i = 0; i < prestring.length(); i ++)
        {
            if (i == lettertoinverse)
            {
                newstring += letterinvert(prestring.charAt(i));
            }
            else
            {
                newstring += prestring.charAt(i);
            }
        }
        return(newstring);
    }
   
     
   
    public String insertsubstring(String originalstring,String insertstring,int indexstart,int indexend)
    {

        String substring1 ="";
        String substring2 ="";
        String substring3= "";
       
        int originalstringlength = originalstring.length();
        int insertstringlength = insertstring.length();
       
        for(int j = 0; j<(insertstringlength) ;j++)
        {        
            substring2 += insertstring.charAt(j);
        }    
        for(int i = 0; i<(originalstringlength) ;i++)
        {          
            if (i < indexstart)
            {
                substring1 +=originalstring.charAt(i);
            }
            else if ( i >= indexend)
            {
                substring3 +=originalstring.charAt(i);
            }
        }      
                     
        return(substring1+substring2+substring3);
    }
   
    public String removesubstring(String originalstring,String insertstring,int indexstart,int indexend)
    {

        String substring1 ="";
        String substring3= "";
       
        int originalstringlength = originalstring.length();
         
        for(int i = 0; i<(originalstringlength) ;i++)
        {          
            if (i < indexstart)
            {
                substring1 +=originalstring.charAt(i);
            }
            else if ( i > indexend)
            {
                substring3 +=originalstring.charAt(i);
            }
        }              
        return(substring1+substring3);
    }    
}
Offline nerb
« Reply #4 - Posted 2013-11-22 00:08:35 »

If it's already heavily used in this way throughout your code, then perhaps you could leave it??? In theory you're not going to gain blistering increases in performance by changing it. But for your future reference keep in mind that utility classes like this are very suited to being static (take java.lang.Math as an example).

*Just to add: It looks like you've made a utility class that will be re-used in other programs? In this case, yes I'd change your methods to static. You should be able to do this without breaking your current code that uses objects.
Offline Rakiayn

Senior Newbie





« Reply #5 - Posted 2013-11-22 00:13:34 »

yes I have thousends of objects that use this object. thanx for the reply. think I will leave it this way
Offline Danny02
« Reply #6 - Posted 2013-11-22 00:58:53 »

Also...

your methods tell me that you are quite new to programming and Java. You duplicate a lot of code which is either present already in the standard library or doing stuff which is extremely unperformant or code size.
Offline Rakiayn

Senior Newbie





« Reply #7 - Posted 2013-11-22 01:11:38 »

yes I am
this is my first project
can you tell me which part is unperformant?
Offline saucymeatman
« Reply #8 - Posted 2013-11-22 01:22:34 »

Hey, you need to clean up your code. (Or at least make the same mistakes consistantly). You sometimes use camel case and then other times you dont. I recomend never starting a method off with a capital letter, that implys that it is a constructor, which it isnt.

Would this work for your intToLetter?

public static String intToLetter(int input) {
String[] alphabet = new String[] {"a","b","c","d","e","f","g","h","i","j","k","l","m","n","o","p","q","r","s","t","u","v","w","x",y","z"};
return alphabet[input];
}

 Grin
Offline saucymeatman
« Reply #9 - Posted 2013-11-22 01:27:59 »

Actually, you could take my previously stated method and adapt it to alot of your needs.
And you could make alphabet a constant, to avoid repetitive code.

Hope I helped.  Smiley
Games published by our own members! Check 'em out!
Legends of Yore - The Casual Retro Roguelike
Offline Danny02
« Reply #10 - Posted 2013-11-22 01:35:17 »

here did some fixes/changes:
1  
2  
3  
4  
5  
6  
7  
8  
9  
10  
11  
12  
13  
14  
15  
16  
17  
18  
19  
20  
21  
22  
23  
24  
25  
26  
27  
28  
29  
30  
31  
32  
33  
34  
35  
36  
37  
38  
39  
40  
41  
42  
43  
44  
45  
46  
47  
48  
49  
50  
51  
52  
53  
54  
55  
56  
57  
58  
59  
60  
61  
62  
63  
64  
65  
66  
67  
68  
69  
70  
71  
72  
73  
74  
75  
76  
77  
78  
79  
80  
81  
82  
83  
84  
85  
86  
87  
88  
89  
90  
91  
92  
93  
94  
95  
96  
97  
98  
99  
100  
101  
102  
103  
104  
105  
106  
107  
108  
109  
110  
111  
112  
113  
114  
115  
116  
117  
118  
119  
120  
121  
122  
123  
124  
125  
126  
127  
128  
129  
130  
131  
132  
133  
134  
135  
136  
137  
138  
139  
140  
141  
142  
143  
144  
145  
146  
147  
148  
149  
150  
151  
152  
153  
154  
155  
156  
157  
158  
159  
160  
161  
162  
163  
164  
165  
166  
167  
168  
169  
170  
171  
172  
173  
174  
175  
176  
177  
178  
179  
180  
181  
182  
183  
184  
185  
186  
187  
188  
189  
190  
191  
192  
193  
194  
195  
196  
197  
198  
public class Physics extends ROGameObject
{
    public Random r = new Random();  
   
   
    @Override
    public void update()
    {
       
    }

    //check if two objcts collide
   public boolean checkCollisions(ROGameObject go1, ROGameObject go2)
    {
        Rectangle r1 = new Rectangle((int)go1.getx(),(int)go1.gety(),(int)go1.getxsize(),(int)go1.getysize());
        Rectangle r2 = new Rectangle((int)go2.getx(),(int)go2.gety(),(int)go2.getxsize(),(int)go2.getysize());
       
        return r1.intersects(r2);
    }  
   
+    //use Math.abs(...)  
-    public int removenegative(int value)
-    {
-        if (value < 0)
-        {
-            return (value*-1);
-        }
-        return value;            
-    }
   
   
   
    //function checks if a value is between minimum and maximum value
   public float floatbetweenfunction(float minvalue, float midvalue, float maxvalue)
    {      
-        if (minvalue < midvalue && midvalue < maxvalue)
-        {
-            return (midvalue);
-        }
        if (midvalue <= minvalue )          
        {    
            return (minvalue);
        }
+        else if (midvalue >= maxvalue )          
        {    
            return (maxvalue);
        }
+   else
             return (midvalue);
+   //but why not use Math.max(Math.min(maxvalue, midvalue), minvalue);
   }
   
+    //no need see above
-    public int intbetweenfunction(int minvalue, int midvalue, int maxvalue)
-    {      
-        ...
-    }    
   
    //returns a random number between minvalue and maxvalue
   public int RandomInt(int minvalue, int maxvalue)
    {          
        return (minvalue + (int)(Math.random() * ((maxvalue - minvalue) + 1)));  
    }
   
+   //never use for Strings for math, pls...
   public String Randombit()
    {
        ...
    }
   
+    //same as aboth
   public char flipbit(char bit)
    {
   ...
    }  
 
+   //why?
   public String flipbitstring(String newstring)
    {
     ...
    }
   
+    //Math.max(...)
   public int MaxInt(int value1, int value2)
    {    
    }    
   
    public static double PytagorasInt(int value1, int value2)
    {    
+   //no need to convert to double, I would say counter productive. If you should convert to long
       double nvalue1 = (double)value1;
        double nvalue2 = (double)value2;        
        double int1 = (nvalue1*nvalue1) + (nvalue2*nvalue2);
       
        return Math.sqrt(int1);
   
    }      
 
    public char inttoletter(int newint)
    {    
+   if(newint < 0 && newint > 33)
+      return 'A';
+   else if(newint < 26)
+      return (char)(newint + 'A');
+   else
+      return (char)(newint - 26 + 'a');        
    }

    public float lettertofloat(char letter)
    {
+   return lettertoint(letter) / 33f;              
    }    
   
    //number between -1 and 1
   public float lettertofloat2(char letter,float tresholdfactor)
    {
+   return tresholdfactor * (-1 + lettertofloat(letter));              
    }        
   
    public int lettertoint(char letter)
    {
+   if(letter >= 'A' && letter <= 'Z')
+      return letter - 'A';
+   else if(letter >= 'a' && letter <= 'h')
+      return letter - 'h';
+   else
+      return 0;              
    }    
   
   
    public char letterup(char letter)
    {
+   if((letter >= 'A' && letter < 'Z') || (letter >= 'a' && letter < 'h'))
+      return letter + 1;
+   else if(letter == 'Z')
+      return 'a';
+   else if(letter == 'h')
+      return 'h';
+   else
+      return letter;                  
    }
   
    public char letterdown(char letter)
    {
+   if((letter > 'A' && letter <= 'Z') || (letter > 'a' && letter <= 'h'))
+      return letter - 1;
+   else if(letter == 'a')
+      return 'Z';
+   else if(letter == 'A')
+      return 'A';
+   else
+      return letter;  
    }
   
    public char letterrandom(char letter)
    {
+   if((letter >= 'A' && letter <= 'Z') || (letter >= 'a' && letter <= 'h')){
+           return inttoletter(RandomInt(0, 33));
+   }    
        return(letter);
    }    
   
    public char letterinvert(char letter)
    {
+   if((letter >= 'A' && letter <= 'Z') || (letter >= 'a' && letter <= 'h')){
+           return inttoletter( 33 - lettertoint(letter) );
+   }    
        return(letter);              
    }    
   
    public String inverseletter(String prestring,int lettertoinverse)
    {
+        String first = prestring.substring(0, lettertoinverse - 1);
+        String last = prestring.substring(lettertoinverse + 1, prestring.length());
+        return first + letterinvert(prestring.charAt(lettertoinverse)) + last;
    }
   
     
   
    public String insertsubstring(String originalstring,String insertstring,int indexstart,int indexend)
    {
        String substring1 = originalstring.subString(0, indexstart);
        String substring3= originalstring.subString(indexenx, originalstring.length());
       
-        for(int j = 0; j<(insertstringlength) ;j++)// wtf
-        {        
-            substring2 += insertstring.charAt(j);
-        }        
                     
+        return substring1 + insertstring2 + substring3;
    }
   
    public String removesubstring(String originalstring,String insertstring,int indexstart,int indexend)
    {

        //also with substring
   }    
}
Offline Rakiayn

Senior Newbie





« Reply #11 - Posted 2013-11-22 23:28:33 »

here did some fixes/changes:

thank you very much. I used most of your improvements.

Actually, you could take my previously stated method and adapt it to alot of your needs.
And you could make alphabet a constant, to avoid repetitive code.

Hope I helped.  Smiley

that improved for performance a bit. why didnt I think of this?
thanks!
Pages: [1]
  ignore  |  Print  
 
 

 

Add your game by posting it in the WIP section,
or publish it in Showcase.

The first screenshot will be displayed as a thumbnail.

xsi3rr4x (81 views)
2014-04-16 03:08:23

BurntPizza (73 views)
2014-04-15 12:46:01

UprightPath (84 views)
2014-04-15 02:39:50

UprightPath (67 views)
2014-04-15 02:35:47

Porlus (84 views)
2014-04-15 00:48:38

tom_mai78101 (107 views)
2014-04-10 13:04:31

BurntPizza (167 views)
2014-04-09 08:06:04

tom_mai78101 (263 views)
2014-04-05 22:34:39

trollwarrior1 (214 views)
2014-04-04 21:06:45

CJLetsGame (223 views)
2014-04-01 11:16:10
List of Learning Resources
by SHC
2014-04-18 12:17:39

List of Learning Resources
by Longarmx
2014-04-08 12:14:44

Good Examples
by matheus23
2014-04-05 22:51:37

Good Examples
by Grunnt
2014-04-04 00:48:46

Good Examples
by Grunnt
2014-04-04 00:48:37

Good Examples
by matheus23
2014-04-02 03:40:51

Good Examples
by matheus23
2014-04-02 03:40:34

Anonymous/Local/Inner class gotchas
by Roquen
2014-03-12 00:22:30
java-gaming.org is not responsible for the content posted by its members, including references to external websites, and other references that may or may not have a relation with our primarily gaming and game production oriented community. inquiries and complaints can be sent via email to the info‑account of the company managing the website of java‑gaming.org
Powered by MySQL Powered by PHP Powered by SMF 1.1.18 | SMF © 2013, Simple Machines | Managed by Enhanced Four Valid XHTML 1.0! Valid CSS!