Mobile Developer
 Аватар для lifestyle
238 / 234 / 18
Регистрация: 10.05.2009
Сообщений: 917

Нужна критика

11.05.2009, 11:26. Показов 1195. Ответов 3
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
собственно хотелось бы узнать насколько граматно написано и можно ли что нить сократить и уменьшить время вычислений
Java
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
public class Primes { 
//------------------------------------------------------ 
   public static boolean isPrime(int num){ 
      boolean b=true; 
      for (int i=2; i <=Math.sqrt(num) && b;i++){ 
         if (num%i==0) b=false; 
      } 
      return (b); 
   } 
//-------------------------------------------------------- 
   public static  int [] primes(int min,int max){ 
      int k=0, size=max-min; 
      int []arr1=new int[size]; 
      for (int i=min;i<(max);i++ ){ 
         boolean d=true; 
         for (int j=2; j<=Math.sqrt(i) && d;j++){ 
          if (i%j==0){ 
             d=false; 
             } 
         }   
            if (d){   
        arr1[k]=i; 
               System.out.print(arr1[k]+","); 
               k++;    
            } 
        }
 return (arr1);    
} 
//---------------------------------------------------------- 
   public static int closestPrime(int num) {  
      int [] arr1=new int [2]; 
      int temp1, temp2; 
      int i=num,j=2; 
      for (i=num-1;i>1;i--){    
         boolean c=true; 
         for(j=2;j<=Math.sqrt(i) && c;j++){    
            if (i%j==0) c=false; 
         } 
         if (c){ 
          arr1[0]=i; 
         break; 
         } 
      } 
      for(int y=num+1;y>num;y++){ 
         boolean e=true; 
         for (int m=2;m<=Math.sqrt(y) && e;m++){ 
            if (y%m==0) e=false; 
         } 
         if(e){ 
            arr1[1]=y; 
         break; 
         } 
      } 
      temp1=(num-arr1[0]); 
      temp2=(arr1[1]-num); 
      if (temp1==temp2){ 
         return (arr1[0]); 
        } 
      else if (temp1<temp2) 
          return (arr1[0]); 
      else
         return (arr1[1]); 
   } 
//---------------------------------------------------------- 
   public static void main(String[] args) { 
    int n=MyConsole.readInt("Enter num>0:"); 
      while (n<=0){ 
       n=MyConsole.readInt("Error n<=0 enter new num:"); 
      } 
      boolean prime = isPrime(n); 
      System.out.println(prime); 
//----------------------------------------------------------       
      int min=MyConsole.readInt("Enter min:"); 
      int max=MyConsole.readInt("Enter max:"); 
      while (min<=1){ 
         min=MyConsole.readInt("Error min<=1 enter new min:"); 
      } 
      while (max<=1){ 
         max=MyConsole.readInt("Error max<=1 enter new max:"); 
      } 
      primes(min,max); 
      System.out.println(); 
//----------------------------------------------------------          
      int m=MyConsole.readInt("Enter num:"); 
      while (m<=1){ 
         m=MyConsole.readInt("Error n<=1 enter new num:"); 
      } 
      System.out.println(closestPrime(m)); 
//---------------------------------------------------------- 
   } 
}
0
IT_Exp
Эксперт
34794 / 4073 / 2104
Регистрация: 17.06.2006
Сообщений: 32,602
Блог
11.05.2009, 11:26
Ответы с готовыми решениями:

Криворукий код и его критика
Необходима критика криворукого кода. Желательно прилагать пояснения что не так, или хотя бы наставить чайник на путь истинный). Генератор...

Друзья нужна критика
Хоть и сказали нам делать его не спеша, но не хочется просрать время до сдачи. Сама задача. Model of a bank. Your bank has...

Почти готовый морской бой (нужна критика кода)
Морской бой 1. код разделён на два пакета logic и swing 2. правильно ли я манипулирую данными: получаю, передаю, изменяю. ? 3. не...

3
 Аватар для Otaka
1857 / 714 / 55
Регистрация: 11.12.2008
Сообщений: 1,019
11.05.2009, 13:41
У тебя везде используются подобные конструкции:
Java
1
for (int j=2; j<=Math.sqrt(i) && d;j++)
зачем каждый раз пересчитывать корень? Ведь только время теряешь.
надо как-то так:
Java
1
2
int sqr=Math.sqrt(i);
for (int j=2; j<=sqr && d;j++)
0
Mobile Developer
 Аватар для lifestyle
238 / 234 / 18
Регистрация: 10.05.2009
Сообщений: 917
11.05.2009, 13:52  [ТС]
дык у меня две пары операторов фор и один растет вверх i++а другой вниз i-- та что мне дает эта строка???
Плюс ко всему это три разные функции ...как их можносвязать этой переменной?
0
16 / 16 / 2
Регистрация: 08.05.2009
Сообщений: 81
12.05.2009, 18:55
их не нужно связывать.

просто есть пара циклов - внешний и внутренний. внешний по i, и в пределах внешнего цикла значение sqrt(i) одно и то же. во внутреннем цикле каждый раз считать его - накладно. соответственно, нужно в каждой функции вынести перед внутренним циклом вычисление корня из счетчика внешнего цикла, как и показал Otaka.
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
BasicMan
Эксперт
29316 / 5623 / 2384
Регистрация: 17.02.2009
Сообщений: 30,364
Блог
12.05.2009, 18:55
Помогаю со студенческими работами здесь

Написал код, нужна критика и советы как можно было бы сделать лучше
Задание №2: Написать программу, копирующую элементы 2-х массивов размером 5 элементов каждый в один массив размером 10 элементов следующим...

Адекватная критика выполнения задачи
Недавно пришло тестовое задание от нашей крупной IT компании. Задание звучало, так: Реализовать объект для учета однотипных...

нужна критика :-)
День добрый. Сайт авиаперевозок. Буду рад любой критике, советам. Спасибо :)

Нужна критика
Здравствуйте! Проанализируйте сайт с точки зрения оптимизации! Сайт: www.fashion-tissues.ру Спасибо!

Нужна критика
Ребята, оцените дизайн, удобство и сео сайт


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
4
Ответ Создать тему
Опции темы

Новые блоги и статьи
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами
8Observer8 20.02.2026
Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .
Конвертировать закладки radiotray-ng в m3u-плейлист
damix 19.02.2026
Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .
Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI
Eddy_Em 18.02.2026
Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .
Камера Toupcam IUA500KMA
Eddy_Em 12.02.2026
Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу
zbw 12.02.2026
И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.
«Знание-Сила»
zbw 12.02.2026
«Знание-Сила» «Время-Деньги» «Деньги -Пуля»
SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров
8Observer8 12.02.2026
Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .
SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image)
8Observer8 11.02.2026
Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru