students_list_yaml_Quartz 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 95c6092475db21fe4c65f122426f93af80bad40716d3c43a612ffd7a9b6700ae
-  data.tar.gz: 33fcb5592ee4b4c9922537ce5c9e9c43d1850714764cbbe23200c156a77789ad
+  metadata.gz: 60a29ec36c1f46d57819b067b74cef537456451c37755f45e4aed25fd3b72551
+  data.tar.gz: a5423f98228dadbbcd6583480fe1fb54baa29860de666eca572e967863702b99
 SHA512:
-  metadata.gz: 8cc4a7aae5c8d4013323fd639e12246ff77e3342742ca44070c576283d46fe842cfbaad8cc5a5ca2682b652daa67e3387b32ac99d9d36ddbecc38e4c9015d90a
-  data.tar.gz: 378290c9c17c7685a217cac342d3a18ec74c53890644836ccc1c58a60d8767cfaf21165047f2dab2f09c73848ac63a65427c83df49e857c00cafe22b8bdadf23
+  metadata.gz: 8b00e3aa69c142229c6215e6d2b2e87dc01186ee1082b3ca93c36fae1faa77a27aadbd21013d7def8db69117e42797f724b9c5242cdcf864d55fc3d16ecb4135
+  data.tar.gz: 30ec41abe8cd8896054e939692e04e471b5d37419c0169a2b470111312aeabee837703c8a207c3de81fe25923ef94b71ea724fa169860bf0df6c9168792a2c7d

data/lib/student.rb

@@ -106,19 +106,6 @@ def contact
 	  @patronymic.to_s <=> other.patronymic.to_s
 	end
 	
-
-  def ==(other)
-    return false unless other.is_a?(Base)
-    
-    contacts_overlap = false
-    [@phone, @email, @telegram].each do |contact|
-      next if contact.nil? || contact.empty?
-      if [other.instance_variable_get(:@phone), 
-          other.instance_variable_get(:@email), 
-          other.instance_variable_get(:@telegram)].include?(contact)
-        contacts_overlap = true
-        break
-      end
     end
     
     comparison_key == other.comparison_key || contacts_overlap

data/lib/students_list_yaml/version.rb

@@ -1,3 +1,3 @@
 module StudentsListYaml
-  VERSION = "0.1.0"
+  VERSION = "0.1.1"
 end

data/lib/students_list_yaml.rb

@@ -1,93 +1,116 @@
-# frozen_string_literal: true
-
-require "yaml"
-require_relative "students_list_yaml/version"
-require_relative "student"
-require_relative "student_short"
-require_relative "data_list"
-require_relative "data_table"
-require_relative "base"
+require 'yaml'
+# Подключаем зависимости, которые должны лежать в папке lib/students_list_yaml/
+require_relative 'students_list_yaml/student'
+require_relative 'students_list_yaml/student_short'
+require_relative 'students_list_yaml/data_list'
+
+class Students_List_YAML
+  attr_reader :file_path
+  
+  def initialize(file_path = nil)
+    @students = []
+    @file_path = file_path
+    load_data if file_path && File.exist?(file_path)
+  end
 
-module StudentsListYaml
-  class StudentsList
-    attr_reader :file_path
+  def load_data
+    return unless @file_path && File.exist?(@file_path)
 
-    def initialize(file_path = nil)
-      @students = []
-      @file_path = file_path
-      load_data if file_path && File.exist?(file_path)
-    end
+    begin
+      file_content = File.read(@file_path)
+      return if file_content.strip.empty?
 
-    def load_data
-      return unless @file_path && File.exist?(@file_path)
+      # YAML.safe_load безопаснее, но требует разрешения на создание ваших классов,
+      # если в YAML хранятся объекты. Если там просто хеши — символизация ключей ок.
+      data = YAML.safe_load(file_content, permitted_classes: [Date, Symbol], symbolize_names: true)
 
-      begin
-        file_content = File.read(@file_path)
-        return if file_content.strip.empty?
-
-        data = YAML.safe_load(file_content, symbolize_names: true)
+      # Проверка: data должен быть массивом
+      if data.is_a?(Array)
         @students = data.map { |student_data| Student.from_hash(student_data) }
-      rescue Psych::SyntaxError
+      else
         @students = []
       end
+    rescue Psych::SyntaxError => e
+      puts "Ошибка парсинга YAML: #{e.message}. Создан пустой список."
+      @students = []
     end
+  end
 
-    def save_data
-      return unless @file_path
-
-      students_data = @students.map(&:to_hash)
-      File.write(@file_path, students_data.to_yaml)
-    end
-
-    def get_student_by_id(id)
-      @students.find { |student| student.id == id }
-    end
+  def student_unic?(student)
+    @students.none? { |unic_student| unic_student == student }
+  end
 
-    def get_k_n_student_short_list(k, n, existing_data_list = nil)
-      start_index = (n - 1) * k
-      selected_students = @students[start_index, k] || []
+  def save_data
+    return unless @file_path
+    students_data = @students.map(&:to_hash)
+    File.write(@file_path, students_data.to_yaml)
+  end
 
-      short_students = selected_students.map { |student| StudentShort.from_student(student) }
+  def get_student_by_id(id)
+    @students.find { |student| student.id == id }
+  end
 
-      if existing_data_list && existing_data_list.is_a?(Data_list)
-        existing_data_list.data = short_students
-        existing_data_list
-      else
-        Data_list_student_short.new(short_students)
-      end
+  # ИСПРАВЛЕНИЕ: метод получения списка с пагинацией
+  def get_k_n_student_short_list(k, n, existing_data_list = nil)
+    start_index = (n - 1) * k
+    # Используем slice вместо range с неопределенным end_index
+    selected_students = @students.slice(start_index, k) || []
+    
+    student_shorts = selected_students.map { |student| StudentShort.from_student(student) }
+
+    if existing_data_list
+      # Убедитесь, что existing_data_list имеет сеттер data=
+      existing_data_list.data = student_shorts
+      existing_data_list
+    else
+      # Убедитесь, что этот класс доступен
+      DataListStudentShort.new(student_shorts) 
     end
+  end
 
-    def sort_by_full_name
-      @students.sort_by! { |student| student.last_name_initials.downcase }
-    end
+  def sort_by_full_name
+    @students.sort_by! { |student| student.last_name_initials.downcase }
+  end
 
-    def add_student(student)
-      new_id = @students.empty? ? 1 : @students.map(&:id).max + 1
-      student_with_id = Student.from_hash(student.to_hash.merge(id: new_id))
-      @students << student_with_id
-      student_with_id
+  def add_student(student)
+    unless student_unic?(student)
+      raise ArgumentError, "Студент не уникальный"
     end
+    # Исправление для пустого массива: если студентов нет, id = 1
+    new_id = @students.empty? ? 1 : (@students.map(&:id).max || 0) + 1
 
-    def replace_student_by_id(id, new_student)
-      index = @students.find_index { |s| s.id == id }
-      return false unless index
+    # Важно: student.to_hash должен возвращать хеш с символьными ключами
+    student_with_id = Student.from_hash(student.to_hash.merge(id: new_id))
 
-      @students[index] = Student.from_hash(new_student.to_hash.merge(id: id))
-      true
-    end
+    @students << student_with_id
+    student_with_id
+  end
 
-    def delete_student_by_id(id)
-      initial_size = @students.size
-      @students.reject! { |s| s.id == id }
-      initial_size != @students.size
+  def replace_student_by_id(id, new_student)
+    # Здесь логика правильная, но проверьте, нужно ли проверять уникальность 
+    # самого себя при замене (иногда это вызывает ошибку, если данные не менялись)
+    unless student_unic?(new_student)
+       # Можно добавить проверку: если это тот же студент, то ошибку не кидать
     end
+    
+    index = @students.find_index { |s| s.id == id }
+    return false unless index
 
-    def get_student_short_count
-      @students.size
-    end
+    @students[index] = Student.from_hash(new_student.to_hash.merge(id: id))
+    true
+  end
 
-    private
+  def delete_student_by_id(id)
+    initial_size = @students.size
+    @students.reject! { |s| s.id == id }
+    initial_size != @students.size
+  end
 
-    attr_writer :students
+  def get_student_short_count
+    @students.size
   end
+
+  private
+
+  attr_accessor :students
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: students_list_yaml_Quartz
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors:
 - Kosta717