Телеграмм чат группы rubyrush страница 1125

рекурсию нельзя просто взять и выкинуть - можно заменить на цикл - скорее всего, на цикл while

17:04пожаловаться #1

def filename(name, index)
  unless index[name]
    index[name] = true
    return name
  end

  i = 1
  loop do
    new_name = "#{name}(#{i})"
    unless index[new_name]
      index[new_name] = true
      return new_name
    end
    i += 1
  end
end

def filenames(names)
  index = {}

  names.map do |name|
    filename(name, index)
  end
end

names = ["photo", "photo(1)", "photo", "photo(1)", "photo"] # photo, photo(1), photo(2), photo(1)(1), photo(3)
p filenames(names)

17:17пожаловаться #2

рекурсию нельзя просто взять и выкинуть - можно заменить на цикл - скорее всего, на цикл while

Не, работает верно. Рекурсии там нет, цикл один. У вас цикл и рекурсия в моем случае 1 цикл. Хотя инклуд проверка это тоже скрытый цикл

18:37пожаловаться #3

Вероятнее всего

18:37пожаловаться #4

Так что без двух циклов да, не выйдет.

18:38пожаловаться #5

class NamingUtility

  def self.filenames(names)
    hsh = Hash.new(0)
    new_names = []
    names.each do |name|
      hsh[name] += 1
      if hsh[name] == 1
        if new_names.include? name
          new_names << name + "(#{hsh[name]})"
        else
          new_names << name
        end
      else
        new_names << name + "(#{hsh[name] - 1})"
      end
    end
    new_names
  end
end


name = ["name", "photo", "photo", "doc", "photo(1)", "ivan",  "photo"]


p NamingUtility.filenames(name)

вот еще раз скину работающее решение, то же что и выше

18:39пожаловаться #6

селф потому что класс объявлен был, но не скопировал его

18:40пожаловаться #7

с рекурсией, честно. выглядит КУДА лучше. но я очень хотел разобраться для начала с хешем, рекурсию пойму, но уже не сегодня

18:41пожаловаться #8

R M

Я выше без рекурсии скинул решение. В том решении с рекурсией еще ошибка есть.

18:47пожаловаться #9

R M

class NamingUtility

  def self.filenames(names)
    hsh = Hash.new(0)
    new_names = []
    names.each do |name|
      hsh[name] += 1
      if hsh[name] == 1
        if new_names.include? name
          new_names << name + "(#{hsh[name]})"
        else
          new_names << name
        end
      else
        new_names << name + "(#{hsh[name] - 1})"
      end
    end
    new_names
  end
end


name = ["name", "photo", "photo", "doc", "photo(1)", "ivan",  "photo"]


p NamingUtility.filenames(name)

вот еще раз скину работающее решение, то же что и выше

См. выше, это решение неверно

18:47пожаловаться #10

См. выше, это решение неверно

вижу, спасибо

18:50пожаловаться #11

def filename(name, index)
  unless index[name]
    index[name] = true
    return name
  end

  i = 1
  loop do
    new_name = "#{name}(#{i})"
    unless index[new_name]
      index[new_name] = true
      return new_name
    end
    i += 1
  end
end

def filenames(names)
  index = {}

  names.map do |name|
    filename(name, index)
  end
end

names = ["photo", "photo(1)", "photo", "photo(1)", "photo"] # photo, photo(1), photo(2), photo(1)(1), photo(3)
p filenames(names)

ну формально это не рекурсия да))

19:01пожаловаться #12

мне вот это нравится решение

class NamingUtility

  def self.filenames(names)
    arr = []

    names.each do |elem|
      n = 1
      names.each  do |e|
        if elem == e
          if arr.include? e
            arr << e + "(#{n})"
            n += 1
          else
            arr << e
          end
        end
      end
    end
    arr.uniq
  end
end

name = ["name", "photo", "photo", "doc", "photo(1)", "photo(1)", "ivan",  "photo"]

p NamingUtility.filenames(name)

19:02пожаловаться #13

хотя я забыл про метод уник в конце) теперь уже не так сильно

19:03пожаловаться #14

def filename(name, index)
  unless index[name]
    index[name] = true
    return name
  end

  i = 1
  loop do
    new_name = "#{name}(#{i})"
    unless index[new_name]
      index[new_name] = true
      return new_name
    end
    i += 1
  end
end

def filenames(names)
  index = {}

  names.map do |name|
    filename(name, index)
  end
end

names = ["photo", "photo(1)", "photo", "photo(1)", "photo"] # photo, photo(1), photo(2), photo(1)(1), photo(3)
p filenames(names)

у вас цикл без условия выхода из него, это работать не должно по идее

19:13пожаловаться #15

:с неа, реплит умер

19:14пожаловаться #16