Updated Comments

.gitignore

@@ -31,5 +31,4 @@ render.experimental.xml
 google-services.json
 
 # Android Profiling
-*.hprof
-
+*.hprof

app/src/main/java/com/example/assignmentone/MainActivity.java

@@ -1,3 +1,9 @@
+/*
+Assignment 1 (part1)
+Assignment_1
+Lew Price
+*/
+
 package com.example.assignmentone;
 
 import android.annotation.SuppressLint;
@@ -27,7 +33,7 @@ public class MainActivity extends AppCompatActivity {
             return insets;
         });
 
-        TextInputLayout tempInput = findViewById(R.id.inputTemp);
+        TextInputLayout tempInput = findViewById(R.id.inputLayoutTemp);
         Button CtoF = findViewById(R.id.but_cToF);
         Button FtoC = findViewById(R.id.but_fToC);
         Button reset = findViewById(R.id.but_reset);
@@ -37,7 +43,8 @@ public class MainActivity extends AppCompatActivity {
 
             @Override
             public void onClick(View v) {
-                assert tempInput.getEditText() != null;
+                tempInput.setError(null);
+                if (tempInput.getEditText() == null) return;
                 String userInput = tempInput.getEditText().getText().toString().trim();
                 if(userInput.isEmpty()) {
                     tempInput.setError("Please enter a temperature");
@@ -50,7 +57,6 @@ public class MainActivity extends AppCompatActivity {
                     result.setText(resultString);
 
                 } catch (NumberFormatException e) {
-                    // Handle cases where the input is not a valid decimal number
                     tempInput.setError("Invalid number format");
                 }
             }
@@ -60,7 +66,8 @@ public class MainActivity extends AppCompatActivity {
 
             @Override
             public void onClick(View v) {
-                assert tempInput.getEditText() != null;
+                tempInput.setError(null);
+                if (tempInput.getEditText() == null) return;
                 String userInput = tempInput.getEditText().getText().toString().trim();
                 if(userInput.isEmpty()) {
                     tempInput.setError("Please enter a temperature");
@@ -83,6 +90,10 @@ public class MainActivity extends AppCompatActivity {
             @Override
             public void onClick(View v) {
                 result.setText("");
+                if (tempInput.getEditText() != null) {
+                    tempInput.getEditText().setText("");
+                }
+                tempInput.setError(null);
             }
         });
     }

app/src/main/res/layout/activity_main.xml

@@ -3,97 +3,120 @@
     xmlns:app="http://schemas.android.com/apk/res-auto"
     xmlns:tools="http://schemas.android.com/tools"
     android:id="@+id/main"
-    android:layout_width="wrap_content"
+    android:layout_width="match_parent"
     android:layout_height="match_parent"
-    android:textAlignment="center"
     tools:context=".MainActivity">
 
-    <TextView
-        android:id="@+id/result"
-        android:layout_width="109dp"
-        android:layout_height="28dp"
-        android:layout_marginStart="4dp"
-        android:layout_marginTop="28dp"
-        android:textSize="20sp"
-        app:layout_constraintStart_toEndOf="@+id/text_conversion"
-        app:layout_constraintTop_toBottomOf="@+id/inputTemp" />
-
     <TextView
         android:id="@+id/textView2"
-        android:layout_width="410dp"
-        android:layout_height="44dp"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:layout_marginTop="16dp"
         android:text="Temperature Conversion"
         android:textAlignment="center"
         android:textSize="24sp"
         app:layout_constraintTop_toTopOf="parent"
-        tools:layout_editor_absoluteX="8dp" />
+        app:layout_constraintStart_toStartOf="parent"
+        app:layout_constraintEnd_toEndOf="parent" />
 
     <TextView
         android:id="@+id/text_Temp"
-        android:layout_width="128dp"
+        android:layout_width="wrap_content"
         android:layout_height="wrap_content"
-        android:layout_marginStart="8dp"
-        android:layout_marginTop="4dp"
-        android:lineSpacingExtra="10sp"
+        android:layout_marginStart="16dp"
         android:text="Temperature:"
-        android:textAlignment="center"
         android:textSize="20sp"
-        android:typeface="sans"
+        app:layout_constraintBottom_toBottomOf="@+id/inputLayoutTemp"
         app:layout_constraintStart_toStartOf="parent"
-        app:layout_constraintTop_toBottomOf="@+id/textView2" />
+        app:layout_constraintTop_toTopOf="@+id/inputLayoutTemp" />
 
     <com.google.android.material.textfield.TextInputLayout
-        android:layout_width="408dp"
+        android:id="@+id/inputLayoutTemp"
+        android:layout_width="0dp"
         android:layout_height="wrap_content"
+        android:layout_marginStart="8dp"
         android:layout_marginEnd="16dp"
-        app:layout_constraintEnd_toEndOf="parent"
-        tools:layout_editor_absoluteY="44dp">
-
-    </com.google.android.material.textfield.TextInputLayout>
-
-    <com.google.android.material.textfield.TextInputEditText
-        android:id="@+id/inputTemp"
-        android:layout_width="280dp"
-        android:layout_height="41dp"
-        android:hint="Enter Temperature"
+        android:layout_marginTop="16dp"
+        app:hintEnabled="false"
         app:layout_constraintEnd_toEndOf="parent"
         app:layout_constraintStart_toEndOf="@+id/text_Temp"
-        app:layout_constraintTop_toBottomOf="@+id/textView2" />
+        app:layout_constraintTop_toBottomOf="@+id/textView2">
+
+        <com.google.android.material.textfield.TextInputEditText
+            android:id="@+id/inputTemp"
+            android:layout_width="match_parent"
+            android:layout_height="wrap_content"
+            android:hint="Enter Temperature"
+            android:textColorHint="#9E9E9E"
+            android:inputType="numberDecimal|numberSigned" />
+    </com.google.android.material.textfield.TextInputLayout>
 
     <TextView
         android:id="@+id/text_conversion"
-        android:layout_width="109dp"
-        android:layout_height="28dp"
-        android:layout_marginTop="40dp"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:layout_marginStart="16dp"
+        android:layout_marginTop="24dp"
         android:text="Conversion:"
         android:textSize="20sp"
-        app:layout_constraintTop_toBottomOf="@+id/text_Temp"
-        tools:layout_editor_absoluteX="4dp" />
+        app:layout_constraintStart_toStartOf="parent"
+        app:layout_constraintTop_toBottomOf="@+id/inputLayoutTemp" />
+
+    <TextView
+        android:id="@+id/result"
+        android:layout_width="0dp"
+        android:layout_height="wrap_content"
+        android:layout_marginStart="8dp"
+        android:layout_marginEnd="16dp"
+        android:textSize="20sp"
+        app:layout_constraintBottom_toBottomOf="@+id/text_conversion"
+        app:layout_constraintEnd_toEndOf="parent"
+        app:layout_constraintStart_toEndOf="@+id/text_conversion"
+        app:layout_constraintTop_toTopOf="@+id/text_conversion" />
 
     <Button
         android:id="@+id/but_cToF"
-        android:layout_width="409dp"
-        android:layout_height="51dp"
+        android:layout_width="0dp"
+        android:layout_height="wrap_content"
+        android:layout_marginStart="16dp"
         android:layout_marginTop="24dp"
+        android:layout_marginEnd="16dp"
+        android:backgroundTint="#A9A7A7"
         android:text="C to F"
-        android:textColorLink="#A09F9F"
-        app:layout_constraintTop_toBottomOf="@+id/text_conversion"
-        tools:layout_editor_absoluteX="34dp" />
+        android:textColor="#FFFFFF"
+        app:cornerRadius="0dp"
+        app:layout_constraintEnd_toEndOf="parent"
+        app:layout_constraintStart_toStartOf="parent"
+        app:layout_constraintTop_toBottomOf="@+id/text_conversion" />
 
     <Button
         android:id="@+id/but_fToC"
-        android:layout_width="409dp"
-        android:layout_height="51dp"
+        android:layout_width="0dp"
+        android:layout_height="wrap_content"
+        android:layout_marginStart="16dp"
+        android:layout_marginEnd="16dp"
+        android:layout_marginTop="8dp"
+        android:backgroundTint="#A9A7A7"
         android:text="F to C"
-        app:layout_constraintTop_toBottomOf="@+id/but_cToF"
-        tools:layout_editor_absoluteX="0dp" />
+        android:textColor="#FFFFFF"
+        app:cornerRadius="0dp"
+        app:layout_constraintEnd_toEndOf="parent"
+        app:layout_constraintStart_toStartOf="parent"
+        app:layout_constraintTop_toBottomOf="@+id/but_cToF" />
 
     <Button
         android:id="@+id/but_reset"
-        android:layout_width="409dp"
-        android:layout_height="51dp"
+        android:layout_width="0dp"
+        android:layout_height="wrap_content"
+        android:layout_marginStart="16dp"
+        android:layout_marginEnd="16dp"
+        android:layout_marginTop="8dp"
+        android:backgroundTint="#A9A7A7"
         android:text="Reset"
-        app:layout_constraintTop_toBottomOf="@+id/but_fToC"
-        tools:layout_editor_absoluteX="-13dp" />
+        android:textColor="#FFFFFF"
+        app:cornerRadius="0dp"
+        app:layout_constraintEnd_toEndOf="parent"
+        app:layout_constraintStart_toStartOf="parent"
+        app:layout_constraintTop_toBottomOf="@+id/but_fToC" />
 
 </androidx.constraintlayout.widget.ConstraintLayout>