Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате
Форум программистов > Java: Общие вопросы > никто не видел Code-Review для *.jsp ?


Автор: polosatij 31.5.2005, 13:55
smile

если еще будет не отдельной программой, а под ант или еклипс, расцелую smile smile

Автор: 3,14 31.5.2005, 14:25
Обьясните мне тупому: "что это такое?"

Автор: polosatij 31.5.2005, 14:45
Цитата(3)
Обьясните мне тупому: "что это такое?"


проверка кода на наличие ошибок..


например,

Checkstyle is a development tool to help programmers write Java code that adheres to a coding standard. It automates the process of checking Java code to spare humans of this boring (but important) task. This makes it ideal for projects that want to enforce a coding standard.

http://checkstyle.sourceforge.net/

но checkStyle похоже не поддерживает *.jsp smile

Добавлено @ 14:49
а именно проверка:

* Unused local variables
* Empty catch blocks
* Unused parameters
* Empty 'if' statements
* Duplicate import statements
* Unused private methods
* Classes which could be Singletons
* Short/long variable and method names

и т.д.

Автор: batigoal 31.5.2005, 14:57
Неужели есть необходимость в подобном инструменте?

Автор: polosatij 31.5.2005, 14:59
Цитата(Lamer @ 31.5.2005, 14:56)
А каких именно ошибок? Ведь можно их откомпилировать.


смотри.. в большом проекте у тебя могут возникнуть куча невызываемых методов, пустых блоков, неиспользованных переменных и прочее..

см. выше...

Автор: batigoal 31.5.2005, 15:03
Да-да, это я не дочитал до конца... уже удалил свой пост. smile

Так неужели имеются настолько сложные страницы, что их нужно автоматизированно проверять? Это нехорошо.

Автор: polosatij 31.5.2005, 15:05
Цитата(Lamer @ 31.5.2005, 14:57)
Неужели есть необходимость в подобном инструменте?


конечно есть!
смотри:

http://java-source.net/open-source/code-analyzers

Автор: 3,14 31.5.2005, 15:07
Lamer George ещё как есть, правда у нас веб компонент это только один из модулей, но при компиляции проекта рекомендуется убрать "unused import warnings" ибо задолбают smile

Автор: polosatij 31.5.2005, 15:07
Цитата(Lamer @ 31.5.2005, 15:03)
Так неужели имеются настолько сложные страницы, что их нужно автоматизированно проверять? Это нехорошо.



представь себе большое предприятие где работает большое количество человек.. так вот.. каждый лепит как хочет.. нужно ограничется от такого.. например сказать всем, ей чуваки, непойдёт ваше строительство, если у вас в коде style, например, читается.. нужно его отдельно выносить.. и таких правил очень много, нужно сделать работу проше.. smile

Автор: batigoal 31.5.2005, 15:15
Ясно, убедили.

У меня-то просто вообще компиляция страниц при сборке проекта отключена - долго.

Автор: simanyay 31.5.2005, 17:53
А Eclipse WTP не подходит?

Автор: 3,14 31.5.2005, 17:54
Вот не совсем от же самое но из этой же серии на глаза попалось: http://www.quest.com/jprobe/try.asp?landing=javalobby_text041905

Автор: polosatij 31.5.2005, 18:02
Цитата(simanyay @ 31.5.2005, 17:53)
А Eclipse WTP не подходит?


что это за хрень? smile

Автор: polosatij 31.5.2005, 18:13
Цитата(3)
Вот не совсем от же самое но из этой же серии на глаза попалось: http://www.quest.com/jprobe/try.asp?landin...obby_text041905



пасиба.. но.. я не могу расширить данную программу:

Download Size (in bytes)
*.exe format

мне нужно рассширить прогу под свой дела..




Цитата(simanyay @ 31.5.2005, 17:53)
А Eclipse WTP не подходит?


не пойму.. что именно ты имел ввиду smile

Автор: simanyay 31.5.2005, 18:18
Цитата(polosatij @ 31.5.2005, 20:13)
не пойму.. что именно ты имел ввиду smile


http://eclipse.org/webtools/index.html

Автор: NotGonnaGetUs 1.6.2005, 10:07
Цитата(polosatij @ 31.5.2005, 13:55)
smile

если еще будет не отдельной программой, а под ант или еклипс, расцелую smile smile

Intellej Idea (cмотрю на bety).
Всё есть вплоть до рефакторингов в коде jsp smile

Автор: polosatij 1.6.2005, 12:35
Цитата(NotGonnaGetUs @ 1.6.2005, 10:07)
Intellej Idea (cмотрю на bety).
Всё есть вплоть до рефакторингов в коде jsp


млин.. она платная (я имею ввиду -> Intellej Idea ) smile + мне надо:

1. под ант
2. под еклипс
3. под командную строку

я не могу весь наш отдел на Intellej Idea перестроить. мы все на еклипсе или IBM Rational Application Software сидим.

в IBM Rational Application Software тоже всё это есть, только я не могу расширить правила smile



Автор: polosatij 1.6.2005, 20:53
Цитата(simanyay @ 31.5.2005, 18:18)
http://eclipse.org/webtools/index.html



млин.. что-то я не пойму..

смотрю:

jst
j2ee Standard Tools Project

там стоит:

The scope of the J2EE Standard Tools subproject is the support of J2EE programming. This includes the support of APIs covered by the J2EE1.4 specifications (e.g. JSP, Servlets, EJBs, JCA, JMS, JNDI, JDBC, Java Web Services, JAX* and related JSRs). Support for JCP specifications commonly used in Web Applications, but not included in J2EE1.4 is to be studied on a case by case basis (ex: JSF,JDO).

потом:

Some Architectural Principles
Seamlessly extend the Eclipse user experience
Accommodate common J2EE application project directory layouts
Avoid need for migration
Apply capabilities orthogonally to source artifacts:
Syntax Coloring
Code Assist
Quick Fixes
Refactoring
Etc.
Extend the Run command to J2EE artifacts, e.g. JSP
Enable hybrid source editing:
Java in JSP
JavaScript in JSP
EJB-QL in Java
Etc.
Enable extension of editors with value-add
Graphical views
Form based views
Provide modular design of tools for enabling separate tool deliveries for differents classes of developers
Enable separate deliveries of servlet/JSP and EJB tools
Etc


скачал:

поставил..

не вижу, что он проверяет *.jsp, *.xml..

где я чего недочитал? smile

Автор: simanyay 1.6.2005, 21:09
тоже самое со мной smile или я гоню, или wtp...
Добавлено @ 21:10
А у тебя можно создать J2EE Web Project? У меня только Simple smile

Автор: polosatij 2.6.2005, 11:09
Цитата(simanyay @ 1.6.2005, 21:09)
тоже самое со мной  или я гоню, или wtp...
Добавлено @ 21:10
А у тебя можно создать J2EE Web Project? У меня только Simple 



у меня добавился "server".. но я нифига не вижу что начался проверяться *.xml/*.jsp код.. smile
млин smile

Автор: polosatij 2.6.2005, 12:05
вот что я нашёл:


http://www.eclipse.org/webtools/jst/components/j2ee/scenarios/ServletScenario.html

и я никак не могу найти: J2EE Web Project (картинка номер 1)

Автор: polosatij 2.6.2005, 12:28
http://www.eclipse.org/webtools/testtutorials/gettingstarted/GettingStarted.html

нифига не Лаботает smile

Автор: simanyay 2.6.2005, 13:57
Цитата(polosatij @ 2.6.2005, 14:05)
и я никак не могу найти: J2EE Web Project (картинка номер 1)


я об этом и говорил. А J2EE Perspective вообщу кошмарно глючит

Автор: polosatij 9.6.2005, 15:43
These are the prerequisites to install WebTools Platform;
They must be downloaded and installed before the WTP can be installed.


The Eclipse driver used in this build is eclipse-SDK-3.1RC1...

сейчас солью 3.1RC1, попробую...

Автор: polosatij 9.6.2005, 17:15
не заЛаботала smile
млин smile

может у кого MyEclipse стоит?
проверяется ли там *.jsp синтаксис smile

здесь нарисовано, что да:

http://www.myeclipseide.com/ContentExpress-display-ceid-52.html

5 картинка...

но я скачал и нифига.. может это только из за демо версии, а в нормальной этого не происходит, или у меня опять что-нибудь глючит smile

Добавлено @ 17:24
заработало smile
могу ли я там как-нибудь свой синтаксис присабачить?

может натыкался кто-нибудь на такое smile

Powered by Invision Power Board (http://www.invisionboard.com)
© Invision Power Services (http://www.invisionpower.com)