matrices_p9 0.0.1 → 1.0.0

data/Rakefile

@@ -5,10 +5,6 @@ require 'rspec/core/rake_task'
 RSpec::Core::RakeTask.new
 task :default => :spec
 
-desc "Run spec"
-task :spec do
-  sh "cd spec \n rspec -I. matrices_spec.rb"
-end
 
 desc "Run UnitTest"
 task :test do

data/lib/fraccion.rb

@@ -1,11 +1,11 @@
-
-#require "./gcd.rb"
-#Programa que trabaja con fracciones
-#Adriana Rolo Inchausti, Luis Antonio Orta Mendes 
-
+#Clase para trabajar con fracciones. 
 class Fraccion
-attr_reader :denominador, :numerador
-
+#Valor de numerador
+attr_reader :denominador
+ 
+#Valor de denominador
+attr_reader :numerador
+#Crea una fraccion con num como numerador y denomin como denominador
 	def initialize(num, denomin)
 		#atributo
 		mcd=gcd(num,denomin)
@@ -24,13 +24,12 @@ attr_reader :denominador, :numerador
 	end
 	
 
-# Metodos getter de Numerador y denominador
-
+# Retorna el valor del numerador
 	def num
 		return @numerador
 	end
 
-
+#Retorna el valor del denominador
 	def denom
 		return @denominador
 	end 
@@ -154,8 +153,7 @@ attr_reader :denominador, :numerador
 	end
 
 
-#Sobrecarga de operadores de comparación devuelve true en caso de que sea cierto y falso en caso contrario 
-#Mayor que 
+#Sobrecarga de operadores de comparación devuelve true en caso de que el numero con el que se invoca es mayor que el segundo y falso en caso contrario 
 	def > (otro)
 		if (@numerador/@denominador > otro.numerador/otro.denominador)
 			return true
@@ -165,7 +163,8 @@ attr_reader :denominador, :numerador
 		
 	end
 	
-#menor que 	
+
+#Sobrecarga de operadores de comparación devuelve true en caso de que el numero con el que se invoca es menor que el segundo y falso en caso contrario 
 	def < (otro)
 		if (@numerador/@denominador < otro.numerador/otro.denominador)
 			return true
@@ -174,7 +173,9 @@ attr_reader :denominador, :numerador
 		end
 		
 	end
-#mayor igual que 
+	
+
+#Sobrecarga de operadores de comparación devuelve true en caso de que el numero con el que se invoca es mayor o igual que el segundo y falso en caso contrario 
 	def >= (otro)
 		if (@numerador/@denominador >= otro.numerador/otro.denominador)
 			return true
@@ -183,7 +184,9 @@ attr_reader :denominador, :numerador
 		end
 		
 	end
-#menos igual que 
+	
+
+#Sobrecarga de operadores de comparación devuelve true en caso de que el numero con el que se invoca es menor que el segundo y falso en caso contrario 
 	def <= (otro)
 		if (@numerador/@denominador <= otro.numerador/otro.denominador)
 			return true
@@ -192,7 +195,8 @@ attr_reader :denominador, :numerador
 		end
 		
 	end
-#Coerce
+	
+#Coerce. se usa para multiplicar numeros por fracciones
 	def coerce(other)
 		[self, other]
 	end

data/lib/matrices_p9.rb

@@ -1,13 +1,21 @@
+require "matrices_p9/version"
 require "fraccion.rb"
 
+#Clase abstracta de la que heredan matriz densa y dispersa. No se puede crear un objeto de esta clase.
 class Matriz
-attr_accessor :fil, :col
+
+#Modulo para comparar
 include Comparable
 
+#Numero de filas
+attr_accessor :fil 
+
+#Numero de columnas 
+attr_accessor :col
+	#Se almacena el valor de las filas y las columnas segun se le pasen por parametros. 
 	def initialize(f, c)
-		#atributo
-	    	@fil=f.to_i; #Numero de filas
-	    	@col=c.to_i; #Numero de columnas
+	    	@fil=f.to_i; 	    	
+	    	@col=c.to_i; 
 	end
 
 
@@ -19,18 +27,27 @@ include Comparable
 				temp = MatrizDensa.new(self.fil, self.col, nil)
 				if other.instance_of?MatrizDensa
 					
-					for i in (0...@fil.to_i)
-						for j in (0...@col.to_i)
-							temp.mat[i][j] = (self.mat[i][j]) + (other.mat[i][j])
+					#for i in (0...@fil.to_i)
+					@fil.to_i.times do |i| 
+					
+						#for j in (0...@col.to_i)
+						@col.to_i.times do |j|
+							temp.mat[i][j] = (@mat[i][j]) + (other.mat[i][j])
+							
 						end
 					end
 				end
 
 				if other.instance_of?MatrizDispersa
-					for i in (0...@fil.to_i)
-						for j in (0...@col.to_i)
+					#for i in (0...@fil.to_i)
+					@fil.to_i.times do |i| 
+					
+						#for j in (0...@col.to_i)
+						@col.to_i.times do |j| 
+					
 							encontrado = 0
-							for k in (0...other.posx.size)
+							#for k in (0...other.posx.size)
+							other.posx.size.times do |k| 
 								if (i==other.posx[k] and j==other.posy[k] and encontrado==0)
 									temp.mat[i][j] = (self.mat[i][j]) + (other.valor[k])
 									encontrado = 1	
@@ -48,10 +65,14 @@ include Comparable
 			if self.instance_of?MatrizDispersa
 				if other.instance_of?MatrizDensa
 					temp = MatrizDensa.new(self.fil, self.col, nil)
-					for i in (0...@fil.to_i)
+					#for i in (0...@fil.to_i)
+						@fil.to_i.times do |i| 
+					
 						for j in (0...@col.to_i)
 							encontrado = 0
-							for k in (0...self.posx.size.to_i)
+							#for k in (0...self.posx.size.to_i)
+							self.posx.size.times do |k| 
+							
 								if (i==self.posx[k] and j==self.posy[k] and encontrado==0)
 									temp.mat[i][j] = (other.mat[i][j]) + (self.valor[k])
 									encontrado = 1	
@@ -71,9 +92,13 @@ include Comparable
 					temp.posx = self.posx
 					temp.posy = self.posy
 
-					for j in (0...other.posx.size.to_i)
+					#for j in (0...other.posx.size.to_i)
+					other.posx.size.to_i.times do |j| 
+					
 						encontrado = false
-						for k in (0...self.posx.size.to_i)
+						#for k in (0...self.posx.size.to_i)
+						self.posx.size.times do |k| 
+							
 							if(other.posx[j] == temp.posx[k] and other.posy[j] == temp.posy[k])
 								temp.valor[k] = temp.valor[k] + other.valor[j]
 								encontrado = true
@@ -94,25 +119,36 @@ include Comparable
 			return nil
 		end
 	end
-
+	#Sobrecarga del operador de resta, recibe como parametros dos matrices y devuelve una matriz con el resultado de la suma de forma A-B= (Aij+Bij)
+	
 	def -(other)	             
 		if(self.fil == other.fil and self.col == other.col)
 			# SELF Matrices densas
 			if self.instance_of?MatrizDensa
 				temp = MatrizDensa.new(self.fil, self.col, nil)
 				if other.instance_of?MatrizDensa		
-					for i in (0...@fil.to_i)
-						for j in (0...@col.to_i)
+					#for i in (0...@fil.to_i)
+					@fil.to_i.times do |i| 
+					
+						#for j in (0...@col.to_i)
+						@col.to_i.times do |j| 
+							
 							temp.mat[i][j] = (self.mat[i][j]) - (other.mat[i][j])
 						end
 					end
 				end
 
 				if other.instance_of?MatrizDispersa
-					for i in (0...@fil.to_i)
-						for j in (0...@col.to_i)
+					#for i in (0...@fil.to_i)
+					@fil.to_i.times do |i| 
+					
+						#for j in (0...@col.to_i)
+						@col.to_i.times do |j| 
+					
 							encontrado = 0
-							for k in (0...other.posx.size)
+							#for k in (0...other.posx.size)
+							other.posx.size.times do |k| 
+							
 								if (i==other.posx[k] and j==other.posy[k] and encontrado==0)
 									temp.mat[i][j] = (self.mat[i][j]) - (other.valor[k])
 									encontrado = 1	
@@ -130,10 +166,16 @@ include Comparable
 			if self.instance_of?MatrizDispersa
 				if other.instance_of?MatrizDensa
 					temp = MatrizDensa.new(self.fil, self.col, nil)
-					for i in (0...@fil.to_i)
-						for j in (0...@col.to_i)
+					#for i in (0...@fil.to_i)
+					@fil.to_i.times do |i| 
+					
+						#for j in (0...@col.to_i)
+						@col.to_i.times do |j| 
+							
 							encontrado = 0
-							for k in (0...self.posx.size.to_i)
+							#for k in (0...self.posx.size.to_i)
+							self.posx.size.times do |k| 
+							
 								if (i==self.posx[k] and j==self.posy[k] and encontrado==0)
 									temp.mat[i][j] = (other.mat[i][j]) - (self.valor[k])
 									encontrado = 1	
@@ -153,9 +195,13 @@ include Comparable
 					temp.posx = self.posx
 					temp.posy = self.posy
 
-					for j in (0...other.posx.size.to_i)
+					#for j in (0...other.posx.size.to_i)
+					other.posx.size.times do |j| 
+							
 						encontrado = false
-						for k in (0...self.posx.size.to_i)
+						#for k in (0...self.posx.size.to_i)
+						self.posx.size.times do |k| 
+							
 							if(other.posx[j] == temp.posx[k] and other.posy[j] == temp.posy[k])
 								temp.valor[k] = temp.valor[k] - other.valor[j]
 								encontrado = true
@@ -179,13 +225,15 @@ include Comparable
 	    
 end
 
-
+#Clase que hereda de la clase matriz. Almacena matrices densas.
 class MatrizDensa < Matriz
-attr_accessor :mat
-	def initialize(f,c,e)
+#Array bidimensional que representa la matriz
+attr_accessor :mat 
+	#crea una matriz con n filas, m columnas y almacena los valores que se le pasan por parámetros
+	def initialize(f,c,e) 
+	
 		super(f,c)
-		@mat = Array.new(@fil.to_i){Array.new(@col.to_i)}
-
+		@mat = Array.new(@fil.to_i){Array.new(@col.to_i)} #k2
 	    	if (e != nil)
 			#Rellenamos la matriz con lo valores recibidos 
 	    		for i in (0...@fil.to_i)
@@ -195,8 +243,8 @@ attr_accessor :mat
 	    		end
 	    	end
 	end
-	# Metodos getter devuelve el valor de una posicion determinada
-
+	
+	# Metodo getter que devuelve el valor de una posicion determinada
 	def pos(a,b)
 		@mat[a][b]
 	end
@@ -205,11 +253,14 @@ attr_accessor :mat
 	def to_s    
 		"#{@mat}"
 	end
-
+	
+	#funcion que calcula el mayor valor que se encuentra en la matriz	
 	def max
 		m = self.mat[0][0]
 		for i in (0...@fil.to_i)
-			for j in (0...@col.to_i)
+			#for j in (0...@col.to_i)
+			@col.to_i.times do |j| 
+							
 				if (self.mat[i][j] > m)
 					m = self.mat[i][j]
 				end
@@ -217,11 +268,13 @@ attr_accessor :mat
 		end
 		return m
 	end
-
+	
+	#funcion que calcula el menor valor que se encuentra en la matriz
 	def min
 		m = self.mat[0][0]
 		for i in (0...@fil.to_i)
-			for j in (0...@col.to_i)
+			#for j in (0...@col.to_i)
+			@col.to_i.times do |j| 
 				if (self.mat[i][j] < m)
 					m = self.mat[i][j]
 				end
@@ -230,8 +283,8 @@ attr_accessor :mat
 		return m
 	end
 	                             
-#Operador de comparacion (Modulo comparable). se comparan por la suma de sus componentes ¡¡¡¡¡¡PONERLO EN LA CLASE MADRE!!!!!!
 
+#Operador de comparacion (Modulo comparable). se comparan por la suma de sus componentes.
 	def <=> (other)
 		return nil unless other.is_a?MatrizDensa
 		c1=0
@@ -252,9 +305,21 @@ end
 		
 
 
+#Clase que hereda de la clase Matriz. Almacena matrices dispersas (matrices que la mayoria de sus posiciones son nulas 0)
 
 class MatrizDispersa < Matriz
-attr_accessor :posx, :posy, :valor
+#Almacena las filas en que se encuentra algun valor
+attr_accessor :posx
+
+#Almacena las columnas en que se encuentra algun valor
+attr_accessor :posy
+
+#Almacena los valores correspondientes
+attr_accessor :valor
+
+
+
+# se recibe el valor de filas, columnas, los dos arrays con las posiciones en que hay valores y los valores correspondientes
 	def initialize(f,c,posx, posy, valor)
 		super(f,c)
 		@posx = posx
@@ -262,7 +327,8 @@ attr_accessor :posx, :posy, :valor
 		@valor = valor
 
 	end
-
+	
+	#Mdevuelve la matriz en forma de string
 	def to_s
 		s=String.new
 		s << "["
@@ -271,7 +337,8 @@ attr_accessor :posx, :posy, :valor
 		end
 		s << "]"
 	end
-
+	
+	#devuelve el mayor valor almacenado en la matriz
 	def max
 		m = self.valor[0]
 		for i in (0...self.valor.size.to_i)
@@ -281,7 +348,8 @@ attr_accessor :posx, :posy, :valor
 		end
 		return m
 	end
-
+	
+	#devuelve el menor valor almacenado en la matriz
 	def min
 		m = self.valor[0]
 		for i in (0...self.valor.size.to_i)
@@ -291,7 +359,7 @@ attr_accessor :posx, :posy, :valor
 		end
 		return m
 	end
-
+	#Retorna el valor correspondiente a las posiciones que se le pasen como parámetros
 	def pos(a,b)
 		for i in (0...self.posx.size)
 			if(posx[i]==a and posy[i]==b)
@@ -304,8 +372,11 @@ attr_accessor :posx, :posy, :valor
 end
 
 
-
-
+#|i,j, mat1,mat2| mat2[i][j] = (mat1[i][j]) + (mat2[i][j]) return mat2
+	
+	
+#	f = Proc.new{|i,j, mat1, mat2| mat2[i][j] = (mat1[i][j]) + (mat2[i][j]) puts "#{mat2[i][j]}"}
+#	f.call(1,1,[[1,2],[1,2]],[[1,2],[1,2]])
 
 
 

data/lib/matrices_p9/version.rb

@@ -1,3 +1,5 @@
+#Modulo version
 module MatricesP9
-  VERSION = "0.0.1"
+  #version 
+  VERSION = "1.0.0"
 end

data/spec/matrices_p9_spec.rb

@@ -1,14 +1,17 @@
 require 'matrices_p9' 
 
+#expectativas para la clase Matriz y sus clase hijas ( Densas y Dispersas) 
 describe Matriz do
 	before :each do	
 		#Matrices densas
 		@m1 = MatrizDensa.new(3,3,[1,2,3,4,5,6,7,8,9])
 		@m2 = MatrizDensa.new(3,3,[1,2,3,4,5,6,7,8,9])
+		@mm1 = MatrizDensa.new(2,2,[1,1,1,1])
 		
 		#Matrices dispersas
 		@m3 = MatrizDispersa.new(3,3,[0,1,2],[0,1,2],[1,2,3])
 		@m4 = MatrizDispersa.new(3,3,[0,1,2],[0,1,2],[1,2,3])
+		@mm2 = MatrizDispersa.new(2,2,[0],[0],[Fraccion.new(1,2)])
 		
 		#Matrices densas con fracciones
 		@m5 = MatrizDensa.new(2,2,[Fraccion.new(1,2),Fraccion.new(1,3),Fraccion.new(1,4),Fraccion.new(1,5)])
@@ -115,7 +118,17 @@ describe Matriz do
 		end
 		
 	end
+#Prueba de modificacion 
+
+	describe "#Operaciones de suma entre matrices densas y dispersas con fracciones." do
 	
+		it "Suma" do
+			(@mm1+@mm2).to_s().should eq("[[3/2, 1], [1, 1]]")
+			
+		end		
+		
+		
+	end	
 	
 	
 	

data/test/tc_MatrizDensa.rb

@@ -1,12 +1,11 @@
-# Pruebas unitarias de la clase matriz densa
 
 require 'fraccion'
 require 'matrices_p9'
 require "test/unit"
 
-
+# Pruebas unitarias de la clase matriz densa
 class Test_MatrizDensa < Test::Unit::TestCase
-	
+	#prueba para suma de matrices densas 
 	def test_1
 	
 		assert_equal "[[2, 4, 6], [8, 10, 12], [14, 16, 18]]", (MatrizDensa.new(3,3,[1,2,3,4,5,6,7,8,9]) + MatrizDensa.new(3,3,[1,2,3,4,5,6,7,8,9])).to_s	

data/test/tc_MatrizDispersa.rb

@@ -1,11 +1,10 @@
-# Pruebas unitarias de la clase matriz dispersa
 require "fraccion"
 require "matrices_p9"
 require "test/unit"
 
-
+# Pruebas unitarias de la clase matriz dispersa
 class Test_MatrizDispersa < Test::Unit::TestCase
-	
+	#Prueba con matrices dispersas
 	def test_1
 	
 		assert_equal "[[0,0,2][1,1,4][2,2,6]]", (MatrizDispersa.new(3,3,[0,1,2],[0,1,2],[1,2,3]) + MatrizDispersa.new(3,3,[0,1,2],[0,1,2],[1,2,3])).to_s	

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: matrices_p9
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 1.0.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-11-19 00:00:00.000000000 Z
+date: 2013-11-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -42,6 +42,53 @@ files:
 - README.md
 - README.md~
 - Rakefile
+- doc/Fraccion.html
+- doc/Gemfile.html
+- doc/Gemfile~.html
+- doc/Guardfile.html
+- doc/LICENSE_txt.html
+- doc/MatricesP9.html
+- doc/Matriz.html
+- doc/MatrizDensa.html
+- doc/MatrizDispersa.html
+- doc/Rakefile.html
+- doc/Rakefile~.html
+- doc/Test_MatrizDensa.html
+- doc/Test_MatrizDispersa.html
+- doc/created.rid
+- doc/images/brick.png
+- doc/images/brick_link.png
+- doc/images/bug.png
+- doc/images/bullet_black.png
+- doc/images/bullet_toggle_minus.png
+- doc/images/bullet_toggle_plus.png
+- doc/images/date.png
+- doc/images/find.png
+- doc/images/loadingAnimation.gif
+- doc/images/macFFBgHack.png
+- doc/images/package.png
+- doc/images/page_green.png
+- doc/images/page_white_text.png
+- doc/images/page_white_width.png
+- doc/images/plugin.png
+- doc/images/ruby.png
+- doc/images/tag_green.png
+- doc/images/wrench.png
+- doc/images/wrench_orange.png
+- doc/images/zoom.png
+- doc/index.html
+- doc/js/darkfish.js
+- doc/js/jquery.js
+- doc/js/quicksearch.js
+- doc/js/thickbox-compressed.js
+- doc/lib/fraccion_rb.html
+- doc/lib/matrices_p9/version_rb.html
+- doc/lib/matrices_p9_rb.html
+- doc/rdoc.css
+- doc/spec/matrices_p9_spec_rb.html
+- doc/test/tc_MatrizDensa_rb.html
+- doc/test/tc_MatrizDispersa_rb.html
+- doc/tmp/rspec_guard_result.html
 - lib/fraccion.rb
 - lib/matrices_p9.rb
 - lib/matrices_p9/version.rb