[SOLUCIONADO] Metodo del perceptron hecho en C#

[fercho1s]

visiten este link y lo podran descargar de ahi

http://sdrv.ms/10ETL2M

Espero que les sea bastante de ayuda

Autor: Fernando Yépez

[razpeitia]

Recomendación usa algo como GitHub o BitBucket para hostear tu código.

[HighCore]

Iniciado por fercho1s

visiten este link y lo podran descargar de ahi

http://sdrv.ms/10ETL2M

Espero que les sea bastante de ayuda

Autor: Fernando Yépez

Capo, no se lo que es el precepton ni lo que hace, pero tu codigo en terminos de calidad de codigo esta bastante feo.

para empezar, tenes una clase que se llama "cl_perceptron". Esto va en contra de las Naming Conventions de C# que especifica que las clases tienen que tener ProperCase, y ningun prefixo. Con lo cual a lo sumo deberia ser

Código:

public class Perceptron

Ademas de eso, tenes un constructor vacio

Código:

public cl_perceptron()
        {

        }

El ReSharper me marca esto en gris ya que el compilador de C# genera exactamente lo mismo por default, con lo cual es codigo redundante.

Despues, tenes un monton de metodos (otra vez todos en minuscula) de los cuales hay varios que devuelven int[] pero el valor de retorno no se usa en ningun lado, con lo cual deberian ser void, y ademas varios de estos metodos reciben parametros que nunca usan (por ejemplo, asignar_nuevo_w_resta() que recibe un parametro int numero_muestra que nunca se usa en ningun lado.

Ademas de todo esto, y creo que es el peor error de todos, tus metodos que realizan la logica reciben como parametros CONTROLES DE LA PANTALLA!!!!!
Tenes que aprender que la logica de tu aplicacion debe estar 100% separada de la capa de presentacion, de lo contrario todo tu codigo va a ser completamente inmantenible y probablemente tenga que ser completamente refactorizado por una persona que si entienda el principio fundamental de la separacion de capas, con lo cual se podria decir que todo tu trabajo fue completamente inutil.

Ademas de esto, tu codigo no tiene contingencia de errores. Le doy F5 y le doy al boton "crear" y me tira un FormatException porque esta tratando de int.Parse() un string vacio de un TextBox.

Creo que si te interesa trabajar como programador alguna vez, deberias poner mas atencion en las buenas practicas de desarrollo de software y menos en cuestiones algebraicas inutiles que no tienen ninguna aplicacion en la realidad.

[fercho1s]

gracias por tus comentarios y esta muy bien tus criticas