spitty / miptshows Goto Github PK
View Code? Open in Web Editor NEWAutomatically exported from code.google.com/p/miptshows
Automatically exported from code.google.com/p/miptshows
Typical modifiers for the constant is:
private static final
If you omit "static" then new variable will be created for every new instance
of your class.
Original issue reported on code.google.com by [email protected]
on 6 Mar 2012 at 3:47
The most part of methods in JcifsController class is redundant and can be
removed.
Original issue reported on code.google.com by [email protected]
on 6 Mar 2012 at 3:57
It is very necessary to finish this class and to provide another one to operate
with prepared statements on inserting data in the DB.
Original issue reported on code.google.com by [email protected]
on 25 Mar 2012 at 3:22
Check if there is no oppotunity to replace List with more common Collection
interface.
Original issue reported on code.google.com by [email protected]
on 6 Mar 2012 at 4:00
Containing all classes in the only package is a very bad practice.
Classes should be union by its place in logical structure of the program
Original issue reported on code.google.com by [email protected]
on 6 Mar 2012 at 4:04
Code always must be self-documented. It means than names of classes, fields,
methods and variables must contain general description of the item.
Original issue reported on code.google.com by [email protected]
on 6 Mar 2012 at 3:46
Firstly, when you calculate string
"MERGE"
+ "INTO files f"
+ "USING temp_data td"
+ "ON (td.file_name = f.file_name)"
+ "WHEN NOT MATCH THEN"
+ "INSERT"
+ "(f.file_id, f.file_name, f.file_size)"
+ "VALUES"
+ "(file_id_generator.nextval, td.file_name, td.file_size)"
you obtain "MERGEINTO files fUSING temp_data tdON (td.file_name =
f.file_name)WHEN NOT MATCH THENINSERT(f.file_id, f.file_name,
f.file_size)VALUES(file_id_generator.nextval, td.file_name, td.file_size)".
You can notice that some words are not divided with spaces.
Second advice is to use (SELECT distinct file_name, file_size) as source table
for merge and join not only by file_name.
Also for executing MERGEs you must use method executeUpdate (not just execute)
of class Statement.
Original issue reported on code.google.com by [email protected]
on 26 Apr 2012 at 11:18
It will be better to extract the part interacting with MyShows in a different
class.
Do not overload a class with complicated logic.
Original issue reported on code.google.com by [email protected]
on 25 Mar 2012 at 2:23
We need to remove redundant items from the trunk directory.
Also the directory 'build' must be removed.
Libraries needed for the project must be put in the directory 'java/lib'.
Original issue reported on code.google.com by [email protected]
on 6 Mar 2012 at 6:35
System.out.println must be forgotten!
Original issue reported on code.google.com by [email protected]
on 6 Mar 2012 at 3:49
Next documents must be added for this component:
1. Program overview
2. Database model
3. Installation notes
4. Functionality of the program
Original issue reported on code.google.com by [email protected]
on 26 Apr 2012 at 11:23
packages "beanpackage" and "testing" may be used in many other projects. To
prevent misunderstooding use packages staring with "org.ncmipt.miptshows" or
something like this.
Original issue reported on code.google.com by [email protected]
on 6 Mar 2012 at 3:43
Returning null value is a bad style.
For the caller an empty collection is more convinient.
Original issue reported on code.google.com by [email protected]
on 6 Mar 2012 at 3:40
Introduce interface to processing data obtaining by JCIFS.
Original issue reported on code.google.com by [email protected]
on 21 Mar 2012 at 9:10
We need to build installation archive with an ant script.
Installation archive must include
project jar/war build file,
documents (e.g. Installation Notes),
resources (e.g. configuration files)
Original issue reported on code.google.com by [email protected]
on 26 Apr 2012 at 11:30
The package "testing" is a good tendency.
Also we need oppotunity to test the part working with MyShows API. One should
create an emulator of api.myshows.ru
Original issue reported on code.google.com by [email protected]
on 6 Mar 2012 at 4:03
A declarative, efficient, and flexible JavaScript library for building user interfaces.
๐ Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
An Open Source Machine Learning Framework for Everyone
The Web framework for perfectionists with deadlines.
A PHP framework for web artisans
Bring data to life with SVG, Canvas and HTML. ๐๐๐
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
Some thing interesting about web. New door for the world.
A server is a program made to process requests and deliver data to clients.
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
Some thing interesting about visualization, use data art
Some thing interesting about game, make everyone happy.
We are working to build community through open source technology. NB: members must have two-factor auth.
Open source projects and samples from Microsoft.
Google โค๏ธ Open Source for everyone.
Alibaba Open Source for everyone
Data-Driven Documents codes.
China tencent open source team.