fixing problems with error handling

app/controllers/articles_controller.rb

@@ -24,13 +24,6 @@ def create
 
   def show
     @article = Article.find_by(id: params[:id])
-
-    if @article
-      render 'show'
-    else
-      flash[:alert] = "Error showing the article."
-      redirect_to articles_path
-    end
   end
 
   def destroy
@@ -56,18 +49,12 @@ def update
     if @article.update(article_params)
       redirect_to @article, notice: 'Article was successfully updated.'
     else
+      flash[:alert] = "Article not found."
       render :edit
     end
   end
   def edit
     @article = Article.find_by(id: params[:id])
-
-    if @article
-      render 'edit'
-    else
-      flash[:alert] = "Article not found."
-      redirect_to articles_path
-    end
   end
 
   def my_articles

app/controllers/subscriptions_controller.rb

@@ -2,7 +2,7 @@ class SubscriptionsController < ApplicationController
   before_action :authenticate_user!
 
   def create
-    result = Subscriptions::Subscribe.new(params,current_user,article).call
+    Subscriptions::Subscribe.new(params,current_user,article).call
     redirect_to article, notice: 'Subscribed to the article.'
   end
 

app/mailers/subscription_mailer.rb

@@ -1,18 +1,12 @@
 class SubscriptionMailer < ApplicationMailer
   def new_subscription_notification(subscriber, article)
-    raise ArgumentError, "Subscriber cannot be nil" if subscriber.nil?
-    raise ArgumentError, "Article cannot be nil" if article.nil?
-
     @subscriber = subscriber
     @article = article
 
 
     mail(to: subscriber.email, subject: "You subscribe new article which title is #{article.title}")
   end
   def delete_subscription_notification(subscriber, article)
-    raise ArgumentError, "Subscriber cannot be nil" if subscriber.nil?
-    raise ArgumentError, "Article cannot be nil" if article.nil?
-
     @subscriber = subscriber
     @article = article
 

app/services/subscriptions/subscribe.rb

@@ -13,25 +13,20 @@ def call
     end
 
     def new_subscription
-      if @article
-        @user.subscribed_articles << @article
-      else
-        raise ArgumentError, "Cannot create subscription without an article"
+      @user.subscribed_articles << @article
+
+      if [email protected]_articles.include?(@article)
+        flash[:alert] = "Error creating the subscription."
+        redirect_to @article
       end
-    rescue StandardError => e
-      puts "Error in new_subscription: #{e.message}"
     end
 
     def adding_points
       @user.update!(points: @user.points + 10)
-    rescue StandardError => e
-      puts "Error in adding_points: #{e.message}"
     end
 
     def sending_mails
       SubscriptionMailer.new_subscription_notification(@user, @article).deliver_now
-    rescue StandardError => e
-      puts "Error in sending_mails: #{e.message}"
     end
   end
 end

app/services/subscriptions/unsubscribe.rb

@@ -22,15 +22,9 @@ def cancel_subscription
     end
     def cancel_points
       @user.update!(points: @user.points - 10)
-
-    rescue StandardError => e
-      puts "Error in cancel_points: #{e.message}"
     end
     def cancel_sending_mails
       SubscriptionMailer.delete_subscription_notification(@user, @article).deliver_now
-
-    rescue StandardError => e
-      puts "Error in sending_mails: #{e.message}"
     end
   end
 end