Rubocop-inspired style tweaks

This commit is contained in:
Maruno17
2020-09-06 01:10:02 +01:00
parent 5073f86190
commit 6c4670eaa3
7 changed files with 329 additions and 365 deletions

View File

@@ -87,8 +87,8 @@ NO_SIGNPOSTS = []
# * The maximum length, in characters, that the player's name can be. # * The maximum length, in characters, that the player's name can be.
#=============================================================================== #===============================================================================
INITIAL_MONEY = 3000 INITIAL_MONEY = 3000
MAX_MONEY = 999999 MAX_MONEY = 999_999
MAX_COINS = 99999 MAX_COINS = 99_999
MAX_PLAYER_NAME_SIZE = 10 MAX_PLAYER_NAME_SIZE = 10
#=============================================================================== #===============================================================================

View File

@@ -7,8 +7,6 @@ class Class
end end
end end
#=============================================================================== #===============================================================================
# module Comparable # module Comparable
#=============================================================================== #===============================================================================
@@ -23,8 +21,6 @@ unless Comparable.method_defined? :clamp
end end
end end
#=============================================================================== #===============================================================================
# class Boolean # class Boolean
#=============================================================================== #===============================================================================
@@ -34,21 +30,19 @@ class Boolean
end end
end end
#=============================================================================== #===============================================================================
# class String # class String
#=============================================================================== #===============================================================================
class String class String
def starts_with?(str) def starts_with?(str)
proc = (self[0...str.length] == str) if self.length >= str.length proc = (self[0...str.length] == str) if self.length >= str.length
return proc ? proc : false return proc || false
end end
def ends_with?(str) def ends_with?(str)
e = self.length - 1 e = self.length - 1
proc = (self[(e-str.length)...e] == str) if self.length >= str.length proc = (self[(e-str.length)...e] == str) if self.length >= str.length
return proc ? proc : false return proc || false
end end
def starts_with_vowel? def starts_with_vowel?
@@ -110,8 +104,6 @@ class String
end end
end end
#=============================================================================== #===============================================================================
# class Numeric # class Numeric
#=============================================================================== #===============================================================================
@@ -122,8 +114,6 @@ class Numeric
end end
end end
#=============================================================================== #===============================================================================
# class Integer # class Integer
#=============================================================================== #===============================================================================
@@ -131,12 +121,10 @@ class Integer
# Returns an array containing each digit of the number in turn. # Returns an array containing each digit of the number in turn.
def digits(base = 10) def digits(base = 10)
quotient, remainder = divmod(base) quotient, remainder = divmod(base)
(quotient==0) ? [remainder] : quotient.digits(base).push(remainder) return (quotient == 0) ? [remainder] : quotient.digits(base).push(remainder)
end end
end end
#=============================================================================== #===============================================================================
# class Array # class Array
#=============================================================================== #===============================================================================
@@ -166,7 +154,6 @@ class Array
end unless method_defined? :shuffle! end unless method_defined? :shuffle!
end end
#=============================================================================== #===============================================================================
# module Enumerable # module Enumerable
#=============================================================================== #===============================================================================
@@ -178,8 +165,6 @@ module Enumerable
end end
end end
#=============================================================================== #===============================================================================
# Kernel methods # Kernel methods
#=============================================================================== #===============================================================================

View File

@@ -1,54 +1,53 @@
################################################################################ #===============================================================================
# "Duel" mini-game # "Duel" mini-game
# Based on the Duel minigame by Alael # Based on the Duel minigame by Alael
################################################################################ #===============================================================================
begin
class DuelWindow < Window_AdvancedTextPokemon class DuelWindow < Window_AdvancedTextPokemon
attr_accessor :hp attr_reader :hp
attr_accessor :name attr_reader :name
attr_accessor :isEnemy attr_reader :is_enemy
def initialize(name,isEnemy) def initialize(name, is_enemy)
@hp = 10 @hp = 10
@name = name @name = name
@isEnemy = isEnemy @is_enemy = is_enemy
super("") super()
self.width = 160 self.width = 160
self.height = 96 self.height = 96
duelRefresh duel_refresh
end end
def hp=(value) def hp=(value)
@hp = value @hp = value
duelRefresh duel_refresh
end end
def name=(value) def name=(value)
@name = value @name = value
duelRefresh duel_refresh
end end
def isEnemy=(value) def is_enemy=(value)
@isEnemy = value @is_enemy = value
duelRefresh duel_refresh
end end
def duelRefresh def duel_refresh
nameColor = @isEnemy ? "<ar><c3=E00808,F8B870>" : "<c3=3050C8,A0C0F0>" name_color = @is_enemy ? "<ar><c3=E00808,F8B870>" : "<c3=3050C8,A0C0F0>"
hpColor = "<c3=209808,90F090>" hp_color = "<c3=209808,90F090>"
self.text = _INTL("{1}{2}\r\n{3}HP: {4}",nameColor,fmtescape(@name),hpColor,@hp) self.text = _INTL("{1}{2}\r\n{3}HP: {4}", name_color, fmtescape(@name), hp_color, @hp)
end end
end end
#===============================================================================
#
#===============================================================================
class PokemonDuel class PokemonDuel
def pbStartDuel(opponent, event) def pbStartDuel(opponent, event)
@event = event
@viewport = Viewport.new(0, 0, Graphics.width, Graphics.height) @viewport = Viewport.new(0, 0, Graphics.width, Graphics.height)
@viewport.z = 99999 @viewport.z = 99999
@sprites = {} @sprites = {}
@event = event
@sprites["player"] = IconSprite.new(-128 - 32, 96, @viewport) @sprites["player"] = IconSprite.new(-128 - 32, 96, @viewport)
@sprites["player"].setBitmap(pbTrainerSpriteFile($Trainer.trainertype)) @sprites["player"].setBitmap(pbTrainerSpriteFile($Trainer.trainertype))
@sprites["opponent"] = IconSprite.new(Graphics.width + 32, 96, @viewport) @sprites["opponent"] = IconSprite.new(Graphics.width + 32, 96, @viewport)
@@ -60,12 +59,12 @@ class PokemonDuel
@sprites["opponentwindow"].x = Graphics.width @sprites["opponentwindow"].x = Graphics.width
@sprites["opponentwindow"].viewport = @viewport @sprites["opponentwindow"].viewport = @viewport
pbWait(Graphics.frame_rate / 2) pbWait(Graphics.frame_rate / 2)
distancePerFrame = 8*20/Graphics.frame_rate distance_per_frame = 8 * 20 / Graphics.frame_rate
while @sprites["player"].x < 0 while @sprites["player"].x < 0
@sprites["player"].x += distancePerFrame @sprites["player"].x += distance_per_frame
@sprites["playerwindow"].x += distancePerFrame @sprites["playerwindow"].x += distance_per_frame
@sprites["opponent"].x -= distancePerFrame @sprites["opponent"].x -= distance_per_frame
@sprites["opponentwindow"].x -= distancePerFrame @sprites["opponentwindow"].x -= distandistance_per_framecePerFrame
Graphics.update Graphics.update
Input.update Input.update
pbUpdateSceneMap pbUpdateSceneMap
@@ -117,19 +116,14 @@ class PokemonDuel
end end
@special[1] = true if action == 3 @special[1] = true if action == 3
pbMessage(_INTL("{1}: {2}", opponent.name, speeches[action * 3 + rand(3)])) pbMessage(_INTL("{1}: {2}", opponent.name, speeches[action * 3 + rand(3)]))
command = rand(4)
list = [ list = [
_INTL("DEFEND"), _INTL("DEFEND"),
_INTL("PRECISE ATTACK"), _INTL("PRECISE ATTACK"),
_INTL("FIERCE ATTACK") _INTL("FIERCE ATTACK")
] ]
if !@special[0] list.push(_INTL("SPECIAL ATTACK")) if !@special[0]
list.push(_INTL("SPECIAL ATTACK"))
end
command = pbMessage(_INTL("Choose a command."), list, 0) command = pbMessage(_INTL("Choose a command."), list, 0)
if command==3 @special[0] = true if command == 3
@special[0] = true
end
if action == 0 && command == 0 if action == 0 && command == 0
pbMoveRoute($game_player, [ pbMoveRoute($game_player, [
PBMoveRoute::ScriptAsync, "moveRight90", PBMoveRoute::ScriptAsync, "moveRight90",
@@ -320,15 +314,15 @@ class PokemonDuel
pbMoveRoute(@event, [ pbMoveRoute(@event, [
PBMoveRoute::DirectionFixOff, PBMoveRoute::DirectionFixOff,
PBMoveRoute::ChangeSpeed, @oldeventspeed]) PBMoveRoute::ChangeSpeed, @oldeventspeed])
fadeTime = Graphics.frame_rate*4/10 fade_time = Graphics.frame_rate * 4 / 10
alphaDiff = (255.0/fadeTime).ceil alpha_diff = (255.0 / fade_time).ceil
fadeTime.times do fade_time.times do
@sprites["player"].opacity -= alphaDiff @sprites["player"].opacity -= alpha_diff
@sprites["opponent"].opacity -= alphaDiff @sprites["opponent"].opacity -= alpha_diff
@sprites["playerwindow"].contents_opacity -= alphaDiff @sprites["playerwindow"].contents_opacity -= alpha_diff
@sprites["opponentwindow"].contents_opacity -= alphaDiff @sprites["opponentwindow"].contents_opacity -= alpha_diff
@sprites["playerwindow"].opacity -= alphaDiff @sprites["playerwindow"].opacity -= alpha_diff
@sprites["opponentwindow"].opacity -= alphaDiff @sprites["opponentwindow"].opacity -= alpha_diff
Graphics.update Graphics.update
Input.update Input.update
pbUpdateSceneMap pbUpdateSceneMap
@@ -339,31 +333,31 @@ class PokemonDuel
def pbFlashScreens(player, opponent) def pbFlashScreens(player, opponent)
i = 0 i = 0
flashTime = Graphics.frame_rate*2/10 flash_time = Graphics.frame_rate * 2 / 10
alphaDiff = (2*255.0/flashTime).ceil alpha_diff = (2 * 255.0 / flash_time).ceil
flashTime.times do flash_time.times do
i += 1 i += 1
if player if player
@sprites["player"].color = Color.new(255,255,255,i*alphaDiff) @sprites["player"].color = Color.new(255, 255, 255, i * alpha_diff)
@sprites["playerwindow"].color = Color.new(255,255,255,i*alphaDiff) @sprites["playerwindow"].color = Color.new(255, 255, 255, i * alpha_diff)
end end
if opponent if opponent
@sprites["opponent"].color = Color.new(255,255,255,i*alphaDiff) @sprites["opponent"].color = Color.new(255, 255, 255, i * alpha_diff)
@sprites["opponentwindow"].color = Color.new(255,255,255,i*alphaDiff) @sprites["opponentwindow"].color = Color.new(255, 255, 255, i * alpha_diff)
end end
Graphics.update Graphics.update
Input.update Input.update
pbUpdateSceneMap pbUpdateSceneMap
end end
flashTime.times do flash_time.times do
i -= 1 i -= 1
if player if player
@sprites["player"].color = Color.new(255,255,255,i*alphaDiff) @sprites["player"].color = Color.new(255, 255, 255, i * alpha_diff)
@sprites["playerwindow"].color = Color.new(255,255,255,i*alphaDiff) @sprites["playerwindow"].color = Color.new(255, 255, 255, i * alpha_diff)
end end
if opponent if opponent
@sprites["opponent"].color = Color.new(255,255,255,i*alphaDiff) @sprites["opponent"].color = Color.new(255, 255, 255, i * alpha_diff)
@sprites["opponentwindow"].color = Color.new(255,255,255,i*alphaDiff) @sprites["opponentwindow"].color = Color.new(255, 255, 255, i * alpha_diff)
end end
Graphics.update Graphics.update
Input.update Input.update
@@ -379,29 +373,19 @@ class PokemonDuel
end end
end end
# Starts a duel. # Starts a duel.
# trainerid - ID or symbol of the opponent's trainer type. # trainer_id - ID or symbol of the opponent's trainer type.
# trainername - Name of the opponent # trainer_name - Name of the opponent
# event - Game_Event object for the character's event # event - Game_Event object for the character's event
# speeches - Array of 12 speeches # speeches - Array of 12 speeches
def pbDuel(trainerID,trainerName,event,speeches) def pbDuel(trainer_id, trainer_name, event, speeches)
trainerID = getID(PBTrainers,trainerID) trainer_id = getID(PBTrainers, trainer_id)
duel = PokemonDuel.new duel = PokemonDuel.new
opponent = PokeBattle_Trainer.new( opponent = PokeBattle_Trainer.new(
pbGetMessageFromHash(MessageTypes::TrainerNames,trainerName),trainerID) pbGetMessageFromHash(MessageTypes::TrainerNames,trainer_name), trainer_id)
speechTexts = [] speech_texts = []
for i in 0...12 for i in 0...12
speechTexts.push(_I(speeches[i])) speech_texts.push(_I(speeches[i]))
end
duel.pbDuel(opponent,event,speechTexts)
end
rescue Exception
if $!.is_a?(SystemExit) || "#{$!.class}"=="Reset"
raise $!
end end
duel.pbDuel(opponent, event, speech_texts)
end end

View File

@@ -261,7 +261,6 @@ class TriadScene
preview.y = 60 preview.y = 60
preview.z = 4 preview.z = 4
index = -1 index = -1
chosenSprites = []
for i in 0...@battle.maxCards for i in 0...@battle.maxCards
@sprites["player#{i}"] = Sprite.new(@viewport) @sprites["player#{i}"] = Sprite.new(@viewport)
@sprites["player#{i}"].x = Graphics.width-92 @sprites["player#{i}"].x = Graphics.width-92
@@ -443,9 +442,8 @@ class TriadScene
return choice return choice
end end
def pbPlayerPlaceCard(card,cardIndex) def pbPlayerPlaceCard(cardIndex)
@sprites["helpwindow"].text = _INTL("Place the card.") @sprites["helpwindow"].text = _INTL("Place the card.")
choice = 0
boardX = 0 boardX = 0
boardY = 0 boardY = 0
doRefresh = true doRefresh = true
@@ -733,9 +731,10 @@ class TriadScreen
for i in 0...@width*@height for i in 0...@width*@height
square = TriadSquare.new square = TriadSquare.new
if @elements if @elements
begin loop do
square.type = rand(PBTypes.maxValue+1) square.type = rand(PBTypes.maxValue+1)
end until !PBTypes.isPseudoType?(square.type) break if !PBTypes.isPseudoType?(square.type)
end
end end
@board.push(square) @board.push(square)
end end
@@ -793,7 +792,7 @@ class TriadScreen
minIndex = minLevel*20 minIndex = minLevel*20
maxIndex = maxLevel*20+20 maxIndex = maxLevel*20+20
opponentCards = [] opponentCards = []
for i in 0...self.maxCards self.maxCards.times do
# generate random card based on level # generate random card based on level
index = minIndex+rand(maxIndex-minIndex) index = minIndex+rand(maxIndex-minIndex)
opponentCards.push(candidates[index][0]) opponentCards.push(candidates[index][0])
@@ -821,7 +820,7 @@ class TriadScreen
while !position while !position
cardIndex = @scene.pbPlayerChooseCard(cards.length) cardIndex = @scene.pbPlayerChooseCard(cards.length)
triadCard = TriadCard.new(cards[cardIndex]) triadCard = TriadCard.new(cards[cardIndex])
position = @scene.pbPlayerPlaceCard(triadCard,cardIndex) position = @scene.pbPlayerPlaceCard(cardIndex)
end end
else else
# Opponent's turn # Opponent's turn
@@ -990,7 +989,7 @@ end
# Card storage # Card storage
#=============================================================================== #===============================================================================
class PokemonGlobalMetadata class PokemonGlobalMetadata
attr_accessor :triads attr_writer :triads
def triads def triads
@triads = TriadStorage.new if !@triads @triads = TriadStorage.new if !@triads

View File

@@ -431,7 +431,7 @@ class VoltorbFlip
# Create and split a string for the number, with padded 0s # Create and split a string for the number, with padded 0s
zeroes=2-num.to_s.length zeroes=2-num.to_s.length
numText="" numText=""
for j in 0...zeroes zeroes.times do
numText+="0" numText+="0"
end end
numText+=num.to_s numText+=num.to_s
@@ -449,7 +449,7 @@ class VoltorbFlip
# Create and split a string for the number, with padded 0s # Create and split a string for the number, with padded 0s
zeroes=2-num.to_s.length zeroes=2-num.to_s.length
numText="" numText=""
for j in 0...zeroes zeroes.times do
numText+="0" numText+="0"
end end
numText+=num.to_s numText+=num.to_s
@@ -466,7 +466,7 @@ class VoltorbFlip
def pbCreateCoins(source,y) def pbCreateCoins(source,y)
zeroes=5-source.to_s.length zeroes=5-source.to_s.length
coinText="" coinText=""
for i in 0...zeroes zeroes.times do
coinText+="0" coinText+="0"
end end
coinText+=source.to_s coinText+=source.to_s

View File

@@ -32,7 +32,7 @@ def pbLottery(winnum,nameVar=2,positionVar=3,matchedVar=4)
winmatched=thismatched winmatched=thismatched
end end
end end
pbEachPokemon { |poke,box| pbEachPokemon { |poke,_box|
thismatched=0 thismatched=0
id=poke.publicID id=poke.publicID
for j in 0...5 for j in 0...5

View File

@@ -339,8 +339,6 @@ class MiningGameScene
prepattern=ITEMS[i[0]][6] prepattern=ITEMS[i[0]][6]
next if provx+provwidth<=prex || provx>=prex+prewidth || next if provx+provwidth<=prex || provx>=prex+prewidth ||
provy+provheight<=prey || provy>=prey+preheight provy+provheight<=prey || provy>=prey+preheight
dx=prex-provx
dy=prey-provy
for j in 0...prepattern.length for j in 0...prepattern.length
next if prepattern[j]==0 next if prepattern[j]==0
xco=prex+(j%prewidth) xco=prex+(j%prewidth)
@@ -359,8 +357,6 @@ class MiningGameScene
prepattern=IRON[i[0]][4] prepattern=IRON[i[0]][4]
next if provx+provwidth<=prex || provx>=prex+prewidth || next if provx+provwidth<=prex || provx>=prex+prewidth ||
provy+provheight<=prey || provy>=prey+preheight provy+provheight<=prey || provy>=prey+preheight
dx=prex-provx
dy=prey-provy
for j in 0...prepattern.length for j in 0...prepattern.length
next if prepattern[j]==0 next if prepattern[j]==0
xco=prex+(j%prewidth) xco=prex+(j%prewidth)