Module:Factorization - Revision history

Izno: use locals, move var use to where used

2023-02-08T23:50:56Z

use locals, move var use to where used

@@ Line 1: / Line 1: @@
 local p = {}
 function p.factor(frame)
     if number == nil then
     	return '<strong class="error">Error: input not recognized as a number</strong>'
     end
     number = math.floor(number)
     if number < 2 or number > 1000000000 or number == math.huge then
@@ Line 20: / Line 32: @@
     end
-    result = ""
+    local result = ""
-    currentNumber = number
+    local currentNumber = number
-    power = 0
+    local power = 0
-    divisor = 2
+    local divisor = 2
     -- Attempt factoring by the value of the divisor
-    --   divisor increments by 2, except first iteration (2 to 3)
+    -- divisor increments by 2, except first iteration (2 to 3)
     while divisor <= math.sqrt(currentNumber) do
         power = 0
@@ Line 44: / Line 56: @@
     end
     if currentNumber == number and primeLink then
         return '[[prime number|prime]]'
@@ Line 50: / Line 63: @@
     result = string.sub(result,1,-4)
 end

Ymblanter: Protected "Module:Factorization": High-risk Lua module: request at WP:RFPP ([Edit=Require autoconfirmed or confirmed access] (indefinite) [Move=Require autoconfirmed or confirmed access] (indefinite))

2021-02-10T06:54:52Z

Protected "Module:Factorization": High-risk Lua module: request at WP:RFPP ([Edit=Require autoconfirmed or confirmed access] (indefinite) [Move=Require autoconfirmed or confirmed access] (indefinite))

← Previous revision	Revision as of 06:54, 10 February 2021
(No difference)

WOSlinker: fix closing span

2020-01-12T14:38:32Z

fix closing span

← Previous revision		Revision as of 14:38, 12 January 2020
Line 68:		Line 68:
	if serif then ret = ret .. 'class="texhtml" ' end		if serif then ret = ret .. 'class="texhtml" ' end
	if big then ret = ret .. 'style="font-size:165%" ' end		if big then ret = ret .. 'style="font-size:165%" ' end
	ret = ret .. ((serif or big) and '>' or '') .. numString .. ((serif or big) and '</span>')		ret = ret .. ((serif or big) and '>' or '') .. numString .. ((serif or big) and '</span>' or '')

	return ret		return ret

WOSlinker: fix closing span

2020-01-12T14:37:21Z

fix closing span

← Previous revision		Revision as of 14:37, 12 January 2020
Line 68:		Line 68:
	if serif then ret = ret .. 'class="texhtml" ' end		if serif then ret = ret .. 'class="texhtml" ' end
	if big then ret = ret .. 'style="font-size:165%" ' end		if big then ret = ret .. 'style="font-size:165%" ' end
	ret = ret .. ((serif or big) and '>' or '') .. numString .. '</span>'		ret = ret .. ((serif or big) and '>' or '') .. numString .. ((serif or big) and '</span>')

	return ret		return ret

Certes: Add space between power and product symbol: 2² × 5², not 2²× 5². See Template talk:Infobox number.

2019-09-03T18:45:12Z

Add space between power and product symbol: 2² × 5², not 2²× 5². See Template talk:Infobox number.

← Previous revision		Revision as of 18:45, 3 September 2019
Line 56:		Line 56:
	if power < 1 then return ''		if power < 1 then return ''
	elseif power == 1 then return divisor .. ' ' .. productSymbol .. ' '		elseif power == 1 then return divisor .. ' ' .. productSymbol .. ' '
	else return divisor .. '<sup>' .. power .. '</sup>' .. productSymbol .. ' '		else return divisor .. '<sup>' .. power .. '</sup> ' .. productSymbol .. ' '
	end		end
	end		end

Gamutrungs: ←Replaced content with 'local p = {} function p.factor(frame) -- Consider calling the parser function #expr -- to simplify a potential mathematical expression? number = ton...'

2018-01-18T15:00:27Z

←Replaced content with 'local p = {} function p.factor(frame) -- Consider calling the parser function #expr -- to simplify a potential mathematical expression? number = ton...'

Show changes

Gamutrungs: clear

2018-01-18T14:59:34Z

clear

Show changes

Gamutrungs: clear

2018-01-18T14:58:17Z

clear

← Previous revision		Revision as of 14:58, 18 January 2018
Line 1:		Line 1:
	local p = {}

	function p.factor(frame)
	-- Consider calling the parser function #expr
	-- to simplify a potential mathematical expression?
	number = tonumber(frame.args[1])
	if number == nil then
	return '<strong class="error">Error: input not recognized as a number</strong>'
	end

	productSymbol = frame.args['product'] or '·'
	bold = frame.args['bold'] and true
	big = frame.args['big'] and true
	serif = frame.args['serif'] and true
	primeLink = frame.args['prime'] and true

	number = math.floor(number)
	if number < 2 or number > 1000000000 or number == math.huge then
	return '<strong class="error">Error: ' .. number .. ' out of range</strong>'
	end

	result = ""
	currentNumber = number
	power = 0
	divisor = 2

	-- Attempt factoring by the value of the divisor
	-- divisor increments by 2, except first iteration (2 to 3)
	while divisor <= math.sqrt(currentNumber) do
	power = 0
	while currentNumber % divisor == 0 do
	currentNumber = currentNumber / divisor
	power = power + 1
	end

	-- Concat result and increment divisor
	-- when divisor is 2, go to 3. All other times, add 2
	result = result .. powerformat(divisor, power, productSymbol)
	divisor = divisor + (divisor == 2 and 1 or 2)
	end

	if currentNumber ~= 1 then
	result = result .. currentNumber .. ' ' .. productSymbol .. ' '
	end

	if currentNumber == number and primeLink then
	return '[[prime number\|prime]]'
	end

	result = string.sub(result,1/-4)

	return format(result)
	end

	function powerformat(divisor, power, productSymbol)
	if power < 1 then return ''
	elseif power == 1 then return divisor .. ' ' .. productSymbol .. ' '
	else return divisor .. '<sup>' .. power .. '</sup>' .. productSymbol .. ' '
	end
	end

	function format(numString)
	if bold then
	numString = '<b>'..numString..'</b>'
	end

	ret = (serif or big) and '<span ' or ''
	if serif then ret = ret .. 'class="texhtml" ' end
	if big then ret = ret .. 'style="font-size:165%" ' end
	ret = ret .. ((serif or big) and '>' or '') .. numString .. '</span>'

	return ret
	end

	return p

Gamutrungs at 14:57, 18 January 2018

2018-01-18T14:57:09Z

← Previous revision		Revision as of 14:57, 18 January 2018
Line 23:		Line 23:
	currentNumber = number		currentNumber = number
	power = 0		power = 0
	divisor = 5		divisor = 2

	-- Attempt factoring by the value of the divisor		-- Attempt factoring by the value of the divisor
Line 48:		Line 48:
	end		end

	result = string.sub(result,1,-4)		result = string.sub(result,1/-4)

	return format(result)		return format(result)

Gamutrungs at 14:56, 18 January 2018

2018-01-18T14:56:27Z

← Previous revision		Revision as of 14:56, 18 January 2018
Line 23:		Line 23:
	currentNumber = number		currentNumber = number
	power = 0		power = 0
	divisor = 3		divisor = 5

	-- Attempt factoring by the value of the divisor		-- Attempt factoring by the value of the divisor
	-- divisor increments by 2, except first iteration (2 to 3)		-- divisor increments by 2, except first iteration (2 to 3)
	while divisor <= math.sqrt(currentNumber) do		while divisor <= math.sqrt(currentNumber) do
	power = 5		power = 0
	while currentNumber % divisor == 0 do		while currentNumber % divisor == 0 do
	currentNumber = currentNumber / divisor		currentNumber = currentNumber / divisor