opal/opal

File string.rb has 1595 lines of code (exceeds 1000 allowed). Consider refactoring.

Open

require 'corelib/comparable'
require 'corelib/regexp'

class ::String < `String`
  include ::Comparable

Method tr_s has 149 lines of code (exceeds 30 allowed). Consider refactoring.

Open

  def tr_s(from, to)
    %x{
      from = $coerce_to(from, #{::String}, 'to_str').$to_s();
      to = $coerce_to(to, #{::String}, 'to_str').$to_s();

Method tr has 130 lines of code (exceeds 30 allowed). Consider refactoring.

Open

  def tr(from, to)
    %x{
      from = $coerce_to(from, #{::String}, 'to_str').$to_s();
      to = $coerce_to(to, #{::String}, 'to_str').$to_s();

Method split has 77 lines of code (exceeds 30 allowed). Consider refactoring.

Open

  def split(pattern = undefined, limit = undefined)
    %x{
      if (self.length === 0) {
        return [];
      }

Method next has 67 lines of code (exceeds 30 allowed). Consider refactoring.

Open

  def next
    %x{
      var i = self.length;
      if (i === 0) {
        return '';

Method [] has 66 lines of code (exceeds 30 allowed). Consider refactoring.

Open

  def [](index, length = undefined)
    %x{
      var size = self.length, exclude, range;

      if (index.$$is_range) {

Method gsub has 61 lines of code (exceeds 30 allowed). Consider refactoring.

Open

  def gsub(pattern, replacement = undefined, &block)
    %x{
      if (replacement === undefined && block === nil) {
        return #{enum_for :gsub, pattern};
      }

Method to_i has 51 lines of code (exceeds 30 allowed). Consider refactoring.

Open

  def to_i(base = 10)
    %x{
      var result,
          string = self.toLowerCase(),
          radix = $coerce_to(base, #{::Integer}, 'to_int');

Method sub has 40 lines of code (exceeds 30 allowed). Consider refactoring.

Open

  def sub(pattern, replacement = undefined, &block)
    %x{
      if (!pattern.$$is_regexp) {
        pattern = $coerce_to(pattern, #{::String}, 'to_str');
        pattern = new RegExp(pattern.replace(/[.*+?^${}()|[\]\\]/g, '\\$&'));

Method index has 39 lines of code (exceeds 30 allowed). Consider refactoring.

Open

  def index(search, offset = undefined)
    %x{
      var index,
          match,
          regex;

Method oct has 37 lines of code (exceeds 30 allowed). Consider refactoring.

Open

  def oct
    %x{
      var result,
          string = self,
          radix = 8;

Method rindex has 35 lines of code (exceeds 30 allowed). Consider refactoring.

Open

  def rindex(search, offset = undefined)
    %x{
      var i, m, r, _m;

      if (offset === undefined) {

Method each_line has 35 lines of code (exceeds 30 allowed). Consider refactoring.

Open

  def each_line(separator = $/, chomp: false, &block)
    return enum_for :each_line, separator, chomp: chomp unless block_given?

    %x{
      if (separator === nil) {

Method upto has 35 lines of code (exceeds 30 allowed). Consider refactoring.

Open

  def upto(stop, excl = false, &block)
    return enum_for :upto, stop, excl unless block_given?
    %x{
      var a, b, s = self.toString();

Method rpartition has 32 lines of code (exceeds 30 allowed). Consider refactoring.

Open

  def rpartition(sep)
    %x{
      var i, m, r, _m;

      if (sep.$$is_regexp) {

Similar blocks of code found in 2 locations. Consider refactoring.

Invalid

    width  = `$coerce_to(#{width}, #{::Integer}, 'to_int')`
    padstr = `$coerce_to(#{padstr}, #{::String}, 'to_str')`.to_s

    if padstr.empty?
      ::Kernel.raise ::ArgumentError, 'zero width padding'

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Invalid

  def ljust(width, padstr = ' ')
    width  = `$coerce_to(#{width}, #{::Integer}, 'to_int')`
    padstr = `$coerce_to(#{padstr}, #{::String}, 'to_str')`.to_s

    if padstr.empty?

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Invalid

      if (from.length == 0 || from === to) {
        return self;
      }

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Invalid

      if (from.length == 0) {
        return self;
      }

• Read up
  Read up