Mensagens enviadas por: Marcelo_SCS

Versão 2.0:

Boa? Não... menos pior... dá para melhorar muito, criar outras classes, mudar nomenclatura de classes, métodos e variáveis, criar exceções não checadas, abstrair um pouco mais... Mas só de quebrar alguns métodos já deu uma outra cara...
Eu não testei, por favor veja se funciona.
Vc não tratava corretamente uns fechamentos de inputs e outputs...

package acesso;

import java.io.BufferedReader;
import java.io.FileNotFoundException;
import java.io.FileOutputStream;
import java.io.IOException;
import java.io.InputStream;
import java.io.InputStreamReader;
import java.io.OutputStream;
import java.io.PrintWriter;
import java.io.UnsupportedEncodingException;
import java.io.Writer;
import java.net.MalformedURLException;
import java.net.URL;
import java.net.URLConnection;
import java.util.Timer;
import java.util.TimerTask;

public class Acesso {

private URL page;

public static final String PAGE_DELETE = "http://tst.fateci.com.br/academico/desenvolve/fotos/delete.php";
	public static final String PAGE_DATA = "http://tst.fateci.com.br/academico/desenvolve/fotos/dados.txt";

public Acesso(String page) throws MalformedURLException {
		this.page = new URL(page);
	}

public InputStream acessaPaginaComTimeout(int timeout) throws IOException {
		URLConnection connection = page.openConnection();
		if(timeout > 0) {
			connection.setConnectTimeout(timeout);
		}
		connection.connect();
		return connection.getInputStream();
	}

public InputStream acessaPagina() throws IOException {
		return acessaPaginaComTimeout(0);
	}

public static void main(String[] args) throws MalformedURLException {

final long TEMPO = (100 * 60);

Timer timer = null;
		if (timer == null) {
			timer = new Timer();
			TimerTask tarefa = new TimerTask() {
				
				private final Acesso pageDelete = new Acesso(PAGE_DELETE);		
				private final Acesso pageData = new Acesso(PAGE_DATA);

public void run() {
					try {
						readAndDeletePage(pageDelete, pageData);
					} catch (IOException e) {
						e.printStackTrace();
					}
				}

private void readAndDeletePage(final Acesso pageDelete,
						final Acesso pageData) throws IOException {
					InputStream input = pageData.acessaPaginaComTimeout(7000);
					String lines = readLine(input);
					if (lines.length() > 0) {
						pageDelete.acessaPagina();
						writeLPT(lines);
					}
				}

private void writeLPT(String lines) throws IOException {
					FileOutputStream outputFile = null;
					PrintWriter writer = null;
					try {
						outputFile = new FileOutputStream("LPT1:");
						writer = new PrintWriter(outputFile);
						writer.print(lines);	
						writer.flush();
					} catch (FileNotFoundException e) {
						e.printStackTrace();
					} finally {
						closeFile(outputFile);
						closeWriter(writer);
					}					
				}

private String readLine(InputStream inputStream) {
					try {
						return readInput(inputStream);												
					} catch (Exception e) {
						e.printStackTrace();
					} finally {
						closeInput(inputStream);
					}
					return "";
				}

private String readInput(InputStream inputStream)
				throws UnsupportedEncodingException, IOException {
					BufferedReader in;
					StringBuilder lines;
					in = new BufferedReader(new InputStreamReader(inputStream,
					"ISO-8859-1"));
					lines = new StringBuilder();
					for (String line = in.readLine(); line != null; line = in.readLine()) {
						lines.append(line).append("\n\r");
					}
					return in.toString();
				}

private void closeFile(OutputStream out) {
					try {
						if(out != null) {
							out.close();
						}
					} catch (IOException e) {
						e.printStackTrace();
					}
				}

private void closeInput(InputStream in) {
					try {
						if(in != null) {
							in.close();
						}
					} catch (IOException e) {
						e.printStackTrace();
					}
				}

private void closeWriter(Writer writer) {
					try {
						if(writer != null) {
							writer.close();
						}
					} catch (IOException e) {
						e.printStackTrace();
					}
				}

};

timer.scheduleAtFixedRate(tarefa, TEMPO, TEMPO);
		}
	}
}

Só de colocar o código no Eclipse, pedir para formatar e eliminando uns warnings, já ficou um pouquinho mais claro...
Como disse o Vini, quebrar sua função em funções menores é o primeiro passo...

"A primeira regra das funções é que tem de ser pequenas.
A segunda regra das funções é que elas têm de ser ainda menores."

http://www.infoq.com/br/articles/clean-code-book-review

Sugiro que você leia CleanCode e Effective Java.

package acesso;

import java.io.*;
import java.net.*;
import java.util.Timer;
import java.util.TimerTask;

public class Acesso {

// Metodos não começam com caixa alta
        // Funções só podem fazer uma coisa por vez (Clean Code)
        // Use Unchecked Exceptions ao invés de Checked (Clean Code) - sugiro vc capturar e lançar uma exceção não checada
	public void Apaga() throws IOException {  
		URL url = new URL(
				"http://tst.fateci.com.br/academico/desenvolve/fotos/delete.php");
		URLConnection amp_apaga = url.openConnection();
		amp_apaga.connect();
	}

public static final long TEMPO = (100 * 60);

public static void main(String[] args) throws MalformedURLException,
			IOException {

final Acesso ap = new Acesso();
		Timer timer = null;
		if (timer == null) {
			timer = new Timer();
			TimerTask tarefa = new TimerTask() {
				public void run() {

try {

URL amplus = new URL(
								"http://tst.fateci.com.br/academico/desenvolve/fotos/dados.txt");
						URLConnection amp = amplus.openConnection();
						amp.setConnectTimeout(7000);
						amp.connect();
						BufferedReader in = new BufferedReader(
								new InputStreamReader(amp.getInputStream(),
										"ISO-8859-1"));
						StringBuilder lines = new StringBuilder();
						String line;
						for (line = in.readLine(); line != null; line = in
								.readLine()) {
							lines.append(line).append("\n\r");
						}
						in.close();

if (lines.length() == 0) {    // O que é isso, um if sem corpo com else em seguida???
						} else {

ap.Apaga();
							FileOutputStream outputFile = new FileOutputStream(
									"LPT1:");
							PrintWriter out = new PrintWriter(outputFile);
							out.print(lines);
							out.flush();
							outputFile.close();
							out.close();

// System.out.println(lines);
						}
					} catch (MalformedURLException me) {
					} catch (IOException ioe) {

}
				} // fechou run
			}; // fechou TimerTask
			timer.scheduleAtFixedRate(tarefa, TEMPO, TEMPO);
		}
	}
}

- A Importancia Dos Testes

Desenvolver software não pode ser comparado a construir uma casa, um carro, etc. O processo se assemelha muito mais à construção de uma nave espacial!
Se uma só peça que constitui a estrutura da nave não estiver em conformidade, corre-se o risco de perder todo o trabalho.
Por isso as agências espaciais aplicam os mais rigorosos padrões de qualidade em todos os componentes da nave, desde os parafusos
até a qualidade do combustível a ser usada.
O desenvolvimento de software deve ser visto da mesma forma. Uma única linha mal escrita, num código com ótimo design pode afetar todo o funcionamento do aplicativo.
Portanto, a abordagem baseada na qualidade total de produção de software é algo que toda equipe de desenvolvimento deve adotar (em primeiro lugar!)

Muito se fala hoje em dia sobre as chamadas técnicas "agéis": RUP, XP, Scrum e a nova onda, Lean. O mais estranho é que se você
perguntar a um perito em (XP,RUP,<coloque aqui sua metodologia preferida>,etc) se a mesma funciona ele vai dizer:

- XP funciona com toda certeza, desde é claro ... que você tenha uma equipe "ágil"!
- Scrum é maravilhoso e funciona muito bem, desde que.... é claro... você tenha uma equipe "ágil"!

Todas as abordagens modernas exigem um denominador comum... "agilidade".
Mas o que é ser ágil? É seguir uma metodologia "ágil"? Não!!! Ser ágil é seguir com devoção o princípio básico da qualidade
total no desenvolvimento de software.
Metodologias e processos vem e vão: Cascata - RUP - XP - Scrum - Lean, não se apeguem demais a essa sopa de letrinhas.
Podem criar novas técnicas ágeis, ninjas, jedi´s, usar CheckStyle, ferramentas de monitoração de testes, PMD, os frameworks de mercado Hibernate, EJB, JSF, Spring, etc... mas nenhuma delas vai funcionar com a máxima eficiência se a equipe não tiver uma mentalidade focada na qualidade.
É inadmissível as pessoas estudarem RUP, XP, Scrum, etc se não sabem nem o que é TDD, DDD, Refactoring, Design Patterns e
conhecer bem POO. Se forem por esse caminho, estão fadadas ao fracasso, pois todas essas metodologias foram criadas
"por/e para" equipes de alta performance/desempenho/qualidade. Se vc é um mal motorista, não importa se guia um Fusca ou uma Ferrari, vai continuar fazendo besteira no transito...

E onde entram os testes nisso?

A indústria de software e o tal movimento ágil provaram por A + B que os testes são requisitos essencias para se garantir
a qualidade. É o primeiro passo para o sucesso real de uma equipe de desenvolvimento de software.
Antes de aprendermos XP, Scrum, RUP, Lean, PMI e outros businness words, devemos ter a humildade de nos perguntarmos:
- "Estou desenvolvendo software da forma correta?"
- "Minha equipe desenvolve software da forma correta?"
- "Existe um jeito melhor de se desenvolver software?"
Não invertam a lógica natural das coisas. Antes de aplicar as metodologias/processos/tecnologias, devemos mudar a cultura de desenvolvimento da equipe, para que ela obtenha um crescimento gradual em qualidade. Lembrem-se de um dos lemas do manifesto ágil: foco nas pessoas, e não nos processos....

Por fim, se há pessoas que acham que testes são enfadonhos, um lembrete: quando compram qualquer produto, exigem que o mesmo tenha a máxima qualidade, e se não tiver, vão ficar revoltadas e "usar os seus direitos de consumidor"...
Se para comprar produtos você exige tal nível de qualidade, tenha o bom senso de desenvolver software com o mesmo rigor, pois outros estarão usando/dependendo dele...

O Mono realmente pode ser considerado robusto para aplicações .NET ou é um wine da vida?

Notícias, artigos e o maior fórum brasileiro sobre Java