X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=blobdiff_plain;f=src%2Flanguage%2Fstats%2Ffactor.c;fp=src%2Flanguage%2Fstats%2Ffactor.c;h=58d5b868eb5be5d5d2fc685ed1466b4ae40dd52e;hb=d0b91eae59319ab2756d0d43b9cb15eb9cd3c234;hp=29e30c16b4b022b3eb26858507562dedc90f6d0e;hpb=ddb7b52128d8f1f54d9632dc3a15c7869e0fbcce;p=pspp-builds.git

diff --git a/src/language/stats/factor.c b/src/language/stats/factor.c
index 29e30c16..58d5b868 100644
--- a/src/language/stats/factor.c
+++ b/src/language/stats/factor.c
@@ -1,5 +1,5 @@
 /* PSPP - a program for statistical analysis.
-   Copyright (C) 2009 Free Software Foundation, Inc.
+   Copyright (C) 2009, 2010 Free Software Foundation, Inc.
 
    This program is free software: you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
@@ -552,6 +552,9 @@ cmd_factor (struct lexer *lexer, struct dataset *ds)
 			      PV_NO_DUPLICATE | PV_NUMERIC))
     goto error;
 
+  if (factor.n_vars < 2)
+    msg (MW, _("Factor analysis on a single variable is not useful."));
+
   while (lex_token (lexer) != '.')
     {
       lex_match (lexer, '/');
@@ -1010,7 +1013,7 @@ static void
 show_factor_matrix (const struct cmd_factor *factor, struct idata *idata, const gsl_matrix *fm)
 {
   int i;
-  const int n_factors = n_extracted_factors (factor, idata);
+  const int n_factors = idata->n_extractions;
 
   const int heading_columns = 1;
   const int heading_rows = 2;
@@ -1225,7 +1228,7 @@ show_explained_variance (const struct cmd_factor * factor, struct idata *idata,
 
       if (factor->print & PRINT_EXTRACTION)
 	{
-	  if ( i < n_extracted_factors (factor, idata))
+	  if (i < idata->n_extractions)
 	    {
 	      /* Sums of squared loadings */
 	      tab_double (t, c++, i + heading_rows, 0, e_lambda, NULL);
@@ -1474,12 +1477,26 @@ do_factor (const struct cmd_factor *factor, struct casereader *r)
   gsl_eigen_symmv_sort (idata->eval, idata->evec, GSL_EIGEN_SORT_ABS_DESC);
 #endif
 
+  idata->n_extractions = n_extracted_factors (factor, idata);
+
+  if (idata->n_extractions == 0)
+    {
+      msg (MW, _("The FACTOR criteria result in zero factors extracted. Therefore no analysis will be performed."));
+      goto finish;
+    }
+
+  if (idata->n_extractions > factor->n_vars)
+    {
+      msg (MW, _("The FACTOR criteria result in more factors than variables, which is not meaningful. No analysis will be performed."));
+      goto finish;
+    }
+    
   {
     const gsl_vector *extracted_eigenvalues = NULL;
     gsl_vector *initial_communalities = gsl_vector_alloc (factor->n_vars);
     gsl_vector *extracted_communalities = gsl_vector_alloc (factor->n_vars);
     size_t i;
-    struct factor_matrix_workspace *fmw = factor_matrix_workspace_alloc (idata->msr->size, n_extracted_factors (factor, idata));
+    struct factor_matrix_workspace *fmw = factor_matrix_workspace_alloc (idata->msr->size, idata->n_extractions);
     gsl_matrix *factor_matrix = gsl_matrix_calloc (factor->n_vars, fmw->n_factors);
 
     if ( factor->extraction == EXTRACTION_PAF)
@@ -1518,10 +1535,9 @@ do_factor (const struct cmd_factor *factor, struct casereader *r)
       }
     else if (factor->extraction == EXTRACTION_PC)
       {
-	for (i = 0 ; i < factor->n_vars; ++i)
-	  {
-	    gsl_vector_set (initial_communalities, i, communality (idata, i, factor->n_vars));
-	  }
+	for (i = 0; i < factor->n_vars; ++i)
+	  gsl_vector_set (initial_communalities, i, communality (idata, i, factor->n_vars));
+
 	gsl_vector_memcpy (extracted_communalities, initial_communalities);
 
 	iterate_factor_matrix (analysis_matrix, extracted_communalities, factor_matrix, fmw);
@@ -1542,6 +1558,8 @@ do_factor (const struct cmd_factor *factor, struct casereader *r)
     gsl_vector_free (extracted_communalities);
   }
 
+ finish:
+
   idata_free (idata);
 
   casereader_destroy (r);